Get rid of btm_ble_send_extended_scan_params am: defbb91f06 am: 896e6cc77e
am: 92829c7b34

Change-Id: Ia0e0c9809a0dbb0e4b6fed3b54f279a824a90d47

Fix set extended scan params command am: 42300f2235 am: bb826d724f
am: 487e762d67

Change-Id: Id1032adbad62fa9cf994e2a6cda0f2ce848fbceb

Get rid of btm_ble_send_extended_scan_params am: defbb91f06
am: 896e6cc77e

Change-Id: I7f78d992842daa591a320b7f6f46bdbf66b2c42e

Fix set extended scan params command am: 42300f2235
am: bb826d724f

Change-Id: Ib1930c9cdeea3d9d5cfa91488f4ed6be3a05112c

Get rid of btm_ble_send_extended_scan_params
am: defbb91f06

Change-Id: I352ab720b1040699bd25ee748d148886a92dcb7d

Fix set extended scan params command
am: 42300f2235

Change-Id: I871bca8c20130b3d736b682674ead62047a063bc

Get rid of btm_ble_send_extended_scan_params

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: I94e62ca0d785b20f97e3397131b70eac06e15003

Fix set extended scan params command

scanning_phys is a bitmap, not a count of PHYs. This error was resulting
in invalid HCI command being sent. This had no impact so far, because
only 1M PHY was used.

Bug: 30622771
Test: manual
Change-Id: I2be4f5ddbe0d453dc49cdee4605821b10853d7d8

Add Service Data GAP EIR data type am: 2a2583bdd3 am: e1403f2990
am: e101d0df0c

Change-Id: Icacd207b1e7b5dc8526861c987a0183e50f9bd6d

Add Service Data GAP EIR data type am: 2a2583bdd3
am: e1403f2990

Change-Id: I263f41199f1e78f91a9e9654cd44e6d97d800433

Add Service Data GAP EIR data type
am: 2a2583bdd3

Change-Id: Id9e77ecd8ae5c101379c16e201e9538ab89458c3

Add Service Data GAP EIR data type

Test: Build
Change-Id: I378883fdeb09fadaac04ace2dc499abd27b7d3ab

Merge "Minor fixes" am: bb952824d1 am: f4ed8ead35
am: 9c707e69c6

Change-Id: I9340d64fd1229e329c3323569528bb7a284ce274

Merge "Minor fixes" am: bb952824d1
am: f4ed8ead35

Change-Id: I6084df1776fd7be3900e0150f229c6cedda0a1ae

Merge "Minor fixes"
am: bb952824d1

Change-Id: I71a561da11481ceddcc53e7051ea568f821124d3

Merge "Minor fixes"

Merge "Advertising data handling improvements" am: caac63a814 am: e883a96a02
am: 5f3e798c09

Change-Id: I15aa1363c18f151f33c85688973b84c0d167adfa

Merge "Advertising data handling improvements" am: caac63a814
am: e883a96a02

Change-Id: Ie099bd7055d376c85968d6a9a6d7fbb7bb6babd9

Merge "Advertising data handling improvements"
am: caac63a814

Change-Id: Ice53320def5bfd56de0329465c42e551828c0661

Merge "Advertising data handling improvements"

Merge "Enable DLOG on eng and userdebug builds" am: 0b8c812427 am: fd3e9557c7
am: a42f1611f7

Change-Id: I4fa902a0ce0d2b20423d292a4e1d6806ca3520ef

Merge "Enable DLOG on eng and userdebug builds" am: 0b8c812427
am: fd3e9557c7

Change-Id: Id934eaf1a73d32eabf9de7ff47c21272241abae1

Merge "Enable DLOG on eng and userdebug builds"
am: 0b8c812427

Change-Id: I67ecd676087b183f749d5ee7495ede92dd938423

Merge "Enable DLOG on eng and userdebug builds"

Minor fixes

- Include <mutex> for heart_rate_server
- Remove unnecessary assignment

Test: Build
Change-Id: Ic2368d08789b95e71c083460148c70b9d0ef0b20

Advertising data handling improvements

* use non-hardcoded advertise data size
* use cache for keeping non-complete advertising data, waiting either
for scan scan response, or secondary channel data.

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: I689edcb4aee30361df15340029559cd19dac552b

Merge "Remove build variables that are no longer used" am: f9ea60e015 am: 23cdcce890
am: f638f75195

Change-Id: I1eb3b444587e7eb01923eea466a71d64b0fc3154

Merge "Remove build variables that are no longer used" am: f9ea60e015
am: 23cdcce890

Change-Id: I7e659d0597425ebb5bada5ae64980bb1cccdba92

Merge "Remove build variables that are no longer used"
am: f9ea60e015

Change-Id: Ic47a45bc196acc715e2152ed95510b9e1afb01c8

Merge "Remove build variables that are no longer used"

Enable DLOG on eng and userdebug builds

Test: manual
Change-Id: I1ff4ad6b2543246351d4d34919ba443e5617a9c0

Use same timestamp for on-disk BT Snoop Log and in-memory BT Snooz Log am: 071b507ad4 am: ae0c098aaa
am: d7f782e2a8

Change-Id: I5eb80d379eeab342ab3f6bcca467b4dfaa0d42b4

Use same timestamp for on-disk BT Snoop Log and in-memory BT Snooz Log am: 071b507ad4
am: ae0c098aaa

Change-Id: I51273b1a197582e4be846cc1980c157721f2536a

Use same timestamp for on-disk BT Snoop Log and in-memory BT Snooz Log
am: 071b507ad4

Change-Id: Ie19de8d98cdf01ac1b66d146742f19a769969f9f

Use same timestamp for on-disk BT Snoop Log and in-memory BT Snooz Log

* Logcat uses gettimeofday for its timestamp, the same as on-disk BT
  Snoop log
* Although in-memory BT Snooz Log uses the same method to get time, it
  is calling it separately, resulting in mismatch between timestamps of
  two snoop logs
* This CL let them uses the same timestamp_us value and put the function
  definition to libosi
* Note that preserved on-disk BT Snoop logs timestamp postfix at
      btsnoop_hci_<timestamp>.log
  will be changed to microsecond since epoch at current device timezone
  instead of the shifted BT Snoop timestamp value
* New unit tests for gettimeofday

Bug: 35113514
Test: Make, unit tests, run BT activities and check both snoop logs
Change-Id: I5b3f87bc523b272ced2c69a4595d0e0cbe29bcb3

Merge "Guard btsnooz ringbuffer access from multiple threads" am: 82fc2a96ef am: 3c7a71dcd2
am: f1b955ef94

Change-Id: Idc4a810c901196a19f5e3d2b103e2d302d408cae

Fix AAC bitrate computation and add missing parameters am: 3b8f12614f am: 495467f9af
am: 34ee971802

Change-Id: I7b34065f41a17a6a5e33c00211a0a583420c74fc

Merge "Guard btsnooz ringbuffer access from multiple threads" am: 82fc2a96ef
am: 3c7a71dcd2

Change-Id: Ie6a1fe91e12bba1df5ad998b3b9f5ca47a298beb

Fix AAC bitrate computation and add missing parameters am: 3b8f12614f
am: 495467f9af

Change-Id: I232ee1e8b0f7b7b2b9a22ea60411497d3f392391

Merge "Guard btsnooz ringbuffer access from multiple threads"
am: 82fc2a96ef

Change-Id: I75ad53c392554ea41694a9d314e1503009c076ce

Fix AAC bitrate computation and add missing parameters
am: 3b8f12614f

Change-Id: I9ecff0dfe4d804678c6dc5c3bc2fc0f5d0a324ed

Merge "Guard btsnooz ringbuffer access from multiple threads"

Remove build variables that are no longer used

* Removed BtHcilpIncluded from BOARD_BLUETOOTH_BDROID_HCILP_INCLUDE
* Removed BtHciUseMct from BLUETOOTH_HCI_USE_MCT

Bug: 35203653, 34951912
Test: Code compilation, no user visible effects
Change-Id: Ifbb0c2165a2b7cb6ec1d5c87a610219dd010206b

Fix AAC bitrate computation and add missing parameters

* Compute the AAC codec bitrate based on the MTU and the AAC
  object type such that the AAC frames are not fragmented.
  The spec recommendation is to avoid fragmentation, and
  some sink devices cannot process fragmented frames.

* Setup/assign AACENC_PEAK_BITRATE as the peak bitrate to
  make sure we never go over the MTU frame size.

* Add missing AAC parameter setup for AACENC_AUDIOMUXVER
  and AACENC_SIGNALING_MODE

* Add a fix for selecting the bitrate if the sink device reports
  capability with zero bitrate - for such devices we negotiate
  our bitrate. For the actual AAC codec bitrate we still compute
  it based on the MTU and the AAC object type.

Test: Tested with numerous AAC sink devices
Bug: 34953385
Change-Id: I9ca6353b0e1ee107f1242803de4e82d568a3394b

Guard btsnooz ringbuffer access from multiple threads

Since moving to HIDL, the btsnooz packet ringbuffer can be accessed from
two separate threads. Thus it should be guarded from concurrent access
to avoid pointer corruption.

Bug: 35182804
Test: manual
Change-Id: I3e6e1a869887a7ad5d87d8bb09ed78a22b3383ae

Merge "Use CHECK() to verify that HIDL is installed" am: c9f353fa78 am: 54c96032e0
am: 0aca2400a7

Change-Id: I8c00c2c44689c026f2e0e1837ab7794da34c0321

Merge "Use CHECK() to verify that HIDL is installed" am: c9f353fa78
am: 54c96032e0

Change-Id: I008018491d3a92a8acb023fd08ddf7bf835a645e

Merge "Use CHECK() to verify that HIDL is installed"
am: c9f353fa78

Change-Id: Ie4c283f10611062df08cfc7f6520b3a31e6363cc

Merge "Use CHECK() to verify that HIDL is installed"

Merge "Fix the implementation of htonll()" am: 9c0d0af2b0 am: 87b1567d58
am: a28c7eb9e6

Change-Id: I773425156d049bc5ea34e1605751245dfbf09b88

Merge "Fix the implementation of htonll()" am: 9c0d0af2b0
am: 87b1567d58

Change-Id: I056cb3496c0c572c710e2c60a5ac5a9812d0cdb9

Merge "Fix the implementation of htonll()"
am: 9c0d0af2b0

Change-Id: I31333bc5c25744a5a438bfc36d6ba347cc761b5f

Merge "Fix the implementation of htonll()"

Fix the implementation of htonll()

Prior the fix, all btsnoop entries had same timestamp.

Bug: 35111777
Test: Examine the btsnoop log file with Wireshark
Change-Id: I9e664475ff5b4ffc543d95b90a47c7f80fdc0370

Merge "AVRCT CT: uint8_t is used for 2byte field" am: a84ef64eb8 am: f474d5f2d1
am: b93026d807

Change-Id: If68786ad9dfda8892a8a4705c86e938e425f4684

HF Client: SCO routing with multiple devices am: ac109dc225 am: 66d065b665
am: 0856aec3f3

Change-Id: I44e8c2996a4334462b8c0703cb929c6242986a69

Merge "AVRCT CT: uint8_t is used for 2byte field" am: a84ef64eb8
am: f474d5f2d1

Change-Id: I63d485dde672adbf2ffce916478e8c2875fb9a40

HF Client: SCO routing with multiple devices am: ac109dc225
am: 66d065b665

Change-Id: I19469783e0304d09b1d342a6cb8ca74681f536d2

Merge "AVRCT CT: uint8_t is used for 2byte field"
am: a84ef64eb8

Change-Id: I0f7fee007ae4a5b3bd19fa2862e0c22aa7f46cae

HF Client: SCO routing with multiple devices
am: ac109dc225

Change-Id: Id8893897e60105f11e7b7637a666aaa6cab957ad

Merge "AVRCT CT: uint8_t is used for 2byte field"

Use CHECK() to verify that HIDL is installed

Test: build, boot test on angler
Change-Id: Ia64dcceedc233ef494ab0fdce7ee649e74fa45a0

HF Client: SCO routing with multiple devices

For SCO it makes sense only to have one active connection at a time for
any device since simultaneous SCO calls in a single device is
un-reasonable. This change changes the SCO state machine in following
ways:
a) SCO is not setup automatically whenever AG wants, the client for this
BTA layer will have to call connect_audio to start SCO
b) Also when we disconnect SCO for a device, it does not go into a
listen state. We are making sure that the upper layer caller always has
to call connect to accept a connection

Bug: b/34518638
Test: Manual testing w/ multiple active calls and SCO switching

Change-Id: I6e6c4c6e6ee19e44e3ff14499fb99083102bde5a

AVRCT CT: uint8_t is used for 2byte field

This causes the upper stack to report incorrect entries if the value is
large

Bug: b/34682784
Test: Check with a list > 4 items large and packet size big enough to
exceed uint8_t size

Change-Id: I73233a74378d9c091e33f204780a9492505d99b4

Merge changes from topic 'bt5_scan_props' am: 1e73f14e77 am: de6a7cbd2a
am: be28cca74e

Change-Id: I0af6abaa511cb43ea84c5f40fb4480f2ebe639e7

Propagate BT5 specific scan result properties up the stack (2/3) am: 7de0f9b78f am: 685d5c61e4
am: ad41dab8aa

Change-Id: I4e27e6507062556f90d12f9a6d9a29ceb1fb6045

Use new event type in BLE scan am: d64bf4f6f8 am: 0ff34d8062
am: e87306b102

Change-Id: Ib59412fca358cffe31ff04ee5b8d8fd6949e8cc8

Merge changes from topic 'bt5_scan_props' am: 1e73f14e77
am: de6a7cbd2a

Change-Id: Iee131acc33feb26afa77a70230042eff6beab234

Propagate BT5 specific scan result properties up the stack (2/3) am: 7de0f9b78f
am: 685d5c61e4

Change-Id: I63538d28f11389dd03ea61eab0446a139168f40a

Use new event type in BLE scan am: d64bf4f6f8
am: 0ff34d8062

Change-Id: I6011791540ed3d6857450738891647cb51a09aec

Merge changes from topic 'bt5_scan_props'
am: 1e73f14e77

Change-Id: I4639f2dc8b6576cbc8c823810e26ce3846fdc010

Propagate BT5 specific scan result properties up the stack (2/3)
am: 7de0f9b78f

Change-Id: I9a273c35111bfc60f40be0d11e24f48c75095abb

Use new event type in BLE scan
am: d64bf4f6f8

Change-Id: Ia3380083b57bf98a6d9188541292b6a081e6ea6b

Merge changes from topic 'bt5_scan_props'

* changes:
  Propagate BT5 specific scan result properties up the stack (2/3)
  Use new event type in BLE scan

Add a mechanism to configure the default A2DP codec priorities am: a6ba5aca4e am: 6b34e4af2a
am: cdbfe874ba

Change-Id: I2a80d5876b5462d0cacc6b467d3d0dae2a67fdf4

Add a mechanism to configure the default A2DP codec priorities am: a6ba5aca4e
am: 6b34e4af2a

Change-Id: I987518130278e2ef51f828cffdfc13cb3a3dc275

Add a mechanism to configure the default A2DP codec priorities
am: a6ba5aca4e

Change-Id: Ifd4cad913643a94b907611d7cf2d1f8b5e5fe2d5

BLE Advertising Manager test improvements am: 3b2765696a am: 0663343f11
am: 40a2d9a5f0

Change-Id: I34c534fb9b1a52ffba2d003274b2e73567c6fab5

BLE Advertising Manager test improvements am: 3b2765696a
am: 0663343f11

Change-Id: Id0962866cf3fd58c2b48d36c1295384853d4bff4

BLE Advertising Manager test improvements
am: 3b2765696a

Change-Id: Ib35e49f00b807af79ba4f711dd1f6154705896c4

Propagate BT5 specific scan result properties up the stack (2/3)

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: Ib9d18ea3a1fc7d95300415a04b551d66767b04d2

Use new event type in BLE scan

Right now new event_type in scan result is translated to legacy
one and used through whole stack. This patch make use of new event_type
through whole stack instead.

Bug: 30622771
Test: sl4a FilteringTest
Change-Id: I7420c040b8c54fae00262c939bcabc6c6fcb16e6

Add a mechanism to configure the default A2DP codec priorities

Previously, the relative codec priorities were hard-codec internally.
The new mechanism uses the following configurable resources in
packages/apps/Bluetooth/res/values/config.xml to re-assign the
default codec priorities per device, or to explicitly disable a codec.

 - a2dp_source_codec_priority_sbc
 - a2dp_source_codec_priority_aac
 - a2dp_source_codec_priority_aptx
 - a2dp_source_codec_priority_aptx_hd
 - a2dp_source_codec_priority_ldac

Those values are assigned on startup.
Also, they can be changed per device by using an overlay:
device/<VENDOR>/<DEVICE>/overlay/packages/apps/Bluetooth/res/values/config.xml

Test: Manually streaming to a headset
Change-Id: I5312299aea4a11c92fbe999d521adb39346b9866

BLE Advertising Manager test improvements

This patch moves mock verification before call to callback. If for any
reason the callback is not properly initialized, this is usually due to
fact that the call didn't reach the mock. Right now in such cases we get
a crash and stack trace. After this patch we will first get information
about the mock not being called properly, then the crash, which would
make debugging such issues much easier.

Bug: 30622771
Test: manual test
Change-Id: Ia182f2eed384924395e8d6b6a2d3d0abd393a4ee

Temporarily disable WBS; to be replaced with eSCO am: f6d79c5d6c am: b785a300ef
am: a955666b0b

Change-Id: Ieede94b3cb6a9a14b4ab3757951db9ad137dd579

Temporarily disable WBS; to be replaced with eSCO am: f6d79c5d6c
am: b785a300ef

Change-Id: Icc65f8c94b8f9fe7bd1400aa9ac245312fbac794

Temporarily disable WBS; to be replaced with eSCO
am: f6d79c5d6c

Change-Id: I39397f3f4788e17073c3994d587a6ae50aa4486a

Temporarily disable WBS; to be replaced with eSCO

HIDL'ized vendor library does not support custom audio parameter
configuration currently. To be replaced with eSCO commands.

Test: manual
Change-Id: I64850a7cbc6a05376f60f669fe43c3955cfcc08c

Fix a memory leak in the osi allocation tracker am: 1aa40037fd am: 41a7d7f0dd
am: e3c21005d3

Change-Id: I4b7d8d42c89ebc4f08ce53fbadb4811ded7d3622

Fix a memory leak in the osi allocation tracker am: 1aa40037fd
am: 41a7d7f0dd

Change-Id: I50f8a9b75d9eeb7e7b0caa870db1a3d74f2e4edc

Fix a memory leak in the osi allocation tracker
am: 1aa40037fd

Change-Id: Ia2163e96693c890005c88ce79726710f378116c7

Fix a memory leak in the osi allocation tracker

Add a missing free(allocation) inside allocation_tracker_notify_free()
when freeing a memory buffer.

Prior to switching the allocation tracker to use C++ unordered_map,
the "allocation" bin was kept within (and reused) by the older hash_map
mechanism. However, after the switch to the C++ unordered_map this
is no longer the case, hence the memory leak.

Test: watch -n 1 "adb shell dumpsys meminfo -d BT-PID | grep Heap"
Bug: 34785845
Change-Id: I737c901a6452d29d18fd8b847e7d5ea10c5ff485

Merge "Use local CCB variable in loop to check GAP CCB pool" am: 064e8d1273 am: e9840b1476
am: 8bcc3620e8

Change-Id: Ibcf55d275bc9f1b468656be75feb26dd9d123629

Merge "Use local CCB variable in loop to check GAP CCB pool" am: 064e8d1273
am: e9840b1476

Change-Id: Iffb41964e9828c6586216a60ab64402d11406cc9

Merge "Use local CCB variable in loop to check GAP CCB pool"
am: 064e8d1273

Change-Id: Ifac39dd2cb84ea065bed2e254aef40bd52f2ab81

Merge "Use local CCB variable in loop to check GAP CCB pool"

Initializing the SDP event message pointer before sending am: 65cca8f2cc am: b722434e5a
am: 8ee915d5e2

Change-Id: Ie894730aff1c4ab39d56b1f473e9113595a2f947

Initializing the SDP event message pointer before sending am: 65cca8f2cc
am: b722434e5a

Change-Id: Ib963b4f53925e9d39f8131a29210b25e93e7f99e

Initializing the SDP event message pointer before sending
am: 65cca8f2cc

Change-Id: Id9a32b5f07e3d8bbe175d99acf4127a2612a4de2

Use local CCB variable in loop to check GAP CCB pool

CCBs are not being properly cleared due to attempting to clear the wrong
CCB. This is fixed by using a local CCB variable in order to preserve the
CCB being freed.

Test: Send multiple files over OPP
Change-Id: I06a8f237047ffb212118b4239e9856fe5c89acb1