am 23453997: (-s ours) am 4c938b5d: am af16d116: am 91f01220: DO NOT MERGE trouble syncing with fitbit and can\'t turn off bluetooth.

* commit '23453997998d19ab3fd66ab5015fc218b3a1bba6':
  DO NOT MERGE trouble syncing with fitbit and can't turn off bluetooth.

am 4c938b5d: am af16d116: am 91f01220: DO NOT MERGE trouble syncing with fitbit and can\'t turn off bluetooth.

* commit '4c938b5d5addd36b5f66bfea93b9db3b0e10adf6':
  DO NOT MERGE trouble syncing with fitbit and can't turn off bluetooth.

am af16d116: am 91f01220: DO NOT MERGE trouble syncing with fitbit and can\'t turn off bluetooth.

* commit 'af16d11685e040b0cf332e729f8f4a0ab6ef0e16':
  DO NOT MERGE trouble syncing with fitbit and can't turn off bluetooth.

am 91f01220: DO NOT MERGE trouble syncing with fitbit and can\'t turn off bluetooth.

* commit '91f012208070d71e6e9e0f38aa36e78e99f2ff4e':
  DO NOT MERGE trouble syncing with fitbit and can't turn off bluetooth.

am 35ed9e1d: (-s ours) am be1d4db0: am 6ca1594d: am b74fe9e2: (-s ours) DO NOT MERGE LE: Add service UUIDs and data to advertising data

* commit '35ed9e1d5854396f8c60554d386f017cbd85cb1a':
  DO NOT MERGE LE: Add service UUIDs and data to advertising data

DO NOT MERGE trouble syncing with fitbit and can't turn off bluetooth.

this is to work around broadcom firmware problem to handle
unsolicited command complete event for HCI_LE_Create_Connection_Cancel
and LE connection complete event with status error code (0x2)
unknown connection identifier from bluetooth controller
the workaround is to release the HCI connection to avoid out of sync
with bluetooth controller, which cause BT can't be turned off.

bug:11614046
Change-Id: Ia5c5ad0c6c9b1a4f03253cac8a7caaf21341c88c

am be1d4db0: am 6ca1594d: am b74fe9e2: (-s ours) DO NOT MERGE LE: Add service UUIDs and data to advertising data

* commit 'be1d4db0862c8d9e40386dc5c9739c1803f25eca':
  DO NOT MERGE LE: Add service UUIDs and data to advertising data

am 6ca1594d: am b74fe9e2: (-s ours) DO NOT MERGE LE: Add service UUIDs and data to advertising data

* commit '6ca1594d1d633a94ea237c136460017151d1d61e':
  DO NOT MERGE LE: Add service UUIDs and data to advertising data

am b74fe9e2: (-s ours) DO NOT MERGE LE: Add service UUIDs and data to advertising data

* commit 'b74fe9e2c9fc0939f5c18282c985622f20b78d2a':
  DO NOT MERGE LE: Add service UUIDs and data to advertising data

am bac0e60f: am 303a1272: am f6918978: Add HID handshake_callback

* commit 'bac0e60f37d577e5b1a47ff52c8f2a96142ee4e7':
  Add HID handshake_callback

am 303a1272: am f6918978: Add HID handshake_callback

* commit '303a12727c96f6e39c31b0ef6a6017b5ef7972c0':
  Add HID handshake_callback

am f6918978: Add HID handshake_callback

* commit 'f691897800e086b6caf238ee78d560526e69c67d':
  Add HID handshake_callback

DO NOT MERGE LE: Add service UUIDs and data to advertising data

Add support for UUIDs and custom service data in the advertising and
scan response records.

Cherry picking ble advertising changes from master to KLP MR2.
b/13137996

Change-Id: I6470427ad194871488cc90f3ab384b9ecf326b15

Fix conditional waits in GKI_wait.

The absolute time calculation uses CLOCK_MONOTONIC, so
make sure pthread_condattr_setclock is called with
CLOCK_MONOTONIC.

Change-Id: I0f20871e36db511961b1f17ecd3225180e893db9

am 8dfc3e29: am 84516150: am be6d58f3: Fix GKI exception of calling free on an already freed buffer

* commit '8dfc3e2992456539ee6c740199f543cdb0d5a9d3':
  Fix GKI exception of calling free on an already freed buffer

am 84516150: am be6d58f3: Fix GKI exception of calling free on an already freed buffer

* commit '845161504098dcfdf8874f6ec592642f40b57547':
  Fix GKI exception of calling free on an already freed buffer

am be6d58f3: Fix GKI exception of calling free on an already freed buffer

* commit 'be6d58f35e66b9e57cf4978be68840cbc8ac2fc9':
  Fix GKI exception of calling free on an already freed buffer

am f6137714: am 3ff3d444: am f9c6502a: Change some printfs in sdp_api.c to SDP_TRACE_DEBUG

* commit 'f6137714438b3ed9e2a29d1ada94b2b531bb940d':
  Change some printfs in sdp_api.c to SDP_TRACE_DEBUG

am 3ff3d444: am f9c6502a: Change some printfs in sdp_api.c to SDP_TRACE_DEBUG

* commit '3ff3d4448da74b395a14c5b88a31747e4ee7c4e0':
  Change some printfs in sdp_api.c to SDP_TRACE_DEBUG

am f9c6502a: Change some printfs in sdp_api.c to SDP_TRACE_DEBUG

* commit 'f9c6502a109b0506916969fe016148bd6addf129':
  Change some printfs in sdp_api.c to SDP_TRACE_DEBUG

Add HID handshake_callback

It is called when the equivalent pkt is received from the HID
client as a normal response to set_report() and set_protocol_mode(),
and for errors for get_report() and get_protocol_mode().

Change-Id: I26ec37348e597f294f83907bbd256c88db5ac2ef
Signed-off-by: Mike J. Chen <mjchen@google.com>

Fix GKI exception of calling free on an already freed buffer

Various parts of btif_hh.c were creating GKI buffers and
keeping references to them and freeing them in odd and
unnecessary ways.  The buffer is freed by lower levels
of the stack once the buffer has been sent to the chip
at the l2c layer and shouldn't be freed by btif_hh itself
since it's possible to double free, and there could
also be race conditions with other threads already processing
the buffer while the reference is freed if the API calls
are invoked again before the previous invocation was completely
processed.

Also added a helper routine to simplify buffer creation and
initialization.

Change-Id: Ia6039983502e2670b2325d90310244edf843b692
Signed-off-by: Mike J. Chen <mjchen@google.com>

Change some printfs in sdp_api.c to SDP_TRACE_DEBUG

The printfs were going to console, which is generally not useful
in Android.

Change-Id: I3b08247c916dc084dfb004ac71fe234d9a541927
Signed-off-by: Mike J. Chen <mjchen@google.com>

am 33428009: am 780bc50d: am 509b2d73: Merge "TARGET_SIMULATOR is long dead."

* commit '3342800947fd6b3167e14c3858179b830dfa0644':
  TARGET_SIMULATOR is long dead.

am 780bc50d: am 509b2d73: Merge "TARGET_SIMULATOR is long dead."

* commit '780bc50ded9e26ad51db651d21e1a0d489300793':
  TARGET_SIMULATOR is long dead.

am 509b2d73: Merge "TARGET_SIMULATOR is long dead."

* commit '509b2d7358545945eb22dde2449d4f8d6e3b3b09':
  TARGET_SIMULATOR is long dead.

Merge "TARGET_SIMULATOR is long dead."

TARGET_SIMULATOR is long dead.

Change-Id: I6ba4a55c865370f12ae30f445661b54bbc714f8a

am de34aea5: am 5303f9d3: resolved conflicts for merge of 277ba466 to klp-modular-dev-plus-aosp

* commit 'de34aea544cb4222f24e4157b96962b09445e2c5':
  Various fixes for LP64

am 5303f9d3: resolved conflicts for merge of 277ba466 to klp-modular-dev-plus-aosp

* commit '5303f9d369946f43167b5e2423799d30c4455df1':
  Various fixes for LP64

resolved conflicts for merge of 277ba466 to klp-modular-dev-plus-aosp

Change-Id: Idc42f72a94c205cfe94feb220651a483cf418e84

Merge "Various fixes for LP64"

am 273071ca: am 6b096f36: am 0d702d25: Merge "Fix warning "suggest braces around empty body in an \'if\' statement""

* commit '273071ca5b91ca490a884ebe3b504085b9b3ca3d':
  Fix warning "suggest braces around empty body in an 'if' statement"

am 6b096f36: am 0d702d25: Merge "Fix warning "suggest braces around empty body in an \'if\' statement""

* commit '6b096f363c8f40bdf5108a2dc27e7f4e9a5ee9e1':
  Fix warning "suggest braces around empty body in an 'if' statement"

am 0d702d25: Merge "Fix warning "suggest braces around empty body in an \'if\' statement""

* commit '0d702d250777d385983b9f7c18467a764b29a7ed':
  Fix warning "suggest braces around empty body in an 'if' statement"

Various fixes for LP64

Changes include
- Cast numbers to their proper length before casting to a pointer.
- Use intptr_t when pointers are stored as integers.
- Eliminate 'implicit declaration' warnings

Signed-off-by: Matthew Leach <matthew.leach@arm.com>
Signed-off-by: Ashok Bhat <ashok.bhat@arm.com>
Signed-off-by: Marcus Oakland <marcus.oakland@arm.com>
Signed-off-by: Kévin PETIT <kevin.petit@arm.com>
Change-Id: I5664a087c5fbc322baa9178552404dbe9a992359

Merge "Fix warning "suggest braces around empty body in an 'if' statement""

Fix warning "suggest braces around empty body in an 'if' statement"

The body of an if statement could become empty if the macro body
is NULL and cause the if to apply to the next line instead.

Signed-off-by: Mike J. Chen <mjchen@google.com>
(cherry picked from dbdc69f4aed8a4eb3af31982c4722b10ee774740)

Change-Id: Ie4a18920e1fc959910451a70edcff6d7fd16e501

am e736740d: am 5834e63b: am 759280ee: Merge "bluetooth: hid: store ssr_max_latency and ssr_min_tout in bonding db" into klp-modular-dev

* commit 'e736740ded718c76c0007baa97b7f55e69ceb1b1':
  bluetooth: hid: store ssr_max_latency and ssr_min_tout in bonding db

am 5834e63b: am 759280ee: Merge "bluetooth: hid: store ssr_max_latency and ssr_min_tout in bonding db" into klp-modular-dev

* commit '5834e63bd410d708d8764f69ede256ca0ed98635':
  bluetooth: hid: store ssr_max_latency and ssr_min_tout in bonding db

am 759280ee: Merge "bluetooth: hid: store ssr_max_latency and ssr_min_tout in bonding db" into klp-modular-dev

* commit '759280ee1f92945ed7c1c13f5ff8a037fa3bf3e4':
  bluetooth: hid: store ssr_max_latency and ssr_min_tout in bonding db

Merge "bluetooth: hid: store ssr_max_latency and ssr_min_tout in bonding db" into klp-modular-dev

am 2ac65164: am 75c7597c: am 4fbbe711: Merge "support peripheral mode feature to connect iphone to android device." into klp-modular-dev

* commit '2ac6516450336e2ec5ccbbefa0221276ea291695':
  support peripheral mode feature to connect iphone to android device.

am 75c7597c: am 4fbbe711: Merge "support peripheral mode feature to connect iphone to android device." into klp-modular-dev

* commit '75c7597c94a7d8ee5c79b75ccaede02f53dd7c9f':
  support peripheral mode feature to connect iphone to android device.

am 4fbbe711: Merge "support peripheral mode feature to connect iphone to android device." into klp-modular-dev

* commit '4fbbe7115dd7c2f8b0df386fad4085f411beeef3':
  support peripheral mode feature to connect iphone to android device.

Merge "support peripheral mode feature to connect iphone to android device." into klp-modular-dev

support peripheral mode feature to connect iphone to android device.

enable BLE_PERIPHERAL_MODE_SUPPORT flag and fix the bug
the discoverability flag is not set in advertisement packet,
which cause the central device(iphone) can not find android device
during scan.
use displayOnly IO cap for dory, so dory can display the pin,
and the iphone(central device) will input the pin.
bug:12797572

Change-Id: Ie15ea9e340e3692df846399b9a85aa6d2a97eba7

am 7d006e6b: am a69e0ae1: am 47128cf9: Merge "Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL"

* commit '7d006e6b176172cb370cc183b4f2eab501e85c7c':
  Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL

am 9d368b68: am 1292dd82: am 6bc62a36: Merge "Bluetooth: GAP: Update bonded device info immediately on NVRAM"

* commit '9d368b68841dee6e6f581a3890433dcdc022fbb8':
  Bluetooth: GAP: Update bonded device info immediately on NVRAM

am a69e0ae1: am 47128cf9: Merge "Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL"

* commit 'a69e0ae1107e3148542a76ac840f3bcaa8ee2661':
  Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL

am 1292dd82: am 6bc62a36: Merge "Bluetooth: GAP: Update bonded device info immediately on NVRAM"

* commit '1292dd82812d76c54568796fbdafd1fdadc816eb':
  Bluetooth: GAP: Update bonded device info immediately on NVRAM

am 47128cf9: Merge "Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL"

* commit '47128cf95b72eedb7a6da29687b1ff6b668b59d6':
  Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL

am 6bc62a36: Merge "Bluetooth: GAP: Update bonded device info immediately on NVRAM"

* commit '6bc62a368da0a434a4e01f7bcb3273bd52a07568':
  Bluetooth: GAP: Update bonded device info immediately on NVRAM

Merge "Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL"

Merge "Bluetooth: GAP: Update bonded device info immediately on NVRAM"

Merge "Bluetooth: Cleanup bt_config.xml during BLE Scan"

Bluetooth: Cleanup bt_config.xml during BLE Scan

BLE Scan needs to clean up the bt_config.xml periodically so that
new entries could be added to bt_config.xml. Also, there is no
need to add BLE Random bd_addr to the bt_config.xml

Change-Id: I00ba2e50dd39a021099052cb569786f6cb2cc8b9

b/12912008

Bluetooth: fix sniff issue with multiple rfcomm connection in same ACL

When there are multiple rfcomm connection with same appid and peer bd
address, we were clearing jv pm control block in addition to closing
connection to service with registered app id. Since appid is common
across all the client and server app registed to jv. we were clearing
entire connection to service for this appid and peer bdadres which was
corrupting sniff logic, when another rfcomm connection exists with same
peer bdaddress. So avoiding closing of connection to the service when
multiple rfcomm connection exists in same ACL.

Change-Id: Id71ef5f7f0e00f1bda3b70a4826c31bef46caafc

Bluetooth: GAP: Update bonded device info immediately on NVRAM

As the new bt-if config updation have delay in writing items
to the NVRAM, some times there is a chance of not updating
incase of battery removal happens before the actual write onto
NVRAM.

As the bonded device info is important data to store, it will
update bonded info immediately onto NVRAM.

Change-Id: I9699c8225a77d9272420334b4c76b3c1df87e459

am 0c880603: am 6a342588: am 136e4b1c: Merge "Don\'t send a dummy 0 HID event when connection is closed" into klp-modular-dev

* commit '0c880603dba9f9f09523509f19c6808ba8f52e12':
  Don't send a dummy 0 HID event when connection is closed

am 6a342588: am 136e4b1c: Merge "Don\'t send a dummy 0 HID event when connection is closed" into klp-modular-dev

* commit '6a342588e9a161ec5347035df7472f1f4a8e86ea':
  Don't send a dummy 0 HID event when connection is closed

am 136e4b1c: Merge "Don\'t send a dummy 0 HID event when connection is closed" into klp-modular-dev

* commit '136e4b1c73af97b3f743395da47f137356776090':
  Don't send a dummy 0 HID event when connection is closed

Merge "Don't send a dummy 0 HID event when connection is closed" into klp-modular-dev

Don't send a dummy 0 HID event when connection is closed

This was put there by Broadcom to try to handle held keys
on disconnect, but the best place to do that is in InputManager.
There's no guarantee that a report_id 1 is keyboard, and
in particular it causes game controllers to get a false
joystick move event to 0,0.

Bug 12785130

Change-Id: I85fbafa44d927082def3848132344550e8b94bb0
Signed-off-by: Mike J. Chen <mjchen@google.com>

am 081ff423: am aa58a6cf: am 13a01401: Merge "Don\'t leak process-private data on Change Local Name command."

* commit '081ff423d4b55098adcdb4d1e7786fb719b410bd':
  Don't leak process-private data on Change Local Name command.

am aa58a6cf: am 13a01401: Merge "Don\'t leak process-private data on Change Local Name command."

* commit 'aa58a6cffbdeed8381d309a4dcb093c42b29fd84':
  Don't leak process-private data on Change Local Name command.

am 13a01401: Merge "Don\'t leak process-private data on Change Local Name command."

* commit '13a01401b8caaa28c407fe21726adc020cfb6169':
  Don't leak process-private data on Change Local Name command.

am dd7e09cf: am 17e40430: am fab294ca: Merge "Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define." into klp-modular-dev

* commit 'dd7e09cf861b0da60a09dfabb665d4001d540bdf':
  Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define.

am 17e40430: am fab294ca: Merge "Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define." into klp-modular-dev

* commit '17e4043072c6860a58c8da57a6e93685bd6bd14a':
  Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define.

am fab294ca: Merge "Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define." into klp-modular-dev

* commit 'fab294ca83b62c8e258f2bcd91018c31daf377bf':
  Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define.

Merge "Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define." into klp-modular-dev

am a89616ab: Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define.

* commit 'a89616ab221841dc6f053672e5e1226201dcf209':
  Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define.

Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define.

The bug caused GATT_INCLUDED to be FALSE if not already defined.

Looks like the bluedroid codebase really can't support no GATT or SMP
if BLE_INCLUDED is true.  Lots of compiler failures (not just warnings but
real failures) if GATT or SMP are not set to TRUE when BLE_INCLUDED is true.

Not cleaning that up at this time but just fixing basic build breakage.

Change-Id: I561cd4dcb2b39e90158ce74e55dca294f55cb540
Signed-off-by: Mike J. Chen <mjchen@google.com>

Fix bug introduced in last change.  GATT define was after BLE_INCLUDED define.

The bug caused GATT_INCLUDED to be FALSE if not already defined.

Looks like the bluedroid codebase really can't support no GATT or SMP
if BLE_INCLUDED is true.  Lots of compiler failures (not just warnings but
real failures) if GATT or SMP are not set to TRUE when BLE_INCLUDED is true.

Not cleaning that up at this time but just fixing basic build breakage.

Change-Id: I561cd4dcb2b39e90158ce74e55dca294f55cb540
Signed-off-by: Mike J. Chen <mjchen@google.com>

am e80e4df1: am b45e54f0: Merge "Fix warnings when BLE_INCLUDED is FALSE" into klp-modular-dev

* commit 'e80e4df1f9b9b60be37ef8ad854d7b46b9dbfb83':
  Fix warnings when BLE_INCLUDED is FALSE

am d382270c: am ef04b8a2: Merge "Set some SMP and GATT to FALSE if BLE is FALSE" into klp-modular-dev

* commit 'd382270cede9188b3cd493cfd9fac093a5efdc9a':
  Set some SMP and GATT to FALSE if BLE is FALSE

am b45e54f0: Merge "Fix warnings when BLE_INCLUDED is FALSE" into klp-modular-dev

* commit 'b45e54f09e5496f298bb5c36354dff8ae27925f2':
  Fix warnings when BLE_INCLUDED is FALSE

am ef04b8a2: Merge "Set some SMP and GATT to FALSE if BLE is FALSE" into klp-modular-dev

* commit 'ef04b8a221df70ac205184c1f5098c409306556d':
  Set some SMP and GATT to FALSE if BLE is FALSE

Merge "Fix warnings when BLE_INCLUDED is FALSE" into klp-modular-dev

Merge "Set some SMP and GATT to FALSE if BLE is FALSE" into klp-modular-dev

Merge "Don't leak process-private data on Change Local Name command."

Don't leak process-private data on Change Local Name command.

The Change Local Name command to the controller doesn't zero out
the contents of the command buffer before setting the contents. As
a result, the controller sees both the new device name and then
random data left over on the heap from previous operations. This
random data may contain sensitive information. Although the device
name is null-terminated, a controller may inadvertently leak the
additional information over the network.

This change fixes the behaviour by zeroing out the entire command
buffer before copying the device name into it. If the device name
doesn't occupy the entire buffer, the rest of it will be zero.

http://b/12949068

Change-Id: I4dd6ce6dd6709589fcd1e23f974db820820bc8e1

Fix warnings when BLE_INCLUDED is FALSE

Change-Id: I27c8ac6d5410159ed6bbef6aca5bf3b9692b0467
Signed-off-by: Mike J. Chen <mjchen@google.com>

Set some SMP and GATT to FALSE if BLE is FALSE

Also add asserts that SMP and GATT are never TRUE
if BLE is FALSE since that would make no sense.

Change-Id: I6530fe0255da0b237daa82b1b0dff286f2c5225d
Signed-off-by: Mike J. Chen <mjchen@google.com>

am 04276619: am 384d6211: Merge "Revert "Make all warnings fatal compilation errors"" into klp-modular-dev

* commit '04276619fc4286c2b2926adfa5a5f87ab5065ee5':
  Revert "Make all warnings fatal compilation errors"

am 384d6211: Merge "Revert "Make all warnings fatal compilation errors"" into klp-modular-dev

* commit '384d62118595b81d081af83d68c7c1de53229768':
  Revert "Make all warnings fatal compilation errors"

Merge "Revert "Make all warnings fatal compilation errors"" into klp-modular-dev

Revert "Make all warnings fatal compilation errors"

This reverts commit ba1ee422cee82449e675a52a785dac1a9200a3df.

When BLE_INCLUDED is FALSE, we still have a bunch of warnings so
revert until that path is cleaned up.

am 3d4f5c71: am b83a8234: Merge changes Ifc373f95,I95eb887a,I1e6be2ab,Ia6ea939a,I147d0138, ... into klp-modular-dev

* commit '3d4f5c715204c3c5eeb21e351aff73072000f61e': (24 commits)
  Make all warnings fatal compilation errors
  Major warnings cleanup
  Add macro UNUSED() to bt_utils.h
  Fix unused parameter warning in static function
  Add "static" to a static function
  Fix a number of unused parameter warnings in static functions
  Fix bug with wrong parameter passed to logu
  Default send_ind_evt should be FALSE
  Fix warning "implicit declaration of function 'GAP_BleReadPeerPrefConnParams'"
  Fix unused parameter warnings in static functions by changing prototype
  Fix some unused parameter warnings in static functions
  Fix warnings about implicit declaration of bdcmp and bdcpy
  Fix warning "comparison is always true due to limited range of data type"
  Fix warning "suggest braces around empty body in an 'if' statement"
  Fix warning "comparison is always true due to limited range of data type"
  Add a function prototype for btm_ble_test_command_complete
  Change argument to return ptr rather than assign to unused local
  Fix warning "comparison is always true due to limited range of data type"
  Fix warnings about implicit declarations
  Fix warning "implicit declaration of function 'bdcmp'"
  ...

am b83a8234: Merge changes Ifc373f95,I95eb887a,I1e6be2ab,Ia6ea939a,I147d0138, ... into klp-modular-dev

* commit 'b83a823466bf93f7d484f7c8d23513e0e7c1a5a8': (24 commits)
  Make all warnings fatal compilation errors
  Major warnings cleanup
  Add macro UNUSED() to bt_utils.h
  Fix unused parameter warning in static function
  Add "static" to a static function
  Fix a number of unused parameter warnings in static functions
  Fix bug with wrong parameter passed to logu
  Default send_ind_evt should be FALSE
  Fix warning "implicit declaration of function 'GAP_BleReadPeerPrefConnParams'"
  Fix unused parameter warnings in static functions by changing prototype
  Fix some unused parameter warnings in static functions
  Fix warnings about implicit declaration of bdcmp and bdcpy
  Fix warning "comparison is always true due to limited range of data type"
  Fix warning "suggest braces around empty body in an 'if' statement"
  Fix warning "comparison is always true due to limited range of data type"
  Add a function prototype for btm_ble_test_command_complete
  Change argument to return ptr rather than assign to unused local
  Fix warning "comparison is always true due to limited range of data type"
  Fix warnings about implicit declarations
  Fix warning "implicit declaration of function 'bdcmp'"
  ...

Merge changes Ifc373f95,I95eb887a,I1e6be2ab,Ia6ea939a,I147d0138, ... into klp-modular-dev

* changes:
  Make all warnings fatal compilation errors
  Major warnings cleanup
  Add macro UNUSED() to bt_utils.h
  Fix unused parameter warning in static function
  Add "static" to a static function
  Fix a number of unused parameter warnings in static functions
  Fix bug with wrong parameter passed to logu
  Default send_ind_evt should be FALSE
  Fix warning "implicit declaration of function 'GAP_BleReadPeerPrefConnParams'"
  Fix unused parameter warnings in static functions by changing prototype
  Fix some unused parameter warnings in static functions
  Fix warnings about implicit declaration of bdcmp and bdcpy
  Fix warning "comparison is always true due to limited range of data type"
  Fix warning "suggest braces around empty body in an 'if' statement"
  Fix warning "comparison is always true due to limited range of data type"
  Add a function prototype for btm_ble_test_command_complete
  Change argument to return ptr rather than assign to unused local
  Fix warning "comparison is always true due to limited range of data type"
  Fix warnings about implicit declarations
  Fix warning "implicit declaration of function 'bdcmp'"
  Fix warning "case label value exceeds maximum value for type"
  Fix missing initializer warning
  Fix missing initializer warning
  Fix warning about missing initializer

Make all warnings fatal compilation errors

Change-Id: Ifc373f954c8019b1c52d0b1b3636362afb99fff4
Signed-off-by: Mike J. Chen <mjchen@google.com>

Major warnings cleanup

Mostly fixing unused parameter warnings.
A few other warnings also fixed like possible use of
unitialized variables (no real issue found, just compiler couldn't
follow the path), signed vs unsigned warning.

Also fixed some typos, indent issues, removal of dead code, etc.

Change-Id: I95eb887aefc4d559d7921f71a0af5f3bfb01ac01
Signed-off-by: Mike J. Chen <mjchen@google.com>

Add macro UNUSED() to bt_utils.h

Also include bt_utils.h in some other header files commonly
included by some subsystems.

Change-Id: I1e6be2abae503c4303c92d97eeeb29981beacd10
Signed-off-by: Mike J. Chen <mjchen@google.com>

Fix unused parameter warning in static function

Just changed the function to not take the unused parameter

Change-Id: Ia6ea939a45583ad9b8ae83125a0294086fde8909
Signed-off-by: Mike J. Chen <mjchen@google.com>

Add "static" to a static function

Change-Id: I147d0138f2871d1fa873184b1e6e8075328a8501
Signed-off-by: Mike J. Chen <mjchen@google.com>

Fix a number of unused parameter warnings in static functions

Since functions were static, just changed them to not take
the unused parameters.

Change-Id: I2f39684d84b1785d60167e617bc5fa79aeac4dda
Signed-off-by: Mike J. Chen <mjchen@google.com>

Fix bug with wrong parameter passed to logu

Caught by compiler warning "passing argument 2 of 'logu' from incompatible pointer type".

Change-Id: If6047654c9862209d3426f832a366ac884b2c9e9
Signed-off-by: Mike J. Chen <mjchen@google.com>

Default send_ind_evt should be FALSE

Bug found when looking at compiler warning about p_mcb might
be used before being initialized.

Change-Id: I6f74a3c9bb46e757d2420cc871bfe78e51256c3b
Signed-off-by: Mike J. Chen <mjchen@google.com>

Fix warning "implicit declaration of function 'GAP_BleReadPeerPrefConnParams'"

Change-Id: I0e4e45d6f0ade8deb070e610baac2d1945ebc1a5
Signed-off-by: Mike J. Chen <mjchen@google.com>