OSDN Git Service
Andre Eisenbach [Tue, 16 Aug 2016 05:50:50 +0000 (05:50 +0000)]
Remove SBC frame scrambling am:
7fcea70080
am:
3753c01648
Change-Id: I2c45a530f9244c87598cd4c2c7e0d716bdf551a5
Andre Eisenbach [Tue, 16 Aug 2016 05:46:52 +0000 (05:46 +0000)]
Remove SBC frame scrambling
am:
7fcea70080
Change-Id: If59587ff211e5ce378e4258aabcb7a32774553a9
Andre Eisenbach [Thu, 23 Jun 2016 22:25:44 +0000 (15:25 -0700)]
Remove SBC frame scrambling
Bug:
29601962
Change-Id: I30ce09f1c7550a1be9fd1c8ed70745f9aa0cdb44
(cherry picked from commit
470aee234664fd75094c16dd2c70313ee28354c1)
Jacky Cheung [Tue, 16 Aug 2016 03:44:13 +0000 (03:44 +0000)]
Merge "New status code for two SMP authorization failures" am:
b29732059e
am:
734ba5b0a8
Change-Id: I9c545f839aff1ca19cb92db4181ea01e35dbb7ac
Andre Eisenbach [Tue, 16 Aug 2016 03:44:13 +0000 (03:44 +0000)]
HID: Check for EINTR in uhid_event() and fix return value evaluation am:
acb093ada7
am:
a319edc546
Change-Id: I28af2d6aa90f187136de6ec5e9ce2c7762da5fb7
Jacky Cheung [Tue, 16 Aug 2016 03:40:18 +0000 (03:40 +0000)]
Merge "New status code for two SMP authorization failures"
am:
b29732059e
Change-Id: I8d05dde66952f263ebfcc81cbc38b0f27675efd7
Andre Eisenbach [Tue, 16 Aug 2016 03:40:15 +0000 (03:40 +0000)]
HID: Check for EINTR in uhid_event() and fix return value evaluation
am:
acb093ada7
Change-Id: I4c7269a3aee069ca725eec0faa885100171e198d
Treehugger Robot [Tue, 16 Aug 2016 03:37:09 +0000 (03:37 +0000)]
Merge "New status code for two SMP authorization failures"
Jacky Cheung [Wed, 6 Jul 2016 22:11:30 +0000 (15:11 -0700)]
New status code for two SMP authorization failures
On BTA_DM_AUTH_SMP_UNKNOWN_ERR and BTA_DM_AUTH_SMP_CONN_TOUT, bond state
status will be notified as BT_STATUS_AUTH_FAILURE. This is observed during
the pairing flow with iOS devices.
Plus minor cleanup on SMP errors and fail reasons.
Bug:
29779689
Change-Id: Ia32fe6bb84b07272d5159409f65217a06a2e5f8e
(cherry picked from commit
6984b19d95736d7205fc6302a21e8be564617287)
Pavlin Radoslavov [Tue, 16 Aug 2016 01:31:05 +0000 (01:31 +0000)]
Removed bogus leftover code that wasn't auto-merged properly
am:
3c48d11354
Change-Id: I01b99c8b2968ab03055bd0f1d379b558f26d2b42
Pavlin Radoslavov [Tue, 16 Aug 2016 00:43:30 +0000 (17:43 -0700)]
Removed bogus leftover code that wasn't auto-merged properly
Change-Id: I41c2869ae32f08bc170cc222f036e0bc207b81f9
Jorge E. Moreira [Mon, 15 Aug 2016 21:26:22 +0000 (21:26 +0000)]
Refactor vendor library to use async manager class am:
0b32c8d8c1
am:
db7cab56f9
Change-Id: I6d23d7a8c5d6f4c82574817e526f31a9eec08f88
Jorge E. Moreira [Mon, 15 Aug 2016 21:26:20 +0000 (21:26 +0000)]
Added the async manager am:
08c214154d
am:
d87ecf6aed
Change-Id: Ia1d7c4774a4433ae8975ebc62eafbfb72442b6ec
Jorge E. Moreira [Mon, 15 Aug 2016 21:21:25 +0000 (21:21 +0000)]
Refactor vendor library to use async manager class
am:
0b32c8d8c1
Change-Id: Ifeb58d21c5343f7fe42b6bce87cb03b2063003ee
Jorge E. Moreira [Mon, 15 Aug 2016 21:21:23 +0000 (21:21 +0000)]
Added the async manager
am:
08c214154d
Change-Id: I583d351023846be861018cd67f5410a50d98a12a
Jorge E. Moreira [Mon, 15 Aug 2016 21:17:32 +0000 (21:17 +0000)]
Added missing #include directives am:
65ee728284
am:
9d427f118f
Change-Id: Icd3c8d63b3ce5ef2f9cba23845f391ecbce1e194
Andre Eisenbach [Mon, 30 May 2016 19:00:46 +0000 (12:00 -0700)]
HID: Check for EINTR in uhid_event() and fix return value evaluation
uhid_event() returns 0 in the normal case. Commit
3a2c2d61
results in the polling thread to be exited.
Bug:
28942565
Change-Id: Id0aff2958bc7c2704ba340aa0ff7848afb8dcf80
(cherry picked from commit
32d66694e9f782a0e9dfa423015284a573f8975c)
Jorge E. Moreira [Mon, 8 Aug 2016 18:39:52 +0000 (11:39 -0700)]
Refactor vendor library to use async manager class
Homogenizes the use of lambdas/binds (uses c++ lambdas only)
Uses STL's time library implementation (std::chrono)
Change-Id: I6194b26c0a7fabffddf96acb5c2379ff52026a84
Jorge E. Moreira [Mon, 15 Aug 2016 21:13:38 +0000 (21:13 +0000)]
Added missing #include directives
am:
65ee728284
Change-Id: I5b293b7ff4665312efdf1bcc00a6f2f5058d2670
Jorge E. Moreira [Mon, 8 Aug 2016 18:39:52 +0000 (11:39 -0700)]
Added the async manager
Change-Id: I968fee42fd610e33e13bf358f54bec1fe7af4a7d
Jorge E. Moreira [Tue, 9 Aug 2016 22:21:01 +0000 (15:21 -0700)]
Added missing #include directives
Change-Id: Ia26cd6775699c65a75d94003c09730926835f7a3
Jorge E. Moreira [Mon, 15 Aug 2016 20:56:18 +0000 (20:56 +0000)]
Replace calls to std::bind by lambda expressions am:
97980c25a1
am:
20641629e6
Change-Id: I925b08e456318959c95a10d3c292f2f6fd9079bc
Jorge E. Moreira [Mon, 15 Aug 2016 20:51:50 +0000 (20:51 +0000)]
Replace calls to std::bind by lambda expressions
am:
97980c25a1
Change-Id: Iad98f599d6bd802c42b583c6bb5d8e75da568839
Jorge E. Moreira [Mon, 15 Aug 2016 19:21:42 +0000 (19:21 +0000)]
No longer uses the macro DISALLOW_COPY_AND_ASSIGN am:
83cf95e0a1
am:
ece79cdeac
Change-Id: Id6e20889627f056a624c02bad3b121aab227d92b
Jorge E. Moreira [Mon, 15 Aug 2016 19:21:41 +0000 (19:21 +0000)]
Using include paths relative to vendor library dir am:
ea3ede2474
am:
c7dd1eae5e
Change-Id: I28b389ee7eb887cd328b59c28a6ddfcd8dc6937c
Jorge E. Moreira [Mon, 15 Aug 2016 19:17:44 +0000 (19:17 +0000)]
No longer uses the macro DISALLOW_COPY_AND_ASSIGN
am:
83cf95e0a1
Change-Id: I6bd4c3d3bb9b12ce8a167769ce4be5dbefebe10a
Jorge E. Moreira [Mon, 15 Aug 2016 19:17:41 +0000 (19:17 +0000)]
Using include paths relative to vendor library dir
am:
ea3ede2474
Change-Id: Ic703fda9a8c7d72f6fb563069d1948f190ac7924
Jorge E. Moreira [Fri, 5 Aug 2016 18:07:07 +0000 (11:07 -0700)]
Replace calls to std::bind by lambda expressions
Change-Id: I5fbe6ae6d5172e87428322084276e593a67fd7d1
Jorge E. Moreira [Thu, 4 Aug 2016 23:55:31 +0000 (16:55 -0700)]
No longer uses the macro DISALLOW_COPY_AND_ASSIGN
Change-Id: I3403cdfa38fabf56ffe0854064eeb0ae38eb5a4a
Jorge E. Moreira [Thu, 4 Aug 2016 23:46:45 +0000 (16:46 -0700)]
Using include paths relative to vendor library dir
Change-Id: I6b5dfa4098cdf2eda21d3fae60729b765532c2f9
Andy Hung [Mon, 15 Aug 2016 18:24:38 +0000 (18:24 +0000)]
Merge "Improve BT HAL write response to BT server reads" am:
f65e83e5b1
am:
268069aab8
Change-Id: I1b592d85515b06b2a4c7534bb54609df9da981da
Andy Hung [Mon, 15 Aug 2016 18:20:11 +0000 (18:20 +0000)]
Merge "Improve BT HAL write response to BT server reads"
am:
f65e83e5b1
Change-Id: Ib48a998e19b3eada7d19897b81626f60309062b3
Treehugger Robot [Mon, 15 Aug 2016 18:15:31 +0000 (18:15 +0000)]
Merge "Improve BT HAL write response to BT server reads"
Pavlin Radoslavov [Mon, 15 Aug 2016 18:03:35 +0000 (18:03 +0000)]
Separate the Alarm unit tests from testing the Wakelocks am:
223cecf8b6
am:
246d21b152
Change-Id: I993ea35259eb6a033408e990748b88dd386d198e
Pavlin Radoslavov [Mon, 15 Aug 2016 17:59:38 +0000 (17:59 +0000)]
Separate the Alarm unit tests from testing the Wakelocks
am:
223cecf8b6
Change-Id: Iceb0a3e4d2f776456cad88a5276dbd9516baf175
Andy Hung [Wed, 15 Jun 2016 02:58:58 +0000 (19:58 -0700)]
Improve BT HAL write response to BT server reads
By polling, we ensure the socket buffer is nearly always full,
as blocking socket writes may not wake in time to avoid underflow.
Bug:
28286313
Change-Id: I056768dbd860993c41d44c33a4846d62c50db933
(cherry picked from commit
5952e40bef1afb310cd3bba7e673163d7e32d62c)
Pavlin Radoslavov [Mon, 8 Aug 2016 18:18:47 +0000 (11:18 -0700)]
Separate the Alarm unit tests from testing the Wakelocks
* Simplify the AlarmTestHarnless unit test setup for Wakelocks.
Now we use wakelock_set_os_callouts() to setup local unit tests and
to keep local state whether the wakelock is acquired.
Also, removed unused timer-related setup inside AlarmTestHarnless.
* Add explicit Wakelock unit tests
Change-Id: I08bd8d457fd48ec45a8a591bd69cb08dbaf5ee7a
Jakub Pawlowski [Thu, 11 Aug 2016 23:10:26 +0000 (23:10 +0000)]
Add LE Secure Connection data parsing (3/4) am:
bd60854e49
am:
2eab00ba4a
Change-Id: I889b02129cc85066ceada7c2d9cfcb74b411b450
Jakub Pawlowski [Thu, 11 Aug 2016 23:06:57 +0000 (23:06 +0000)]
Add LE Secure Connection data parsing (3/4)
am:
bd60854e49
Change-Id: Ia33a911e973fe22b72a4bf4f3ce89d20b9b71fea
Jakub Pawlowski [Thu, 28 Jul 2016 12:58:35 +0000 (05:58 -0700)]
Add LE Secure Connection data parsing (3/4)
Bug:
30460956
Change-Id: I216142090fe99b25ef7697fceceb278b761a182b
Satomi [Tue, 9 Aug 2016 20:26:45 +0000 (20:26 +0000)]
Revert "Register for Service Changed notifications for untrusted devices" am:
42657c2f23
am:
ef259dd0fb
Change-Id: I7bc5b5a8681629c6c16431e654feaffd1dae8b49
Satomi [Tue, 9 Aug 2016 20:23:51 +0000 (20:23 +0000)]
Revert "Register for Service Changed notifications for untrusted devices"
am:
42657c2f23
Change-Id: I01ff859d5158904695965a0038b252fc98c9096d
Satomi [Tue, 9 Aug 2016 15:19:03 +0000 (08:19 -0700)]
Revert "Register for Service Changed notifications for untrusted devices"
This reverts commit
5f0e23d532f4f90d081ccc0e294f404eb5e5b688.
Instead of reusing service discovery data, gatt profile initiates
discovery by service uuid before regular service discovery is started,
and both discovery procedures interfere with each other.
As a result, link is dropped.
Bug:
30462943
Bug:
30481143
Change-Id: Ic63339f4f46eda70e575a4103974766d54a9d624
Signed-off-by: Satomi <michitsuta@casio.co.jp>
Satish Kodishala [Tue, 9 Aug 2016 18:19:00 +0000 (18:19 +0000)]
strlcpy call size fixes, memory clearing am:
4fd9ce1b4d
am:
6157dc7459
Change-Id: Ib258a1623314aefcb1479b8efa0042719e968fb3
Satish Kodishala [Tue, 9 Aug 2016 18:13:45 +0000 (18:13 +0000)]
strlcpy call size fixes, memory clearing
am:
4fd9ce1b4d
Change-Id: I563fa0319297ae930efc5eb114ecf639b0a5096e
Satish Kodishala [Fri, 5 Feb 2016 07:05:54 +0000 (12:35 +0530)]
strlcpy call size fixes, memory clearing
Usecase:
1. Run PTS test cases IIA BV-03-I/IIC BV-02-I/IID BV-01-I
Failure:
Randomly errors are returned for AT+BIA from PTS.
Cause:
Command parsing advances the check beyond the string end,
which is not guaranteed to be zeroed.
Fix:
Break string parsing when the end of string is encountered
after a AT+BIA argument.
Bug:
29615016
Change-Id: Ifc77435e780fba34006372694a7da4758e4f8c36
Jakub Pawlowski [Tue, 9 Aug 2016 10:58:42 +0000 (10:58 +0000)]
Merge "Fix descriptor write" am:
a8be6caf45
am:
d3e28d696a
Change-Id: Ic305147474f43c87c3b0693061ba12cd66fc82de
Jakub Pawlowski [Tue, 9 Aug 2016 10:56:15 +0000 (10:56 +0000)]
Merge "Fix descriptor write"
am:
a8be6caf45
Change-Id: I6346731e787457bbd013c958ccb306f470e9bc00
Jakub Pawlowski [Tue, 9 Aug 2016 10:48:53 +0000 (10:48 +0000)]
Merge "Fix descriptor write"
Jakub Pawlowski [Tue, 9 Aug 2016 10:44:05 +0000 (03:44 -0700)]
Fix descriptor write
This patch fixes error introduced in commit
bb920a65b959fbb593403587f89bc3fc7e831732
Writing descriptor was broken, ACTS tests were failing.
Change-Id: Ied2ff4a12df1053961677fdcb824838b7946dbaa
Jacky Cheung [Mon, 8 Aug 2016 17:49:08 +0000 (17:49 +0000)]
Add dumpsys support for LE connection parameter updates am:
8ec8ca4a87
am:
6fdfdef14f
Change-Id: I5e1a8ae04e0d88a9dfcc3f4459a187954e0729eb
Jacky Cheung [Mon, 8 Aug 2016 17:46:40 +0000 (17:46 +0000)]
Add dumpsys support for LE connection parameter updates
am:
8ec8ca4a87
Change-Id: I0966798416b8a7c784f4b1e6d5af593e91006990
Jacky Cheung [Thu, 4 Aug 2016 17:20:39 +0000 (10:20 -0700)]
Add dumpsys support for LE connection parameter updates
Change-Id: Ie203eec9d8efec0d55b1b10208c64a98449044a2
(cherry picked from commit
4e5514b69b95ee1416fb0d0b5e7783bcfe1612ab)
Marie Janssen [Sat, 6 Aug 2016 00:05:41 +0000 (00:05 +0000)]
resolve merge conflicts of
d19e078 to stage-aosp-master
am:
b91e97ba44
Change-Id: I76bc52887f10e59f5c5fbe8296a50e2f4781af65
Marie Janssen [Sat, 6 Aug 2016 00:02:23 +0000 (17:02 -0700)]
resolve merge conflicts of
d19e078 to stage-aosp-master
Change-Id: I901ce1a7cafc2f526d8a7e515e9fe8ffdd20f33e
Marie Janssen [Fri, 15 Jul 2016 19:48:27 +0000 (12:48 -0700)]
Use standard types, consistent ifdef style everywhere
Remove the typedefs in stack/include/bt_types.h
Use standard types everywhere.
Use standard style for #if statements:
- #if (VAR_NAME == TRUE)
- #if (VAR_NAME1 == TRUE && VAR_NAME2 == TRUE)
Use __func__ instead of __FUNCTION__
Fix some debug statements to use __func__
Update script to be less disruptive to aligned assignment blocks.
Change-Id: I8f8f068e6c26ce74fd3b3707e1e31fd0b919cdd0
Jakub Pawlowski [Fri, 5 Aug 2016 19:24:03 +0000 (19:24 +0000)]
Fix APR crashes in read rssi callback am:
7294d958dc
am:
e449d19a74
Change-Id: I2a3f0294fe2578e28e546b7aedc4562dda34ecad
Jakub Pawlowski [Fri, 5 Aug 2016 19:21:35 +0000 (19:21 +0000)]
Fix APR crashes in read rssi callback
am:
7294d958dc
Change-Id: I5e7daa5962e92c8fb2c6adf683fe616f0af294d1
Jakub Pawlowski [Wed, 3 Aug 2016 22:43:56 +0000 (15:43 -0700)]
Fix APR crashes in read rssi callback
When read RSSI timeouts, callback is called with NULL result. This was
causing crash.
Bug:
30562545
Change-Id: I05b7eb2808c58655525ed5c7019ae175181c837a
Greg Hartman [Thu, 4 Aug 2016 23:29:48 +0000 (23:29 +0000)]
Implements some additional commands am:
03c2c7c22a
am:
4c95eabead
Change-Id: Ic7050227fde19166acc3d316b06d934f6e3423a5
Greg Hartman [Thu, 4 Aug 2016 23:27:29 +0000 (23:27 +0000)]
Implements some additional commands
am:
03c2c7c22a
Change-Id: I5a0f2011966daad1679a4f196996552db40f5e56
Greg Hartman [Tue, 26 Jul 2016 20:19:02 +0000 (13:19 -0700)]
Implements some additional commands
- Added/Improved responses to certain opcodes on bt_vendor.cc
- Added handlers for a couple of commands used in L
Change-Id: I85c02553057e4ea3a3e51795c2e1a2bb679fd5e1
Mudumba Ananth [Thu, 4 Aug 2016 01:47:47 +0000 (01:47 +0000)]
HFP 1.7 profile update (1/4) am:
4de9bbcde7
am:
381ffde672
Change-Id: I5d8fc1329bf2712a3bf7bb5462cb0ef793af8791
Mudumba Ananth [Thu, 4 Aug 2016 01:37:03 +0000 (01:37 +0000)]
HFP 1.7 profile update (1/4)
am:
4de9bbcde7
Change-Id: I52433802594e44702152b57f00f7d5511f4741ac
Mudumba Ananth [Mon, 29 Feb 2016 10:21:43 +0000 (02:21 -0800)]
HFP 1.7 profile update (1/4)
-> Added AG support for HFP 1.7 version upgrade which
adds new type of indicator called HF(headset) indicators.
-> Added support for two new AT commands AT + BIND and AT + BIEV
which have been appended to Hands-Free SLC sequence.
-> Added support to propagate the above commands and their data to
the upper layers.
Bug:
19983867
Change-Id: I93d5b2af949f9fb99507a954e623cd0927ddc976
(cherry picked from commit
e5bed64926d6ae45501617052f90932287da135b)
Jakub Pawlowski [Thu, 4 Aug 2016 00:12:50 +0000 (00:12 +0000)]
Rmove write type from GATT descriptor writes (4/4) am:
bb920a65b9
am:
bd4199ca15
Change-Id: I5e4e9fdba783959170082d0d9b65a9ddee2be070
Jakub Pawlowski [Thu, 4 Aug 2016 00:01:51 +0000 (00:01 +0000)]
Rmove write type from GATT descriptor writes (4/4)
am:
bb920a65b9
Change-Id: I9301e43850d5e97256f3eca0a47047c47757c08b
Jakub Pawlowski [Tue, 21 Jun 2016 23:16:09 +0000 (16:16 -0700)]
Rmove write type from GATT descriptor writes (4/4)
According to the Bluetooth Core specification v4.2, Vol 3, Part G,
section 4.12.3: "The Attribute Protocol WRITE REQUEST is used
for this sub-procedure".
Change-Id: I60bb0b1489b417af96975265a023c356ace293bd
Jakub Pawlowski [Wed, 3 Aug 2016 13:05:56 +0000 (13:05 +0000)]
Merge changes from topic 'beautiful-gatt-server' am:
3cf12a6cfc
am:
27efd47cb6
Change-Id: Ie6a077a34377f4bac0aba6e996c56fbd10786681
Jakub Pawlowski [Wed, 3 Aug 2016 13:00:52 +0000 (13:00 +0000)]
Merge changes from topic 'beautiful-gatt-server'
am:
3cf12a6cfc
Change-Id: I542982eb13daa23c2ee75c74c3a0a04a94449de0
Jakub Pawlowski [Wed, 3 Aug 2016 10:21:43 +0000 (10:21 +0000)]
Merge changes from topic 'beautiful-gatt-server'
* changes:
GATT Server refactoring (4/4)
Add C++ Parcelable types for Service, Characteristic, Descriptor
Jakub Pawlowski [Mon, 1 Aug 2016 22:39:50 +0000 (22:39 +0000)]
Merge "Fix build when SMP_DEBUG=TRUE plus logging improvements" am:
980147c889
am:
d0bdd83a9c
Change-Id: Ia1b60529fc2b4ca20ff9db3e32508f4d8b5e187a
Jakub Pawlowski [Mon, 1 Aug 2016 22:36:53 +0000 (22:36 +0000)]
Merge "Fix build when SMP_DEBUG=TRUE plus logging improvements"
am:
980147c889
Change-Id: I97ffc1ef3c2a953f486829b57da9f558c2b3ecd5
Treehugger Robot [Mon, 1 Aug 2016 22:29:31 +0000 (22:29 +0000)]
Merge "Fix build when SMP_DEBUG=TRUE plus logging improvements"
Pavlin Radoslavov [Fri, 29 Jul 2016 17:20:43 +0000 (17:20 +0000)]
Relax some of the timing parameters in the Alarm unit tests am:
9713b9476c
am:
b63bbb0fbe
Change-Id: Ie58c059606e80a4915a553dfbfb24eb23cecde19
Pavlin Radoslavov [Fri, 29 Jul 2016 17:18:14 +0000 (17:18 +0000)]
Relax some of the timing parameters in the Alarm unit tests
am:
9713b9476c
Change-Id: Ibb7599098fd10e2b005622488160ffdce10aea3f
Jakub Pawlowski [Sat, 26 Mar 2016 07:47:23 +0000 (00:47 -0700)]
GATT Server refactoring (4/4)
Right now in order to add gatt service, we need to add it piece by
piece - each characteristic and descriptor separately. This causes lots
of concurrency errors in higher layers. Instead accept whole service
definition and start whole service at once.
Also, use handles to identify GATT server attributes. It's already done
on clinet side.
Bug:
27999121
Change-Id: I42c796809423d116fe6755ef8b1c40da270427a5
Jakub Pawlowski [Tue, 24 May 2016 19:42:45 +0000 (12:42 -0700)]
Add C++ Parcelable types for Service, Characteristic, Descriptor
This types will be used in upcoming GATT Client/Server code. They will
match ones you can found in frameworks/base.
Change-Id: I6080310a726bcbcd0cd36970712928b188f7bd24
Jakub Pawlowski [Thu, 28 Jul 2016 13:11:36 +0000 (06:11 -0700)]
Fix build when SMP_DEBUG=TRUE plus logging improvements
Bug:
30460956
Change-Id: I2611501d1961b104c8e66b1a3fca414936dc321f
Pavlin Radoslavov [Thu, 28 Jul 2016 20:11:10 +0000 (13:11 -0700)]
Relax some of the timing parameters in the Alarm unit tests
In certain environments (e.g., virtual machines), the alarm timeouts
can late by tens of ms (40ms have been observed in some cases).
Relaxing some of the unit tests timing parameters, so we can
continue testing for correctness as covered by the AlarmTest.
Bug:
28907913
Change-Id: I211c36a689774c5cdf427039e4c9184cf4147604
Ayan Ghosh [Thu, 28 Jul 2016 06:27:12 +0000 (06:27 +0000)]
Proper handling of AV connection collision am:
09d91c0ceb
am:
bea5f8bbec
Change-Id: I78475f289871c346fcd1674fed817d5ce54beb0d
Ayan Ghosh [Thu, 28 Jul 2016 06:24:04 +0000 (06:24 +0000)]
Proper handling of AV connection collision
am:
09d91c0ceb
Change-Id: I7ace6b2005e544c8821271a9d575ee49f883ada1
Ayan Ghosh [Fri, 22 Jul 2016 09:41:11 +0000 (15:11 +0530)]
Proper handling of AV connection collision
Use case:
1. Pair to Remote
2. Turn OFF/ON BT on DUT
3. After DUT's BT is turned ON, Remote would reconnect to DUT
Failure:
Bluetooth settings UI continues to display Connecting when
BT was Turned OFF and ON.
Root cause:
- This is a connection collision case where remote is not responding
to DUT initiated SDP attribute fetch request for AV. AV holds outgoing
connection for sometime and meanwhile remote starts AV connection
from its end. Then DUT disconnects the SDP channel after a second
without waiting to fetch requested info, and this confuses DUT's AV
state machine. The outgoing connection could not proceed as incoming
AV L2CAP connects, because DUT's AV state machine is not equipped
with handling this scenario. On incoming connection timer expiry, it
does not start AV media channel as it finds AV Open has not been done
from upper layer after incoming L2CAP got connected.
- When the incoming path 2s timer expires, the DUT checks whether Open
has been called after incoming state is set. This is not the case here,
because Open was triggered earlier as as part of the outgoing
connection. As a result, A2DP cannot connect at all.
Fix:
- Properly set collision flags for handling AV connection collision.
- Update BTA AV handle even when BTIF AV state machine is yet to be built.
Bug:
30362987
Change-Id: I02a3adb62479b0f762bc792a5727d06e11eaaa52
Greg Hartman [Tue, 26 Jul 2016 20:01:52 +0000 (20:01 +0000)]
Allow the Bluetooth test library to be used as a vendor library am:
6bfe283db7
am:
854af1efaa
Change-Id: Idfedb9126f62abfc0148c1601f65e5a4107fb139
Greg Hartman [Tue, 26 Jul 2016 19:58:58 +0000 (19:58 +0000)]
Allow the Bluetooth test library to be used as a vendor library
am:
6bfe283db7
Change-Id: I0be79f4e37637a8af166a8b307c4b664bc777036
Greg Hartman [Tue, 26 Jul 2016 03:22:48 +0000 (20:22 -0700)]
Allow the Bluetooth test library to be used as a vendor library
Change-Id: I479fd72482399f878ef045271c7b493c94aa5ab7
Greg Kaiser [Mon, 25 Jul 2016 22:48:08 +0000 (22:48 +0000)]
btif: Fix logic bug within register_application am:
696996c115
am:
f2f5deb39f
Change-Id: I92a4edfe482827b211234c6f24a086cd93d8c295
Greg Kaiser [Mon, 25 Jul 2016 22:45:12 +0000 (22:45 +0000)]
btif: Fix logic bug within register_application
am:
696996c115
Change-Id: Id2c2d4d2c171dfa9d8389a901e1f0f627628fa8e
Greg Kaiser [Mon, 27 Jun 2016 18:27:09 +0000 (11:27 -0700)]
btif: Fix logic bug within register_application
We were checking for the SINK bit twice in the same condition,
and would incorrectly report a SINK only mask as a DUAL device,
and would never hit the logic path to report a device as SINK.
Bug:
29763285
Change-Id: Ibc39db6396c09ddcd63896302dabf72841304cf2
Jakub Pawlowski [Wed, 20 Jul 2016 15:13:15 +0000 (15:13 +0000)]
Don\\'t use move in FOR_EACH_* macro am:
e84c4bdb5f
am:
e7eed81e43
Change-Id: I9bb69590224c39702e295e56934f87181ff6df8c
Jakub Pawlowski [Wed, 20 Jul 2016 15:10:48 +0000 (15:10 +0000)]
Don\'t use move in FOR_EACH_* macro
am:
e84c4bdb5f
Change-Id: Ibda5234458e95627b16fcc94edb4331b2c8f960a
Jakub Pawlowski [Wed, 20 Jul 2016 10:12:49 +0000 (03:12 -0700)]
Don't use move in FOR_EACH_* macro
FOR_EACH_* macros are resolved to for loop. Using std::move inside it
will cause already moved object to be used when more than one observer
is registered. This patch fixes that.
Change-Id: Iaaa314f5eeed4094d1e0fc951d3664fb1514ba15
Andre Eisenbach [Mon, 18 Jul 2016 22:25:45 +0000 (22:25 +0000)]
Make parameter set S4 default for eSCO am:
3edefef0ce
am:
9d2429f288
Change-Id: I5f8039120a79bbac9e7b5e7c6572cc622974bdbd
Andre Eisenbach [Mon, 18 Jul 2016 22:23:16 +0000 (22:23 +0000)]
Make parameter set S4 default for eSCO
am:
3edefef0ce
Change-Id: I2beae8e74d45d29ca8868b9baa62ef3c818ed1de
Andre Eisenbach [Fri, 15 Jul 2016 17:50:32 +0000 (10:50 -0700)]
Make parameter set S4 default for eSCO
Bug:
29615817
Change-Id: I5256b31a746433f10c2e7de1629952f668c99d16
(cherry picked from commit
af5b8248593c80acd8b1afb695eafe5801b44f7d)
Subramanian Srinivasan [Mon, 18 Jul 2016 09:19:03 +0000 (09:19 +0000)]
Process Gatt read multiple response properly am:
089cd116de
am:
d91d423599
Change-Id: Ie8242f1d9c4ab4d985c1419d076d6cf8b94ae2d5
Subramanian Srinivasan [Mon, 18 Jul 2016 09:16:05 +0000 (09:16 +0000)]
Process Gatt read multiple response properly
am:
089cd116de
Change-Id: Ieff85e5040d7fa3db24a7946a05d4958480ddd08
Subramanian Srinivasan [Mon, 16 May 2016 18:14:03 +0000 (11:14 -0700)]
Process Gatt read multiple response properly
The Gatt server multi response queue is made NULL after each Gatt
server response is sent by BT stack in response to the remote Gatt
client's request. But the multi response queue is not initialized
again to process subsequent read multiple Gatt server responses.
Hence, the BT stack does not send Gatt server responses for the
subsequent read multiple requests from remote device which inturn
leads to disconnection. This change would process read multiple
responses by initializing the queue when it is null.
Some of Bluetooth certification test cases involve testing
Gatt server's capability to process and send read multiple
response.
Test case: TC_GAR_SR_BV_05_C
This change also cleans up unnecessary initialization and freeing
of multi rsp queue. This change makes sure that the stack initializes
multi rsp queue only when it is necessary(ie only when the Gatt
multiple read response is being processed).
Bug:
29011255
Change-Id: Ia3e2e1569ee16cac9c518f95501945257b9fb7fd
Jakub Pawlowski [Thu, 14 Jul 2016 19:39:13 +0000 (19:39 +0000)]
Fix linux build, missing strlcpy am:
3a2dc880ce
am:
c703cf4dd9
Change-Id: If30b8e1a856bb4ff7ed0ae086a735a62afbf2743
Jakub Pawlowski [Thu, 14 Jul 2016 19:36:45 +0000 (19:36 +0000)]
Fix linux build, missing strlcpy
am:
3a2dc880ce
Change-Id: I306f667701519fed87b97138e0b47aab0cf2cc3d