OSDN Git Service
Treehugger Robot [Tue, 16 Apr 2019 00:06:53 +0000 (00:06 +0000)]
Merge "Mark these yacc files as creating headers"
Treehugger Robot [Mon, 15 Apr 2019 21:49:01 +0000 (21:49 +0000)]
Merge "Implement Queue"
Treehugger Robot [Mon, 15 Apr 2019 20:37:28 +0000 (20:37 +0000)]
Merge "Change storage metadata type to byte array"
Myles Watson [Fri, 12 Apr 2019 21:30:51 +0000 (14:30 -0700)]
RootCanal: Open network ports only if there is a console
Bug:
128355308
Test: toggle airplane mode and check for opening sockets
Change-Id: I8c4581b708c05a8c72dba3604c02adea97c1653b
Dan Willemsen [Fri, 12 Apr 2019 22:16:12 +0000 (15:16 -0700)]
Mark these yacc files as creating headers
So that they're properly represented in the build graph.
Test: treehugger
Change-Id: If7141171117260b870dc6a0c6608dcae696b3189
Chienyuan [Thu, 4 Apr 2019 17:29:02 +0000 (10:29 -0700)]
Implement Queue
* Implement Queue which provides a flow control mechanism for data
transmission between modules. This queue streaming data with
EnqueueCallback and DequeueCallback based on the reactor pattern.
Enqueue end should register callback when data is ready to be sent
and unregister when no data ready to send. Dequeue end should
register callback when ready to handle data and unregister when not
ready to handle data.
* Implement ReactiveSemaphore, a wrapper for an event_fd work in
non-blocking and Semaphore mode used by Queue to handle readable
signal in the reactor pattern.
Test: run bluetooth_test_gd
Change-Id: Ia7019cdbe271d193c92f1a0b405ecced41a2d84b
Hansong Zhang [Tue, 9 Apr 2019 15:55:27 +0000 (08:55 -0700)]
Simple stack bring up
Just bring up the HAL transportation layer
Test: unit test
Change-Id: I94641d3008ce155d13df6ffe78f3710b3efca645
Hansong Zhang [Wed, 10 Apr 2019 22:46:25 +0000 (15:46 -0700)]
HCI HAL adapter: separate initialization and set_callback
Test: run bluetooth_test_gd
Change-Id: I1275be3020e34c56e0ef439e0f748db127d9f8e2
Hansong Zhang [Thu, 11 Apr 2019 21:24:37 +0000 (14:24 -0700)]
Add .gitignore
Test: compile
Change-Id: I0e4dd26c07ba5ed5e6ea4acf0fa9c9f831cba534
Treehugger Robot [Thu, 11 Apr 2019 05:46:26 +0000 (05:46 +0000)]
Merge "packet: Enforce View inheritance and validation"
Jakub Pawlowski [Wed, 10 Apr 2019 14:48:30 +0000 (14:48 +0000)]
Merge "PDL: SMP packet definitions"
Myles Watson [Thu, 4 Apr 2019 22:33:43 +0000 (15:33 -0700)]
packet: Enforce View inheritance and validation
Add was_validated_ flag to Views that can be constructed from bytes.
Check was_validated_ in every generated Get* function.
Test: bluetooth_packet_parser_test --gtest_filter=*Validate*Death*
Change-Id: I1de8d4e5e60bdd7b51562ffd258300e808db4adb
Zach Johnson [Tue, 9 Apr 2019 21:47:23 +0000 (21:47 +0000)]
Merge "Remove Ajay from owners"
Zach Johnson [Tue, 9 Apr 2019 21:43:41 +0000 (14:43 -0700)]
Remove Ajay from owners
Moved to a different team.
Test: none
Change-Id: I2df8efa1dd59a98ae5c0d9bcd801955e5c85f8d7
Hansong Zhang [Mon, 8 Apr 2019 21:26:53 +0000 (14:26 -0700)]
Bluetooth HCI HAL adapter for Android HIDL
Test: unit test
Change-Id: I60339d3f0845437e64705edc744e6091a2c3a171
Treehugger Robot [Tue, 9 Apr 2019 17:14:53 +0000 (17:14 +0000)]
Merge "Switch to clang-r353983b."
Xin Li [Tue, 9 Apr 2019 04:29:22 +0000 (04:29 +0000)]
Merge "DO NOT MERGE -- Merge PPRL.190405.003 into master"
Myles Watson [Mon, 11 Mar 2019 23:48:08 +0000 (16:48 -0700)]
PDL: SMP packet definitions
Change-Id: Ia6e2da10a959e9476c4f3f32398ac5e815996669
Treehugger Robot [Tue, 9 Apr 2019 04:06:16 +0000 (04:06 +0000)]
Merge "hci: CreateConnectionRoleSwitch is 8 bits"
Myles Watson [Mon, 8 Apr 2019 23:41:33 +0000 (16:41 -0700)]
hci: CreateConnectionRoleSwitch is 8 bits
Test: Manual comparison with the spec
Vol 2 Part E 7.1.5 Command Parameter Allow_Role_Switch
Change-Id: Iaa7df5c3c946e3740115df79f14bfae0ddf872b6
Xin Li [Tue, 9 Apr 2019 01:56:18 +0000 (18:56 -0700)]
DO NOT MERGE -- Merge PPRL.190405.003 into master
Bug:
130175407
Change-Id: I643bb187b1555598121d8b4a15b0acd73fe1f741
Treehugger Robot [Tue, 9 Apr 2019 01:50:55 +0000 (01:50 +0000)]
Merge "packet/parser: Add pragma once to generated files"
Treehugger Robot [Tue, 9 Apr 2019 00:26:30 +0000 (00:26 +0000)]
Merge "no forward_list in hci/acl_builder_test.cc"
Myles Watson [Mon, 8 Apr 2019 19:08:33 +0000 (12:08 -0700)]
no forward_list in hci/acl_builder_test.cc
Test: bluetooth_test_gd --gtest_filter="*Acl*"
Change-Id: I6a764ec276102df59897102d6380110b5dd266ce
Myles Watson [Fri, 5 Apr 2019 15:45:43 +0000 (08:45 -0700)]
packet/parser: Add pragma once to generated files
Test: bluetooth_packet_parser_test
Change-Id: Ie0a2717cdbf74859fd457ee68533caded16a5d20
Hansong Zhang [Fri, 29 Mar 2019 21:27:05 +0000 (14:27 -0700)]
Adapter layer for Bluetooth HCI HAL
Add adapter layer for HIDL HAL and host rootcanal HAL
Test: with cert and rootcanal
Change-Id: I1a84076d3df3176b75a499ac160e503e51fad8b3
android-build-team Robot [Mon, 8 Apr 2019 18:10:47 +0000 (18:10 +0000)]
Snap for
5310204 from
817facf4bc2a956fafa8a38c15ab5963f2d0bc0e to pi-platform-release
Change-Id: I704240f3f6bd8da8e23865ff7a6eb85b2233bcb1
Stephen Hines [Fri, 5 Apr 2019 23:06:55 +0000 (16:06 -0700)]
Switch to clang-r353983b.
This is the current toolchain version being used in
build/soong/cc/config/global.go, so it should be used here as well.
Ideally, this python file would be updated to pick up the value directly
from the build configuration.
Test: N/A
Change-Id: I1c615a0d0c6dc4dedd82d1ced0381bab273f9e61
Yi Kong [Fri, 29 Mar 2019 22:42:58 +0000 (15:42 -0700)]
Modernise code to use override specifier
Generated by clang-tidy.
Test: m checkbuild
Change-Id: Icba58009f6db2a0e9e1360326b0b66fc10d04a5a
Treehugger Robot [Fri, 5 Apr 2019 05:05:25 +0000 (05:05 +0000)]
Merge changes Ica38a448,I526fb846
* changes:
Add namespace in bluetooth_packetgen
Add multi file support for bluetooth_packetgen
Treehugger Robot [Fri, 5 Apr 2019 02:19:38 +0000 (02:19 +0000)]
Merge "Add packet parser tests to TEST_MAPPING"
Treehugger Robot [Fri, 5 Apr 2019 01:35:30 +0000 (01:35 +0000)]
Merge "Fix handling of startup_timer_expired"
Hansong Zhang [Thu, 4 Apr 2019 23:23:32 +0000 (16:23 -0700)]
Add namespace in bluetooth_packetgen
::bluetooth is our top level namespace. Each subdirectory is a
namespace.
Test: bluetooth_packet_parser_test
Change-Id: Ica38a448c692396e6a34dd66eabc3eaa8df884e0
Hansong Zhang [Thu, 4 Apr 2019 20:24:43 +0000 (13:24 -0700)]
Add multi file support for bluetooth_packetgen
Test: bluetooth_packet_parser_test
Change-Id: I526fb8462f67f2ce099aab4e63ca695797f36723
Zach Johnson [Fri, 5 Apr 2019 00:21:31 +0000 (17:21 -0700)]
Add packet parser tests to TEST_MAPPING
Test: atest bluetooth_packet_parser_test
Change-Id: If82370a834825cd265fac3c7e100176b94c79bb0
Myles Watson [Mon, 25 Mar 2019 22:25:08 +0000 (15:25 -0700)]
PDL: L2CAP packet definition
Test: Comparison with the SPEC
bluetooth_test_gd --gtest_filter="*L2cap*"
Change-Id: If29036fa7147b27d5a79458773c5eb772eb15817
Myles Watson [Mon, 25 Mar 2019 22:24:43 +0000 (15:24 -0700)]
PDL: HCI packet definition
Test: Comparison with the SPEC
bluetooth_test_gd --gtest_filter="*Acl*"
Change-Id: I48fcd2655edb54d831f8b395de9a209c46049189
Ajay Panicker [Wed, 6 Feb 2019 22:02:36 +0000 (14:02 -0800)]
packet: Add the PDL Packet Generator
Test: bluetooth_packet_parser_test
Change-Id: Ia741e608aebff7b55d005a37e0f0eaeae63767a8
Chienyuan [Sat, 30 Mar 2019 00:33:41 +0000 (17:33 -0700)]
Fix memory leak of reactor and reactor_unittest
* Fix memory leak happen in below two cases
Case 1
1. Reactable1 register
2. Reactable2 register
3. Reactable2 unregister during the callback event of Reactable2
4. Reactable1 unregister
In step 3, reactable_removed_ will set to true due to unregister while
reactable->is_executing_ is true, it makes sure we delete Reactable2
after the callback is executed. But it will cause Reactable1 to not be
deleted in step 4.
To avoid this, we can reset reactable_removed_ to false after Reactable
deleted.
Case 2
1. Reactable1 register
2. Reactable2 register
3. Reactable2 unregister during the callback event of Reactable2
4. Reactable1 unregister from different thread during step 3 processing
In step 3, although we reset reactable_removed_ to false after Reactable2
deleted immediately for case 1, if other thread unregister Reactable1
before Reactable2 deleted, Reactable1 will fail to be deleted.
To avoid thie, we add a local variable to check if the Reacable is
executing. If not, deleted the Reactable directly.
* Add unittest for these two cases
* Fix memory leak in reactor_unittest
* Turn on cfi and address flag
Test: sudo ./bluetooth_test_gd
Change-Id: I0a0ca79b439fd3a1bf3ec0fa2b2a43a88e037fbb
Chienyuan [Wed, 3 Apr 2019 23:12:36 +0000 (16:12 -0700)]
PacketBuilderTest: Modify member array
Test: sudo ./bluetooth_test_gd
Change-Id: Ibf015899c9430a25d7ae64f5bcc82a13e8f80871
Treehugger Robot [Wed, 3 Apr 2019 18:45:12 +0000 (18:45 +0000)]
Merge "Handle Bluetooth HAL service died"
Ugo Yu [Tue, 2 Apr 2019 13:55:14 +0000 (21:55 +0800)]
Handle Bluetooth HAL service died
Bug:
129320921
Test: kill Bluetooth HAL
Change-Id: I83c897f5a9def52b2050a3e73d97f99b1b71aa3b
Sunny Kapdi [Sat, 30 Mar 2019 00:42:38 +0000 (17:42 -0700)]
Fix handling of startup_timer_expired
1. The startup_timer needs to be less than
3 sec to allow the wakelock to be held by
the stack and needs to be less than the
BLE_START_TIMEOUT of framework.
2. On expiry of startup_timer, just abort
as there is no proper recovery steps here.
Test: "adb shell cat dev/ttyHS0" when Bluetooth is OFF
Turn on Bluetooth
Bug:
129394377
Change-Id: I1a397bab5c88fe7ef8ccbc369897903ee5d0a36a
Merged-In: I1a397bab5c88fe7ef8ccbc369897903ee5d0a36a
Treehugger Robot [Wed, 3 Apr 2019 03:40:38 +0000 (03:40 +0000)]
Merge "RootCanal: Add beacons by default"
Treehugger Robot [Wed, 3 Apr 2019 01:01:06 +0000 (01:01 +0000)]
Merge "packet: Add Packet classes and tests"
Myles Watson [Mon, 1 Apr 2019 20:21:25 +0000 (13:21 -0700)]
packet: Add Packet classes and tests
Test: atest bluetooth_test_gd
Change-Id: I8e124d5123cff3354f5c451b3806904a2d60b38d
weichinweng [Wed, 20 Mar 2019 10:53:11 +0000 (18:53 +0800)]
Add service change handle for Hearing Aids
When receive service changed indication from Hearing Aid (which indicates
Hearing Aid service changed), it will refresh the hearing aid attribute
handle to ensure the attribute handle is correct.
Bug:
122008481
Test: 1.run unit test
2.After Pair old version FW HearingAid, Disconnect/Reconnect new version
FW HearingAId, then check whether HearingAid is working fine.
3.After Pair old version FW HearingAid, Bluetooth off/on to reconnect
new version FW HearingAId, then check whether HearingAid is working
fine.
Change-Id: I48eae10a3016429f35f4f904752be93bb419d515
Jakub Pawlowski [Tue, 2 Apr 2019 17:21:14 +0000 (19:21 +0200)]
Fix for GATT not returning disconnect callback.
Before recent refactor GATT was using the L2CAP layer to start LE
connections. Since the refactor, GATT talks directly to added
connection_manager, just like L2CAP.
L2CAP does have it's own timer for direct connections, that was
returning conneciton failure after timeout. After switching to
connection_manager, GATT no longer receives those callbacks.
This patch makes the connection_manager send the on_connection_timed_out
callback, and wire it to same handler that L2CAP layer was using before.
Test: unit tests added
Bug:
125553095
Change-Id: Iaf8be46fe8eed49f78ddbc6a2f3516bc8279e91f
Myles Watson [Tue, 2 Apr 2019 20:57:46 +0000 (13:57 -0700)]
RootCanal: Add beacons by default
Bug:
129546852
Test: Scan for devices using RootCanal as the Bluetooth chip
Change-Id: Ia17535bffbcdde28db5120f99aad45a915f1a7ca
Treehugger Robot [Tue, 2 Apr 2019 20:07:54 +0000 (20:07 +0000)]
Merge "Common: remove GD related stuff from common"
Jack He [Mon, 1 Apr 2019 23:20:14 +0000 (16:20 -0700)]
Common: remove GD related stuff from common
* Remove reactor, thread, handler, alarm, repeating alarm from common/
since they are already copied into gd/os
* Extract thread and alarm benchmarks into gd:bluetooth_benchmark_gd
* Enable host side unit tests for system/bt in TEST_MAPPING
Bug:
129537494
Test: run benchmark, run unit tests
Change-Id: Id5d7592067bab3275976dfc710e4d751c43b9df8
Greg Kaiser [Mon, 1 Apr 2019 13:47:23 +0000 (06:47 -0700)]
Avoid using erase()d iterator
After calling erase(it), 'it' is no longer valid and should not be
used as it was in the 'it++' of the 'for' loop. We change to
update 'it' to the result of erase(), and only perform 'it++' if
we're not erasing it.
Test: TreeHugger
Change-Id: If7889fdca76a6883aeb4341be1a6fff466585e29
Greg Kaiser [Tue, 26 Mar 2019 16:09:28 +0000 (09:09 -0700)]
btif_storage: Avoid extra std::string copies
When a function takes a const std::string reference for an
argument, it's inefficient to pass in std::string::c_str().
Test: TreeHugger
Change-Id: Ibd00c10a08ab853875fd16739bce5851f6b42639
Yi Kong [Tue, 2 Apr 2019 08:04:02 +0000 (08:04 +0000)]
Merge "Fix use of uninitialised variable"
Treehugger Robot [Tue, 2 Apr 2019 04:14:33 +0000 (04:14 +0000)]
Merge "Disable Link layer socket device test in rootcanal"
Weichin Weng [Tue, 2 Apr 2019 01:48:09 +0000 (01:48 +0000)]
Merge "Revert "DO NOT MERGE Hearing Aid: Add JNI API for add/remove whitelist""
Hansong Zhang [Mon, 1 Apr 2019 23:33:52 +0000 (16:33 -0700)]
Disable Link layer socket device test in rootcanal
Test: presubmit
Bug:
129537494
Change-Id: I852da314ab65052a0e1a794f5faa9f21e1249efe
Zach Johnson [Mon, 1 Apr 2019 20:12:03 +0000 (20:12 +0000)]
Merge "Fix assert macro logical inversion"
Zach Johnson [Fri, 29 Mar 2019 19:45:50 +0000 (12:45 -0700)]
Fix assert macro logical inversion
Rename FATAL_WHEN to ASSERT and LOG_FATAL_WHEN to ASSERT_LOG
Test: atest
Change-Id: I0e6e15ce7cfbb89e76910181f164e8ff32abc033
weichinweng [Fri, 29 Mar 2019 06:07:52 +0000 (14:07 +0800)]
Revert "DO NOT MERGE Hearing Aid: Add JNI API for add/remove whitelist"
This reverts commit
d65f697c011cc6e5eaad767de3ec726cfd25b27f.
This CL is merge into AOSP in error due to the block copy from
pi-dev-plus-aosp
Test: run unit test
Change-Id: I9e206465e832d263084895b7c7b808df4573f986
Merged-In: Iab11567fa326b6837a621c1edc93e0eceb04ec43
Yi Kong [Fri, 29 Mar 2019 23:15:50 +0000 (16:15 -0700)]
Fix use of uninitialised variable
The fallthrough code path on L264 does not initialise the type variable
but it is used on L270.
Test: m checkbuild
Bug:
129566813
Change-Id: I18eb3b9be1712bac844aeb804ab76b218580c4b6
Hansong Zhang [Fri, 29 Mar 2019 15:55:25 +0000 (15:55 +0000)]
Merge "Fix potential null point dereference"
Ted Wang [Fri, 29 Mar 2019 08:20:27 +0000 (16:20 +0800)]
Fix potential null point dereference
Use find() to check element in map to avoid unknow element been insert
into map causing null point dereference.
Bug:
121310341
Test: Manual connect/disconnect headset
Change-Id: If127086ec4beb9b735ef3c72456e8e2209826b57
Zach Johnson [Fri, 29 Mar 2019 01:15:40 +0000 (01:15 +0000)]
Merge changes Ice4cee21,I6e579811,Ieae0cb21
* changes:
RootCanal: Add more scripts
RootCanal: Desktop simulation envrionment
RootCanal: New Directory Structure
Zach Johnson [Fri, 29 Mar 2019 00:37:33 +0000 (00:37 +0000)]
Merge "Get gd os tests running on host"
Myles Watson [Thu, 7 Feb 2019 23:21:25 +0000 (15:21 -0800)]
RootCanal: Add more scripts
Test: Connect to HCI sockets and LinkLayer sockets
Change-Id: Ice4cee21f295bbb3a3fe35cfe44073028e65d811
Myles Watson [Wed, 13 Feb 2019 20:36:32 +0000 (12:36 -0800)]
RootCanal: Desktop simulation envrionment
Test: nativetest64/root-canal/root-canal [TEST_PORT] [HCI_PORT] [LINK_PORT]
python scripts/test_channel.py
Change-Id: I6e57981182c392366d7d97249d837694b49dfa4e
Myles Watson [Fri, 18 Jan 2019 19:42:33 +0000 (11:42 -0800)]
RootCanal: New Directory Structure
model/controller/ contains the simulated chip
model/devices/ contains simulated devices
model/setup/ contains the test model
packets/link_layer/ contains the fake LMP packets
Test: cts-verifier Insecure RFCOMM test
LinkLayerPacketBuilderTest
rootcanal-packets_test_host
Change-Id: Ieae0cb21f7d57c03797f800797cedae59dd70e49
Zach Johnson [Thu, 28 Mar 2019 02:15:38 +0000 (19:15 -0700)]
Get gd os tests running on host
* Run os tests on host
* Fix flaky hander clear test
* Move to linux_generic
* Remove dependency on libchrome
* Allow alarms to run on linux host where we can't get CAP_WAKE_ALARM
Test: atest bluetooth_gd_test_os
Change-Id: Ia95ce39169ac8c963052e3252e7b39ef32bd5bb1
Treehugger Robot [Thu, 28 Mar 2019 21:42:33 +0000 (21:42 +0000)]
Merge "Run code coverage only on host target"
Hansong Zhang [Wed, 27 Mar 2019 22:49:13 +0000 (15:49 -0700)]
Run code coverage only on host target
Bug:
129421924
Test: run code coverage
Change-Id: I1dc681c72d8af29e43ccb1441ffd218059b8056c
Treehugger Robot [Thu, 28 Mar 2019 16:37:55 +0000 (16:37 +0000)]
Merge "BQR: Fix for the issue that Bluetooth keeps on crashing if the controller firmware does not support the BQR feature."
Zach Johnson [Thu, 28 Mar 2019 03:09:39 +0000 (03:09 +0000)]
Merge "Only use the new format in gd/"
Treehugger Robot [Wed, 27 Mar 2019 22:32:09 +0000 (22:32 +0000)]
Merge "Copy reactor classes to new directory"
Myles Watson [Wed, 27 Mar 2019 22:26:49 +0000 (15:26 -0700)]
Only use the new format in gd/
Test: Upload a change with long lines
Change-Id: Iba4a08f9556219bfada9a13272463acbf7ddbdce
Treehugger Robot [Wed, 27 Mar 2019 21:07:29 +0000 (21:07 +0000)]
Merge "A2DP: Initialize the UIPC HAL only when those binderized HALs are not enabled"
Hansong Zhang [Tue, 19 Mar 2019 23:17:42 +0000 (16:17 -0700)]
Copy reactor classes to new directory
Test: compile and run unit test
Change-Id: Ie704bf6b80f24bcae86d557ca351a893dcf8bdfa
Treehugger Robot [Wed, 27 Mar 2019 17:26:06 +0000 (17:26 +0000)]
Merge "Hearing aid: use new common/Timer"
Cheney Ni [Wed, 27 Mar 2019 12:58:49 +0000 (20:58 +0800)]
A2DP: Initialize the UIPC HAL only when those binderized HALs are not enabled
When using BluetoothA2dp / BluetoothAudio HAL, the UIPC won't be used,
and is no need to do the initialization. If it is up, there will be an
uipc-main thread running, and we have to release by the UIPC_Close API
before the A2DP source restarting. If we are acquiring a new one before
released, some of its resource will be leaked, and causes the stack
abnormal.
Bug:
128256722
Test: Check uipc-main is running when using the legacy HAL only
Change-Id: Icc8ea102d29c92b58c77099979d17e85e5cb9a83
Chienyuan [Sat, 23 Mar 2019 00:12:52 +0000 (17:12 -0700)]
Add test about post/clear task from callback for handler
Test: sudo ./bluetooth_test_common
Change-Id: Ibbf8622a705d3523638ef6cc9e6a62130b877637
Hansong Zhang [Thu, 21 Mar 2019 18:21:31 +0000 (11:21 -0700)]
Hearing aid: use new common/Timer
Previously osi/alarm cannot be cancelled or freed while in a callback.
Replace it with common/Timer. It's safe to be cancelled in a callback
and well tested.
Bug:
119533256
Test: manual
Change-Id: I73c7c98d683c85ec8f0f4256c3e4bccf3e11b56a
Treehugger Robot [Fri, 22 Mar 2019 07:46:23 +0000 (07:46 +0000)]
Merge "Fallback to the legacy HAL when IBluetoothAudioProvidersFactory is unsupported"
Ray Kuo [Thu, 21 Mar 2019 09:55:47 +0000 (17:55 +0800)]
BQR: Fix for the issue that Bluetooth keeps on crashing if the
controller firmware does not support the BQR feature.
Considering for the Treble case, it might upgrade the system image
only and the BQR feature will be enabled on a device whose Bluetooth
controller firmware does not support the BQR feature. The change is
not to crash the system if the controller firmware does not handle the
BQR VSC.
Bug:
129037162
Test: Enabled the BQR feature on the device whose Bluetooth controller
firmware does not support the BQR feature. The Bluetooth can be turned
on and works normally.
Change-Id: I6e9310a2b604679ba300e558ae4482887501e022
Treehugger Robot [Thu, 21 Mar 2019 02:03:49 +0000 (02:03 +0000)]
Merge "Refine the log messages of audio_bluetooth_hal"
Treehugger Robot [Thu, 21 Mar 2019 02:03:11 +0000 (02:03 +0000)]
Merge "Add a workaround to play A2DP SBC Mono"
Treehugger Robot [Thu, 21 Mar 2019 00:33:08 +0000 (00:33 +0000)]
Merge "Add event_value to BluetoothClassicPairingEventReported"
Elliott Hughes [Wed, 20 Mar 2019 19:41:28 +0000 (19:41 +0000)]
Merge "Use PLOG rather than LOG strerror(errno)."
Cheney Ni [Wed, 20 Mar 2019 11:53:35 +0000 (19:53 +0800)]
Refine the log messages of audio_bluetooth_hal
This CL follows the review suggestions at aosp/920718 and aosp/925233 to
refine logs.
Bug: none
Test: check logs manually
Change-Id: I3c57afaf6d416752687a7a697a310fe98c8ee495
Cheney Ni [Mon, 11 Mar 2019 12:49:11 +0000 (20:49 +0800)]
Add a workaround to play A2DP SBC Mono
There is a similar WAR of aosp/522661 at A2DP legacy HAL. In order to
suport MONO channel mode, the PCM audio is pulled as STEREO and mixed
into MONO by the Bluetooth Audio HAL.
Test: Playing SBC mono with Headset
Bug:
127593318
Change-Id: I78f3973ba6c8c733dc18122288a915daed97be65
Jack He [Wed, 20 Mar 2019 11:04:36 +0000 (04:04 -0700)]
Add event_value to BluetoothClassicPairingEventReported
* Add an event_value field to log status value such as
- encryption enabled state
- simple pairing mode
- secure connection host support
- delete all flag
* Use the event_value field to log above information during
classic pairing process instead
Bug:
128966402
Test: make
Change-Id: I2a87c3837754bfc5bcd55f72325400c428c2ca25
Cheney Ni [Thu, 14 Mar 2019 12:58:59 +0000 (20:58 +0800)]
Fallback to the legacy HAL when IBluetoothAudioProvidersFactory is unsupported
Because the stack may run without the new BluetoothAudio HAL like GSI
under old devices, it will be nullptr to getService from the
IBluetoothAudioProvidersFactory in such condition. We take nullptr as
unsupported, and fallback to the legacy HAL.
Bug:
128419724
Test: Manually running A2DP and hearing aid with / without the HAL
Change-Id: I606abc3e5b63b7857c3307c879fd4cbe46dd05d9
Stanley Tng [Mon, 25 Feb 2019 20:05:22 +0000 (12:05 -0800)]
Add Rx RSSI logs for Hearing Aids
When there is a data buffer flush (which indicates data congestion),
the received RSSI will be queried for this connection multiple times and
logged in the dumpsys.
Bug:
124331686
Test: Manual test
Change-Id: I686f4e34bda3f8067b42d6b41ca8bf316a5bf6f1
Elliott Hughes [Fri, 15 Mar 2019 03:22:17 +0000 (20:22 -0700)]
Use PLOG rather than LOG strerror(errno).
Test: builds
Change-Id: I75c3311ce00fccc79b8efc198ce69ffed96e805a
Jorge Moreira Broche [Thu, 14 Mar 2019 20:19:03 +0000 (20:19 +0000)]
Merge "Allow to disable the rootcanal test console with a property"
Treehugger Robot [Thu, 14 Mar 2019 18:08:12 +0000 (18:08 +0000)]
Merge "No need to abort the audio HAL / Bluetooth when failed to suspend"
Ugo Yu [Thu, 14 Mar 2019 07:32:44 +0000 (15:32 +0800)]
Change storage metadata type to byte array
Bug:
124448651
Test: runtest bluetooth
Change-Id: I10489c4aa93c2c3463456bcba210036bc20082a4
Merged-In: I10489c4aa93c2c3463456bcba210036bc20082a4
Treehugger Robot [Thu, 14 Mar 2019 03:51:30 +0000 (03:51 +0000)]
Merge "Move controller_properties.json to the vendor image"
Nick Kralevich [Wed, 13 Mar 2019 18:24:59 +0000 (18:24 +0000)]
Merge "compiler based array initialization"
Joseph Pirozzo [Wed, 13 Mar 2019 16:04:19 +0000 (16:04 +0000)]
Merge "Recover from HandsFree client connection collision"
Cheney Ni [Thu, 7 Mar 2019 06:15:27 +0000 (14:15 +0800)]
No need to abort the audio HAL / Bluetooth when failed to suspend
When the headset was disconnecting, the audio HAL may not be able to
suspend the stream successfully, and it is no need to abort the process
for such acceptable failure. This change also adds extra log messages
about HIDL status.
Bug:
127654107
Test: Play / pause music via the Bluetooth manually
Change-Id: I335fcf75708343c7971ebd97514fea5db50f17a0