Revert "Fix data reporting state when using HID device"
am: 763930636d

* commit '763930636d0af0123fe5728a504ec5e625d638a5':
  Revert "Fix data reporting state when using HID device"

Revert "Fix data reporting state when using HID device"

Original commit breaks Fugu remote (amongst other things).

This reverts commit 44efbb13a748a276de994456cffc02e8e9da03b0.

Change-Id: Ifa4ab421e0b14f59988d2e3ac7f338f0844aaa2d

Write Protobuf in Base64
am: e064f0846d

* commit 'e064f0846d382180a7955b7e45fcb73fa13ff992':
  Write Protobuf in Base64

Fix ParcelHelpersTest.ScanResult unit test
am: 512fabd174

* commit '512fabd174f38ba423bd057a3687623ea3b9deb4':
  Fix ParcelHelpersTest.ScanResult unit test

Add support for collecting A2DP Protobuf metrics
am: 4e6b1c82b7

* commit '4e6b1c82b78aa17c887376c14cf323bed02e1fd4':
  Add support for collecting A2DP Protobuf metrics

Write Protobuf in Base64

Change-Id: I3868365e7fd166517b13ff03a15d629271bd47b4

Fix ParcelHelpersTest.ScanResult unit test

Fix a failing unit test that was broken by some
earlier cleanup work.

Bug: 26879229
Bug: 27152031
Change-Id: I244d078a23a69c82fbc7cb4468649ab933131977

Add support for collecting A2DP Protobuf metrics

* Add new function metrics_a2dp_session() to
  connect A2DP metrics into Protobuf

* Add new function btif_update_a2dp_metrics()
  that is called when the metrics should be
  callected

* Add few more statistics to the BTIF media stats.

Change-Id: I5862ac82e55b3546dc1fbde0c78cb4683e40d19e

Instrument and fix calls to strncmp()
am: df5c69caa3

* commit 'df5c69caa3c2683779d48739e1571c13e5f7ab2e':
  Instrument and fix calls to strncmp()

Instrument and fix calls to strncmp()

Bug: 27222508
Change-Id: I0075b4480aff52dcdc326b7bdb7154ca919ccee9

Instrument and fix calls to strncmp()
am: 0e888661a5  -s ours

* commit '0e888661a5dc23e516429b3e80f1f6a1aac12015':
  Instrument and fix calls to strncmp()

Suspend properly when receiving events without response
am: b1f50e1ff3

* commit 'b1f50e1ff32b58437ff1c02e541447319baa12df':
  Suspend properly when receiving events without response

Additional logging for bond information
am: a72c96f8db

* commit 'a72c96f8dbfcb4db1e45c4954fc96d3da93a4008':
  Additional logging for bond information

Fix data reporting state when using HID device
am: 44efbb13a7

* commit '44efbb13a748a276de994456cffc02e8e9da03b0':
  Fix data reporting state when using HID device

service: Use built-in byte vector parser
am: b6806d0fd5

* commit 'b6806d0fd5232d3eae4ffa928162bd04351e3807':
  service: Use built-in byte vector parser

Remove the need for -Wno-constant-logical-operand
am: 302bae5e92

* commit '302bae5e924ec33768a21bc92dcb6930df50659b':
  Remove the need for -Wno-constant-logical-operand

Remove Porsche car-kit pairing workaround
am: 8ee098cd6b

* commit '8ee098cd6bca17fc016dd3cc519dc5990be22429':
  Remove Porsche car-kit pairing workaround

Fix compile error due to interop_feature_t not being public
am: dc0b89cc7f

* commit 'dc0b89cc7f7a8685045d4ebec4a948385d42777f':
  Fix compile error due to interop_feature_t not being public

Add new HAL functions to fake Bluetooth interface
am: f79d9cb591

* commit 'f79d9cb5918a80e4b5f96dfe7ceddd8a2cb709a4':
  Add new HAL functions to fake Bluetooth interface

Add ability to add interop entries dynamically (2/2)
am: a0c02f62a4  -s ours

* commit 'a0c02f62a49c4ae9de5789731cdbb99b686a938d':
  Add ability to add interop entries dynamically (2/2)

Fix compilation warnings/errors on fugu
am: 3aa58c5fad

* commit '3aa58c5fad5be75c559109ddce3ccd79ac807af9':
  Fix compilation warnings/errors on fugu

Fix C/C++ compiler generated warnings
am: 12265e39b5  -s ours

* commit '12265e39b511f28425236b0dc97c78e7d837304b':
  Fix C/C++ compiler generated warnings

service: Add services removed/added handlers
am: 0bd8fa5b24

* commit '0bd8fa5b24103f5376afb4ddd593ecb1a6a2588c':
  service: Add services removed/added handlers

Cleanup C and C++ compiler flags
am: 0b60bb04b9  -s ours

* commit '0b60bb04b966030fd1dfdffe68099da8f280ade0':
  Cleanup C and C++ compiler flags

Pick proper addres type for unknown devices
am: 0ec558b58d

* commit '0ec558b58d55eb05f2b42bc29a6c1c93786ac090':
  Pick proper addres type for unknown devices

Fix discovery status handling
am: 0e36a4d161

* commit '0e36a4d1617da1c5ec713c85a5bef4f6af89148e':
  Fix discovery status handling

Validates the PDU length of GATT Prepare Write attribute requests
am: 4c1e089a33

* commit '4c1e089a33a0e8d7e030007a79cf7523ef19f1d5':
  Validates the PDU length of GATT Prepare Write attribute requests

Create backup for bt_config.conf
am: f621aa868f

* commit 'f621aa868f2d980d7a8a1afaa6d70dcf42866bc5':
  Create backup for bt_config.conf

Removed function bte_load_ble_conf
am: d0e384c346

* commit 'd0e384c3463fd30cccedd805fe957846f0f7246f':
  Removed function bte_load_ble_conf

Add handle range to BTA_GATTC_GetGattDb
am: 12339a8011

* commit '12339a801146d45f8aab010b38c099082bfc80e5':
  Add handle range to BTA_GATTC_GetGattDb

Removed checks for NULL returns after osi_calloc() / osi_malloc()
am: 5fe6f0cf6b  -s ours

* commit '5fe6f0cf6b223e3ed6be4912d55b3ed5b41ce0cd':
  Removed checks for NULL returns after osi_calloc() / osi_malloc()

osi: Don\'t use -std=c99 to compile
am: 75be8ac297  -s ours

* commit '75be8ac2970284b8dbd7cdb2323bd3cf80f9c6ff':
  osi: Don't use -std=c99 to compile

Remove code related to legacy config file
am: 37f7f8380f  -s ours

* commit '37f7f8380f91221f5f49248ee09be927434b0875':
  Remove code related to legacy config file

Add Privacy 1.0/1.1 check for BG connection
am: 7dc23e85a6

* commit '7dc23e85a659853c4b2eadb6d179232459c9e123':
  Add Privacy 1.0/1.1 check for BG connection

binder: fix possible stack corruption
am: be4ae78ad9  -s ours

* commit 'be4ae78ad94311389227b1788cf11d36c81d6d22':
  binder: fix possible stack corruption

Fix possible memory leak
am: 4bdc3535a7

* commit '4bdc3535a788dc708a1b6c364abf1ee594087494':
  Fix possible memory leak

Add Developer menu entry to disable Bluetooth absolute volume
am: 90b71eac9b

* commit '90b71eac9b45f90ff3267fa0a400cc1daaec88d4':
  Add Developer menu entry to disable Bluetooth absolute volume

Replaced osi_getbuf()/osi_freebuf() with osi_malloc()/osi_free()
am: abd70abb5e  -s ours

* commit 'abd70abb5e42c9431df94fe9d2c4a78a0d8d9af9':
  Replaced osi_getbuf()/osi_freebuf() with osi_malloc()/osi_free()

Use dynamic memory for keeping GATT cache
am: 9df3192b94  -s ours

* commit '9df3192b94a186207976ecc579d390f3cfd547c6':
  Use dynamic memory for keeping GATT cache
  Add missing "\" at end of line

Removed function osi_get_buf_size()
am: ca57b25df8  -s ours

* commit 'ca57b25df827286c4996e17aefc251ecd79b33fc':
  Removed function osi_get_buf_size()

Disable gnu-variable-sized-type-not-at-end warning
am: 74c3783d72  -s ours

* commit '74c3783d729292ed6111688d02cf2af4fd2867e0':
  Disable gnu-variable-sized-type-not-at-end warning

Add missing log tags
am: a41dbb1560  -s ours

* commit 'a41dbb156068a33e7cccafe93c4b520406781c9f':
  Add missing log tags

Remove BCM_STR* macros
am: 66209db61f  -s ours

* commit '66209db61f94128f19ec09891ed99b1d10e4ebfb':
  Remove BCM_STR* macros

Add list_back_node method to osi list
am: 0a03a42c20

* commit '0a03a42c20ebe33ce00a55154ac0931a9b010544':
  Add list_back_node method to osi list

Fix auto disconnect right after connecting
am: 7adaa5a3f3

* commit '7adaa5a3f33476bf3442ef948b037ea93136ff04':
  Fix auto disconnect right after connecting

Set GATT link inactivity timer to 1
am: 5c52fece85

* commit '5c52fece85240dae419ac58227afca8fe9539cae':
  Set GATT link inactivity timer to 1

Instrument and fix calls to strncmp()

Bug: 27222508
Change-Id: I0075b4480aff52dcdc326b7bdb7154ca919ccee9

Suspend properly when receiving events without response

In the BTU handler for messages from the controller, rn event may be
received that the stack does not have to respond to. In this case, the
stack should go to sleep.

This patch calls the provided function in the BTU to enter sleep.

Signed-off-by: Jaganath Kanakkassery <jaganathx.kanakkassery@intel.com>
Signed-off-by: Yingkai Cheng <yingkaix.cheng@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>
Change-Id: Ie7c3fcae418eb2480338509e86b7cc870177a965

Additional logging for bond information

Log the bond state, the funciton that updated the bond state, as well as the
device that was updated. Also added in functionality for protobuff logging.

Example output
Bond Events:
  Total Number of events: 8
  Time          BD_ADDR            Function             State
  00:01:12.353  ac:9e:17:07:58:d8  btif_dm_create_bond  BOND_STATE_NONE
  00:01:12.354  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDING
  00:01:13.999  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDING
  00:01:14.778  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDED
  00:01:14.827  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDED
  00:01:31.534  34:36:3b:c4:bf:99  btif_dm_create_bond  BOND_STATE_NONE
  00:01:31.535  34:36:3b:c4:bf:99  bond_state_changed   BOND_STATE_BONDING
  00:01:32.771  34:36:3b:c4:bf:99  bond_state_changed   BOND_STATE_BONDING

Change-Id: I01ab3a232300203704a356a17aa045e638f97463

Fix data reporting state when using HID device

Start and Stop are called on HID driver probe/unprobe.
Open/Close are called when driver is interesed/not-interested
in data reports, typically when user-space open/close the input
device. So we can have several open/close calls for one start/stop
cycle.

https://www.kernel.org/doc/Documentation/hid/hid-transport.txt

Problem is that current implementation disables data reporting on close
but never enables it on open. This patch fixes this issue to match the
expected behavior.

Change-Id: Id5d4627dba389338b06df60a6e1d993ac3e3e5ec
Signed-off-by: Loic Poulain <loic.poulain@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

service: Use built-in byte vector parser

Change-Id: I72369c0b7678338fc8ccf520b59a95affdfd2994

Remove the need for -Wno-constant-logical-operand

Change-Id: Iea44eeed487df3236d9b482b7e5bbc92f7f46797

Remove Porsche car-kit pairing workaround

Bug: 26551752
Change-Id: I14c5e3fcda0849874c8a94e48aeb7d09585617e1

Fix compile error due to interop_feature_t not being public

Change-Id: I3c6d2d28d98098aea1854ebd2726fdaac9cc14e9

Add new HAL functions to fake Bluetooth interface

Change-Id: I5b2b5a495ea14fd743885c4061a6bcd3e7c39c23

Add ability to add interop entries dynamically (2/2)

Add ability and interface for adding dynamic entries to the interop
workaround database.

Bug: 26548845
Change-Id: Id886e4233fc1548727e79f1581cdc0c6f0738d59

Fix compilation warnings/errors on fugu

Fix compilation warnings/errors on fugu about unused
variables.

Change-Id: Ib68097122f27c631ab36b71a6815d4e1f3b60558

Fix C/C++ compiler generated warnings

 * Remove unused functions: -Wno-unused-function
 * Fix logical operations that are missing parentheses:
   -Wno-logical-not-parentheses, -Wno-parentheses
 * Add missing braces in initializers: -Wno-missing-braces
 * Remove unused variables: -Wno-unused-variable
 * Fix printf()-like formatting issues: -Wno-format
 * Avoid using unitialized variables:
   -Wno-maybe-uninitialized, -Wno-uninitialized
 * Fix a bug inside bta_gattc_get_gatt_db_impl()
   when the processed GATT DB attribute type is unknown.
 * Fix warnings about missing field initializers:
   -Wno-missing-field-initializers
 * Re-implement macro COMPILE_ASSERT(COND) to fix a compilation warning:
   -Wno-non-literal-null-conversion
 * Fix sign mismatch comparison warnings:
   -Wno-sign-compare
 * Fix warnings related to enum conversion:
   -Wno-enum-conversion
 * Fix warnings related to incompatible pointer types:
   -Wno-incompatible-pointer-types
 * Fix warnings related to enum conversion:
   -Wno-enum-conversion

Bug: 26879229
Change-Id: I522931fe156aeab23ae841051a9e25ceab00b1c2

service: Add services removed/added handlers

Change-Id: Ida3f232b4e77bfb95dec78ea4af7677c9c12b411

Cleanup C and C++ compiler flags

Cleanup the setting of C and C++ compiler flags:
 * (Almost) all compiler flags are set uniformly
   in the system/bt/Android.mk file.
 * Enable by default breaking the compilation if there is a
   compilation warning: -Werror
 * Enable most compilation warnings: -Wall -Wextra
 * Renamed Android.mk related flags:
   - bdroid_C_INCLUDES -> bluetooth_C_INCLUDES
   - bdroid_CFLAGS -> bluetooth_CFLAGS
 * Introduce variables for C-only and C++ only compiler:
   - bluetooth_CFLAGS: common C and C++ compiler flags
   - bluetooth_CONLYFLAGS: C only compiler flags
   - bluetooth_CPPFLAGS: C++ only compiler flags
 * Disable warnings for existing issues - to be removed as issues are
   resolved
 * Add a workaround for libchrome and -DNDEBUG usage.

Bug: 26879229
Change-Id: Ie7595965ca0c8ead0e95e983e76c327e7891b2c3

Pick proper addres type for unknown devices

Change-Id: I46641d0fe065b5e5a6620dc34c770429ac8dbe11

Fix discovery status handling

This typo was introduced in commit 48db2d25 3years ago.
This cause unnecesary purge of cache and rediscovery even though
discovery was successfull.

Change-Id: I828e148ad489af1e231d33d7807bbe0e46f445c5

Validates the PDU length of GATT Prepare Write attribute requests

Validates whether the PDU length of GATT Prepare Write
requests satisfy the minimum length requirements of
GATT write attribute request format.

Change-Id: Id18d32b050389ff8c71341bb738dd77225029acd

Create backup for bt_config.conf

Move current config file to a backup when a new config file is
saved. This backup can be used in case there is an error loading
the current config file.

Change-Id: Icc210c457dc71a72fc2e82ff72c91207b25deaf1

Removed function bte_load_ble_conf

The functionality provided by this function is not needed anymore.

Change-Id: I59696c7a7e1d7ddb095f3a450f58ce1db97927b6

Add handle range to BTA_GATTC_GetGattDb

Change-Id: Iaf0280853278a4f28dde5b952b646096cfb12d97

Removed checks for NULL returns after osi_calloc() / osi_malloc()

Removed explicit checks for NULL pointer returns after calls
to osi_calloc() and osi_malloc(), because those are not needed.
If the memory allocation fails, osi_calloc() and osi_malloc()
will trigger an assert.

Bug: 27048759
Change-Id: I2791eb2f69c08f991f8fcdef10e101a41568cd95

osi: Don't use -std=c99 to compile

Using -std=c99 to compile breaks when we are using C++ files, which
metrics uses for protobufs.

Bug: 27077250
Change-Id: Ie94e3ea5d8b219671329108e7795cdc02ecc38b4

Remove code related to legacy config file

Change-Id: Ia82d9bbffcc5df1ec4cddff28d812ca221505ab9

Add Privacy 1.0/1.1 check for BG connection

Check for privacy 1.0 and 1.1 controller and do not start
background connection, if RPA offloading is not supported,
since it will not connect after change of random address

Bug: 22102400
Change-Id: Ida618f8dc70e0a3860f76f856bb88de8b55b90d7

binder: fix possible stack corruption

The stack could be corrupted by crafting a IPC call in interesting ways
when a character buffer was passed.

This patch also removes code duplication where these would occur.

Bug: 26917241
Change-Id: Ib6c149a293abf01f31c69a94c8f6dd91d8a2fff2

Fix possible memory leak

Fix possible memory leak when allocating reused server cache control
block.

Change-Id: I3aab727e4f60e55530f49b0b05ccf2c891c72760

Add Developer menu entry to disable Bluetooth absolute volume

Bug: 27078729
Change-Id: I24ac605c6d02dc3fe3904c441ec416cb66b63168

Replaced osi_getbuf()/osi_freebuf() with osi_malloc()/osi_free()

Removed the alternative buffer allocation osi_getbuf() / osi_freebuf()
and use instead osi_malloc() / osi_free().

Correspondingly, replaced usage of osi_freebuf_and_reset()
with osi_free_and_reset().

Bug: 24914560
Change-Id: I7a9599ba7fa900321f087da684428133eb0ddd6b

Use dynamic memory for keeping GATT cache

Change-Id: I60674c47246d8fe2094fe78ce31b752fa2acb89e

Add missing "\" at end of line

Change-Id: I6b8a275769387cc731f4bf5ec9ec4d10700fc0ba

Removed function osi_get_buf_size()

Refactored code that uses function osi_get_buf_size(), and removed
the need for that function.

Bug: 24914560
Change-Id: I0d002635024a9703acb78f47735aafc957a2b761

Disable gnu-variable-sized-type-not-at-end warning

This warning is triggered unnecessary in too many places.
It spills through the build log and cause very useful warnings
to stay unnoticed.

Change-Id: Iee75327d05cbece4b9dc9c13005a9bfdf94c0d56

Add missing log tags

When building for Linux, missing log tags cause errors.

Change-Id: I0b3cda3333c059251b00a25bbfc7e764057286be

Remove BCM_STR* macros

This also fixes very incosistent null termination and various string
length issues.

Bug: 27069905
Change-Id: I527256d224ac5b18e3943d6038b9c912b247676f

Add list_back_node method to osi list

Change-Id: I919ce97373701cbdea03b8228b3a90263d7ef180

Fix auto disconnect right after connecting

Bug: 27207638
Change-Id: Ia36cd898c21058b9a3ed56a0d300c3ea8384a5cf

Set GATT link inactivity timer to 1

GATT_LINK_IDLE_TIMEOUT_WHEN_NO_APP timer was zero. Due to this zero
inactivity timer after the authentication is complete the link is brought
down immediately. Set it to 1 instead.

Bug: 22100369
Change-Id: I6ba4f63b9e48759e4255f230d6bbf426eda31c6f

Write Protobuf in Base64
am: 1d00dd26c3

* commit '1d00dd26c35590d6fff06555a53cd8c7acffeb67':
  Write Protobuf in Base64

Write Protobuf in Base64

Change-Id: I3868365e7fd166517b13ff03a15d629271bd47b4

Fix ParcelHelpersTest.ScanResult unit test
am: db49d08101

* commit 'db49d08101d414f8e3ebe646e83438c9899e8ff7':
  Fix ParcelHelpersTest.ScanResult unit test

Fix ParcelHelpersTest.ScanResult unit test

Fix a failing unit test that was broken by some
earlier cleanup work.

Bug: 26879229
Bug: 27152031
Change-Id: I244d078a23a69c82fbc7cb4468649ab933131977

Add support for collecting A2DP Protobuf metrics
am: a1bccb9b39

* commit 'a1bccb9b394abf4614bf64259375545f0059972e':
  Add support for collecting A2DP Protobuf metrics

Add support for collecting A2DP Protobuf metrics

* Add new function metrics_a2dp_session() to
  connect A2DP metrics into Protobuf

* Add new function btif_update_a2dp_metrics()
  that is called when the metrics should be
  callected

* Add few more statistics to the BTIF media stats.

Change-Id: I5862ac82e55b3546dc1fbde0c78cb4683e40d19e

Merge "Additional logging for bond information"
am: ddf2af1497

* commit 'ddf2af14973968f0dc0e32b457cf6deced34fc43':
  Additional logging for bond information

Suspend properly when receiving events without response
am: d722e2ff1e

* commit 'd722e2ff1e9b54aee0c373ac8a6fadadfd0a84d9':
  Suspend properly when receiving events without response

Merge "Additional logging for bond information"

Suspend properly when receiving events without response

In the BTU handler for messages from the controller, rn event may be
received that the stack does not have to respond to. In this case, the
stack should go to sleep.

This patch calls the provided function in the BTU to enter sleep.

Signed-off-by: Jaganath Kanakkassery <jaganathx.kanakkassery@intel.com>
Signed-off-by: Yingkai Cheng <yingkaix.cheng@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>
Change-Id: Ie7c3fcae418eb2480338509e86b7cc870177a965

Fix data reporting state when using HID device
am: 5f30560dbb

* commit '5f30560dbb80f0231fb9f351ab5b0180513dffe7':
  Fix data reporting state when using HID device

Additional logging for bond information

Log the bond state, the funciton that updated the bond state, as well as the
device that was updated. Also added in functionality for protobuff logging.

Example output
Bond Events:
  Total Number of events: 8
  Time          BD_ADDR            Function             State
  00:01:12.353  ac:9e:17:07:58:d8  btif_dm_create_bond  BOND_STATE_NONE
  00:01:12.354  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDING
  00:01:13.999  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDING
  00:01:14.778  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDED
  00:01:14.827  ac:9e:17:07:58:d8  bond_state_changed   BOND_STATE_BONDED
  00:01:31.534  34:36:3b:c4:bf:99  btif_dm_create_bond  BOND_STATE_NONE
  00:01:31.535  34:36:3b:c4:bf:99  bond_state_changed   BOND_STATE_BONDING
  00:01:32.771  34:36:3b:c4:bf:99  bond_state_changed   BOND_STATE_BONDING

Change-Id: I01ab3a232300203704a356a17aa045e638f97463

Fix data reporting state when using HID device

Start and Stop are called on HID driver probe/unprobe.
Open/Close are called when driver is interesed/not-interested
in data reports, typically when user-space open/close the input
device. So we can have several open/close calls for one start/stop
cycle.

https://www.kernel.org/doc/Documentation/hid/hid-transport.txt

Problem is that current implementation disables data reporting on close
but never enables it on open. This patch fixes this issue to match the
expected behavior.

Change-Id: Id5d4627dba389338b06df60a6e1d993ac3e3e5ec
Signed-off-by: Loic Poulain <loic.poulain@intel.com>
Signed-off-by: Zhiquan Liu <zhiquan.liu@intel.com>

service: Use built-in byte vector parser
am: 5eedb96edd

* commit '5eedb96edd61f31294f5038adac22f4c76621028':
  service: Use built-in byte vector parser

service: Use built-in byte vector parser

Change-Id: I72369c0b7678338fc8ccf520b59a95affdfd2994

Remove the need for -Wno-constant-logical-operand
am: e6e73fa499

* commit 'e6e73fa499d09696c33390e91ec17cba488e49cb':
  Remove the need for -Wno-constant-logical-operand

Remove the need for -Wno-constant-logical-operand

Change-Id: Iea44eeed487df3236d9b482b7e5bbc92f7f46797