Merge "Fix bad GATT client state machine state after successfull cache load" am: 69b916046d am: 0776cdf3e6
am: 42acb81619

Change-Id: Ia726080e38eb634668665b07e9bd4ea91b6ae340

Merge "Fix bad GATT client state machine state after successfull cache load" am: 69b916046d
am: 0776cdf3e6

Change-Id: I2094e041dc9d38c1ceb1ea5a649d62de16a9ae0a

Merge "Fix bad GATT client state machine state after successfull cache load"
am: 69b916046d

Change-Id: Ib8fedecce74779755c2ebb285db6ca31b8714b68

Merge "Fix bad GATT client state machine state after successfull cache load"

Replace log/log.h with android/log.h am: 2f0c1fff33 am: 53dc5ba848
am: 7a3ec37304

Change-Id: I49cd4f5249dc65c74cf828254f3191a017eb3ad2

Replace log/log.h with android/log.h am: 2f0c1fff33
am: 53dc5ba848

Change-Id: I1009dd3e5413e8c84608bc6e473cd4127fbd4f9e

Replace log/log.h with android/log.h
am: 2f0c1fff33

Change-Id: I137539cc1f2cdeda6ceb6f7e4106fb379dc50555

Replace log/log.h with android/log.h

Actually, remove reference, android logging is not used here.

Test: compile
Bug: 26552300
Bug: 31289077
Change-Id: I37731dc37f61c129f3440e5d9ff1d0e71fe193d0

Fix bad GATT client state machine state after successfull cache load

If the GATT service cache was successfully loaded from a file, the
state must be reset to idle.

Bug: 31175159
Change-Id: I92ebf1bec9d2e7467d7412b41923614ec5a13b6d

test_vendor: Refactor scheduling of dual_mode_controller event handling am: 3c4f59fc27 am: 81f2249e94
am: f8626b0721

Change-Id: I8bf88152127337435a7fa5330be45c610f30f65b

test_vendor: Refactor scheduling of dual_mode_controller event handling am: 3c4f59fc27
am: 81f2249e94

Change-Id: I9e6660f68a78f200d00ccc8f895b59599901138a

test_vendor: Refactor scheduling of dual_mode_controller event handling
am: 3c4f59fc27

Change-Id: Ia9a9860a9744580349921a5d9cb0ad9771bdefaf

test_vendor: Refactor scheduling of dual_mode_controller event handling

Moved all controller actions out of read callbacks to task callbacks to
avoid race conditions when modifying internal controller state.

Change-Id: Ie8bb762992755db2dddb6b7bb6a4fe52ab2d0020

Remove redundant logic from system/bt/bta am: e143d941d2 am: b9554aed8a
am: 1a8056a222

Change-Id: I278f26b30e5e05391fb7920f57fe1d6149f9c7be

Remove redundant logic from system/bt/bta am: e143d941d2
am: b9554aed8a

Change-Id: Id26fd245bb90aeeee658a61adee595d30ef0ba15

Remove redundant logic from system/bt/bta
am: e143d941d2

Change-Id: I4fbe813cfa2809efd083cd36d567860fc7fe5ceb

Remove redundant logic from system/bt/bta

Bug: 31274290
Test: Manual
Change-Id: I5a4bff5e25b08a32709cdc81bac86a641d80f113

Merge "Do not request remote name from non-connectable dev" am: 49109aefb0 am: 13e05a1959
am: 3ab83c181e

Change-Id: I206199b593ad932e6603ffe4e34322cdfb3295f7

Merge "Do not request remote name from non-connectable dev" am: 49109aefb0
am: 13e05a1959

Change-Id: Id6c15d6012a7a2d6edfbee1e24ef81fba25e93ac

Merge "Fix warnings of implicit function declaration in Linux vendor lib" am: c4c9923724 am: feaf001c2d
am: 7ed28e9b28

Change-Id: Ie041880ef40b5a0f5a873caa0332ba5b4da20aec

Merge "Do not request remote name from non-connectable dev"
am: 49109aefb0

Change-Id: Iae4d72d5ed0c452085adf36b0c27daa61c45a8ae

Merge "Fix warnings of implicit function declaration in Linux vendor lib" am: c4c9923724
am: feaf001c2d

Change-Id: I18d5825d09aba13e37ece6e8b63b5087bd729d08

Merge "Do not request remote name from non-connectable dev"

Merge "Fix warnings of implicit function declaration in Linux vendor lib"
am: c4c9923724

Change-Id: I1d4d89be2d72f13e9bd5ae8b4b90856b21e3a0b9

Merge "Fix warnings of implicit function declaration in Linux vendor lib"

Fix NULL pointer crash in send_at_cmd am: 5c8fa2b199 am: 907d1e9cdc
am: 0df96a52b2

Change-Id: I575c15a4cc39000c494cb69ef1647d3161032a74

Fix NULL pointer crash in send_at_cmd am: 5c8fa2b199
am: 907d1e9cdc

Change-Id: Iccd3eb172e6bdae93aab0dbf7724d5a73beba379

Fix NULL pointer crash in send_at_cmd
am: 5c8fa2b199

Change-Id: I64b04cbf63438f25e58b700a9af6fe1d51a3905b

Do not request remote name from non-connectable dev

if an LE device is found during an LE scan, the BT host will try to fetch its name (creating an LE
connection) even though the advertising report received indicates that it is a non-connectable
advertising.
This is pointless and should not happen.

Change-Id: I246e4795b4f4e33f3202908ebff3724e8e89523c

Fix warnings of implicit function declaration in Linux vendor lib

Change-Id: I482f1096e87966f33ba8584ea60188abab3bcc42

Fix NULL pointer crash in send_at_cmd

If the string argument of send_at_cmd is a null pointer,
then the function should not crash: indeed this is a valid
parameters which can be used to send an AT cmd which doesn't
require any extra parameter.

Test: trivial fix. Existing unit tests still pass.
Change-Id: I3d83eebed660d703cd5a93a19dc73f4f354ab7bf
Signed-off-by: Sebastien Griffoul <sebastien.griffoul@intel.com>

Enable libbase logging for whole stack am: 75b83661ea am: 8d81059e2f
am: 1f161a621d

Change-Id: If12246cb7f0a7ecc7814d8b70e543c6a2b1d883b

Enable libbase logging for whole stack am: 75b83661ea
am: 8d81059e2f

Change-Id: I7d89ff65ad5de2d0756cab4ffdb6cf77cf34b9ac

Enable libbase logging for whole stack
am: 75b83661ea

Change-Id: Ic09c0c3bfefbad2e8949f29b9d36c8fa13b266ae

Enable libbase logging for whole stack

Currently, only C++ code in the service/ folder can use libbase logging.
This patch makes sure that logging can be used and is properly
configured for usage when the stack is running as part of Bluetooth.apk.

Bug: 31806042
Change-Id: I1f8be79ba9999b53ece0b5217a893b4bd20ecafc

Merge "Support TARGET_TRANSLATE_2ND_ARCH case" am: d09a76a935 am: b874d655ef
am: f0105e5a76

Change-Id: I7486a2559590d1f9d3840ddaeb5f0ea62f643c4d

Merge "Support TARGET_TRANSLATE_2ND_ARCH case" am: d09a76a935
am: b874d655ef

Change-Id: I9d091454c236d3f5bd6c3f96725869f2c563bbb0

Merge "Support TARGET_TRANSLATE_2ND_ARCH case"
am: d09a76a935

Change-Id: Ib9c919256db089ed4ef859016d46ae8ec7791bdd

Merge "Support TARGET_TRANSLATE_2ND_ARCH case"

Fix 100 LE device connection limitation am: 426da5ad3c am: 9856bc202b
am: 302e9575d9

Change-Id: Id03b9a333606ac01d7283fa7de0aa92d81c53fee

Fix 100 LE device connection limitation am: 426da5ad3c
am: 9856bc202b

Change-Id: Ieb9e1d2870574c8cf0efdf9fa86bed634db38c96

Fix 100 LE device connection limitation
am: 426da5ad3c

Change-Id: Ie9d7439eda57bfaeac544bc35f6100413b1fe354

Merge commit '7e701dbd4375ee00c3b72d4bda970d444f20c470' into manual_merge_7e701db

Change-Id: I12e74839725fe33e6739517e454a0941aa4bd986

Fix 100 LE device connection limitation

Currently after 100 devices are added to the device security database,
any subsequent LE connection to a newly discovered LE device address
will fail as entries in the security record database are not reused.

This patch removes a device record if the device itself is removed and
also ensures that the oldest device security record is deleted if the
limit is reached to ensure a new record can be allocated.

Bug: 31625900
Test: SL4A multi-device connection test + regression
Change-Id: I22f6c82c64a9a9bfb2a16d79182903e5aa011355
(cherry picked from commit 013c32bf98b8d511dc29fcd7de7578a421a52590)

Fix 100 LE device connection limitation
am: 013c32bf98

Change-Id: I64d57c1a19e41bcf35569eb707f58d31c9a54d63

Support TARGET_TRANSLATE_2ND_ARCH case

Build-only change to support the multilib case where the second arch
is translated (and thus bluetooth library is needed).

"True" multilib case is explicitly not supported.

Test: Build system refactoring CL. Existing unit tests still pass.

BUG=31422117

Change-Id: I09f239d39f5dbe0848a89367327db1ea1074ca39

Separate the definition of BTM layer types from control blocks am: dc3c1f2afa am: f7af15140a
am: b5a4a143de

Change-Id: Ifc520da330610371c16e2d1b5e418f1ecfff8b02

Separate the definition of BTM layer types from control blocks am: dc3c1f2afa
am: f7af15140a

Change-Id: I328bc6141dcc895530b93dca2981bd4bfeeb3da1

Separate the definition of BTM layer types from control blocks
am: dc3c1f2afa

Change-Id: Iea26880f80f623744cd96b40c27188428b7d9860

Separate the definition of BTM layer types from control blocks

Right now, data types, control blocks, and functions used in the BTM
layer are defined in the same header files. This means that if someone
wants to write a test that uses those data types, they must also define
all control blocks, or compile the whole module.

This patch separates the data types from other definitions. Thanks to
it, we will be able to write unit tests, once other dependencies get
separated.

Change-Id: Ibc089e273cc37642fbb8672964b266c20f8d825d

Fix 100 LE device connection limitation

Currently after 100 devices are added to the device security database,
any subsequent LE connection to a newly discovered LE device address
will fail as entries in the security record database are not reused.

This patch removes a device record if the device itself is removed and
also ensures that the oldest device security record is deleted if the
limit is reached to ensure a new record can be allocated.

Bug: 31625900
Change-Id: I22f6c82c64a9a9bfb2a16d79182903e5aa011355

Use better callbacks in LE multi advertising am: 0a009c382f am: 2f7dd70c11
am: 1127103c3b

Change-Id: Ib24640b13cb8c2f4978b05ce68ebba106be89238

Use better callbacks in LE multi advertising am: 0a009c382f
am: 2f7dd70c11

Change-Id: Id0d07632d059a984dc350b69f8548aad88b61a3c

Use better callbacks in LE multi advertising
am: 0a009c382f

Change-Id: Ia9e39fe19443641b6d17cc8e78e8156b5304619f

Use better callbacks in LE multi advertising

Right now, LE multi advertising related code uses it's own
implementation of queue on top of static array. Use std::queue, and
base::Callback instead.

Changing function pointers to base::Callback instances is required for
further refactoring, which will turn this code into class.

Bug: 30622771
Test: sl4a BleAdvertiseApiTest ConcurrentBleAdvertisingTest
Change-Id: I839e036c6edb4e19a17def4d68d351296468f88c

Fix unnecessary error message during GATT discovery am: f2e3bea844
am: 7edffcd474  -s ours

Change-Id: Ic99ab24d1000f5f9fbd75f937d9675d96bfac57e

Fix unnecessary error message during GATT discovery
am: f2e3bea844

Change-Id: Ied4363b52980f4a1e146c6186ffe920770f65866

Update BUILD.gn files am: 29796eb6de am: d42287dd61
am: 59a4789cd8

Change-Id: I089264ea9600e498df77c2f01df9f851c70c3c79

Update BUILD.gn files am: 29796eb6de
am: d42287dd61

Change-Id: Ic0c3dfae0e723d55ea53feb8c024579e66dbb443

Update BUILD.gn files
am: 29796eb6de

Change-Id: I41391c0764e9def86ff32e72e65222e0c3f6043f

Update BUILD.gn files

Fix unnecessary error message during GATT discovery

Bug: 31637098
Change-Id: Ib2545e7827bd338d2e22c88836818fce23e7a057

Bluetooth: Prevent gatt DB copy if server cache is absent am: c695f2578d am: 1b85fb000a
am: ec5ff3d956

Change-Id: I0c6e040a8b9e8256e55145a8b929fb99d77135c4

Bluetooth: Prevent gatt DB copy if server cache is absent am: c695f2578d
am: 1b85fb000a

Change-Id: If0b5f4f2e397cf269ed25b684eacf5ea3fb65e51

Bluetooth: Prevent gatt DB copy if server cache is absent
am: c695f2578d

Change-Id: Ic25dfb446c5133823523e3b707177739d1296a9a

Bluetooth: Prevent gatt DB copy if server cache is absent

In case the server cache does not exist for a specific
connection, the API used to copy the server DB into the
DB structure needs to return immediately. This change
makes sure of that and prevents dereferencing of a null
block causing the runtime error.

Change-Id: Iec3040a1280ef9d80b1b9c76eca8071dff499411

Merge "Convert BLE multi-advertising code to C++" am: 6cbb6f8c37 am: b2d733b15b
am: 0f174ee3a8

Change-Id: I8198ec8bb793e675bc6d3673e4d58d784d7fd8dd

Merge "Convert BLE multi-advertising code to C++" am: 6cbb6f8c37
am: b2d733b15b

Change-Id: I33565f33735b1143a8c6971aed4a25e2ce71a924

Merge "Convert BLE multi-advertising code to C++"
am: 6cbb6f8c37

Change-Id: I917c9c38fafc5b8a4e537c89038e5bdd056e7c4c

Merge "Convert BLE multi-advertising code to C++"

Fix the BLE connection failure with some mice am: 58c27b48aa
am: 1b97714aa8

Change-Id: I3cd24bae9b2015b7776d9b4aad96f643344cfcc4

Fix the BLE connection failure with some mice
am: 58c27b48aa

Change-Id: I45bdf4a316fca554c8c85cefdb87cff0cd2d21e6

Merge "Move btsnoop_hci.log to /data/misc/bluetooth/logs" am: b64d604c0c am: 271a06c67d
am: f5c6c7b801

Change-Id: I66fe55ba8ba938e097a321921a29abc4f40cf791

Merge "Move btsnoop_hci.log to /data/misc/bluetooth/logs" am: b64d604c0c
am: 271a06c67d

Change-Id: I7f27d2f0bd6b0bf303e570cc376d90397a36d390

Merge "Move btsnoop_hci.log to /data/misc/bluetooth/logs"
am: b64d604c0c

Change-Id: I87d5ef9224da76d569c2e4b32782f839686590df

Merge "Move btsnoop_hci.log to /data/misc/bluetooth/logs"

Convert BLE multi-advertising code to C++

This patch converts btm_ble_multi_adv.c file to .cc. It also separate
header definitions from rest of BTM layer. It is another step towards
making BLE advertising-related code testable.

Bug: 30622771
Change-Id: I89965e10fbf773e48176ee19ef613dd0df2fb951

Fix the BLE connection failure with some mice

If enable the "connection parameters update" during GATT discovery,
it may cause some device connection failure due to connection timeout.
Since the the "connection parameter update" is enabled again when GATT
discovery completed, remove the enable of "connection parameter update"
after SMP complete to fix the issue.

This issue was already fixed in commit 2d41fe1c, but then it was
re-introduced in commit 444a8da8, which also removed comment explaining
the issue, added back in this patch.

Bug: 29060797
Change-Id: I42b0b36056821c30d887484e22bfcbd04ea7ca03
(cherry picked from commit 1ec17e06c73c67592aeda29e0aaa03580fdecce9)

Removed usage of A2DP codec_type as a separate field am: b3e5af36e9 am: a21c8ebcd9
am: 809d61b769

Change-Id: Id8f093ab2e77b1368d8fc60b41882663eec0da2a

Removed usage of A2DP codec_type as a separate field am: b3e5af36e9
am: a21c8ebcd9

Change-Id: I12bee5b45667e7e95e94b463050709becd078776

Removed usage of A2DP codec_type as a separate field
am: b3e5af36e9

Change-Id: I886c6b3cbaa9cd1abdfc8340e586f5cd2dc33cc2

Fix the BLE connection failure with some mice am: 1ec17e06c7 am: 3047313009
am: de7bd173ea

Change-Id: I38e941f96497a19bdc138cbb6a24c5c1c5de69d7

Merge "Fix improper null check when handling rc features" am: e90972c9f2 am: 025f04f15c
am: aa9f7f5cec

Change-Id: I7b2dc0e5e0946d2ee1d36571b86332dcc04dc421

Fix the BLE connection failure with some mice am: 1ec17e06c7
am: 3047313009

Change-Id: I91c3d42d8a00e8b7daa163d648085926868ef725

Fix the BLE connection failure with some mice
am: 1ec17e06c7

Change-Id: Ib5b03ac545fee9c71ae53d869615a1f0fc155182

Merge "Remove net_bt_stack group and replace it with bluetooth" am: b85d405374 am: 2c189b27ba
am: f41808ab56

Change-Id: I3be0afd27e9e395627c68c3338a61eb0f7e7db38

Removed usage of A2DP codec_type as a separate field

The value of codec_type can be extracted from the codec_info,
so it is not needed to pass it around as an argument, or keep
it as an field.

Also:
* Added new APIs and the corresponding unit tests:
  - A2D_GetPacketTimestamp()
  - A2D_BuildCodecHeader() - replacement for bta_av_sbc_bld_hdr()
  - A2D_CodecName()
* Replaced tBTA_AV_SEP.codec_type with tBTA_AV_SEP.codec_info

Bug: 30958229
Test: A2DP tested manually. Unit tests included in the CL.
Change-Id: I3bfd4750e8639d77a9938ab3c274c9d6ea859fb6

Move btsnoop_hci.log to /data/misc/bluetooth/logs

Bug: 31466840
Change-Id: Ibd8f8b85eb59be8bfbb8a7c83b5935802624a748

Fix the BLE connection failure with some mice

If enable the "connection parameters update" during GATT discovery,
it may cause some device connection failure due to connection timeout.
Since the the "connection parameter update" is enabled again when GATT
discovery completed, remove the enable of "connection parameter update"
after SMP complete to fix the issue.

This issue was already fixed in commit 2d41fe1c, but then it was
re-introduced in commit 444a8da8, which also removed comment explaining
the issue, added back in this patch.

Bug: 29060797
Change-Id: I42b0b36056821c30d887484e22bfcbd04ea7ca03

Merge "Fix improper null check when handling rc features" am: e90972c9f2
am: 025f04f15c

Change-Id: Ib00b67cf8b74258d31322d5f6c5241c2ed629b28

Merge "Remove net_bt_stack group and replace it with bluetooth" am: b85d405374
am: 2c189b27ba

Change-Id: Ia6fda2cd85dfdacc60d5d3e5ef7944798802de01

Blacklist Motorola Roadster for Absolute Volume am: e89122a9c9 am: c3ceceaa8b
am: 27962021e4

Change-Id: I8b931c35b7aa10c95412343544816fcb00f25e71

Merge "Fix improper null check when handling rc features"
am: e90972c9f2

Change-Id: I228b1a0befa5becf1c60ca5eaa4b8b1a1e2292f8

Merge "Remove net_bt_stack group and replace it with bluetooth"
am: b85d405374

Change-Id: Iea39edaa3c6294001384597a151906aacf423980

Merge "Fix improper null check when handling rc features"

Merge "Remove net_bt_stack group and replace it with bluetooth"

Fix improper null check when handling rc features

Bug: 31533166
Change-Id: Ia04e9c51280a3e379c0670b39a84eb5a53c5332b

Blacklist Motorola Roadster for Absolute Volume am: e89122a9c9
am: c3ceceaa8b

Change-Id: I93ea5f243a16bc51471a98ec00d12906566ec4ad

Blacklist Motorola Roadster for Absolute Volume
am: e89122a9c9

Change-Id: Id7753a999d4e7c283c901c1eb9a80666f5dda767

Blacklist Motorola Roadster for Absolute Volume

Motorola Roadster is advertizing Absolute Volume but it has a
buggy implementation.

Bug: 29740052
Change-Id: I18c1a2f03617669138be2e7e1d9c7bb391cd1129