am 1d5fe5c4: am 958f09b6: am 4602429d: Fix incorrect SetConnScanParam function mapping

* commit '1d5fe5c41e57f70727735f0e94dfd17d780c6bb4':
  Fix incorrect SetConnScanParam function mapping

am 1948336c: am a887e765: am 11026214: Enable clang compilation.

* commit '1948336c4025489b61f725f0357ab71a79a22da7':
  Enable clang compilation.

am c8eb3f77: am 7deacad6: am 3858011b: Send current RPA instead of pseudo-address on connect.

* commit 'c8eb3f77e15017187685180cdbca51697b1da54f':
  Send current RPA instead of pseudo-address on connect.

am a4d32b59: am 6106a1b3: am f34d336a: (-s ours) A2DP audio HAL: implement out_get_presentation_position()

* commit 'a4d32b59192b985a13a2f3fada0cf79a2b1ba1d1':
  A2DP audio HAL: implement out_get_presentation_position()

am 5e20685e: am 22c1d4fe: am 28f294b9: Load factory Bluetooth address from system property

* commit '5e20685e4dae1cefa0a6ad05be9f3c0e0e41f192':
  Load factory Bluetooth address from system property

am 97018a4c: am 09e9dec4: am 6a42cbdb: Add a SCO routing command to net_hci.

* commit '97018a4ccd797c8ada1ebc8cdf85cc4da0d634ca':
  Add a SCO routing command to net_hci.

am e37a7807: am 7fbf06c2: am 2af0bb6e: Fix merge discrepencies and resulting compile errors

* commit 'e37a780798cf0179f10b7baec1f48114944b7749':
  Fix merge discrepencies and resulting compile errors

am 958f09b6: am 4602429d: Fix incorrect SetConnScanParam function mapping

* commit '958f09b6a0e8af5ef9e26f3cb6eec5117f5e8b4d':
  Fix incorrect SetConnScanParam function mapping

am a887e765: am 11026214: Enable clang compilation.

* commit 'a887e765da53bb49b7cb834e930cd793732dbcc9':
  Enable clang compilation.

am 7deacad6: am 3858011b: Send current RPA instead of pseudo-address on connect.

* commit '7deacad6f54e08296527769b514183532d6fda29':
  Send current RPA instead of pseudo-address on connect.

am 6106a1b3: am f34d336a: (-s ours) A2DP audio HAL: implement out_get_presentation_position()

* commit '6106a1b3217ff15a62498e706812a0a548a72261':
  A2DP audio HAL: implement out_get_presentation_position()

am 4602429d: Fix incorrect SetConnScanParam function mapping

* commit '4602429dbcf5a5d6076f65450354be4db9f0f114':
  Fix incorrect SetConnScanParam function mapping

am 11026214: Enable clang compilation.

* commit '110262140c4ab383a27d19c08e33f5937bd09c11':
  Enable clang compilation.

am 3858011b: Send current RPA instead of pseudo-address on connect.

* commit '3858011befbf62ac8c38af1493d9c61d43f999da':
  Send current RPA instead of pseudo-address on connect.

am f34d336a: (-s ours) A2DP audio HAL: implement out_get_presentation_position()

* commit 'f34d336a16a6788f667355b70a4d53b1325a25ed':
  A2DP audio HAL: implement out_get_presentation_position()

am 22c1d4fe: am 28f294b9: Load factory Bluetooth address from system property

* commit '22c1d4fe04bea076b793d370cd8fe9ab00660eb5':
  Load factory Bluetooth address from system property

am 09e9dec4: am 6a42cbdb: Add a SCO routing command to net_hci.

* commit '09e9dec4a8011803ba3343ad0eeb347583b41c11':
  Add a SCO routing command to net_hci.

am 7fbf06c2: am 2af0bb6e: Fix merge discrepencies and resulting compile errors

* commit '7fbf06c2ba7a57d38e567c36941d5aca8c906b71':
  Fix merge discrepencies and resulting compile errors

am 28f294b9: Load factory Bluetooth address from system property

* commit '28f294b94f6e05f4572672671535f66bb60f0c0a':
  Load factory Bluetooth address from system property

am 6a42cbdb: Add a SCO routing command to net_hci.

* commit '6a42cbdb53d856a364a0c0fac28a732b55bc1cd9':
  Add a SCO routing command to net_hci.

am 2af0bb6e: Fix merge discrepencies and resulting compile errors

* commit '2af0bb6e5f6b380f374d510c54eabdf74c32bdf2':
  Fix merge discrepencies and resulting compile errors

DO NOT MERGE Revert "DO NOT MERGE Use POSIX timer API for wake alarms instead of OSI callouts."

Also reverts:
  "Fix build: use unsigned long long in arithmetic expression."
  "Fix build: add LOG_TAG parameter to LOG_* macros."

These CLs cause build failures on various platforms, an alternate
approach is being developed.

Fix incorrect SetConnScanParam function mapping

BTA_DM_API_BLE_CONN_SCAN_PARAM_EVT is defined before
BTA_DM_API_BLE_SCAN_PARAM_EVT in the enum but the corresponding
entries in bta_dm_action were inverted. This resulted in incorrect
invocation of set_conn_scan_params while trying to set parameters
for scanning. The call was hence failing here and was not
resulting in setting of the new scan parameters in the Controller
irrespective of the scan mode selected from the App. This would
essentially result in the breakage of ScanSettings.SCAN_MODE_<xx>
Android SDK APIs.

Bug: 23602042
Change-Id: I17e1b14a32250e3ccb7631a67690ec2e0a6bc321

Enable clang compilation.

Files under system/bt/device do not use atomic operation.
Old setting of LOCAL_CLANG is unnecessary.

Other atomic operations in system/bt/bt/core were changed
and osi/include/atomic.h was removed, in
https://android-review.googlesource.com/#/c/166762

Change-Id: I807e1fd1792e801392f40ab92a8990c826a5785a

Send current RPA instead of pseudo-address on connect.

Bug: 23079777

Change-Id: If6da80392748cdaad2a4833dd31fe299b62c28e3

A2DP audio HAL: implement out_get_presentation_position()

And complete implementation of out_get_render_position()

Bug: 21199150
Change-Id: I95038ad68433e401b096a0159282a2ae372570e1

Load factory Bluetooth address from system property

Implemented a check to grab the ro.boot.btmacaddr property in case the
device fails to receive an address beforehand.

Bug: 22618015
Change-Id: Ie322888e114a732f8e85c81793e3e5e5eacffc7a

Add a SCO routing command to net_hci.

This allows us to change the SCO routing parameters at runtime while
debugging / doing device bringup.

Change-Id: I8bafb7c7fb7ad7470d378cd14a7ee4aad63f3b9b

Fix merge discrepencies and resulting compile errors

Change-Id: I29fbec9f668ee9218b90a222801c9ddcb9352017

am 629f9555: am d3bfaecc: am ff1469f0: service/doc: Fixed some post-review comments

* commit '629f9555ee20a310b62bcc13427aec1fc2a0efcf':
  service/doc: Fixed some post-review comments

am 8ca512c3: am 3f6cfff7: am 9fc7d81c: service: Fix broken UUID advertising

* commit '8ca512c35d72edb44b049cca700735eeac211410':
  service: Fix broken UUID advertising

am d3bfaecc: am ff1469f0: service/doc: Fixed some post-review comments

* commit 'd3bfaecca37776ba15ac3b2acdc6fea9524172b4':
  service/doc: Fixed some post-review comments

am 614ee0e7: am 1e18ddc4: am 87222e0e: service/client: Add start-adv -u flag

* commit '614ee0e7c9c1c2915ab66d28d18ea3db8ce1f296':
  service/client: Add start-adv -u flag

am a7d32ade: am de312356: am de5cc703: service: Improve creating UUIDs from string

* commit 'a7d32ade6cf04a32c23bf97d6c1380be260e0a1b':
  service: Improve creating UUIDs from string

am 3f6cfff7: am 9fc7d81c: service: Fix broken UUID advertising

* commit '3f6cfff74c5b9cbcbf2edd1fbbaa13cf66f3b965':
  service: Fix broken UUID advertising

am ff1469f0: service/doc: Fixed some post-review comments

* commit 'ff1469f0e762840c0c745e7306b1edd96167a4c6':
  service/doc: Fixed some post-review comments

am 1e18ddc4: am 87222e0e: service/client: Add start-adv -u flag

* commit '1e18ddc491ef945b278df497b2d8372da0fb3275':
  service/client: Add start-adv -u flag

am de312356: am de5cc703: service: Improve creating UUIDs from string

* commit 'de3123563665c352a50ffb1b4e0e16da5c082887':
  service: Improve creating UUIDs from string

am 9fc7d81c: service: Fix broken UUID advertising

* commit '9fc7d81c2dfa3f9b7c4381cd83e5e8f788e235f3':
  service: Fix broken UUID advertising

service/doc: Fixed some post-review comments

Change-Id: I62e4f26cc55763f5880922bd06373db47329def9

am 87222e0e: service/client: Add start-adv -u flag

* commit '87222e0e826216c69f6a9a5bfe77689561067474':
  service/client: Add start-adv -u flag

am de5cc703: service: Improve creating UUIDs from string

* commit 'de5cc7034275bf0dfc0124d07b28c70ec634aed7':
  service: Improve creating UUIDs from string

service: Fix broken UUID advertising

The BTIF API expects a single 128-bit UUID in little-endian form to be
passed for its multi-adv service UUID field. The system service never
did this conversion so UUIDs were never properly advertised. This CL
fixes this.

Bug: 24247700
Change-Id: I541921da85bdabd907bfa9f75fbde7a3477ff593

service/client: Add start-adv -u flag

Added the -u flag that allows passing a UUID in string representation
(16-bit, 32-bit, and 128-bit) to be advertised as long as all fields
fit inside the data.

Bug: 24247700

Change-Id: I3e572db31b84f1e010ba19ab5a51d00c563bb3b7

service: Improve creating UUIDs from string

- When UUIDs are initialized from a 128-bit UUID, they are now parsed
  from the canonical UUID string representation.

- Invalid strings now cause the UUID to be marked as invalid, rather than
  causing a crash.

- Input such as "0x180d" is now valid.

- Added the GetShortestRepresentationSize() method to use in determining whether
  a UUID is within Base BT SIG UUID domain.

Bug: 24247700

Change-Id: Id28255ec963bc6da27a8e871598016d41920b688

am ea6195da: am 3e88d02a: am 79d1678a: Add net_admin group capability to bluetoothtbd

* commit 'ea6195da8ce42ddfdf8e84e5463127c365f5c359':
  Add net_admin group capability to bluetoothtbd

am 3e88d02a: am 79d1678a: Add net_admin group capability to bluetoothtbd

* commit '3e88d02a8523e90e25d0a2055abfbabca17f857a':
  Add net_admin group capability to bluetoothtbd

am 79d1678a: Add net_admin group capability to bluetoothtbd

* commit '79d1678a7714cb457cdb3435db41ef2336bf429c':
  Add net_admin group capability to bluetoothtbd

Add net_admin group capability to bluetoothtbd

The net_admin group capability is needed for accessing the rfkill
device in the kernel.

Change-Id: Ic5c2c2717fbad6428fc75ff00a0a3afcd2c5dec6

am 56e82aed: am 48d488d6: am 91c59c9f: service/client: Handle EOF

* commit '56e82aed62b27e4f9b880b1c623761b0e0f8085d':
  service/client: Handle EOF

am 48d488d6: am 91c59c9f: service/client: Handle EOF

* commit '48d488d6035b20d67f6763f408cd99bd2d84b77e':
  service/client: Handle EOF

am 91c59c9f: service/client: Handle EOF

* commit '91c59c9febbf4a49664b8a06620d4a4882605131':
  service/client: Handle EOF

service/client: Handle EOF

Added handling for EOF to the REPL so we gracefully exit if
Ctrl+D is pressed.

Change-Id: I706f3f46a7092a30b1e9d83c32805dbefc5180c9

am 9e5166c4: am 2ffa6718: am f6fc0c48: service: Added Binder API documentation

* commit '9e5166c4574ec8903afcb40d8a3be518d2ffa47f':
  service: Added Binder API documentation

am 2ffa6718: am f6fc0c48: service: Added Binder API documentation

* commit '2ffa67184bb825522c80a33f450de339fe289336':
  service: Added Binder API documentation

am f6fc0c48: service: Added Binder API documentation

* commit 'f6fc0c48c2b4c8a9965f0f86275896da386ebdb2':
  service: Added Binder API documentation

service: Added Binder API documentation

Added the service/doc directory with documentation for the
currently supported Binder interfaces in AIDL format.

Change-Id: I81bec12b7da7fbee110b1df0ffafcc3dc7d6eec3

am 08ea3b20: am 872a3a0c: am 852bf498: Add Flic smart button to interop database

* commit '08ea3b20067fb87a3e6236688492e0e02e1345c2':
  Add Flic smart button to interop database

am 872a3a0c: am 852bf498: Add Flic smart button to interop database

* commit '872a3a0ceebc123ecf6f6f20aff6f5f68bca7ea9':
  Add Flic smart button to interop database

am 852bf498: Add Flic smart button to interop database

* commit '852bf4980ea87a916e5f3037c9b897fd84c2417b':
  Add Flic smart button to interop database

Add Flic smart button to interop database

Bug: 24737963
Change-Id: If3b59437439305b490c5fc260a7080cee04f09c5

am e65f1a6b: am b6543387: Add missing LOG_TAG as the first argument to LOG_ERROR()

* commit 'e65f1a6beed9915ca6c2c6b76f34c8a87a534f32':
  Add missing LOG_TAG as the first argument to LOG_ERROR()

am 47eb1e23: am 3834c085: Merge commit \'955abac95ae2f1b8cdb269c5ab8c53d93f2fa48f\' into HEAD

* commit '47eb1e236c3f67956bfda3a9646b20afef4476a9':

am b6543387: Add missing LOG_TAG as the first argument to LOG_ERROR()

* commit 'b654338743916e5b86de7125bfc2e6739d06629b':
  Add missing LOG_TAG as the first argument to LOG_ERROR()

am 3834c085: Merge commit \'955abac95ae2f1b8cdb269c5ab8c53d93f2fa48f\' into HEAD

* commit '3834c0853006a0302569bd263ab33a5d78f63f16':

Add missing LOG_TAG as the first argument to LOG_ERROR()

NOTE: This fix is needed as a side-effect of code merged
from different branches.

Change-Id: I1d25e719605d75899253732bd3d10ca0b552d244

Merge commit '955abac95ae2f1b8cdb269c5ab8c53d93f2fa48f' into HEAD

Add missing LOG_TAG as the first argument to LOG_ERROR()

NOTE: This fix is needed as a side-effect of code merged
from different branches.

Change-Id: I1d25e719605d75899253732bd3d10ca0b552d244

Update the usage of fixed_queue.

 * Relax non-NULL requirements for fixed_queue by eliminating
   some of the asserts.
   Now, when semantically possible, fixed_queue - related function
   will return the appropriate value even if the queue pointer is NULL.
   This reduces clutter in the code where we had to do anyway
   "if (queue != NULL)" checks.

 * Add non-NULL guards in the few remaining places where
   fixed_queue_get_list(). For now, we need to use this function,
   hence the extra check.
   That function should be eliminated in the future, because all the
   code where it is used violates the semantics of using a queue.

Bug: 24723840
Change-Id: I47632a3515f3d27856d4870e18723d345c040d64

am 8f51b034: am 234138e2: service: Create a client library for applications

* commit '8f51b034342af28dc64806bd9742f53218c1a4e8':
  service: Create a client library for applications

am 234138e2: service: Create a client library for applications

* commit '234138e2606dd7a54fbcc540643511abc0a3598d':
  service: Create a client library for applications

resolved conflicts for 955abac9 to master

Change-Id: If3f90984c5ca906bab2fd6df7d77b4dfec47115e

am 9128e253: am 6c25b3c0: Raise BTU and HCI thread priorities

* commit '9128e25326fcd297c15c866e7d37b2b837006ea2':
  Raise BTU and HCI thread priorities

am 6c25b3c0: Raise BTU and HCI thread priorities

* commit '6c25b3c02c863cbb047e4cf8b4ec88d8335dac30':
  Raise BTU and HCI thread priorities

am 22e229cd: am ff651b7f: Allow parsing of UUID and service data for advertising

* commit '22e229cd94c1126e4145af0ffc5fe0a358df0bee':
  Allow parsing of UUID and service data for advertising

am ff651b7f: Allow parsing of UUID and service data for advertising

* commit 'ff651b7fd11daaa29732cb8735d248910d9248f9':
  Allow parsing of UUID and service data for advertising

service: Create a client library for applications

This CL creates the libbluetooth-client static library that external
code can link against to interact with the Bluetooth daemon using
Binder. To this end, all service/ code that is common to the client
library and the daemon itself have been moved to service/common/bluetooth
and service/common is exported in LOCAL_EXPORT_C_INCLUDE_DIRS.

bluetooth-cli and bt-example-hr-server have been update to demonstrate
how an app should use this library (using library includes).

Bug: 23169160
Change-Id: Ie6841bd6c48846eecec824ff1e80a9971385d1fc

Allow parsing of UUID and service data for advertising

Added code so that bluetooth advertisement packets will now
parse UUID data and service data. Also added unit tests to
confirm data is being parsed correctly. Also added constants for
the type fields for the different service UUID and service data
types.

Bug: 24247700

Change-Id: I53f5b639adb30e781ef155c8d55004547a26672d

Raise BTU and HCI thread priorities

The audio sub-system and audio related tasks run at elevated thread
priorities in order to ensure timely handling of audio packets. This
patch raises the thread priority of lower layer Bluetooth tasks involved
in delivering audio packets to avoid thread pre-emption and subsequent
audio skipping.

Bug: 24570959
Change-Id: I5e19ee4590207df753f7b8bfc22174959722ec2d

am 4ff8a3be: am 48d96ea3: service: Enable all tests for target

* commit '4ff8a3beac94ff1dbf40e9b3315a384ce0c86f48':
  service: Enable all tests for target

am 48d96ea3: service: Enable all tests for target

* commit '48d96ea34700455dde8d5baaa1ded2317d418cf7':
  service: Enable all tests for target

service: Enable all tests for target

Made some changes to the test structure following the instructions
at http://go/brillo/brillo-team/testing:

  - Renamed the existing host and target tests to bluetoothtbd-host_test
    and bluetoothtbd_test respectively.
  - The target suite now includes ALL daemon unit tests + binder tests.
  - Added custom test/main.cpp to set up logging and command-line. Now
    we can run tests with the -v flag to increase log verbosity level.

Bug: 24708686
Change-Id: Ia4ce5d7028ce6cb0efd4a1c51ffebdf3bbe6921e

am a81c51ff: am 9ba212b9: Merge "brillo: run bluetoothtbd with correct permissions"

* commit 'a81c51ffb0a21c1d4d2747f820d7395ff1cf25f0':
  brillo: run bluetoothtbd with correct permissions

am 9ba212b9: Merge "brillo: run bluetoothtbd with correct permissions"

* commit '9ba212b923bee11a6369dff4916e90d3839de899':
  brillo: run bluetoothtbd with correct permissions

Merge "brillo: run bluetoothtbd with correct permissions"

brillo: run bluetoothtbd with correct permissions

Adding net_bt_admin group grabs more correct permissions.

Change-Id: I3764e6d242fd79bb78cda620f4ec65f0b732aedd

GKI cleanup - removed PORT_SCHEDULE_LOCK / UNLOCK macros

Removed macros PORT_SCHEDULE_LOCK and PORT_SCHEDULE_UNLOCK
and replaced them with direct calls to mutex_global_lock()
and mutex_global_unlock() respectively.

Change-Id: I84a365feec239612b2224f029ae59f260cdb24a4

am f83d8fd5: am c40433d1: am 2846af52: service: Add missing build includes

* commit 'f83d8fd5039fbec127b6ab67f69417be5d6ac403':
  service: Add missing build includes

am c40433d1: am 2846af52: service: Add missing build includes

* commit 'c40433d1c4ea471e33c7f0a98e06ec0833b10fe6':
  service: Add missing build includes

am 2846af52: service: Add missing build includes

* commit '2846af5215215c12ec162b76075769f395e41434':
  service: Add missing build includes

service: Add missing build includes

Added missing files to service/Android.mk to fix a build breakage on
master.

Change-Id: Ic6a3e3511fa25cbc8f2c546e46a91f8665d2f654

GKI cleanup - moved GKI buffer allocation wrappers to OSI

* Moved the following GKI buffer allocation functions to OSI:
  - GKI_getbuf() -> osi_getbuf()
  - GKI_freebuf() -> osi_freebuf()
  - GKI_get_buf_size() -> osi_get_buf_size()

  For now we need the osi_getbuf() / osi_freebuf() allocation wrapper,
  because we need to be able to call osi_get_buf_size() on the allocated
  buffer.
  In the future those should be replaced with osi_malloc() / osi_free().
  Currently, the osi_malloc() buffer size internal allocation tracker
 does not always track the size, hence we need the osi_getbuf() wrapper.

* Replaced GKI_MAX_BUF_SIZE with BT_DEFAULT_BUFFER_SIZE

* Added new file include/bt_common.h that can be usee to include
  few files that should be included alost everywhere (e.g. bt_target.h"
  NOTE: This file might be removed in the future and we should include
  everywhere the right set of header files.

* Removed some of the GKI-related references

* Removed file include/gki_target.h

Change-Id: Ie87830e73143de200746d54235aa99f228a95024

Merge "GKI cleanup - Moved struct TIMER_LIST_ENT to OSI"

am d1e6587a: am e7d9c796: am 1c162dc3: service/example: Complete HR simulation

* commit 'd1e6587aed35c3277f4c55157760f1ab96078fbe':
  service/example: Complete HR simulation

GKI cleanup - Moved struct TIMER_LIST_ENT to OSI

* Moved struct TIMER_LIST_ENT to file osi/include/non_repeating_timer.h
  and renamed it to timer_entry_t
  NOTE: This is a short-term solution. timer_entry_t should be
  removed, and its usage everywhere should be replaced by
  struct non_repeating_timer_t .
* Renamed TIMER_CBACK to timer_callback_t
* Renamed TIMER_PARAM_TYPE to timer_param_t

Change-Id: I9ca830718bf900195f9c0a513a97f6995322693b

am e7d9c796: am 1c162dc3: service/example: Complete HR simulation

* commit 'e7d9c796ecde8a8da6d985931c2d2910ada5f57d':
  service/example: Complete HR simulation

am 1c162dc3: service/example: Complete HR simulation

* commit '1c162dc31bad03ee21a93e5afb9c7a572bf21ec7':
  service/example: Complete HR simulation

service/example: Complete HR simulation

Added the remaining logic for an example Heart Rate server app.
Also added more annotations throughout the code.

Bug: 24245347
Change-Id: I46184d2585eeb7683efdc5ea7122d0472dd7f05b

am fe3640ec: am 93b2956f: am cd644e3c: service: Support GattServer notifications/indications

* commit 'fe3640ecdac51519c4101d33a4b8701aabb7e836':
  service: Support GattServer notifications/indications

am 2b1f7707: am 8f8ab311: am 4ebcbd9e: service: Support GATT server async write requests.

* commit '2b1f77079f8dcb77074c9a8aaf489fb11604ab68':
  service: Support GATT server async write requests.