OSDN Git Service
Pavlin Radoslavov [Fri, 4 Aug 2017 22:21:40 +0000 (22:21 +0000)]
Protect the btif config state with a lock during cleanup am:
ecadb0d01d am:
6de527247e
am:
0d1cd29a83
Change-Id: I6990123f22ce83946f4303e6acf88464f869bb6f
johnshamoon [Fri, 4 Aug 2017 22:21:04 +0000 (22:21 +0000)]
test_vendor: Use Custom Iterators in L2CAP Classes am:
6f1401153f am:
590df2b3f1 am:
6acf4aacf5 am:
f3c1d04aeb
am:
1ce5874a41
Change-Id: I3e804313af99dfe03ef21271b743613085cc4040
Pavlin Radoslavov [Fri, 4 Aug 2017 22:20:31 +0000 (22:20 +0000)]
Protect the btif config state with a lock during cleanup am:
ecadb0d01d
am:
6de527247e
Change-Id: I82a101d7dced93e4e1bbaa4edd262f04eec7b000
johnshamoon [Fri, 4 Aug 2017 22:14:32 +0000 (22:14 +0000)]
test_vendor: Use Custom Iterators in L2CAP Classes am:
6f1401153f am:
590df2b3f1 am:
6acf4aacf5
am:
f3c1d04aeb
Change-Id: If96ac50ffb63500a21020a7e09fec1fd13293d76
Pavlin Radoslavov [Fri, 4 Aug 2017 22:14:30 +0000 (22:14 +0000)]
Protect the btif config state with a lock during cleanup
am:
ecadb0d01d
Change-Id: I279e7a496ec3742b19073a0610953c4a68cd37d1
johnshamoon [Fri, 4 Aug 2017 22:08:03 +0000 (22:08 +0000)]
test_vendor: Use Custom Iterators in L2CAP Classes am:
6f1401153f am:
590df2b3f1
am:
6acf4aacf5
Change-Id: Iaed989f6bd0f2c1679a568c610f9f36f1110769a
johnshamoon [Fri, 4 Aug 2017 22:02:04 +0000 (22:02 +0000)]
test_vendor: Use Custom Iterators in L2CAP Classes am:
6f1401153f
am:
590df2b3f1
Change-Id: I1949b8181ea1c3c5b4bebbb111a316f2e2e9b76d
johnshamoon [Fri, 4 Aug 2017 21:56:01 +0000 (21:56 +0000)]
test_vendor: Use Custom Iterators in L2CAP Classes
am:
6f1401153f
Change-Id: Ie5fcdbe89d35ab391fd756f675b264d1dd2233ed
johnshamoon [Wed, 26 Jul 2017 17:28:07 +0000 (10:28 -0700)]
test_vendor: Use Custom Iterators in L2CAP Classes
Use custom bounded iterators instead of vector iterators in L2capPacket and
L2capSdu.
Test: Passes unit tests in test/l2cap_test.cc
Change-Id: I892083ec6155b18c4b222f3c38d17d2d9fef686c
johnshamoon [Fri, 4 Aug 2017 20:43:35 +0000 (20:43 +0000)]
Merge "test_vendor: Add HciPacket and Iterator classes" am:
3e1a3d0ddc am:
ff5beab4fd am:
808c25199a am:
7059b2bb00
am:
40bdf9401a
Change-Id: If78cab9e89c5b2a935c6764587b2d50448520a4d
Pavlin Radoslavov [Fri, 4 Aug 2017 20:42:58 +0000 (20:42 +0000)]
Protect the btif config state with a lock during cleanup am:
a4febd6d04 am:
407754ba3a am:
762e953d59 am:
0e39a26e55
am:
bc67337c02
Change-Id: Ief46f0bbfad6fa6a4c515cb85198eab1ce51598d
johnshamoon [Fri, 4 Aug 2017 20:33:13 +0000 (20:33 +0000)]
Merge "test_vendor: Add HciPacket and Iterator classes" am:
3e1a3d0ddc am:
ff5beab4fd am:
808c25199a
am:
7059b2bb00
Change-Id: I89d03320eae1656d194ecd3caea53ff9a70ed0f9
Pavlin Radoslavov [Fri, 4 Aug 2017 20:32:55 +0000 (20:32 +0000)]
Protect the btif config state with a lock during cleanup am:
a4febd6d04 am:
407754ba3a am:
762e953d59
am:
0e39a26e55
Change-Id: I86a584cb73f50c39c23753864958d4f41c2bdea2
johnshamoon [Fri, 4 Aug 2017 20:27:23 +0000 (20:27 +0000)]
Merge "test_vendor: Add HciPacket and Iterator classes" am:
3e1a3d0ddc am:
ff5beab4fd
am:
808c25199a
Change-Id: I639d24fb6a6b68f6280191495ca247ad7c978700
Pavlin Radoslavov [Fri, 4 Aug 2017 20:26:55 +0000 (20:26 +0000)]
Protect the btif config state with a lock during cleanup am:
a4febd6d04 am:
407754ba3a
am:
762e953d59
Change-Id: Icaa34698cb98f593e55519efd2072bf11f06c951
johnshamoon [Fri, 4 Aug 2017 20:20:39 +0000 (20:20 +0000)]
Merge "test_vendor: Add HciPacket and Iterator classes" am:
3e1a3d0ddc
am:
ff5beab4fd
Change-Id: Ie44e518117764d92cb67d23a0b3d1a059e708540
Pavlin Radoslavov [Fri, 4 Aug 2017 20:20:24 +0000 (20:20 +0000)]
Protect the btif config state with a lock during cleanup am:
a4febd6d04
am:
407754ba3a
Change-Id: I8cb723ca43c3917faa690c191078e0575aa21625
johnshamoon [Fri, 4 Aug 2017 20:14:51 +0000 (20:14 +0000)]
Merge "test_vendor: Add HciPacket and Iterator classes"
am:
3e1a3d0ddc
Change-Id: Ide1490901c49ae9e38e69ba6315c69df92c7bb27
Pavlin Radoslavov [Fri, 4 Aug 2017 20:14:24 +0000 (20:14 +0000)]
Protect the btif config state with a lock during cleanup
am:
a4febd6d04
Change-Id: I47a3b5efc680954afc11e58f16c440284ba016a9
Treehugger Robot [Fri, 4 Aug 2017 20:13:17 +0000 (20:13 +0000)]
Merge "test_vendor: Add HciPacket and Iterator classes"
Pavlin Radoslavov [Fri, 4 Aug 2017 18:26:21 +0000 (11:26 -0700)]
Protect the btif config state with a lock during cleanup
Bug:
64186300
Test: Code compilation, enable/disable Bluetooth
Change-Id: Ic2397a128d59c9cc69cbd8252734b69eb7c02501
(cherry picked from commit
a4febd6d0460ee0f5834822ca406a4b8bab28cf1)
Pavlin Radoslavov [Fri, 4 Aug 2017 18:26:21 +0000 (11:26 -0700)]
Protect the btif config state with a lock during cleanup
Bug:
64186300
Test: Code compilation, enable/disable Bluetooth
Change-Id: Ic2397a128d59c9cc69cbd8252734b69eb7c02501
Jakub Pawlowski [Fri, 4 Aug 2017 18:24:28 +0000 (18:24 +0000)]
Advertisement parsing fix for zero padding am:
35a9ea352d am:
b3bf1285f4 am:
04e74ae1f5 am:
504f4bf2d1
am:
4473fde113
Change-Id: Iab9bf77497b79187db8d369f99fde2688ac9f848
Jakub Pawlowski [Fri, 4 Aug 2017 18:13:15 +0000 (18:13 +0000)]
Advertisement parsing fix for zero padding am:
35a9ea352d am:
b3bf1285f4 am:
04e74ae1f5
am:
504f4bf2d1
Change-Id: I5262e0777c6924b8ad84bab603e0c168a8216bbf
Jakub Pawlowski [Fri, 4 Aug 2017 18:07:14 +0000 (18:07 +0000)]
Advertisement parsing fix for zero padding am:
35a9ea352d am:
b3bf1285f4
am:
04e74ae1f5
Change-Id: Ic99b3dbc6be1463ddfa618564f665c24f1dd188a
Jakub Pawlowski [Fri, 4 Aug 2017 18:01:14 +0000 (18:01 +0000)]
Advertisement parsing fix for zero padding am:
35a9ea352d
am:
b3bf1285f4
Change-Id: Ica2250b89a1f72bf15bf16d1eaf159fe71049c44
Jakub Pawlowski [Fri, 4 Aug 2017 17:55:13 +0000 (17:55 +0000)]
Advertisement parsing fix for zero padding
am:
35a9ea352d
Change-Id: I59bf8524b2c95c5f8c7f4261166057078dd25b7c
Jakub Pawlowski [Fri, 4 Aug 2017 15:56:53 +0000 (08:56 -0700)]
Advertisement parsing fix for zero padding
When AD data is zero padded, and Scan Response is appended at end, data
becomes invalid. Instead, zero paddning must be removed first.
Test: AdvertiseDataParserTest.RemoveTrailingZeros
Bug:
38489707
Change-Id: I229ca3db6c92bc06bc1429e72412417010721063
Pavlin Radoslavov [Fri, 4 Aug 2017 06:26:32 +0000 (06:26 +0000)]
Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database am:
427da91563 am:
b7471f0173 am:
e998b84e8c am:
7919765c57
am:
52b49e4fe0
Change-Id: I1803bd74661430b2b267c2214efa9d01b077f743
Pavlin Radoslavov [Fri, 4 Aug 2017 06:20:31 +0000 (06:20 +0000)]
Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database am:
427da91563 am:
b7471f0173 am:
e998b84e8c
am:
7919765c57
Change-Id: I43703678f2c9c3dd4306345c6045da2daac107ee
Pavlin Radoslavov [Fri, 4 Aug 2017 06:14:32 +0000 (06:14 +0000)]
Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database am:
427da91563 am:
b7471f0173
am:
e998b84e8c
Change-Id: Ie246041d26910c9547402836e20a845a1fb0b4c7
Pavlin Radoslavov [Fri, 4 Aug 2017 06:08:00 +0000 (06:08 +0000)]
Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database am:
427da91563
am:
b7471f0173
Change-Id: I953492a7be219d4b15f9b7ae581af3bceb5f47ef
Pavlin Radoslavov [Fri, 4 Aug 2017 06:02:00 +0000 (06:02 +0000)]
Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database
am:
427da91563
Change-Id: I7412713b0c0104309b8e5fcffa7df2fde1eaa7f2
Pavlin Radoslavov [Fri, 4 Aug 2017 04:13:38 +0000 (04:13 +0000)]
Merge "Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev am:
1dc69f4180 am:
f46e978d11" into oc-mr1-dev-plus-aosp
am:
20d915000a
Change-Id: Icd852fd14d45281ddf1f929a9ba7eb479e38acba
Pavlin Radoslavov [Fri, 4 Aug 2017 04:13:20 +0000 (04:13 +0000)]
Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev am:
1dc69f4180 am:
e952831d48
am:
c2c245f9eb
Change-Id: Ia2abed5ac1ad424d45ef607f1f71158661b10c74
Android Build Merger (Role) [Fri, 4 Aug 2017 04:07:34 +0000 (04:07 +0000)]
Merge "Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev am:
1dc69f4180 am:
f46e978d11" into oc-mr1-dev-plus-aosp
Pavlin Radoslavov [Fri, 4 Aug 2017 04:07:19 +0000 (04:07 +0000)]
Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev am:
1dc69f4180
am:
e952831d48
Change-Id: I85c870f574f0ed482d6071bd0ac8333cfaf70ba0
Pavlin Radoslavov [Fri, 4 Aug 2017 04:07:19 +0000 (04:07 +0000)]
Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev am:
1dc69f4180
am:
f46e978d11
Change-Id: I6ecf94364c1ce6cc6cf5379222b79b9998da0664
Pavlin Radoslavov [Fri, 4 Aug 2017 04:01:20 +0000 (04:01 +0000)]
Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev
am:
1dc69f4180
Change-Id: I17f6180ebfb596d74ec6c76b6ec3573c37a9ffdc
Pavlin Radoslavov [Fri, 4 Aug 2017 04:01:19 +0000 (04:01 +0000)]
Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev
am:
1dc69f4180
Change-Id: I9704aeb9017d0cae455470879c404d05b4199b4b
Pavlin Radoslavov [Fri, 4 Aug 2017 03:50:45 +0000 (03:50 +0000)]
Merge "Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database" into oc-dr1-dev
Pavlin Radoslavov [Fri, 4 Aug 2017 00:34:07 +0000 (17:34 -0700)]
Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database
Bug:
64357126
Test: Audio streaming to the affected carkit
Change-Id: I160eb3775c77fe400798f038d2aae7dd7825aacb
(cherry picked from commit
a5f0a7717d129ca8164a0712e63f41cc4b60dd72)
Pavlin Radoslavov [Fri, 4 Aug 2017 00:34:07 +0000 (17:34 -0700)]
Added another entry to the INTEROP_2MBPS_LINK_ONLY interoperability database
Bug:
64357126
Test: Audio streaming to the affected carkit
Change-Id: I160eb3775c77fe400798f038d2aae7dd7825aacb
Pankaj Kanwar [Fri, 4 Aug 2017 00:18:02 +0000 (00:18 +0000)]
Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev am:
233a2b47b2 am:
e214c9bbf3
am:
300284af57
Change-Id: Ic46bffad955a2662806e60c54d40c7d4dca2e013
Pankaj Kanwar [Fri, 4 Aug 2017 00:09:00 +0000 (00:09 +0000)]
Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev am:
233a2b47b2 am:
575b50b936
am:
9d80fdb3e9
Change-Id: Ib61c7e26072c853f9b560b6200afacaf5d53a03d
Joseph Pirozzo [Fri, 4 Aug 2017 00:08:40 +0000 (00:08 +0000)]
Merge "AVRCP Controller request metadata on playback." into oc-mr1-dev am:
a685868941 -s ours
am:
8d91165cbc -s ours
Change-Id: I2f3e14dc20a8b1741c77f4447f00ed833fb67536
Pankaj Kanwar [Fri, 4 Aug 2017 00:06:38 +0000 (00:06 +0000)]
Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev am:
233a2b47b2
am:
e214c9bbf3
Change-Id: I4a301e090f00d8154dc9b18d6dc0e0d221f2ac6f
Pankaj Kanwar [Fri, 4 Aug 2017 00:04:04 +0000 (00:04 +0000)]
Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev am:
233a2b47b2
am:
575b50b936
Change-Id: Ib37340caf7a5e7c7ed9cea7027a615976a60bec7
Joseph Pirozzo [Fri, 4 Aug 2017 00:01:24 +0000 (00:01 +0000)]
Merge "AVRCP Controller request metadata on playback." into oc-mr1-dev
am:
a685868941 -s ours
Change-Id: I977749419c440c61070bcfedcae64e1b530fe8d3
Pankaj Kanwar [Fri, 4 Aug 2017 00:00:43 +0000 (00:00 +0000)]
Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev
am:
233a2b47b2
Change-Id: Iad6f3b180335b51fb24935aaa87243fc597f64ed
Pankaj Kanwar [Fri, 4 Aug 2017 00:00:42 +0000 (00:00 +0000)]
Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev
am:
233a2b47b2
Change-Id: If4d5777b4835b0a7d5e26833debdd666075f5950
Pankaj Kanwar [Thu, 3 Aug 2017 23:52:09 +0000 (23:52 +0000)]
Merge "Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"" into oc-dr1-dev
Joseph Pirozzo [Thu, 3 Aug 2017 23:46:53 +0000 (23:46 +0000)]
Merge "AVRCP Controller request metadata on playback." into oc-mr1-dev
johnshamoon [Wed, 19 Jul 2017 19:06:54 +0000 (12:06 -0700)]
test_vendor: Add HciPacket and Iterator classes
HciPacket is the abstract base class for all packet types.
Iterator is the custom iterator class that all packet types will use
to iterate through. The iterators in this class are bounded from
[start_of_packet, length_of_packet]. This prevents incrementing the iterator
passed the end sentinel.
Test: Passes all of the tests in test/iterator_test.cc
Change-Id: If9fec9ae32d9a23575ad3e9219b8249d164bc0d5
Pankaj Kanwar [Thu, 3 Aug 2017 20:41:44 +0000 (20:41 +0000)]
Revert "Allow collection of Bluetooth firmware dumps in bugreports (3/3)"
This reverts commit
0580dd639cb10fafb6953d8a724db030df5030db.
Change-Id: I37f9ba5a9ace655d1d12404926a158fb9a4001a7
Jakub Pawlowski [Thu, 3 Aug 2017 18:49:29 +0000 (18:49 +0000)]
Workaround for Traxxas am:
c6e8384203 -s ours
am:
40eafe1ff3 -s ours
Change-Id: I275a96816cf8170d2f209a1505756796aec2557b
Jakub Pawlowski [Thu, 3 Aug 2017 18:41:10 +0000 (18:41 +0000)]
Workaround for Traxxas
am:
c6e8384203 -s ours
Change-Id: I90f9e9aad65e4c309e0fc916b51f28dea45fcbef
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
Merged-In: I1471e9e5f0f36ec931f8d86c927abbc1137e2b94
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
Merged-In: Iae6969ff230d91335b6af41955f07f1151d79de2
(cherry picked from commit
7e08d96dc3b58e4d796c901b92d35185da005548)
Pavlin Radoslavov [Thu, 3 Aug 2017 11:14:46 +0000 (11:14 +0000)]
Don't reuse AVDTP internal state for different purposes am:
8da87bea5e am:
42544cb250 am:
f5465c5a81 am:
441f8f8ec1
am:
e96a53d23f
Change-Id: I6f1a0cda8407f8253392fc1dcbb503a48b453061
Pavlin Radoslavov [Thu, 3 Aug 2017 11:14:41 +0000 (11:14 +0000)]
Add A2DP_DumpCodecInfo() API am:
219b85f136 am:
ae550b31c2 am:
f3f19bb10d am:
2e9afaa3b5
am:
906be65f81
Change-Id: I0f7aa42f1e2c3a1f4b0716a440e225e44168629a
Pavlin Radoslavov [Thu, 3 Aug 2017 11:07:33 +0000 (11:07 +0000)]
Don't reuse AVDTP internal state for different purposes am:
8da87bea5e am:
42544cb250 am:
f5465c5a81
am:
441f8f8ec1
Change-Id: Ie02b8ff2f469cb7f723feb31710f7acbd701d15b
Pavlin Radoslavov [Thu, 3 Aug 2017 11:07:27 +0000 (11:07 +0000)]
Add A2DP_DumpCodecInfo() API am:
219b85f136 am:
ae550b31c2 am:
f3f19bb10d
am:
2e9afaa3b5
Change-Id: I8c4c7dc88b8bbc39dd7639e8dff7353b3560660b
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:33:42 +0000 (03:33 +0000)]
Don't reuse AVDTP internal state for different purposes am:
c64dc1d284 am:
2456fdcb2b
am:
f06f51e314
Change-Id: I50717826a00a4e5ad9eeee66df3b4888af60d211
Pavlin Radoslavov [Thu, 3 Aug 2017 03:33:29 +0000 (03:33 +0000)]
Don't reuse AVDTP internal state for different purposes am:
c64dc1d284 am:
318943e5ca
am:
d561d343c4
Change-Id: Ic80d7e3e93172d973cad25475e82f513a04d2c62
Pavlin Radoslavov [Thu, 3 Aug 2017 03:24:14 +0000 (03:24 +0000)]
Don't reuse AVDTP internal state for different purposes am:
c64dc1d284
am:
2456fdcb2b
Change-Id: I0c492cb36cbbc9dbee665440ec03c30fa2edb51b
Pavlin Radoslavov [Thu, 3 Aug 2017 03:23:39 +0000 (03:23 +0000)]
Don't reuse AVDTP internal state for different purposes am:
c64dc1d284
am:
318943e5ca
Change-Id: Ib40c880b43c4c11f8fa277c5a2b29b4839e06b25
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: Ia42b343751f2fa48a0b8f601256828d3c15718b2
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:57:23 +0000 (02:57 +0000)]
Merge "Add A2DP_DumpCodecInfo() API am:
81c2234305 am:
98f5272ea0" into oc-mr1-dev-plus-aosp
am:
1cff4e27f4
Change-Id: Ic7fe2a3566441ccbb086422173c74277756959d6
Pavlin Radoslavov [Thu, 3 Aug 2017 02:57:09 +0000 (02:57 +0000)]
Add A2DP_DumpCodecInfo() API am:
81c2234305 am:
6fc20ca428
am:
6fe9f51724
Change-Id: Ia7c153633cce4ecc0257f1e08855bffd17e74ddb
Android Build Merger (Role) [Thu, 3 Aug 2017 02:52:20 +0000 (02:52 +0000)]
Merge "Add A2DP_DumpCodecInfo() API am:
81c2234305 am:
98f5272ea0" into oc-mr1-dev-plus-aosp
Pavlin Radoslavov [Thu, 3 Aug 2017 02:52:08 +0000 (02:52 +0000)]
Add A2DP_DumpCodecInfo() API am:
81c2234305
am:
6fc20ca428
Change-Id: I3a03be1d40f279b22b954063e6de4110d8bc5681
Pavlin Radoslavov [Thu, 3 Aug 2017 02:52:08 +0000 (02:52 +0000)]
Add A2DP_DumpCodecInfo() API am:
81c2234305
am:
98f5272ea0
Change-Id: If6de2527d408b0fda383e52bef85b82f743b3bf1
Pavlin Radoslavov [Thu, 3 Aug 2017 02:47:37 +0000 (02:47 +0000)]
Add A2DP_DumpCodecInfo() API
am:
81c2234305
Change-Id: I9775e3cf2ca822cb9b8386f8d4a63f65b20b5952
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:22:43 +0000 (22:22 +0000)]
Workaround for Traxxas am:
02d1ec171f am:
9f4e534074 am:
7571a640eb am:
112f439ab9
am:
15426a21a6
Change-Id: I04e0b4512c60f00d84ce380b27cbab2ed8313c2d
Jakub Pawlowski [Wed, 2 Aug 2017 22:17:41 +0000 (22:17 +0000)]
Workaround for Traxxas am:
02d1ec171f am:
9f4e534074 am:
7571a640eb
am:
112f439ab9
Change-Id: I5445c02f580a82489c12c5737aaa092b48f017b8
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 20:03:14 +0000 (20:03 +0000)]
Merge "AE: Set secondary adv max skip to zero" into oc-dr1-dev am:
427ea2398a am:
ac903f82ff
am:
308e13ea84
Change-Id: I43393feac3254d4eb1005bad6b5f63aadcdb9209
Joseph Pirozzo [Wed, 2 Aug 2017 20:03:00 +0000 (20:03 +0000)]
Merge "AVRCP Controller request metadata on playback." am:
39b6c45264 am:
9cc7daf831 am:
d1fd1f38c9 am:
464e16a369
am:
93d956f76a
Change-Id: I254a43d6e92190e750f2b17beb52729c1ff29638
Sunny Kapdi [Wed, 2 Aug 2017 20:02:43 +0000 (20:02 +0000)]
Merge "AE: Set secondary adv max skip to zero" into oc-dr1-dev am:
427ea2398a am:
1881614fc9
am:
00e9b1d20a
Change-Id: I3fdbcff21ae002c64a653b505a0264cd100d5f66
Sunny Kapdi [Wed, 2 Aug 2017 20:00:55 +0000 (20:00 +0000)]
Merge "AE: Set secondary adv max skip to zero" into oc-dr1-dev am:
427ea2398a
am:
ac903f82ff
Change-Id: Ia06487d9dadb5e192db99887e856d02a091e41e2
Joseph Pirozzo [Wed, 2 Aug 2017 20:00:41 +0000 (20:00 +0000)]
Merge "AVRCP Controller request metadata on playback." am:
39b6c45264 am:
9cc7daf831 am:
d1fd1f38c9
am:
464e16a369
Change-Id: Iae6969ff230d91335b6af41955f07f1151d79de2
Sunny Kapdi [Wed, 2 Aug 2017 19:56:41 +0000 (19:56 +0000)]
Merge "AE: Set secondary adv max skip to zero" into oc-dr1-dev am:
427ea2398a
am:
1881614fc9
Change-Id: I9c98f53d7ddd40c2af6a19a0e6bd5c308c2e9772
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