Bluetooth: Handle get RSSI timeout gracefully am: b0014dae41
am: 8fe142c111

Change-Id: Ib667ace75390ab1c969aac216e3babb58ea82dfa

BTA HF CLIENT: Unset initial SCO value am: 882760d186
am: 7aec0a91c5

Change-Id: I238b52168be022d5112b70bcff06d2a5381a7c19

Bluetooth: Handle get RSSI timeout gracefully
am: b0014dae41

Change-Id: I55f9d891c58b82e12073fe57d1d19bdf64f6019d

BTA HF CLIENT: Unset initial SCO value
am: 882760d186

Change-Id: I1342f805da9d37e542e137bad9eda54bab8287c2

Bluetooth: Handle get RSSI timeout gracefully

Usecase:
1. Turn ON BT
2. Pair and connect with Headset
3. Play Music over A2DP
4. Make GetRssi timeout
5. Bt app crashes

Failure:
BT app crashes.

Rootcause:
Calling callback with NULL and in callback handling NULL check with assert.

Fix:
Call the callback with error instead of NULL.

Change-Id: I20a9e504a513697945cd76d218ec12a0ef1e3330

BTA HF CLIENT: Unset initial SCO value

Initialize the block with invalid SCO state so that while allocating the
if condition does not ignore the block. This was causing the stack to
not initiate a SCO connection to the remote (AG role)

Bug: b/33555377
Test: Check for SCO initialied on audio call
Change-Id: I3f5619c69288483bfa45c2a1adcf22008e353841

Fix A2DP Metrics Logging Capacity am: a3f831cc74
am: 1cac42b5da

Change-Id: I85fd2f52849365593106e7f45bdcdb9c493d2f0a

Fix A2DP Metrics Logging Capacity
am: a3f831cc74

Change-Id: I6a790aae682f6b8282de219a286f82bdca4503b5

Fix A2DP Metrics Logging Capacity

* Set the maximum number of wake events logged to 1000
* Stop logging wake log name as it takes too much memory
* Add counters for each of the repeated values in BluetoothLog so that
  the true number of events can be determined while oldest event get
  dropped
* Log Bluetooth session disconnect reasons using enum instead of string
  in order to save memory usage
* Apply other branch changes to bluetooth.proto on system/bt

Bug: 33694310
Test: Code compilation and unit tests
Change-Id: I2cc6f9304725938b63b211d615eb1941eac60edf
(cherry picked from commit 7ab4b59672013eddcb706e288962ab7309a75628)

test_vendor: Remove extra byte in LE Features am: a72fc53164
am: e13ce6b773

Change-Id: Ibf2b67f0165db5ff470bbadc7979931caff5b713

test_vendor: Remove extra byte in LE Features
am: a72fc53164

Change-Id: If703d5f52b0ee55338f0f3c24c5c6cad113fe108

test_vendor: Remove extra byte in LE Features

Bug: 34357009
Test: push test-vendor lib as libbt-vendor.so, Bluetooth starts
Change-Id: Id3a5f85f6b3da5a723dba04527db8b66de0e84a8

Fix bad UUID comparison when adding GATT service am: 33f424d493
am: bd6197aae4

Change-Id: I8290d2055f06a869b964840bc205b8b8b89925b6

Fix bad UUID comparison when adding GATT service
am: 33f424d493

Change-Id: I8552b9aa41c0852672a5ecf63ea39b8a289d27df

Fix bad UUID comparison when adding GATT service

This patch fixes bad comparison in add_service_impl. Instead of checking
wether service is one of two excluded services, it was excluding all
services.

Test: sl4a GatReadTest
Bug: 34280184
Change-Id: I3195e94b6fafe4d2e3c61fc7bfd7398c2855b78e

Merge "test_vendor: Make tryStartThread return 0 on success" am: 91b994faa6
am: 0e462b4c9b

Change-Id: If36b548d7b58060d920caef2fea0d13f3fe3e596

Merge "test_vendor: Make tryStartThread return 0 on success"
am: 91b994faa6

Change-Id: I54671e84b39953a5e38e7b41a5275800de9c06b4

Merge "test_vendor: Make tryStartThread return 0 on success"

test_vendor: Make tryStartThread return 0 on success

Test: Start the test vendor library with no error message
Bug: 34357009
Change-Id: Id2821199ad675e2b95a9f333afa803dcced811a3

Fix parser for BOARD_BLUETOOTH_BDROID_BUILDCFG_INCLUDE_DIR am: a643b5b0df
am: 4c56d8f849

Change-Id: I5dcd31ca425b25286fb86ca0e48015c4514f3b8d

Fix parser for BOARD_BLUETOOTH_BDROID_BUILDCFG_INCLUDE_DIR
am: a643b5b0df

Change-Id: I3c4d7ccfcfbcb0faa151776ecd325891d14f38f5

Fix parser for BOARD_BLUETOOTH_BDROID_BUILDCFG_INCLUDE_DIR

* Current parser does not support multi-directory lists, which may cause
  a build failure if BoardConfig.mk defines one.
* This CL splits the list of directories using whitespace and adds them
  individually to the include list

Bug: 34396100
Test: Code compilation, no user visible effect
Change-Id: I59e901d76d1bc20f50b54941a3007c5521ded214

Convert build system to soong using Android.bp am: e2eeff4f7f
am: bfd636ce15

Change-Id: Id194d1c543cc52f52e93016f3ae37a3207527219

Convert build system to soong using Android.bp
am: e2eeff4f7f

Change-Id: I1caf32ca845b33d414582f972e89229949c439cf

Convert build system to soong using Android.bp

* Convert top level Android.mk into build templates in build/Android.bp
  and build/fluoride.go
* Initial conversion is done by "androidmk Android.mk > Android.bp"
* Android.bp does not allow source inclusion from external directories
  and therefore they have to be made in to cc_library_static in their
  respective sub-directories and linked using whole_static_libs in the
  modules where they are used
* As Android.bp does not allow multiple modules of the same name,
  same-name mudules for different target are merged into one definition
  with target specific setup
* Generated proto header path has to be changed in osi/src/metrics.cc as
  Android.bp only generate header path relative to the Android.bp file
  instead of top-level directory such as system/bt
* Android.bp does not support resource copying yet and hence conf files
  are left un-touched.
* Android.bp does support conditional module declaration and therefore
  test-vendor libs are left untouched except for unit tests
* The goal of this CL is to direct (almost) translate Android.mk to
  Android.bp first with Android.bp specific optimizations coming later

Bug: 32958753
Test: Code compilation, manual testing by test team

Change-Id: I5249e1f2135c4121205619b1d735ce448feb7499

Merge "HF CLIENT BTA: Reduce logging for non-allocated blocks" am: 7b70d1e420
am: 10f9c1d487

Change-Id: Id529230bcbaa5c6d730fd080da1b9373bd10397b

Merge "HF CLIENT BTA: Reduce logging for non-allocated blocks"
am: 7b70d1e420

Change-Id: I4a196410b5517c2a4231deea4b12c79c6b1e3b3f

Merge "HF CLIENT BTA: Reduce logging for non-allocated blocks"

HID: Prevent crash by Cancelling SDP during cleanup am: 8ed9a77792
am: 378900c9aa

Change-Id: Ibfb0232678d95aa0dc362a56914bfb2c6257a2c4

HID: Prevent crash by Cancelling SDP during cleanup
am: 8ed9a77792

Change-Id: I6020ab9443f3e674511ef88116fcaa0ea9a727ba

HF CLIENT BTA: Reduce logging for non-allocated blocks

Bug: b/33552136
Test: adb shell dumpsys bluetooth_manager
Change-Id: I3cb6c28bffe6b335cf82cac63dc78ce4957d78f4

HID: Prevent crash by Cancelling SDP during cleanup

Usecase:
1) User tried to connect to HID Device.
2) SDP is internally performed by DUT. SDP is at stage,
   where L2CAP connection, configuration is done, and data
   fetch is ongoing.
3) BT was turned off from UI.
Observation:
BT crashed while accessing memory that was freed already because BT turn off,
caused ACL disconnection, leading to L2CAP disconnect indication in stack,
leading to sdp disconnect indication.
backtrace:
    #00 pc 000f98d4  /system/lib/hw/bluetooth.default.so (SDP_FindServiceUUIDInDb+51)
    #01 pc 000b5dbd  /system/lib/hw/bluetooth.default.so (hidh_search_callback+0x40)
    #02 pc 000f770b  /system/lib/hw/bluetooth.default.so (sdp_disconnect_ind+0x5e)
    #03 pc 00107a5f  /system/lib/hw/bluetooth.default.so (l2c_csm_execute+3446)
    #04 pc 001080e7  /system/lib/hw/bluetooth.default.so (l2c_link_hci_disc_comp+122)
    #05 pc 000fda81  /system/lib/hw/bluetooth.default.so (btu_hcif_process_event+588)
    #06 pc 000fec81  /system/lib/hw/bluetooth.default.so (btu_hci_msg_ready+96)
    #07 pc 00118191  /system/lib/hw/bluetooth.default.so
    #08 pc 0011917f  /system/lib/hw/bluetooth.default.so
    #09 pc 00041993  /system/lib/libc.so (_ZL15__pthread_startPv+30)
    #10 pc 000192b5  /system/lib/libc.so (__start_thread+6)
Register Dump
pid: 15740, tid: 15761, name: bluedroid wake/  >>> com.android.bluetooth <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x20000
    r0 815a5cac  r1 a1a2f370  r2 00000000  r3 85d4e541
    r4 00020000  r5 815a5cac  r6 a1a2f370  r7 b6d3ae40
    r8 00000000  r9 b6d3ae40  sl 00000002  fp 00000013
    ip a228c050  sp a1a2f360  lr a20eddc1  pc a21318d4  cpsr 200e0030
Rootcause
The above scenario could lead to crash we see as below, as bta_hh_cb.p_disc_db would be freed
during HID Host cleanup, and would be accessed via callback received for parsing SDP results on
SDP completion.
Fix:
While cleaning up HID Host, Cancel SDP search before freeing and resetting bta_hh_cb.p_disc_db.
This will internally send L2CAP disconnect request for SDP, and would lead to sdp_disconnect_cfm
call when L2CAP is disconnected, and would lead to call of hidh_search_callback with result code
as SDP_CANCEL.

Change-Id: I63563cb23dd69946f87a70cafa203c44edc9b753

Add conditional unit testing for codecs that might not be on the device am: 4decac33f6
am: 2ead0be903

Change-Id: Ibb8af4cac0986f2b387be06c585a7593400e0e2d

Add conditional unit testing for codecs that might not be on the device
am: 4decac33f6

Change-Id: I9ac89df8897be966206020361ec3b4681aeb9681

Add conditional unit testing for codecs that might not be on the device

Codecs like aptX and aptX-HD are supported on the device if the
corresponding shared library is installed.
Fix the unit tests to conditionally test for aptX/aptX-HD.

Also, update function descriptions in files a2dp_sbc.h and
a2dp_vendor_ldac.h .

Test: Running unit tests on a device that doesn't have aptX/aptX-HD
Change-Id: I44dc275ca6e0cb5438ee6ed0b6213213d56592f4

Fix a NULL-pointer access before check in connect_int am: 9b2e40df05
am: 73af061500

Change-Id: I596eaaa5f8e1919e75f4332534f480057465cb54

Fix a NULL-pointer access before check in connect_int
am: 9b2e40df05

Change-Id: I3217f206ef56d73dbac01b19eeb123b6cfb68870

Fix a NULL-pointer access before check in connect_int

This fixes a potential NULL-pointer access in case cb is returned NULL

Test: mm -j 8
Change-Id: Iaf7aa0ba736b5cb161811726b4aaea7f1ab7e5b3

Merge "Reduced the log verbosity level for aptX/aptX-HD data packets" am: 5256f1feea
am: 76efc23846

Change-Id: I38cf5bdaff222dbd7328f04bf82da3c5cc056ab6

Don't allow GATT and GAP service override am: 8a06ff5ce2
am: 3b672538ba

Change-Id: I235d976c6e1e1ab80fdc3a457f5ea776281a309a

Merge "Reduced the log verbosity level for aptX/aptX-HD data packets"
am: 5256f1feea

Change-Id: I762b7e65001740fec55515f7121f5217c4dbb455

Don't allow GATT and GAP service override
am: 8a06ff5ce2

Change-Id: I9eaff8a251269b3f073845c80284fefde35822b4

Merge "Reduced the log verbosity level for aptX/aptX-HD data packets"

Fix NPE when discovering invalid GATT datatbase am: 8957fdca4f
am: 1637dc7990

Change-Id: I1e9710d05b0276106c0fd19604db2b9db5cde257

Fix NPE when discovering invalid GATT datatbase
am: 8957fdca4f

Change-Id: I1e196651928314494a696e5a3ef126272dd3208a

Don't allow GATT and GAP service override

Those services are automatically added by the stack, and overriding them
results in invalid GATT database being created.

Bug: 34280184
Test: Manual test
Change-Id: Ie6a09e9d606e40066bef6f6a2c934eacd17bf53b

Fix NPE when discovering invalid GATT datatbase

When doing a discovery of invalid GATT database, where two services are
added with same start handle, characteristics are added to the first
one, where descriptors are added to the later. This cause NPE when
adding descriptor.

Bug: 34280184
Test: manual test
Change-Id: I27619098df33a641cb089b82cf4d4ffd3c6e6aea

Reduced the log verbosity level for aptX/aptX-HD data packets

Also, fixed a typo in a log message.

Test: A2DP streaming to headset, and capturing "adb logcat" output
Change-Id: I525d78e3ebde334857261f0cbd256a7f6a946e96

Fix for Extended Scan parameter length am: 0c86aad3f5
am: a41493f386

Change-Id: Id57ec583ff0a29a5dfdaf2aee449fd7e6d5414c6

Fix for Extended Scan parameter length
am: 0c86aad3f5

Change-Id: I561461bd6b119912ef53dfbca923f2aac9b3fad7

Fix for Extended Scan parameter length

With BT 5.0 extended advertisement scan parameter length
is configured with correct value.

Test: BT scan from settings menu
Change-Id: I749099282a9f2cf190d4f9576649aefe88564809
Signed-off-by: Reddy Praveen <praveen.reddy@intel.com>

Removed the feeding_init call from tA2DP_ENCODER_INTERFACE am: f8e2670d12
am: a7d87c4c12

Change-Id: I4b28e24b3fad9248028bae23038a12d8c4b67ce8

Removed the feeding_init call from tA2DP_ENCODER_INTERFACE
am: f8e2670d12

Change-Id: I716d1c077b089345c08ca76ccf2823ac1ff6d8e3

Removed the feeding_init call from tA2DP_ENCODER_INTERFACE

The feeding_init call is not used anymore - it has been superseded
by the newer A2dpCodecs.setCodecAudioConfig() mechanism.

Also, moved aptx_init_framing_params() and aptx_hd_init_framing_params()
calls to the corresponding encoder_update functions.

Test: A2DP streaming to Bluetooth headsets
Change-Id: I52f42b4701d04fe2ddb8f115bb9cd57fe38e2b38

Integrated A2DP LDAC support with the LDAC encoder library am: d556b752bd
am: 95c78b6741

Change-Id: I7c1f807c2ede174d4682e2acf6ced396b1b19356

Integrated A2DP LDAC support with the LDAC encoder library
am: d556b752bd

Change-Id: Ib0d6db87f7f30ac7544bf2e5764b7dda0d1a24f9

Integrated A2DP LDAC support with the LDAC encoder library

* Add compile-time dependency on the libldacBT_enc module
  in external/libldac
* Use the <ldacBT.h> header and remove local typedefs and defines

Also, fix an initialization issue of the LDAC handler
inside file a2dp_vendor_ldac_encoder.cc

Bug: 30958229
Test: A2DP streaming to LDAC headset
Change-Id: I3757230c9901509daab36f2b2018faa05490171d

Merge "Serialize stack shutdown, state change callback and cleanup" into cw-f-dev
am: 718b632f2b

Change-Id: I59546b88705ca7a9748bc3a41ad0a382caada0fe

Serialize stack shutdown, state change callback and cleanup
am: 352b8e89ea

Change-Id: Ie1262e4b3604a5e04910201d330419873a656f2f

Merge "Serialize stack shutdown, state change callback and cleanup" into cw-f-dev

Merge "Add "connecting" and "disconnecting" states to HID Device." am: 35ae4adf21
am: 0d5910c685

Change-Id: Ibd5437b69871d1dbf5159152fa488892523510a6

Merge "Add "connecting" and "disconnecting" states to HID Device."
am: 35ae4adf21

Change-Id: I5eef753ca30b57848ebeaeb3b73aa134cb6b6970

Merge "Add "connecting" and "disconnecting" states to HID Device."

resolve merge conflicts of 316589aa to nyc-mr1-dev-plus-aosp

Change-Id: I5758549f4c2e1f26d02ff087e240698b13343bbe

Remove position dependent lookup tables in AT command parser
am: 8d3cf5988a

Change-Id: I0a690f1538c49fbed743ccf324d240dbb8c927c8

Batch scan simplification am: eab3651db4
am: 58dde764e0

Change-Id: Idffdb211a7084a21789f1c2bbad4c634809be842

LE filters without custom data types am: 2ffd8da9af
am: 4d227fc750

Change-Id: Ib7f7de32290dd8efc4b97185324011135ec53229

Get rid of BTA_DmBleCfgFilterCondition am: ad6afdf96c
am: fbb8d006a6

Change-Id: I1b4b2e071a0101ed7db0eecd7fc9c457a6093f2d

Simplify LE scan filter memory management am: ae2aabef09
am: fad449ee33

Change-Id: I447b69cd3bc0ee2ad9e4d33473831b9ed2013ae8

Remove reduntant re-schedule am: 2c1522e2b8
am: cbaeb50923

Change-Id: I62f949769fd07a9adabe3ba2b83ab3d2b12ae1d5

Use callbacks for LE filter functions (BTM layer) am: 1febda9706
am: 8cf65e4d0e

Change-Id: I24828a1130fc54d0d72845751b5725b72db5c30c

Batch scan simplification
am: eab3651db4

Change-Id: Iab508f656ceaaa726edff3e04945146a3d4762d4

LE filters without custom data types
am: 2ffd8da9af

Change-Id: I787c473f2c5839674fbf0ab3c646f210d15ab717

Get rid of BTA_DmBleCfgFilterCondition
am: ad6afdf96c

Change-Id: I9d5464bc0199759ef83cac4a2b7b6a94635053a4

Simplify LE scan filter memory management
am: ae2aabef09

Change-Id: I654160519dd7b03fa11d58a6ce6e5dbc24dd5d88

Remove reduntant re-schedule
am: 2c1522e2b8

Change-Id: Id7a3ed5b5fe1460e14450616bad422b39f8cbc9d

Use callbacks for LE filter functions (BTM layer)
am: 1febda9706

Change-Id: I218fb9df79b2b4df4d8a1774273a951c63042772

Batch scan simplification

Bug: 30622771
Test: sl4a UniqueFilteringTest BleOpportunisticScanTest
Change-Id: I0e6ca8ed3d7d98057277a5187e684246f410d978

LE filters without custom data types

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

Get rid of BTA_DmBleCfgFilterCondition

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

Simplify LE scan filter memory management

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

Remove reduntant re-schedule

Calls to BleScannerInterface are already made using jni thread.
Also all this calls do is scheduling to BTA thread, so remove
unnecessary call to do_in_jni_thread

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

Use callbacks for LE filter functions (BTM layer)

This patch use callbacks, instead of custom event queue when scheduling
LE filter VSC.

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

Fix bad connectable advertising constant am: 35675d0e73
am: 4c8198cfd9

Change-Id: Idb7596057acbad7fb30e1719261cfe17e29440c0

Zero is a valid advertiser id am: daa8a99913
am: 17a68bf7df

Change-Id: I90e4ce2f7e0005de2d0bd5541883f3587f2a96dc

Remove redundant dependency on libpower am: 4302060551
am: 51a1abd14d

Change-Id: Id809a19186f9f31964fa4d66f19535413b1fd3bf

Fix bad connectable advertising constant
am: 35675d0e73

Change-Id: I02605b0df8e6d46db0ca2f81e6c5e172337614a7

Zero is a valid advertiser id
am: daa8a99913

Change-Id: I05b1d504381932959877a0c856b1bb7ef82292a1

Remove redundant dependency on libpower
am: 4302060551

Change-Id: Ie78d7e79d9f1b9043eceae84865b4eb5274fd4db

Fix bad connectable advertising constant

Test: start connectable advertising using bluetoothtbd
Change-Id: Ie7e42570748d43a27bc7691378aee85ab1565a05

Zero is a valid advertiser id

Test: start advertising using bluetooth-cli
Change-Id: Iead31c5e73a70850c6a209b6bbbeaeb792202608

Remove redundant dependency on libpower

* libpower dependency was introduced in change
  I1e8d49b087eff6301373e1e90e8d868f15847c34
* It is never used in any part of the system/bt stack and therefore
  should be removed from the shared library dependency list

Bug: 34079981
Test: code compilation, no user visible effect
Change-Id: Ic8e3ad0c369839fb9ced446695c72fdf0ffad1ca

Fix Heart Rate Service example am: e3051fec06
am: fb326faf46

Change-Id: I8367ba5e2238fddd5636a3b0ae7c896db9a0f1ff

Fix Heart Rate Service example
am: e3051fec06

Change-Id: I8ff34d31fe85899a0657ce437f4bf614a24528c8

Add "connecting" and "disconnecting" states to HID Device.

Currently the upper java layers have full support for all four
connection states, yet the lower stack only reports about
"connected" and "disconnected".
This patch adds sending "connecting" and "disconnecting" messages
when bta_hd_connect_act and bta_hd_disconnect_act finish
successfully.

Test: build, run, logcat
Change-Id: I00f9f414daa3854198ff9eee19e9c98a2e18f603

Fix Heart Rate Service example

Name must be appended at end, not beginning.

Test: run bt-example-hr-server --advertise
Change-Id: I70909e0bfc94521c396908d0dcb78cf4439a7e63

Remove unnecessary cutils dependency am: 19e3d703b2
am: c1f7859ee3

Change-Id: Iea809f705c041d384b514df146f510e21c185a0a

Remove unnecessary cutils dependency
am: 19e3d703b2

Change-Id: Id0b5878395183458449cb0c89b515659b5e86133

Update Linux BUILD.gn files am: a9743d8101
am: 5c4642ab77

Change-Id: I5e38f3e901431873e9d3ed19a5ffebd868850e4e

Update Linux BUILD.gn files
am: a9743d8101

Change-Id: I846fa35f6f6ec7ccdc242d77b2f7e5ce41e46f9b