OSDN Git Service
Pavlin Radoslavov [Thu, 3 Aug 2017 10:55:42 +0000 (10:55 +0000)]
Don't reuse AVDTP internal state for different purposes am:
8da87bea5e am:
42544cb250
am:
f5465c5a81
Change-Id: Ie075a5abcece437a40b92b8684f2a352a97e9370
Pavlin Radoslavov [Thu, 3 Aug 2017 10:55:36 +0000 (10:55 +0000)]
Add A2DP_DumpCodecInfo() API am:
219b85f136 am:
ae550b31c2
am:
f3f19bb10d
Change-Id: I1136a8cb666a00ca31747440a2235963953fce54
Pavlin Radoslavov [Thu, 3 Aug 2017 10:45:44 +0000 (10:45 +0000)]
Don't reuse AVDTP internal state for different purposes am:
8da87bea5e
am:
42544cb250
Change-Id: I5245bb4d02d11efbdd1bc06fbaf0ef865a9a9a94
Pavlin Radoslavov [Thu, 3 Aug 2017 10:45:40 +0000 (10:45 +0000)]
Add A2DP_DumpCodecInfo() API am:
219b85f136
am:
ae550b31c2
Change-Id: I711e1ceac6c0c44d34f9e5e8b0d2508a8e6af6f5
Pavlin Radoslavov [Thu, 3 Aug 2017 10:40:13 +0000 (10:40 +0000)]
Don't reuse AVDTP internal state for different purposes
am:
8da87bea5e
Change-Id: I1b35a8e6ba0f3d109731a1128c4ccb6363a9404c
Pavlin Radoslavov [Thu, 3 Aug 2017 10:40:09 +0000 (10:40 +0000)]
Add A2DP_DumpCodecInfo() API
am:
219b85f136
Change-Id: I1bc85fcf214ab75e130954245d585e3015b08bec
Pavlin Radoslavov [Wed, 2 Aug 2017 20:52:14 +0000 (13:52 -0700)]
Don't reuse AVDTP internal state for different purposes
A number of issues were exposed by carkits that proactively try
to discover the codecs supported by the Phone and explicitly select
the A2DP codec.
* Don't reuse p_scb->p_cap for storing the most recent fetched capability
and for storing the result configuration while reconfiguring the stream.
Instead, use p_scb->cfg for storing the result configuration.
* Don't try to reconfigure the audio stream before all peer's capabilities
have been retrieved.
* Don't reset p_scb->sep_info_idx after fetching all capabilities inside
bta_av_save_caps(), otherwise the stream reconfiguration logic is
confused. Apparently, p_scb->sep_info_idx is overused for both:
(a) Pointing where to store the next capability while fetching the
capabilities.
(b) Storing the selected the SEP after the codec selection is completed.
Resetting p_scb->sep_info_idx to 0 creates logical confusion when the
actual selected SEP index is 0.
* Add extra debug log messages
Test: Connecting to and streaming with various carkits
Bug:
64128712
Change-Id: I5f57e9069d5402bcbb17613470ac0d29c8e45069
(cherry picked from commit
c64dc1d28496b5e917d0f82317d40d3c60e6495d)
Pavlin Radoslavov [Wed, 2 Aug 2017 20:43:44 +0000 (13:43 -0700)]
Add A2DP_DumpCodecInfo() API
The API can be used to display A2DP codec info when using |LOG_DEBUG|.
Test: Code compilation
Bug:
64128712
Change-Id: I04379b45f24adb7008365d38ccc8aa7f49ed4b73
(cherry picked from commit
81c2234305d4c84eb38f0d5a92605e3b42366f84)
Pavlin Radoslavov [Thu, 3 Aug 2017 03:17:38 +0000 (03:17 +0000)]
Don't reuse AVDTP internal state for different purposes
am:
c64dc1d284
Change-Id: Id2640ae54ffe2da7cb5a5fc0275a1df8853d7045
Pavlin Radoslavov [Thu, 3 Aug 2017 02:47:37 +0000 (02:47 +0000)]
Add A2DP_DumpCodecInfo() API
am:
81c2234305
Change-Id: Ia7166677a094f498667a3508ce45d979830d32f4
Pavlin Radoslavov [Wed, 2 Aug 2017 20:52:14 +0000 (13:52 -0700)]
Don't reuse AVDTP internal state for different purposes
A number of issues were exposed by carkits that proactively try
to discover the codecs supported by the Phone and explicitly select
the A2DP codec.
* Don't reuse p_scb->p_cap for storing the most recent fetched capability
and for storing the result configuration while reconfiguring the stream.
Instead, use p_scb->cfg for storing the result configuration.
* Don't try to reconfigure the audio stream before all peer's capabilities
have been retrieved.
* Don't reset p_scb->sep_info_idx after fetching all capabilities inside
bta_av_save_caps(), otherwise the stream reconfiguration logic is
confused. Apparently, p_scb->sep_info_idx is overused for both:
(a) Pointing where to store the next capability while fetching the
capabilities.
(b) Storing the selected the SEP after the codec selection is completed.
Resetting p_scb->sep_info_idx to 0 creates logical confusion when the
actual selected SEP index is 0.
* Add extra debug log messages
Test: Connecting to and streaming with various carkits
Bug:
64128712
Change-Id: I5f57e9069d5402bcbb17613470ac0d29c8e45069
Pavlin Radoslavov [Wed, 2 Aug 2017 20:43:44 +0000 (13:43 -0700)]
Add A2DP_DumpCodecInfo() API
The API can be used to display A2DP codec info when using |LOG_DEBUG|.
Test: Code compilation
Bug:
64128712
Change-Id: I04379b45f24adb7008365d38ccc8aa7f49ed4b73
Jakub Pawlowski [Wed, 2 Aug 2017 22:12:20 +0000 (22:12 +0000)]
Workaround for Traxxas am:
02d1ec171f am:
9f4e534074
am:
7571a640eb
Change-Id: Ie8b9c333c21c4e7b74a432238a7e57aa6be227ba
Jakub Pawlowski [Wed, 2 Aug 2017 22:07:20 +0000 (22:07 +0000)]
Workaround for Traxxas am:
02d1ec171f
am:
9f4e534074
Change-Id: I34c64ec33c5cb09d839416f81110b3594e125bb4
Jakub Pawlowski [Wed, 2 Aug 2017 22:02:51 +0000 (22:02 +0000)]
Workaround for Traxxas
am:
02d1ec171f
Change-Id: If547b412aabc0cf86194116c9a0641e700209336
Jakub Pawlowski [Tue, 1 Aug 2017 22:12:04 +0000 (15:12 -0700)]
Workaround for Traxxas
Bug:
64252588
Test: net_test_stack_ad_parser
Change-Id: I1471e9e5f0f36ec931f8d86c927abbc1137e2b94
Sunny Kapdi [Wed, 2 Aug 2017 19:56:32 +0000 (19:56 +0000)]
Merge "AE: Set secondary adv max skip to zero" into oc-dr1-dev
am:
427ea2398a
Change-Id: I7fddeb8c364d2253c8d8916458148ee6a5dbd09f
Joseph Pirozzo [Wed, 2 Aug 2017 19:56:12 +0000 (19:56 +0000)]
Merge "AVRCP Controller request metadata on playback." am:
39b6c45264 am:
9cc7daf831
am:
d1fd1f38c9
Change-Id: Ie8331109bfb153f3ef9c98b4a72a0ffaa8807abc
Joseph Pirozzo [Wed, 2 Aug 2017 19:51:39 +0000 (19:51 +0000)]
Merge "AVRCP Controller request metadata on playback." am:
39b6c45264
am:
9cc7daf831
Change-Id: Ie860ca5d4c8b2330f77279ec0eba84496da48f54
Joseph Pirozzo [Wed, 2 Aug 2017 19:47:11 +0000 (19:47 +0000)]
Merge "AVRCP Controller request metadata on playback."
am:
39b6c45264
Change-Id: Ia5000440315c0f8a1bc75142e4463606e9fc931c
Joseph Pirozzo [Wed, 2 Aug 2017 19:33:05 +0000 (19:33 +0000)]
Merge "AVRCP Controller request metadata on playback."
TreeHugger Robot [Wed, 2 Aug 2017 19:27:48 +0000 (19:27 +0000)]
Merge "AE: Set secondary adv max skip to zero" into oc-dr1-dev
Sunny Kapdi [Wed, 2 Aug 2017 17:24:01 +0000 (17:24 +0000)]
Merge "AE: Set secondary adv max skip to zero" am:
b92d9f3530 am:
149053fe3d
am:
483293d6e7
Change-Id: Iff3604eb976a06f22f54e91d90783cf77274c013
Sunny Kapdi [Wed, 2 Aug 2017 17:19:19 +0000 (17:19 +0000)]
Merge "AE: Set secondary adv max skip to zero" am:
b92d9f3530
am:
149053fe3d
Change-Id: Ie7f516abf4a778b4e288ef5ab069b9a64fa0ca50
Sunny Kapdi [Wed, 2 Aug 2017 17:13:59 +0000 (17:13 +0000)]
Merge "AE: Set secondary adv max skip to zero"
am:
b92d9f3530
Change-Id: I54710829a6ba81d69bbee4571c00492d4a612e32
Treehugger Robot [Wed, 2 Aug 2017 17:08:20 +0000 (17:08 +0000)]
Merge "AE: Set secondary adv max skip to zero"
Sunny Kapdi [Tue, 13 Jun 2017 19:20:04 +0000 (12:20 -0700)]
AE: Set secondary adv max skip to zero
Setting secondary adv max skip to zero to guarantee
when the AUX Tx happens. This way the App can use the
primary advertising interval to dictate the AUX Tx
interval.
Bug:
62552121
Change-Id: Id66222e4f8a583c42acafb6532f81a817504e606
Joseph Pirozzo [Tue, 1 Aug 2017 21:58:03 +0000 (14:58 -0700)]
AVRCP Controller request metadata on playback.
Track metadata is only provided when requested, and the current
implementation only requested it when there was a track change
notification. Add a request for metadata whenever there is an
update to indicate that the play status is playing.
Bug:
63998350
Test: Start playing media, make a phone call, end a phonecall observe
metadata is correct.
Change-Id: Iea5cdacfe95c1a98cdaf73836ee6359a923f481b
jruthe [Wed, 2 Aug 2017 02:23:07 +0000 (02:23 +0000)]
test_vendor: Rename discoverable devices. am:
974b1f9a91 am:
6e928fcdd7
am:
05c66c947e
Change-Id: I627213c146569a20ae9635e64c6debb40dc3c0c8
jruthe [Wed, 2 Aug 2017 02:18:36 +0000 (02:18 +0000)]
test_vendor: Rename discoverable devices. am:
974b1f9a91
am:
6e928fcdd7
Change-Id: I483622c7d27b5acb12d8040199f96496cdad998f
jruthe [Wed, 2 Aug 2017 02:14:34 +0000 (02:14 +0000)]
test_vendor: Rename discoverable devices.
am:
974b1f9a91
Change-Id: I2252dd34fa376c4aa3e6da49b0f5f75f59d05837
jruthe [Tue, 1 Aug 2017 23:44:15 +0000 (16:44 -0700)]
test_vendor: Rename discoverable devices.
Test: compiles
Change-Id: I1fc1f6845995e9e816556679622db6f6d447b7b3
Sunny Kapdi [Tue, 13 Jun 2017 19:20:04 +0000 (12:20 -0700)]
AE: Set secondary adv max skip to zero
Setting secondary adv max skip to zero to guarantee
when the AUX Tx happens. This way the App can use the
primary advertising interval to dictate the AUX Tx
interval.
Bug:
62552121
Change-Id: Id66222e4f8a583c42acafb6532f81a817504e606
Satish Kodishala [Tue, 1 Aug 2017 08:04:51 +0000 (08:04 +0000)]
Merge "Use the correct AVDTP version-specific features" into oc-dr1-dev
am:
3568ed17b5
Change-Id: I224d44cf863d63bf06790eacb640849000326106
TreeHugger Robot [Tue, 1 Aug 2017 07:59:36 +0000 (07:59 +0000)]
Merge "Use the correct AVDTP version-specific features" into oc-dr1-dev
Subramanian Srinivasan [Tue, 1 Aug 2017 06:17:16 +0000 (06:17 +0000)]
Merge "Fix processing of Inquiry results data" into oc-dr1-dev
am:
60a5b95359
Change-Id: I09bea5fc3de0ed2e28c68d22cdf6c5d1d12a73a2
Subramanian Srinivasan [Tue, 1 Aug 2017 06:13:57 +0000 (06:13 +0000)]
Merge "Fix processing of Inquiry results data" am:
67da9a5e19 am:
6718f5d924
am:
39c7208fe8
Change-Id: I92c848148fd148406d8a1d8db937f7432b2bb1bd
Subramanian Srinivasan [Tue, 1 Aug 2017 06:10:14 +0000 (06:10 +0000)]
Merge "Fix processing of Inquiry results data" am:
67da9a5e19
am:
6718f5d924
Change-Id: I78efe07301f3f6747dcdae95c5e5bab6cc4801f2
TreeHugger Robot [Tue, 1 Aug 2017 06:07:23 +0000 (06:07 +0000)]
Merge "Fix processing of Inquiry results data" into oc-dr1-dev
Subramanian Srinivasan [Tue, 1 Aug 2017 06:06:15 +0000 (06:06 +0000)]
Merge "Fix processing of Inquiry results data"
am:
67da9a5e19
Change-Id: If6b7523e9096c114263baf7a065f023186818c82
Treehugger Robot [Tue, 1 Aug 2017 05:59:41 +0000 (05:59 +0000)]
Merge "Fix processing of Inquiry results data"
Satish Kodishala [Fri, 24 Feb 2017 09:39:01 +0000 (15:09 +0530)]
Use the correct AVDTP version-specific features
Failure:
A2DP is not reconnected after power cycling DUT BT.
Root cause:
AVDTP v1.3 feature request (AVDT_GetAllCapReq) sent by DUT
as remote supports the same, but this 1.3 feature is not
completely supported by DUT which leads to erroneous behavior.
Fix:
Use AVDT_GetAllCapReq request only when both host and remote
AVDTP version are 1.3 and above.
Test: 1.Pair to CarKit; 2.Connect to CarKit; 3.Do BT power cycle @ DUT
Bug:
35657623
Change-Id: I66c2d7d8fe8506b74282bcca93595e1cbf5c2565
(cherry picked from commit
9cd9ef4c3818d2c7949f52703b3c2ba0e0f43f2f)
Satish Kodishala [Tue, 1 Aug 2017 03:33:04 +0000 (03:33 +0000)]
Use the correct AVDTP version-specific features am:
9cd9ef4c38 am:
3a7e4f8a30
am:
eef3c66bac
Change-Id: I46a1051b50bac3df3ad362bca384d1560562efe9
Satish Kodishala [Tue, 1 Aug 2017 03:29:03 +0000 (03:29 +0000)]
Use the correct AVDTP version-specific features am:
9cd9ef4c38
am:
3a7e4f8a30
Change-Id: If55547328805868f3cbb84af5b96f85f6dde791c
Satish Kodishala [Tue, 1 Aug 2017 03:25:34 +0000 (03:25 +0000)]
Use the correct AVDTP version-specific features
am:
9cd9ef4c38
Change-Id: I85e816084a49a0a3438c8cc88fe2ab0ef32097f2
Tyler Freeman [Tue, 1 Aug 2017 01:59:48 +0000 (01:59 +0000)]
DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3) am:
fead72d124 -s ours am:
672ca0faf5 -s ours am:
0e379aaef1 -s ours
am:
0b9aac4eb5 -s ours
Change-Id: I1d5adc561db542384cb298e4343ab0b8a6a570ba
Tyler Freeman [Tue, 1 Aug 2017 01:53:47 +0000 (01:53 +0000)]
DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3) am:
fead72d124 -s ours am:
672ca0faf5 -s ours am:
0e379aaef1 -s ours
am:
2008a94da6 -s ours
Change-Id: I9e6ac30fa7b713d80dee19a469adc1565c7f6880
Tyler Freeman [Tue, 1 Aug 2017 01:48:21 +0000 (01:48 +0000)]
DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3) am:
fead72d124 -s ours am:
672ca0faf5 -s ours
am:
0e379aaef1 -s ours
Change-Id: I1fdf1d2c32c4ef79d7e56d2a3b34f8ecc6687bfb
Tyler Freeman [Tue, 1 Aug 2017 01:40:07 +0000 (01:40 +0000)]
DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3) am:
fead72d124 -s ours am:
672ca0faf5 -s ours
am:
0e379aaef1 -s ours
Change-Id: I7cf536bdf1ab3c56ca5e1357749ef1b939dd703a
Tyler Freeman [Tue, 1 Aug 2017 01:33:50 +0000 (01:33 +0000)]
DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3) am:
fead72d124 -s ours
am:
672ca0faf5 -s ours
Change-Id: Icf9b2a90b0adb3d9645bd6839ed9c244ba8c8c26
Tyler Freeman [Tue, 1 Aug 2017 01:24:13 +0000 (01:24 +0000)]
DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3)
am:
fead72d124 -s ours
Change-Id: I96daa5c94b8ac75e2e7faf00911274ba72dff51b
Satish Kodishala [Fri, 24 Feb 2017 09:39:01 +0000 (15:09 +0530)]
Use the correct AVDTP version-specific features
Failure:
A2DP is not reconnected after power cycling DUT BT.
Root cause:
AVDTP v1.3 feature request (AVDT_GetAllCapReq) sent by DUT
as remote supports the same, but this 1.3 feature is not
completely supported by DUT which leads to erroneous behavior.
Fix:
Use AVDT_GetAllCapReq request only when both host and remote
AVDTP version are 1.3 and above.
Test: 1.Pair to CarKit; 2.Connect to CarKit; 3.Do BT power cycle @ DUT
Bug:
35657623
Change-Id: I66c2d7d8fe8506b74282bcca93595e1cbf5c2565
Tyler Freeman [Fri, 28 Jul 2017 01:21:18 +0000 (18:21 -0700)]
DO NOT MERGE ANYWHERE Allow the Bluetooth MAC address to be updated asynchronously (3/3)
There are intermittent issues where either the returned Bluetooth
MAC address to Java framework is uninitialized or this address update
arrives too late. This fix will do 2 things:
(1) Returns error when MAC address is unavailable in the native code.
(2) Updates the MAC address later by adding a new broadcast event.
Test: Check address for these cases: factory reset, system reboot, and
Bluetooth re-enable.
Bug:
36709382
(cherry picked from commit
7dc0525365025e17e289869288b1b7d5146306cc)
Merged-In: I7b1c526ee227897c003ac921078f317f96b92604
Change-Id: I9810484a136c69d487a4b415c9265913db6c5cce
Subramanian Srinivasan [Fri, 26 May 2017 00:05:11 +0000 (17:05 -0700)]
Fix processing of Inquiry results data
Process inquiry results data based on the data
length sent from the lower stack layers and not
by using HCI_EXT_INQ_RESPONSE_LEN.
Bug:
62589724
Change-Id: Iee4a2c8f0dfb459404b28ba21061f7ced9f8455c
(cherry picked from commit
9bd6250eba5e624c94c66f053f5603dca8e0e4f1)
Subramanian Srinivasan [Fri, 26 May 2017 00:05:11 +0000 (17:05 -0700)]
Fix processing of Inquiry results data
Process inquiry results data based on the data
length sent from the lower stack layers and not
by using HCI_EXT_INQ_RESPONSE_LEN.
Bug:
62589724
Change-Id: Iee4a2c8f0dfb459404b28ba21061f7ced9f8455c
Pavlin Radoslavov [Mon, 31 Jul 2017 18:14:52 +0000 (18:14 +0000)]
Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry am:
b60955f9c7 am:
7093baab90
am:
c824624aa6
Change-Id: Ic2ec5a88b31e4f1ca5acb5e0b815d75ce1ee0aef
Pavlin Radoslavov [Mon, 31 Jul 2017 18:11:22 +0000 (18:11 +0000)]
Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry am:
b60955f9c7
am:
7093baab90
Change-Id: I0e90bc82991f31c520be480c3f14a61901fa2827
Pavlin Radoslavov [Mon, 31 Jul 2017 18:08:21 +0000 (18:08 +0000)]
Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry
am:
b60955f9c7
Change-Id: I06a99e795b0ddfd6978fca2ccca09f20de3d299f
Pavlin Radoslavov [Mon, 31 Jul 2017 18:08:20 +0000 (18:08 +0000)]
Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry
am:
3507060c58
Change-Id: I36bb63bd24c595b68976d89da8164b9ac7c1a96c
Pavlin Radoslavov [Mon, 31 Jul 2017 03:18:57 +0000 (20:18 -0700)]
Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry
Bug:
62394764
Test: Code compilation
Change-Id: I3e6d8bb8258d6cc56ce05d7f17dda445859f011a
(cherry picked from commit
b60955f9c72d8481a1a2d92d46d59e2fd31139c9)
Pavlin Radoslavov [Mon, 31 Jul 2017 03:18:57 +0000 (20:18 -0700)]
Fix the MAC address for INTEROP_2MBPS_LINK_ONLY entry
Bug:
62394764
Test: Code compilation
Change-Id: I3e6d8bb8258d6cc56ce05d7f17dda445859f011a
Pavlin Radoslavov [Fri, 28 Jul 2017 21:53:48 +0000 (21:53 +0000)]
Add interoperability mechanism to restrict the AVDTP MTU for AAC codec
am:
7cff5f55bf
Change-Id: Ifea61a89133cc420d255a675d4f80e39df05185d
Pavlin Radoslavov [Fri, 28 Jul 2017 19:38:40 +0000 (19:38 +0000)]
Add interoperability mechanism to restrict the AVDTP MTU for AAC codec am:
b866da0bc1 am:
ec7b336583
am:
a84c2057b6
Change-Id: I03015c3754f317f14fac387393f47974f0b189ce
Pavlin Radoslavov [Fri, 28 Jul 2017 19:34:08 +0000 (19:34 +0000)]
Add interoperability mechanism to restrict the AVDTP MTU for AAC codec am:
b866da0bc1
am:
ec7b336583
Change-Id: Id94f4643bec9e052d51e54950bab9ef8b062a13a
Pavlin Radoslavov [Fri, 28 Jul 2017 19:29:43 +0000 (19:29 +0000)]
Add interoperability mechanism to restrict the AVDTP MTU for AAC codec
am:
b866da0bc1
Change-Id: I79d73807f18ca5f21050f72b16c0e6f6f972f0fb
Pavlin Radoslavov [Fri, 28 Jul 2017 02:19:32 +0000 (19:19 -0700)]
Add interoperability mechanism to restrict the AVDTP MTU for AAC codec
Also, add an interoperability entry for devices that are known
to have AAC-related issues.
Test: A2DP streaming to Headsets
Bug:
63725151
Change-Id: Ic9b6640da44ebc28bfa63a19a87f0543d69bc576
(cherry picked from commit
b866da0bc1254103146912077bcb49128963dee7)
Pavlin Radoslavov [Fri, 28 Jul 2017 02:19:32 +0000 (19:19 -0700)]
Add interoperability mechanism to restrict the AVDTP MTU for AAC codec
Also, add an interoperability entry for devices that are known
to have AAC-related issues.
Test: A2DP streaming to Headsets
Bug:
63725151
Change-Id: Ic9b6640da44ebc28bfa63a19a87f0543d69bc576
Matadeen Mishra [Thu, 27 Jul 2017 02:38:36 +0000 (02:38 +0000)]
Merge "Retry authentication in key-missing or pairing collision use case" into oc-dr1-dev
am:
0b95c58b0d
Change-Id: I5639a930b06858f5f780c8842649eabc6163b820
Matadeen Mishra [Thu, 27 Jul 2017 02:36:05 +0000 (02:36 +0000)]
Retry authentication in key-missing or pairing collision use case am:
a95a6a3608 am:
27457a8b85
am:
06a13195af
Change-Id: Ie5dbfb47dc879508c1799619735972ffb457cf74
Matadeen Mishra [Thu, 27 Jul 2017 02:34:06 +0000 (02:34 +0000)]
Retry authentication in key-missing or pairing collision use case am:
a95a6a3608
am:
27457a8b85
Change-Id: I343dbedf4a3aefe61c4e37fe8d0e2f0f26b02a49
TreeHugger Robot [Thu, 27 Jul 2017 02:30:27 +0000 (02:30 +0000)]
Merge "Retry authentication in key-missing or pairing collision use case" into oc-dr1-dev
Matadeen Mishra [Thu, 27 Jul 2017 02:29:36 +0000 (02:29 +0000)]
Retry authentication in key-missing or pairing collision use case
am:
a95a6a3608
Change-Id: Iad1f7b2b5cc32fe6bb7e7237a4652ba7b2ed4e78
Stanley Tng [Thu, 27 Jul 2017 02:14:34 +0000 (02:14 +0000)]
Merge "Remove unwanted space characters from dialing number" into oc-dr1-dev
am:
332808df52
Change-Id: I2cea8eab02198b9b141e54a585032dc547320818
TreeHugger Robot [Thu, 27 Jul 2017 02:07:35 +0000 (02:07 +0000)]
Merge "Remove unwanted space characters from dialing number" into oc-dr1-dev
Matadeen Mishra [Fri, 17 Feb 2017 07:10:53 +0000 (12:40 +0530)]
Retry authentication in key-missing or pairing collision use case
Problem:
A2DP connection failing with connection timeout
Steps:
1. Pair and connect car kit from DUT
2. Disconnect the car kit from DUT
3 Unpair DUT from car kit menu and initiate pairing from car kit.
Pairing and A2DP connection should be successful.
Failure:
Paring and A2DP connections fail
Root Cause:
As the stack posting authentication failure to upper layers
when SOC gives key missing error code remote removed from
DUT paired list, but same time lower layers keep retrying
for missing key and leading to connection failure.
Fix:
Don't post authentication complete to upper layers when stack
re trying for security in key missing or transaction collision cases
Test: manual
Bug:
35448920
Change-Id: I970c8679bf27801fe46f8dd757d6435ed500f77f
(cherry picked from commit
35752e32fb5d859c012f6d06f2c22fa6b1f84174)
Matadeen Mishra [Fri, 17 Feb 2017 07:10:53 +0000 (12:40 +0530)]
Retry authentication in key-missing or pairing collision use case
Problem:
A2DP connection failing with connection timeout
Steps:
1. Pair and connect car kit from DUT
2. Disconnect the car kit from DUT
3 Unpair DUT from car kit menu and initiate pairing from car kit.
Pairing and A2DP connection should be successful.
Failure:
Paring and A2DP connections fail
Root Cause:
As the stack posting authentication failure to upper layers
when SOC gives key missing error code remote removed from
DUT paired list, but same time lower layers keep retrying
for missing key and leading to connection failure.
Fix:
Don't post authentication complete to upper layers when stack
re trying for security in key missing or transaction collision cases
Test: manual
Bug:
35448920
Change-Id: I970c8679bf27801fe46f8dd757d6435ed500f77f
(cherry picked from commit
35752e32fb5d859c012f6d06f2c22fa6b1f84174)
Pavlin Radoslavov [Wed, 26 Jul 2017 21:37:26 +0000 (21:37 +0000)]
Use a mutex to protect the consistency of the tBTA_PM_TIMER state
am:
dbe6451648
Change-Id: If64141be51bf876038283daca40ceadb274d4780
Eric Laurent [Wed, 26 Jul 2017 20:33:53 +0000 (20:33 +0000)]
Merge "Revert "bt: Update libchrome APIS to r456626."" am:
6241dd5522 am:
afc7f05f72
am:
1e541be8c5
Change-Id: Id2c114e6685eafd196fc84d9cbb7e1d43cb51efa
Eric Laurent [Wed, 26 Jul 2017 20:29:11 +0000 (20:29 +0000)]
Merge "Revert "bt: Update libchrome APIS to r456626."" am:
6241dd5522
am:
afc7f05f72
Change-Id: I2f8ceb0afa199d5faee032b4b6f03206589d5421
Eric Laurent [Wed, 26 Jul 2017 20:19:29 +0000 (20:19 +0000)]
Merge "Revert "bt: Update libchrome APIS to r456626.""
am:
6241dd5522
Change-Id: I2ab440ec5601a01717428b0ad8faf135437af971
Pavlin Radoslavov [Wed, 26 Jul 2017 20:04:49 +0000 (20:04 +0000)]
Merge "Use a mutex to protect the consistency of the tBTA_PM_TIMER state" am:
02ce3f8f52 am:
6baf3931ae
am:
6843fcc1b8
Change-Id: I9421702bbf69be463345377fc54a8bc85e3a0562
Pavlin Radoslavov [Wed, 26 Jul 2017 20:02:19 +0000 (20:02 +0000)]
Merge "Use a mutex to protect the consistency of the tBTA_PM_TIMER state" am:
02ce3f8f52
am:
6baf3931ae
Change-Id: I2d2f4cb5b997af3dcf18eb6d1792163ff0aa90d5
Eric Laurent [Wed, 26 Jul 2017 20:02:18 +0000 (20:02 +0000)]
Merge "Revert "bt: Update libchrome APIS to r456626.""
Pavlin Radoslavov [Wed, 26 Jul 2017 19:56:48 +0000 (19:56 +0000)]
Merge "Use a mutex to protect the consistency of the tBTA_PM_TIMER state"
am:
02ce3f8f52
Change-Id: I61dbfa463046af52f30385bb2593709b0f8e7730
Treehugger Robot [Wed, 26 Jul 2017 19:48:05 +0000 (19:48 +0000)]
Merge "Use a mutex to protect the consistency of the tBTA_PM_TIMER state"
Joseph Pirozzo [Wed, 26 Jul 2017 18:43:50 +0000 (18:43 +0000)]
Merge "HFP Client connect Audio" am:
d568679105 am:
d9401dfa06
am:
54dfa9bdc0
Change-Id: Iaad34e25ebfb7dacc163c19c2749bcaa24874475
Joseph Pirozzo [Wed, 26 Jul 2017 18:39:19 +0000 (18:39 +0000)]
Merge "HFP Client connect Audio" am:
d568679105
am:
d9401dfa06
Change-Id: I985b2d6e8d09e35bde57eb75f54024f8e51d438c
Joseph Pirozzo [Wed, 26 Jul 2017 18:35:19 +0000 (18:35 +0000)]
Merge "HFP Client connect Audio"
am:
d568679105
Change-Id: Ie8b2a8b3b0d54d10fb99017d6f34649004033a5c
Treehugger Robot [Wed, 26 Jul 2017 18:30:08 +0000 (18:30 +0000)]
Merge "HFP Client connect Audio"
Stanley Tng [Wed, 26 Jul 2017 18:20:57 +0000 (18:20 +0000)]
Merge "Remove unwanted space characters from dialing number" am:
ac3faef9da am:
86470eebc8
am:
56f2268904
Change-Id: I469fb5bafc2e74afa4d125efc79c95e4f1c6c99b
Pavlin Radoslavov [Wed, 26 Jul 2017 06:55:04 +0000 (23:55 -0700)]
Use a mutex to protect the consistency of the tBTA_PM_TIMER state
Bug:
63689331
Test: Code compilation
Change-Id: Id8ab78473e1f766d5a76b967024b7ff4af499667
(cherry picked from commit
f470def4e91301b7b161b88b0428476ecce4f324)
Stanley Tng [Wed, 26 Jul 2017 18:11:58 +0000 (18:11 +0000)]
Merge "Remove unwanted space characters from dialing number" am:
ac3faef9da
am:
86470eebc8
Change-Id: I46b2bc514c28a59e59e2ff2542c99a18bd8e71b5
Stanley Tng [Wed, 26 Jul 2017 18:04:58 +0000 (18:04 +0000)]
Merge "Remove unwanted space characters from dialing number"
am:
ac3faef9da
Change-Id: I1209591467468e7b5b5cc77b04c759b02b3b7b67
Stanley Tng [Tue, 25 Jul 2017 17:07:05 +0000 (10:07 -0700)]
Remove unwanted space characters from dialing number
Certain car kits might send an AT command with space characters in
the dialing number. This commit adds a workaround to trim these unwanted
space characters.
Bug:
6340999
Test: Manual test with a car kit but purposely added numbers with spaces.
Change-Id: I9c238bc625fff8503c45cbbfaeb8bdd15b8a7d9a
(cherry picked from commit
54a1461a29a8c71dd010d418ec1b23012bf32805)
Treehugger Robot [Wed, 26 Jul 2017 17:50:46 +0000 (17:50 +0000)]
Merge "Remove unwanted space characters from dialing number"
Luis Hector Chavez [Wed, 26 Jul 2017 17:36:20 +0000 (17:36 +0000)]
Revert "bt: Update libchrome APIS to r456626."
This reverts commit
014890b74878022db7d23be47db179ba06e01856.
Reason for revert: Broke the mac_sdk
Exempt-From-Owner-Approval: Fixing mac_sdk
Change-Id: I11652839faf8430a7ee6ad64af0a7bd6985478e8
Pavlin Radoslavov [Wed, 26 Jul 2017 06:55:04 +0000 (23:55 -0700)]
Use a mutex to protect the consistency of the tBTA_PM_TIMER state
Bug:
63689331
Test: Code compilation
Change-Id: Id8ab78473e1f766d5a76b967024b7ff4af499667
Jay Civelli [Wed, 26 Jul 2017 02:01:23 +0000 (02:01 +0000)]
bt: Update libchrome APIS to r456626. am:
014890b748 am:
c05cd556df
am:
2096bedc0c
Change-Id: I9868b517e218c0a3aa68fc4df6e6b8580b4a5e7a
Jay Civelli [Wed, 26 Jul 2017 01:57:26 +0000 (01:57 +0000)]
bt: Update libchrome APIS to r456626. am:
014890b748
am:
c05cd556df
Change-Id: I92cda49c91c18bf0ca080d3ea9b27cff4fbb7240
Jay Civelli [Wed, 26 Jul 2017 01:53:56 +0000 (01:53 +0000)]
bt: Update libchrome APIS to r456626.
am:
014890b748
Change-Id: Iaba9b51fa55e72df01bcdb46c10dcf46d11a6856