Commit graph

3150 commits

Author SHA1 Message Date
Sanket Agarwal
9d0d6700b1 Add API for AVRCP CT get_play_status_cmd
Bug: b/31684931
Change-Id: I5e47e3b1ab30f457ee2581d02a384ffb76454a21
(cherry picked from commit de0f636db2035deff127fdd5f62ed2043650b270)
2016-11-01 19:59:36 +00:00
Sanket Agarwal
7798d9ddfc Merge "Implement set addressed player." 2016-11-01 17:22:49 +00:00
Sanket Agarwal
8f8171ac23 Implement set addressed player.
Bug: b/31554234
Change-Id: I32b5d33605b06c70fea1c20942af82a1f9561b46
(cherry picked from commit f01b2391410485aa4cb82bc2d91085d10350c84d)
2016-11-01 15:55:32 +00:00
Treehugger Robot
5096f9c288 Merge "[AVRCP1.6 CT] Add reply status to get_folder_items_callback." 2016-11-01 00:43:42 +00:00
Sanket Agarwal
f65a26b962 [AVRCP1.6 CT] Add reply status to get_folder_items_callback.
It is useful to know what error code the get_folder_items_cmd gets back
with since it can help the UI decide if to keep fetching more. For ex.
if the return code is out of range then the UI does not need to keep
fetching anymore. This is useful for folders where we do not know the
size before hand (Media Player List or Now Playing List).

Bug: b/31253501

Change-Id: Ia6544de4845b8b72feaa359ef1481290dc54348a
(cherry picked from commit fdc7153966f9aecdbfd962b8a7655b5db3e464e0)
2016-10-31 16:17:48 -07:00
Treehugger Robot
86f508d45d Merge "Separate LE scanner and GATT client BTIF interfaces (2/3)" 2016-10-28 20:45:10 +00:00
Jakub Pawlowski
1e24ce9167 Separate LE scanner and GATT client BTIF interfaces (2/3)
In order to properly separate low level structures associated with LE
scanner and GATT client, their interfaces must be separated first.

Test: sl4a BleScanApiTest
Bug: 30622771
Change-Id: Idf8c5b7285b68e59f18f5216380488bfe8404b22
2016-10-28 12:07:45 -07:00
Jakub Pawlowski
1f4fc31367 Merge "Separate LE scanner from GATT client (3/4)" 2016-10-27 23:18:54 +00:00
Jakub Pawlowski
8cb1baefa1 Separate LE scanner from GATT client (3/4)
Right now, LE scanning functionality is combined with the GATT client.
This is the source of various bugs, like scans suddenly stoppinging when
a GATT client is killed. It also increases memory consumption, because
we associate many structures with a GATT client, which are not necessary
when just scanning.

Test: sl4a BleScanApiTest ConcurrentBleScanTest
Bug: 30622771
Change-Id: I5c2608b769bd6aec54900357cdccfa22987962c9
2016-10-27 23:10:20 +00:00
Dmitry Shmidt
d20fe45ed5 Merge "audio: Fix size_t related warnings" 2016-10-26 22:48:41 +00:00
Dmitry Shmidt
a898f93475 audio: Fix size_t related warnings
Test: manual

Change-Id: I34961243f2d4b95c47d954fc616404391302595a
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>
2016-10-26 13:54:25 -07:00
Jakub Pawlowski
81f7d7e8bf Merge "Change how Advertise Data is passed to BTIF (1/3)" 2016-10-26 00:10:50 +00:00
Jakub Pawlowski
6b76de1ef0 Change how Advertise Data is passed to BTIF (1/3)
The way that AD data is passed right now put some additional
limitations, i.e. only one manufacturer specific data can be set, or
only one service UUID. By moving AD generation to upper layers, another
set of limitations is removed.

Test: Covered by BleAdvertiseApiTest sl4a test
Bug: 30622771
Change-Id: I34e8d731141caaf4497565f6e929992f6e929913
2016-10-19 14:57:05 -07:00
Jakub Pawlowski
77a4c05a7e Merge "Get rid of dependency on core from linux build (2/2)" 2016-10-19 18:46:21 +00:00
Jakub Pawlowski
7b4adb1dbc Get rid of dependency on core from linux build (2/2)
When building bluetooth stack for linux, don't include
dependencies from core.
2016-10-18 20:33:24 -07:00
Treehugger Robot
6b6400720a Merge "AVRCP 1.6: media browsing support on Controller" 2016-10-17 21:16:21 +00:00
Sanket Agarwal
f93277695b AVRCP 1.6: media browsing support on Controller
Bug: 28791287
Change-Id: I52519e91c9f8f6d8db05ebc8a79e118582701150
(cherry picked from commit dd4825b6ab)
2016-10-17 12:35:18 -07:00
Treehugger Robot
68669afe4d Merge "Convert advertising HAL from struct into class (1/3)" 2016-10-10 22:34:50 +00:00
Jakub Pawlowski
4a635288b8 Convert advertising HAL from struct into class (1/3)
Bug: 30622771
Test: all related tests were updated
Change-Id: I61381cc57a9c8cd725478ad4e52abce9f857a373
2016-10-08 16:42:37 -07:00
Colin Cross
e1582496f9 Merge "Make gralloc enums unsigned" 2016-10-08 22:33:50 +00:00
Colin Cross
92bba13dd1 Make gralloc enums unsigned
hardware/libhardware/include was being included with -isystem, which
hides warnings.  The gralloc enum values are used as bitmasks on
unsigned types, explicitly mark them values as unsigned in order to
force the type of the resulting enum to be unsigned.

Bug: 32018017
Test: m -j
Change-Id: Iea281480574e0a7a8413273983c76c2c0f9f7049
2016-10-08 11:40:17 -07:00
Colin Cross
3b08202c1b Merge changes If68bf155,I35f950ba,Idb430c45,I71a1d33a
* changes:
  Avoid C-style cast warnings
  Add name for typedef
  Fix enum switch warnings
  Fix doc warnings
2016-10-08 00:32:04 +00:00
Colin Cross
cc8d9f9fcb Avoid C-style cast warnings
hardware/libhardware/include was being included with -isystem, which
hides warnings.  Use a macro to convert modules to hw_device_t** to call
the open function that works in C++ and C.

Bug: 32018017
Test: m -j
Change-Id: If68bf15581975f1217fcab366cef7bc784894641
2016-10-07 13:10:20 -07:00
Colin Cross
64cd91274f Add name for typedef
hardware/libhardware/include was being included with -isystem, which
hides warnings.  Add the missing name for a typedef.

Bug: 32018017
Test: m -j
Change-Id: I35f950ba3e26836cb51312ef28e98748f29d8cd8
2016-10-07 13:10:07 -07:00
Colin Cross
248ec3da8b Fix enum switch warnings
hardware/libhardware/include was being included with -isystem, which
hides warnings.  -Wswitch requires that all cases in a switch statement
on an enum be enum values.  For enums being used as bit flags, cast the
enum to an int.

Bug: 32018017
Test: m -j
Change-Id: Idb430c451d017bb7ec61a7119d6abd87b6dcdc87
2016-10-07 13:10:07 -07:00
Colin Cross
867e1e3043 Fix doc warnings
hardware/libhardware/include was being included with -isystem, which
hides warnings.  Fix use of doxygen function comments on on function
pointers, which are not considered functions or methods.

Bug: 32018017
Test: m -j
Change-Id: I71a1d33a9ae90b646bf22f32e052bbe22e8aca76
2016-10-07 13:10:07 -07:00
Treehugger Robot
04302ea2fa Merge "[sensor] Clarification of sensor HAL behavior at deactivation" 2016-10-06 00:10:02 +00:00
Peng Xu
aaeeaa41bd [sensor] Clarification of sensor HAL behavior at deactivation
Sensor HAL should abandon stale data at deactivation.

Test: documentation change, spell checked.
Bug: 28443809

Change-Id: I29b1d324fc96d0448cdefbf2130161f71c3d3310
2016-10-05 15:17:33 -07:00
Treehugger Robot
4074ce1408 Merge "Fix an incorrect description of OOB data field" 2016-09-22 18:57:24 +00:00
Jakub Pawlowski
9caabe1563 Fix an incorrect description of OOB data field
Change-Id: Ia3712d9af9d5521e2a3cc9ecbcf7b37fd0755c9a
2016-09-22 17:10:12 +00:00
Treehugger Robot
9e1a02cc5e Merge "Bluetooth: split setting params and enabling advertising (2/3)" 2016-09-13 08:30:27 +00:00
Jakub Pawlowski
bb84c86f5a Bluetooth: split setting params and enabling advertising (2/3)
Proper order of commands when starting advertising:
* set parameters
* set data
* enable advertising

This cannot be achieved when setting advertising parameters and enabling
advertising is put together in one function. Enabling before setting
data might cause first advertisements to be empty. If a device with
hardware filter pick up such advertisement, it might fail to properly
recognize our device.

Bug: 30622771
Bug: 19372779
Change-Id: Iac154beb5129526fa3cd1d1b31eba269459bc9a0
2016-09-13 06:26:14 +00:00
Dan Stoza
e7a66b9abc Merge "Re-add gralloc 1.0 header" 2016-09-09 21:21:19 +00:00
Dan Stoza
a2d5b7d898 Re-add gralloc 1.0 header
Adds the header for version 1.0 of the gralloc HAL, now that the
framework properly supports it.

Bug: 28401203
Change-Id: I2ef28ca1e5ed5e9e71a14c4d1c97245051acce6c
2016-09-09 12:04:29 -07:00
Dan Stoza
799c3fc422 Merge changes I90234d91,Idf5f86c9
* changes:
  HWC2: Add SKIP_CLIENT_COLOR_TRANSFORM capability
  HWC2: Redefine retire fence as present fence
2016-09-09 16:51:40 +00:00
Dan Stoza
d2168f7e47 HWC2: Add SKIP_CLIENT_COLOR_TRANSFORM capability
Adds HWC2_CAPABILITY_SKIP_CLIENT_COLOR_TRANSFORM, which indicates
that the client should never apply the color transform, even if all
layers are being composed by the client. This allows the device to
use its color management hardware in all cases to avoid any potential
mismatches between client and device application of the transform.

Bug: 29870302
Change-Id: I90234d91858206714e650eadb03227a4a180d975
2016-09-08 15:01:49 -07:00
Dan Stoza
ef264828fe HWC2: Redefine retire fence as present fence
Redefines the retire fence returned by presentDisplay to be a present
fence with more strictly defined semantics: this fence should fire
at the vsync when the frame is picked up by hardware, either for
scanout (in the case of video-mode panels) or for transfer to panel
memory (in the case of command-mode panels).

Bug: 29771461
Change-Id: Idf5f86c92a8748426cf3368242c08876a48d81dd
2016-09-08 15:01:34 -07:00
Treehugger Robot
8bc59ec625 Merge "Stop hard coding gmock things." 2016-09-06 19:42:34 +00:00
Dan Albert
b55da5dd3f Stop hard coding gmock things.
The headers are exported.

Change-Id: I72f8dda3f441bc72accce3c9e94a7160fcefc54c
Test: mma
(cherry picked from commit a26bcd5424)
2016-09-06 11:07:26 -07:00
Ajay Panicker
b576a06972 Merge "AVRCP 1.6: media browsing support on Target(2/3)" 2016-09-02 20:46:01 +00:00
Avish Shah
f20c24e86b AVRCP 1.6: media browsing support on Target(2/3)
-> As a part of Avrcp 1.6 upgrade, added support
   for following features:
   [1] setAddressedPlayer
   [2] setBrowsedPlayer
   [3] getFolderItems scope=VFS/NowPlaying/Search
   [4] changePath
   [5] getItemAttributes
   [6] playItem
   [7] getTotalNumberOfItems
   [8] search string in current browsed path.
   [9] AddToNowPlaying
-> Defined functions and enhanced structures to
   support above mentioned features.
-> Added support for dual RC.

Bug: 19361366
Merged-In: I0ab7f0c7d87c06fe4f454151d20494c56aceae12
Change-Id: I0ab7f0c7d87c06fe4f454151d20494c56aceae12
2016-09-02 19:51:25 +00:00
Greg Kaiser
f7888e1130 Merge "modules/camera: Fix compiler warnings" 2016-08-31 21:55:30 +00:00
Jakub Pawlowski
7a3148bc41 Merge "Separate advertiser from GATT client (3/4)" 2016-08-31 18:05:27 +00:00
Jakub Pawlowski
6ecdb8c0cd Separate advertiser from GATT client (3/4)
Bug: 30622771
Change-Id: Ib9f9a27a2d1a7945b05a1c985cf78b46f4b97026
2016-08-30 10:46:53 -07:00
Chih-hung Hsieh
81a5640cf9 Merge "Fix misc-macro-parentheses warnings in context-hub.h." 2016-08-27 00:40:59 +00:00
Chih-Hung Hsieh
dc57369f96 Fix misc-macro-parentheses warnings in context-hub.h.
* Add parentheses around macro parameters.
Bug: 28705665
Test: build with WITH_TIDY=1

Change-Id: Ib1f4a99b331a844d50271a795b1582ce034a4f0c
2016-08-26 16:08:07 -07:00
Greg Kaiser
cbd7b77367 Merge "modules/power: Silence compiler warnings" 2016-08-25 16:25:34 +00:00
Greg Kaiser
5e53f5d85e modules/camera: Fix compiler warnings
Change-Id: I9a58f45cbfc09e4e1a243fa657dd0955648bf101
2016-08-25 06:20:32 -07:00
Greg Kaiser
854bf684af modules/power: Silence compiler warnings
Change-Id: Ib70ea23bd70ac39990cee5e1c435e346ab8f244d
2016-08-25 06:13:52 -07:00
Chih-Hung Hsieh
5f366ef448 Merge "Fix clang-tidy performance warnings in libhardware."
am: 1c1e311e06

Change-Id: I00ee76484a102dc1577566fe370af16f9283fc3e
2016-08-04 22:03:49 +00:00