OSDN Git Service
Elliott Hughes [Wed, 20 Mar 2019 19:52:40 +0000 (12:52 -0700)]
Merge "Use PLOG rather than LOG strerror(errno)."
am:
fa450772a8
Change-Id: I4fa20e7921ba336ddc866e3596e1f1e602eea5d6
Elliott Hughes [Wed, 20 Mar 2019 19:41:28 +0000 (19:41 +0000)]
Merge "Use PLOG rather than LOG strerror(errno)."
Stanley Tng [Mon, 18 Mar 2019 20:34:55 +0000 (13:34 -0700)]
Add Rx RSSI logs for Hearing Aids
am:
10c80b5ce2
Change-Id: I1bbded7fff9ddf1c4d1bb637df1d86d565371d07
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:35:43 +0000 (13:35 -0700)]
Merge "Allow to disable the rootcanal test console with a property"
am:
ddd6a6f7c0
Change-Id: Id29f2fbec572e3cbab7273a8223caf7394414add
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"
Cheney Ni [Thu, 14 Mar 2019 18:24:59 +0000 (11:24 -0700)]
Merge "No need to abort the audio HAL / Bluetooth when failed to suspend"
am:
0cc83418b1
Change-Id: Ia71c87eb6fe33267eee4339d759ee96edaf3ee0a
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"
Jorge E. Moreira [Thu, 14 Mar 2019 04:05:39 +0000 (21:05 -0700)]
Merge "Move controller_properties.json to the vendor image"
am:
6f80399e14
Change-Id: I0810d6c2249370828dd442b8c953932902d01dbf
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:36:12 +0000 (11:36 -0700)]
Merge "compiler based array initialization"
am:
e9220762b0
Change-Id: I03324d9d61a934dc142153f6e64b0f7d9aca53cd
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:15:07 +0000 (09:15 -0700)]
Merge "Recover from HandsFree client connection collision"
am:
3081c2e81a
Change-Id: I9cddf7fc2a827b20f18393ff62d1cb3fd0032320
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
Cheney Ni [Wed, 13 Mar 2019 05:31:51 +0000 (22:31 -0700)]
Merge "Release encoder state when cleaning up the hearing aids instance"
am:
714a57639a
Change-Id: Iebef7301770d31cd38df3007b49aa52873aa0e29
Xin Li [Wed, 13 Mar 2019 05:21:51 +0000 (22:21 -0700)]
[automerger skipped] Merge "DO NOT MERGE - Merge PPRL.190305.001 into master"
am:
01f16ea9f7 -s ours
am skip reason: subject contains skip directive
Change-Id: I23b7ed3c6c0b6f9af14443698fe7918d58269595
The Android Open Source Project [Wed, 13 Mar 2019 04:47:15 +0000 (21:47 -0700)]
[automerger skipped] DO NOT MERGE - Merge PPRL.190305.001 into master
am:
145c825e98 -s ours
am skip reason: subject contains skip directive
Change-Id: I23b7e1711f6351d09973e25d9defd9a6fefd4032
Treehugger Robot [Wed, 13 Mar 2019 02:43:26 +0000 (02:43 +0000)]
Merge "Release encoder state when cleaning up the hearing aids instance"
Xin Li [Wed, 13 Mar 2019 02:26:03 +0000 (02:26 +0000)]
Merge "DO NOT MERGE - Merge PPRL.190305.001 into master"
Jorge E. Moreira [Wed, 13 Mar 2019 00:41:11 +0000 (17:41 -0700)]
Allow to disable the rootcanal test console with a property
Rootcanal's test console is in a tcp socket which is forbiden for HALs
by the Android security policy.
Bug:
128355308
Test: run in cuttlefish, ensure no 'avc: denied' messages show up
Change-Id: I74bf07c34166d2df399f33194cbb41e57178edc2
Jorge E. Moreira [Tue, 12 Mar 2019 22:16:42 +0000 (15:16 -0700)]
Move controller_properties.json to the vendor image
Bug:
128355308
Test: build
Change-Id: I3cb9140b34af18569a983122eef3bd642516440c
Nick Kralevich [Tue, 12 Mar 2019 20:19:02 +0000 (13:19 -0700)]
compiler based array initialization
Have the caller null out the array rather than rely on the callee
performing a memset.
Bug:
121194976
Test: compiles. No runtime tests performed.
Change-Id: I1ce3f0530080769b97be0af313e822b74fd375ce
Joseph Pirozzo [Tue, 12 Mar 2019 19:30:19 +0000 (12:30 -0700)]
Recover from HandsFree client connection collision
If a HF client fails to connect (due to collision), clean up state such
that subsequent connection attempts will pass.
Bug:
118422992
Test: Connect HF client to AG
Change-Id: I093cf3d229956c66d9e8501eb882071600321555
Cheney Ni [Fri, 8 Mar 2019 15:41:42 +0000 (23:41 +0800)]
Release encoder state when cleaning up the hearing aids instance
When the Bluetooth state changes from ON to BLE_ON, the hearing aid
instance was cleaned up, but not encoder state. Since the Bluetooth
process is kept at BLE_ON but not exited, the new instance of the
hearing aids would see the encoder as initialized without starting a new
Bluetooth audio session, and caused the audio HAL to be unable to talk
to the stack. We now reset the encoder state when cleaning up the
instance, so it will start a session next time during the first
connection of a new hearing aid instance.
Bug:
127610666
Test: ON / OFF BT with BLE_ON and switch active device manually
Change-Id: I426fed4ea22c0b858bee273727fca6e2e7481e84
Weichin Weng [Tue, 12 Mar 2019 06:29:40 +0000 (23:29 -0700)]
Merge "Hearing Aid Dump Audio Tool: Add No Start Cmd feature"
am:
812ea5b2a4
Change-Id: I4ac4ba987795a949cb455f6e7159351045b77dd7
Weichin Weng [Tue, 12 Mar 2019 06:12:19 +0000 (06:12 +0000)]
Merge "Hearing Aid Dump Audio Tool: Add No Start Cmd feature"
The Android Open Source Project [Mon, 11 Mar 2019 18:55:08 +0000 (11:55 -0700)]
DO NOT MERGE - Merge PPRL.190305.001 into master
Bug:
127812889
Change-Id: I733a17b7dd1e24a6f4a19aac2d45e312b15e0199
Xin Li [Mon, 11 Mar 2019 16:09:42 +0000 (16:09 +0000)]
Merge "DO NOT MERGE - Merge PPRL.190305.001 into stage-aosp-master" into stage-aosp-master
weichinweng [Thu, 7 Mar 2019 07:25:43 +0000 (15:25 +0800)]
Hearing Aid Dump Audio Tool: Add No Start Cmd feature
When the HCI Snoop logs wraparound, the Hearing Aid Audio Control
Command "Start" is lost. For the case, we can enable No Start Cmd
feature to set a fake "Start" to extract audio data.
Bug:
127745964
Test: ./dump_hearingaid_audio.py -c1 1 -c2 3 -ns true btsnoop_hci.log
Change-Id: Id7f67da0d5476faf38352ab57339fed8e358bac6
Ajay Panicker [Sat, 9 Mar 2019 00:29:03 +0000 (16:29 -0800)]
Merge "Use a weak pointer to deliver updates to AVRCP devices."
am:
80eeba2af3
Change-Id: Ie9d8501153264be4a755ae88956b585df3c1bdad
Treehugger Robot [Sat, 9 Mar 2019 00:18:09 +0000 (00:18 +0000)]
Merge "Use a weak pointer to deliver updates to AVRCP devices."
The Android Open Source Project [Fri, 8 Mar 2019 23:57:00 +0000 (15:57 -0800)]
DO NOT MERGE - Merge PPRL.190305.001 into stage-aosp-master
Bug:
127812889
Change-Id: I433e1bf1f64c8be7187bda2c22a2554a936e4d43
Martin Brabham [Fri, 8 Mar 2019 22:33:29 +0000 (14:33 -0800)]
Merge "osi: explicitly release wakelock during turn off"
am:
b5414b30a0
Change-Id: Ib3b886cfca8edc7faf4a3484e3fa80e14d799616
Treehugger Robot [Fri, 8 Mar 2019 22:11:05 +0000 (22:11 +0000)]
Merge "osi: explicitly release wakelock during turn off"
android-build-team Robot [Fri, 8 Mar 2019 18:50:37 +0000 (18:50 +0000)]
Snap for
5240760 from
b9dd3863033facdd8608904e4dd59ff3cc52871b to pi-platform-release
Change-Id: I8098c6f610e684be5b84a984b163088d255420b8
Ajay Panicker [Fri, 14 Dec 2018 22:55:02 +0000 (14:55 -0800)]
Use a weak pointer to deliver updates to AVRCP devices.
If a device disconnects right before a update message gets queued, the
device becomes null and there is a crash when the callback for the
update executes on the disconnected device. This patch switches the
device reference from being Unretained to using a weak pointer so that
the callback just doesn't execute if the device is disconnected.
Bug:
120431125
Test: Use the same test as b/
120477414 as that bug causes a disconnect
at the same time as a media update.
Change-Id: I1dcc08e5c9866106e7ec0dad52505e34b42da600
Martin Brabham [Tue, 26 Feb 2019 22:54:58 +0000 (14:54 -0800)]
osi: explicitly release wakelock during turn off
Ensures we release the wakelock when turning off if any module hasn't stopped timers properly
Bug:
123289721
Test: atest net_test_bluetooth
Change-Id: Ibec5c262d2ec33ce22f2c8bb2a9b838e8d6b90a7
Joseph Pirozzo [Thu, 7 Mar 2019 19:13:18 +0000 (11:13 -0800)]
AVRCP Controller Play Position Changed
am:
6f22c6e83c
Change-Id: I0aeb82a4a8fc1619626d1b546adb8803bd2c1525
Joseph Pirozzo [Thu, 7 Mar 2019 00:00:21 +0000 (16:00 -0800)]
AVRCP Controller Play Position Changed
Implement code to register for the AVRCP notification
EVENT_PLAYBACK_POS_CHANGED such that playback position gets update on
skip to beginning, and during audio scrubbing from phone.
Bug:
119119967
Test: Play music, scrub audio to another position on phone, observe
playback position is in sync.
Change-Id: Ib8459dece7629622649ba6ed340dcd697c700b09
Jakub Pawlowski [Wed, 6 Mar 2019 21:01:46 +0000 (13:01 -0800)]
Fix for connection attempt after connection cancel
am:
27870c9803
Change-Id: I53d5bce8962f5e3789f0ae41c39bc916ffe85ca7
Jakub Pawlowski [Wed, 6 Mar 2019 11:47:59 +0000 (12:47 +0100)]
Fix for connection attempt after connection cancel
When connection attempt is canceled, we put the transport control block
(p_tcb) into CLOSING state, but we never close or free it.
For LE devices that have not established the connection, it's safe to
go straight to CLOSED state and free the p_tcb.
This issue was introduced during connection manager refactor.
Bug:
127242749
Test: GattConnectTest
Change-Id: I8e45ead9b071c60e95d2e60dcea8afdbaefb92b1
Jakub Pawlowski [Wed, 6 Mar 2019 06:51:59 +0000 (22:51 -0800)]
Merge "Clear the link key when releasing security records"
am:
6c228c5b58
Change-Id: I5989cec3929a07dbbf36d112c329ffb5c2d37433
Jakub Pawlowski [Wed, 6 Mar 2019 06:40:54 +0000 (06:40 +0000)]
Merge "Clear the link key when releasing security records"
Alistair Strachan [Tue, 5 Mar 2019 23:04:15 +0000 (15:04 -0800)]
Merge "Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails"
am:
bbec743e47
Change-Id: I831f7c3f113b807e4ba73169d3ad00d2770e02b8
Treehugger Robot [Tue, 5 Mar 2019 20:57:51 +0000 (20:57 +0000)]
Merge "Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails"
Jakub Pawlowski [Tue, 5 Mar 2019 11:10:02 +0000 (12:10 +0100)]
Clear the link key when releasing security records
Test: compilation
Bug:
127479372
Change-Id: Ia571a2048fa5ca85c45b03b39d398e480aadb55a
Fukai Wang [Tue, 5 Mar 2019 14:50:23 +0000 (06:50 -0800)]
Add btm_free() to clean up btm properly
am:
91b7bb254f
Change-Id: I8e17143167c49d9b938ee7026aea94ea17a6ec79
Fukai Wang [Wed, 26 Jul 2017 01:35:21 +0000 (09:35 +0800)]
Add btm_free() to clean up btm properly
Test: Run AdapterRepeatedEnableDisable in system/bt/test with
kTestRepeatCount set to 1000.
Change-Id: I6d6650e735697bf97098827b5a9d39552c6616d8
Alistair Strachan [Sat, 2 Mar 2019 01:45:09 +0000 (17:45 -0800)]
Fall back to CLOCK_BOOTTIME if CLOCK_BOOTTIME_ALARM fails
If the cuttlefish device does not have an rtc device (such as the crosvm
VMM) the bt osi layer can promote crashes due to it not being able to
create a CLOCK_BOOTTIME_ALARM timer. Bring back a fallback but enable it
at runtime instead of compile time.
Bug:
126955943
Test: run with cuttlefish
Change-Id: I3ab0282b3e8fde776aa7b37d5772c8f62cf957bf
Greg Kaiser [Mon, 4 Mar 2019 19:33:20 +0000 (11:33 -0800)]
Merge "bta_jv_act: Avoid writing through NULL/freed pointer"
am:
bed859917b
Change-Id: I30585d79f53f3647aa626f268f31cb16e29aaf0c
Greg Kaiser [Mon, 4 Mar 2019 19:32:01 +0000 (11:32 -0800)]
Merge "bta_jv_act: Avoid potential NULL dereference"
am:
b81c6e7f04
Change-Id: I6dcb774ab24978fb56d9d1e9c502e4fccad3dfd8
Treehugger Robot [Mon, 4 Mar 2019 18:49:24 +0000 (18:49 +0000)]
Merge "bta_jv_act: Avoid writing through NULL/freed pointer"
Treehugger Robot [Mon, 4 Mar 2019 18:45:11 +0000 (18:45 +0000)]
Merge "bta_jv_act: Avoid potential NULL dereference"
Stanley Tng [Mon, 4 Mar 2019 16:39:15 +0000 (08:39 -0800)]
Merge "When stopping Hearing Aids, send Stop cmd to devices"
am:
937988bbf8
Change-Id: I08f9b717d39d3dec7589b407579ad51450ae25fe
Treehugger Robot [Mon, 4 Mar 2019 16:20:17 +0000 (16:20 +0000)]
Merge "When stopping Hearing Aids, send Stop cmd to devices"
Stanley Tng [Sat, 2 Mar 2019 00:54:11 +0000 (16:54 -0800)]
When stopping Hearing Aids, send Stop cmd to devices
This fixes the bug where the Stop command is not send to the Hearing
Aids devices which switching audio away from Hearing Aid.
Bug:
126465843
Test: Manual switching between A2DP and Hearing Aids while
playing audio.
Change-Id: Ifd8eaed7496e7e16c3948683c518d7627824d3e8
Cheney Ni [Sat, 2 Mar 2019 00:58:13 +0000 (16:58 -0800)]
Merge "Reland BluetoothAudio HAL: flip to be enabled by default"
am:
86c4e25f06
Change-Id: I2bb2b5e4665ad38da3704ec73fc26704d6454748
Treehugger Robot [Sat, 2 Mar 2019 00:37:41 +0000 (00:37 +0000)]
Merge "Reland BluetoothAudio HAL: flip to be enabled by default"
Greg Kaiser [Fri, 1 Mar 2019 13:54:56 +0000 (05:54 -0800)]
bta_jv_act: Avoid writing through NULL/freed pointer
Before writing to 't->init_called', we know that 't' might be
NULL (there's an explicit check), so we add another check here
to assure it's not NULL. Furthermore, we're possibly freeing
't' prior to writing to 't->init_called'. So we set 't' NULL
in that case, so our new NULL check will let us avoid corrupting
memory we no longer own.
Test: TreeHugger
Change-Id: Iaa246d5190f5f99610dace24707e74a846df3cf3
Greg Kaiser [Fri, 1 Mar 2019 13:34:26 +0000 (05:34 -0800)]
bta_jv_act: Avoid potential NULL dereference
When 'tc' is NULL (a condition we check for), 't' is never
set to a non-NULL value, yet is still dereferenced. We move
our NULL check of 't' to make this code safe.
Test: TreeHugger
Change-Id: Ibfea74963dd763401ea18ccdd24a51ecd0baefc5
Cheney Ni [Fri, 1 Mar 2019 02:48:43 +0000 (18:48 -0800)]
Merge "Reland Start the media encoder for a successful BTA_AV_START_EVT"
am:
a55c26ab5f
Change-Id: Iec8d2e35dfc2eae1c841069b372de81079928f05
Treehugger Robot [Fri, 1 Mar 2019 02:33:47 +0000 (02:33 +0000)]
Merge "Reland Start the media encoder for a successful BTA_AV_START_EVT"
Cheney Ni [Sat, 23 Feb 2019 18:14:20 +0000 (02:14 +0800)]
Reland BluetoothAudio HAL: flip to be enabled by default
Bug:
126304229
Test: Switch BluetoothAudio HAL and A2DP offload manully
Change-Id: I936e6c9e9fa3e35508ec435dcff8fa54b3f4d883
Merged-In: I936e6c9e9fa3e35508ec435dcff8fa54b3f4d883
Cheney Ni [Tue, 26 Feb 2019 03:56:25 +0000 (11:56 +0800)]
Reland Start the media encoder for a successful BTA_AV_START_EVT
When phone is A2DP source and receives BTA_AV_START_EVT, we need to
proceed to start the encoder of software or offload, and ack back to
audio HAL. When issue happened, the remote as the AVDTP_START initiator
triggered BTA_AV_START_EVT, and caused we did not start the encoder to
stream the audio. This change simplified the handler of BTA_AV_START_EVT
to always start the encoder for a successful event, and ack to audio HAL
if we are AVDTP initiator.
Bug:
126136429
Test: A2DP play and paused manually on Pixel 2 and Pixel 3
Change-Id: Ieb479fd6f42da1bf37f8f32af7794d86e04cac1b
Merged-In: Ieb479fd6f42da1bf37f8f32af7794d86e04cac1b
Yi Kong [Fri, 1 Mar 2019 00:59:28 +0000 (16:59 -0800)]
Merge "Fix a bunch of incorrect syntax statements"
am:
84cf37a3c4
Change-Id: Ifd68da108a3cfb58322e4bc35f6d15ba0c2f23aa
Yi Kong [Fri, 1 Mar 2019 00:30:53 +0000 (00:30 +0000)]
Merge "Fix a bunch of incorrect syntax statements"
Yi Kong [Fri, 1 Mar 2019 00:05:09 +0000 (16:05 -0800)]
Fix a bunch of incorrect syntax statements
Found by the upcoming compiler update.
Test: m checkbuild
Change-Id: Ieafcddde7c91de990aaf7bf42de58039ba5555cd
Sal Savage [Thu, 28 Feb 2019 19:33:24 +0000 (11:33 -0800)]
Merge "Audio HAL: Check if ProviderFactory is null before printing debugging"
am:
32202930cd
Change-Id: I047574f44d3194f23774c3c75839657674174217
Sal Savage [Thu, 28 Feb 2019 19:09:49 +0000 (19:09 +0000)]
Merge "Audio HAL: Check if ProviderFactory is null before printing debugging"
Sal Savage [Wed, 27 Feb 2019 20:23:07 +0000 (12:23 -0800)]
Audio HAL: Check if ProviderFactory is null before printing debugging
Test: Build and flash on auto hardware to make sure the error went away
Change-Id: I90a8f3a0e0cbc23e7e4a2edcead46402f44ef3a4
Hansong Zhang [Tue, 26 Feb 2019 23:53:42 +0000 (15:53 -0800)]
Reactor-based alarm implementation
am:
1499636274
Change-Id: I412be63f28f9f4d26b7d7bd9a58c831d597ea0f1
Hansong Zhang [Tue, 26 Feb 2019 23:53:24 +0000 (15:53 -0800)]
Reactor-based threading model
am:
7972cd5401
Change-Id: Ibf74ba5506089a6b7b2326e44db963238113f247
Hansong Zhang [Wed, 12 Dec 2018 22:52:00 +0000 (14:52 -0800)]
Reactor-based alarm implementation
* Use Reactor+Thread (common/thread.h) to replace existing
libchrome-based message_loop_thread
* Use Alarm/RepeatingAlarm to replace existing Timer, by using
kernel-based timerfd, to make all components unified
Bug:
110303473
Test: run unit test, and run benchmark
Change-Id: I6d6bc8dc3897649d0f6cc00ce0aa7054a3ddc09d
Hansong Zhang [Wed, 12 Dec 2018 22:52:00 +0000 (14:52 -0800)]
Reactor-based threading model
* Use Reactor+Thread (common/thread.h) to replace existing
libchrome-based message_loop_thread
* Use Handler to implement multiple message queue per thread, by using
kernel-based eventfd
Test: run unit test, and run benchmark
Change-Id: Idd2e4ef99fb9a7b2c0956de0e372c67a1098f1b6
Xin Li [Sat, 23 Feb 2019 00:39:20 +0000 (00:39 +0000)]
Merge "DO NOT MERGE - Merge pi-dev@
5234907 into stage-aosp-master" into stage-aosp-master
Hansong Zhang [Fri, 22 Feb 2019 06:54:07 +0000 (22:54 -0800)]
Merge "clang: Set to 120 characters per line"
am:
b228330d0e
Change-Id: I28e6c28cdc9ec2a56fe0cc4c9d1bf9f7d7c84d5e
Treehugger Robot [Fri, 22 Feb 2019 06:41:12 +0000 (06:41 +0000)]
Merge "clang: Set to 120 characters per line"
Hansong Zhang [Wed, 9 Jan 2019 03:39:22 +0000 (19:39 -0800)]
clang: Set to 120 characters per line
Modern terminals and text editors can display 120 characters
Test: compile
Change-Id: I799de73e622d045ba997e749eb15d5f786764fdd
Cheney Ni [Thu, 21 Feb 2019 19:54:46 +0000 (11:54 -0800)]
Don't require A2DP codec information when initialize audio_hal_interface
am:
8552e95c46
Change-Id: I4f64760c7f30ca670637d7dfecea7b70e015054f
Xin Li [Thu, 21 Feb 2019 19:27:44 +0000 (11:27 -0800)]
DO NOT MERGE - Merge pi-dev@
5234907 into stage-aosp-master
Bug:
120848293
Change-Id: I1ef532e784869beb11b0487c266063f903523e95
Cheney Ni [Thu, 21 Feb 2019 11:54:10 +0000 (19:54 +0800)]
Don't require A2DP codec information when initialize audio_hal_interface
Before set active device, it is normal that can't get the current codec
information, and will be setup later. If we query the codec and break
the init, it will cause the stack can't use Bluetooth Audio HAL for
A2DP, so remove here.
Bug:
125245738
Test: Connect / disconnect multiple A2DP manually when playing
Change-Id: Ic5493141cfba60115783c03fac4faf56a7a64b07
Kim Schulz [Wed, 20 Feb 2019 20:44:47 +0000 (12:44 -0800)]
Fix crash when more than 2 PHYs specified for connection
am:
d20051b26a
Change-Id: I05749d14830cf2217dab8ae8fc30660cc30b2950
Cheney Ni [Wed, 20 Feb 2019 20:28:19 +0000 (12:28 -0800)]
Merge changes I58ceea50,I73319018
am:
fcbe660320
Change-Id: Id62babcd79cd2565e3a537b838dc3814dfeb048e
Kim Schulz [Wed, 20 Feb 2019 15:33:38 +0000 (16:33 +0100)]
Fix crash when more than 2 PHYs specified for connection
Bug:
124793834
Change-Id: Ide304e22d4b390f252ba9577aeb5d9a67fa1a1cd
Treehugger Robot [Wed, 20 Feb 2019 17:30:52 +0000 (17:30 +0000)]
Merge changes I58ceea50,I73319018
* changes:
Check A2DP UIPC state before feeding data
Break the control early when the stream state of Bluetooth Audio changed
Xin Li [Wed, 20 Feb 2019 16:54:51 +0000 (08:54 -0800)]
[automerger skipped] DO NOT MERGE - Merge PPRL.190205.001 into master
am:
b5d88246be -s ours
am skip reason: subject contains skip directive
Change-Id: I7fbd5d8dca44de835df547ec2336571b3ee6f2d0
Cheney Ni [Wed, 20 Feb 2019 13:41:33 +0000 (21:41 +0800)]
Check A2DP UIPC state before feeding data
When using the new Bluetooth Audio HAL, A2DP UIPC won't be initialized
and can't be accessed. This change checks the UIPC is valid before
accessing, and prevents unexpected SIGSEGV.
Bug:
124645960
Test: Start / pause A2DP manually
Change-Id: I58ceea503d0c159817167bb3ca3b99b6f933bf23
Cheney Ni [Wed, 20 Feb 2019 06:27:06 +0000 (14:27 +0800)]
Break the control early when the stream state of Bluetooth Audio changed
When switching active device, the Bluetooth stack would first stop the
datapath while the audio HAL was still streaming, and the failure by
busy was returned to HAL. Since the control status was returned, HAL
should not ignore the result or kept waiting, because the stack wouldn't
queue the control.
Bug:
124838461
Test: switch the codec and active device manually
Change-Id: I73319018f726590c52c07547af90fac57114b58a
Xin Li [Mon, 18 Feb 2019 23:58:54 +0000 (15:58 -0800)]
DO NOT MERGE - Merge PPRL.190205.001 into master
Bug:
124234733
Change-Id: I260dc329f1d83ef8a9cab7e9caee32ba47277477
Satish kumar sugasi [Mon, 18 Feb 2019 03:12:19 +0000 (19:12 -0800)]
Merge "Add support for a2dp offload HAL ver2.0"
am:
10cad005f5
Change-Id: Ic77b64e6703e89599399c8f857647d58c29d3310
Treehugger Robot [Mon, 18 Feb 2019 03:01:45 +0000 (03:01 +0000)]
Merge "Add support for a2dp offload HAL ver2.0"
Hansong Zhang [Sat, 16 Feb 2019 04:10:36 +0000 (20:10 -0800)]
Merge "Rename a2dp_software_encoding to a2dp_encoding"
am:
ad392e481f
Change-Id: I24ca935afc634a93f62ed9b72ea23bbe244cece6
Treehugger Robot [Sat, 16 Feb 2019 04:05:34 +0000 (04:05 +0000)]
Merge "Rename a2dp_software_encoding to a2dp_encoding"
Satish kumar sugasi [Thu, 7 Feb 2019 22:40:35 +0000 (14:40 -0800)]
Add support for a2dp offload HAL ver2.0
Test: manual
Change-Id: Id5323b3c22d6aff6c194e3835eefeb22e32145c9
Hansong Zhang [Sat, 16 Feb 2019 00:19:44 +0000 (16:19 -0800)]
Rename a2dp_software_encoding to a2dp_encoding
Change-Id: Ief1c8cb1a768902a719c4956947d16081ad20add
Cheney Ni [Fri, 15 Feb 2019 19:27:27 +0000 (11:27 -0800)]
Remove unnecessary unlock
am:
9ddb2adeb6
Change-Id: Ibba705841e26a19698ce8405e3335dd375be2b31
Cheney Ni [Fri, 15 Feb 2019 15:47:15 +0000 (23:47 +0800)]
Remove unnecessary unlock
Bug:
124480689
Test: manual A2DP test
Change-Id: Id4cd4e2c97a9c4ed7219d8f1b4c15bf82b1031be
Xin Li [Thu, 14 Feb 2019 00:41:16 +0000 (16:41 -0800)]
DO NOT MERGE - Merge pi-platform-release (PPRL.190205.001) into stage-aosp-master
Bug:
124234733
Change-Id: I2ad9e0da67801687409b5e755d2436ba3c99770a