Use proper type in filter init am: 803fb5db92
am: 1fdd0b9e81

Change-Id: Icc80bdb3fa60d892463a5784d2d00fa740ca87f6

Use proper type in filter init
am: 803fb5db92

Change-Id: I63283f1cf5acea4ab2feb4b0ce1b991b9b1e4022

Use proper type in filter init

Change-Id: I429ea832f32cb4c67525dda71f9e05ba60110e1e

Merge "Updating QDID to reflect the certfication done for N." am: 66244795b0
am: 92f129a72c

Change-Id: I5e6c5e17f9ba8a8da6685ed64a689e94e4b40eb5

Merge "Updating QDID to reflect the certfication done for N."
am: 66244795b0

Change-Id: I218d1208aa3975ee8e61226ed632f766fba76068

Merge "Updating QDID to reflect the certfication done for N."

test_vendor: Refactor test channel am: e3ab96a689
am: 828f404082

Change-Id: I0dcf1b7e1c1c2a4d2dd14e5762cfe9c2800daf25

test_vendor: Refactor test channel
am: e3ab96a689

Change-Id: Ib7c83ff478b40244038faf52c91c0f18496957cc

Updating QDID to reflect the certfication done for N.

Change-Id: Ia80cb82c833df341f832d2ab8cc71a868e2c8f5e
(cherry picked from commit a1e4dd95ea389036c9e59872de09059726723fb4)

test_vendor: Refactor test channel

The test channel can always be enabled when using the AsyncManager.
 - Remove enabled_ and port_
 - Allow multiple connections
 - Pass file descriptors as parameters instead of private variables
 - Add a clean up function to the Test Channel
 - Add a static class in bt_vendor.cc and always EXPORT the entry point
 - Move the vendor callbacks and the global pointer to the vendor
   manager object from vendor_manager to bt_vendor

Change-Id: I3e0dee846eb89f434893603a705c2b13219272be
Signed-off-by: Myles Watson <mylesgw@google.com>
Signed-off-by: Jorge E. Moreira <jemoreira@google.com>

Merge "gatt_db: Fix uuid_to_str for UUID_32" am: b4f8b75e88
am: def1b8d418

Change-Id: I50e6d4e3e4ffd76e80c114a0b69f956eaad59e75

Merge "gatt_db: Fix uuid_to_str for UUID_32"
am: b4f8b75e88

Change-Id: I0ddbf8e4dfe0010786334a722c7938b031e710c1

Merge "gatt_db: Fix uuid_to_str for UUID_32"

Merge "Deregister from notifications when an HOGP background connection is cancelled" am: 629f814b9a
am: 2c77487f81

Change-Id: I9e614b3d873d99232b9523a9112464c5831fbc2b

Merge "Deregister from notifications when an HOGP background connection is cancelled"
am: 629f814b9a

Change-Id: Ia0ad02b5b43f5f2ccadbd082f53a3c7a72dd19f1

Merge "Deregister from notifications when an HOGP background connection is cancelled"

gatt_db: Fix uuid_to_str for UUID_32

A typo in a conditional had us treating LEN_UUID_32 as
an unknown length in this function.

Change-Id: Iec0190fd70859d9b72295e91798af5fdc4dfae93

Merge "AVRCP 1.6: media browsing support on Target(1/3)" am: 6d488f5272
am: 52c4699341

Change-Id: I23dc6ce863460af96cf5da733270644aa7076d45

Merge "AVRCP 1.6: media browsing support on Target(1/3)"
am: 6d488f5272

Change-Id: If9a2b360f190a1799499a71e0ab0429180b8c348

Merge "Remove BTA layer components for LE advertising" am: ee1e3df872
am: 87decb7aa9

Change-Id: Iecf36597ac502299aef1f1d99842886eddabe716

test_vendor: Add BtAddress unit tests am: b3a80b682f
am: 2cb9770708

Change-Id: Id666378631d06563fbb01e349085e70f3a42230b

test_vendor: Add a BtAddress class am: 6f8593ec98
am: 0e0f3abf74

Change-Id: I89be0eb31990ede581edd39c8511f8ff221af4af

Merge "AVRCP 1.6: media browsing support on Target(1/3)"

Deregister from notifications when an HOGP background connection is cancelled

Bug: 29111689
Change-Id: I5b57da9e08dea372e9b236b360ffeeae4f6b49af

Merge "Remove BTA layer components for LE advertising"
am: ee1e3df872

Change-Id: I6c397d8b87dafc8ba8067e8694a77506e9218a72

test_vendor: Add BtAddress unit tests
am: b3a80b682f

Change-Id: Iab8e73b89459a701ebec1cbc3df83f7f6a3ee0c3

test_vendor: Add a BtAddress class
am: 6f8593ec98

Change-Id: I07cfa4b9156807c0d3d20a90eca547ad022c6dbf

Merge "Remove BTA layer components for LE advertising"

AVRCP 1.6: media browsing support on Target(1/3)

-> As a part of Avrcp 1.6 upgrade, added support
   for following features:
   [1] setAddressedPlayer
   [2] setBrowsedPlayer
   [3] getFolderItems scope=VFS/NowPlaying/Search
   [4] changePath
   [5] getItemAttributes
   [6] playItem
   [7] getTotalNumberOfItems
   [8] search string in current browsed path.
   [9] AddToNowPlaying
-> Defined functions and enhanced structures to
   support above mentioned features.
-> Added Dual RC support.

Bug: 19361366
Merged-In: I45ed60e337b57c686d9693e19993fee1ce3c2504
Change-Id: I45ed60e337b57c686d9693e19993fee1ce3c2504

test_vendor: Add BtAddress unit tests

Change-Id: I23d27a1a5c258de330f4eac23d314ef8b9200d2a
Signed-off-by: Myles Watson <mylesgw@google.com>

test_vendor: Add a BtAddress class

The class provides IsValidBtAddress(string), functions to convert
from vectors and strings, and some operators.

Change-Id: I553c47e698ec46ed0e24656f89141c23f7369380
Signed-off-by: Myles Watson <mylesgw@google.com>

Remove BTA layer components for LE advertising

Advertising is so simple, it doesn't need the BTA layer.
Values returned by BTM_VendorSpecificCommand shouldn't be handled as
success or failure, it's just to inform wether we sent the callback.

Bug: 30622771
Change-Id: I104c4fbf84d3818b76fe8c527fbde4e6c4336c3a

Unify LE advertising instance identifiers am: 6a96661166
am: 1951d644d6

Change-Id: Iaceca9a422ce960e98232bdf624e03b017617911

Unify LE advertising instance identifiers
am: 6a96661166

Change-Id: If5385cb981686925dd42fa6616185f5c282557ee

Unify LE advertising instance identifiers

Up till now, we had three numbers associated with each advertising
instance: client_id, inst_id, and cbindex. We also had special structure
mapping each of those numbers to another.

From now on, when registering advertiser, we'll grab next free
advertising instance id and make advertiser_id equal to it. Thanks to
this, we can remove all the mapping and have just one number associated
with each advertising instance.

This also means we no longer need to pass *p_ref value to the BTM layer
with each request, as advertiser_id is equal to inst_id.

Bug: 30622771
Change-Id: Ied71bff36e30d6c6ce4ca3e62d46ba96320cf597

Cleanup in the bta/av layer am: d8c312e624
am: cb04cb59c7

Change-Id: Idfdd8d290074d202200dd3e1ebdef2d6fffd52ec

Cleanup in the bta/av layer
am: d8c312e624

Change-Id: I33c18706dc9490bddf482a447baaf29506b994b3

Cleanup in the bta/av layer

* Removed unused defines and function declarations
* Removed unused tBTA_AV_Q_INFO.vdp
* Cleanup of some of the log messages

Also:
* Added constant AVDT_TSEP_INVALID instead of using a hard-coded value

Change-Id: I9ce7b5b08dec7c9ea9699d9f6c598acb39fb105a

Added A2DP codec-related APIs am: 7e5862624f
am: d162323cbc

Change-Id: I19ebc778c84b1b722b962b9ba52a97c016fbada6

Added A2DP codec-related APIs
am: 7e5862624f

Change-Id: I4841099c07df6353cf9bfd16bc845d2b0ea01b18

Added A2DP codec-related APIs

Those APIs can be used to check various codec-related characterictics,
and minimize the codec-specific knowledge in the rest of the code.

The added APIs are:
 - A2D_GetCodecType()
 - A2D_IsCodecSupported()
 - A2D_UsesRtpHeader()
 - A2D_IsVendorCodecSupported()
 - A2D_VendorCodecGetVendorId()
 - A2D_VendorCodecGetCodecId()
 - A2D_VendorUsesRtpHeader()

Also:
 - Added unit tests for the new API calls
 - Removed redundant definition of BTA_AV_CODEC_TYPE_IDX
 - Removed unused A2D_MEDIA_CT_* codec type defines

Bug: 30958229
Change-Id: Id9ab01fd93b0b0a9b55a0af9bfb482310ae0ade4

Separate advertiser from GATT client (4/4) am: 67d5a25233
am: 93f37868fa

Change-Id: Iaec886c54568b2392efcc6e5286a7cb5d854d42d

Separate advertiser from GATT client (4/4)
am: 67d5a25233

Change-Id: I1640cd73003ab4d37b751ecb8e47f7da5bbf4588

Separate advertiser from GATT client (4/4)

Bug: 30622771
Change-Id: I761f0bff09aec1047bd512915960021e4fbdc0c8

Use closures in advertising related code am: 5b12a83ba9
am: 8b4db1a134

Change-Id: Ia93ee57dd132e3795feee06775d15072d2647303

Use closures in advertising related code
am: 5b12a83ba9

Change-Id: I2ad76a71fc3af35c5c60a5d44be6eff5cc827d6c

Use closures in advertising related code

This patch is a preparation to get rid of BTA layer for advertising, and
for further refactoring required to keep this code maintainable.

Tests: SL4A/ConcurrentBleAdvertisingTest
Bug: 30622771
Change-Id: I574b57159390d090b16d264d3c1fba2f07b70d4c

Merge "Disable auto-pairing for hands-free devices" into nyc-dev
am: fb309b3ce7

Change-Id: Ie6a548d7d0bdcad3a30ef7be3868df1b2a264873

Disable auto-pairing for hands-free devices
am: d2a37effc2

Change-Id: If399b79af277ec9c7c99934ff4b320de4daf903f

Merge "Disable auto-pairing for hands-free devices" into nyc-dev

audio_a2dp_hw: Always update frame counter in out_write
am: 5274ab7a31

Change-Id: I1749b616522346a733b2bcfc0273851b7babe437

audio_a2dp_hw: Always update frame counter in out_write

Required now that we always return success.

Bug: 30025777
Change-Id: I7db51321672c128039545cc0fb604c615f169bc3
(cherry picked from commit f4a37b255d1d6fa4cf7cefe2aeb1c3f4fdf33f3c)

Cleanup and removed BTIF-related unused code am: a62c5a302e
am: 7a84515916

Change-Id: Ia7a15974a43929b3e73dba0cf380aa7cac87f1a2

Cleanup and removed BTIF-related unused code
am: a62c5a302e

Change-Id: I449704fafd17f331fac8c201eabdf9f3af30fc4e

Cleanup and removed BTIF-related unused code

* Removed field tBTIF_MEDIA_CB.av_sm_hdl, because it is never used
  after an assignment.
* Removed the corresponding function btif_av_get_sm_handle() which
  is not used/needed anymore.
* Declared btif_hf_is_call_idle() function in a new header file
  "btif_hf.h"
* Removed tBTIF_STATUS, BTIF_SUCCESS and BTIF_ERROR_SRV_* error codes
  which are never used.
* Removed duplicate conditional define of BTA_AV_CO_CP_SCMS_T
* Removed unused typedef tBTA_AV_AUDIO_CODEC_INFO, and unused
  BTA_AV_CO_* defines and enums.

Change-Id: I7bb49957fad94073d474fea6ea0d466f36df37fb

Merge "Update linux BUILD.gn files" am: c143b4a3fa
am: 0c3164c963

Change-Id: I0d9dd8ac08e6e20eb0078240000898611f3cf839

Merge "Update linux BUILD.gn files"
am: c143b4a3fa

Change-Id: Iad689fa7313868a072f7e1836d39d47ca4e471aa

Merge "Update linux BUILD.gn files"

Free av_open_on_rc_timer on cleanup am: 8981fb2f35
am: 5a79d6da20

Change-Id: I3f8f9724c571afcb10b98123b8cffdd795701fdd

Free av_open_on_rc_timer on cleanup
am: 8981fb2f35

Change-Id: I4a429c099df2ca47c1759499127aeb267bb22937

Update linux BUILD.gn files

Change-Id: Iccec195d35b5a62d7b45834670229a68deeb4add

Remove log spam in btif_gattc_update_properties am: 831bec7af3
am: e678af1282

Change-Id: I6bf3ea3065403197f46edef8817a9a8a8b7eaf31

Remove log spam in btif_gattc_update_properties
am: 831bec7af3

Change-Id: I1c967f90cbb87551b9a50ff2cc6da019871afbfc

Free av_open_on_rc_timer on cleanup

Bug: 27602473
Change-Id: I4b0f1563952be75a6dee238607e6674ce98ea623

Remove log spam in btif_gattc_update_properties

Bug: 29601962
Change-Id: Ibcccbe20a4b7c79edcaff162e4f4a19078ffb75d
(cherry picked from commit ffdb33b5fcfa0a652e9fa368f7f8c3bb0654b46e)

Merge "Don't set the A2DP sink data callback when registering A2DP source" am: e1211822cc
am: 245e7ee851

Change-Id: Ifd9a276afed3d4051ed0d0e54074b8523a543c48

Merge "Don't set the A2DP sink data callback when registering A2DP source"
am: e1211822cc

Change-Id: I4f984c5ebdfdb7c5d03f92c05bf98209bbd82a9d

Merge "Don't set the A2DP sink data callback when registering A2DP source"

Removed duplicate and unused BTA_AV_ and BTIF_AV_ constants am: 89330a3888
am: cce0a0674a

Change-Id: I3bbebe617d401e18467298cfcdf7caf3ea9002a0

Removed duplicate and unused BTA_AV_ and BTIF_AV_ constants
am: 89330a3888

Change-Id: I4d2b8e6816b589d5f83978098a3754f66c6e896c

Removed duplicate and unused BTA_AV_ and BTIF_AV_ constants

* Replaced usage of BTA_AV_CODEC_SBC and BTIF_AV_CODEC_SBC
  with A2D_MEDIA_CT_SBC
* Removed remaining unused BTA_AV_CODEC_* re-defines
* Removed other BTA_* constants and re-defines that are never used
* Removed unused BTIF_AV_ constants and typedefs

Change-Id: I38090acee3dfdcd1196185e0b060cdad7572003a

Don't set the A2DP sink data callback when registering A2DP source

The p_sink_data_cback pointer should be NULL when registering
A2DP source with BTA_AvRegister()

Change-Id: I1a709c4e0d3cfe6faf0368a013b391c6e6a53a82

Merge "Removed unused log-related defines and typedefs" am: fc32808f2f
am: 1d1f81df8d

Change-Id: I204a6f1f1e270dd84bdfcf4faa0123189a14ed76

Merge "Removed unused log-related defines and typedefs"
am: fc32808f2f

Change-Id: Ie48e32bafe7a0ba7baedc1138b525062d136127d

Merge "Removed unused log-related defines and typedefs"

Make it possible to post closures to BTA message queue am: e5c78954fb
am: b981c12c3b

Change-Id: Ia3f3f48479eee6fc0cb6ac177b2b3cc8499b8428

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD am: 2cb07a3468
am: 66e7938a30

Change-Id: Id92e90db29d3692028861bc672e194ee6892d86d

test_vendor: Build independently from the stack am: 8f981328ad
am: 735ce72d77

Change-Id: Ib29e7ad0b6fda22e9ea6892060cd91aa4d0fb691

test_vendor: Fix style for UNUSED_ATTR, whitespace am: 98dcd0babf
am: 46fedfd9bb

Change-Id: I3cfc9ba5570ea50596f0f7c58c4b0906d602a35c

Make it possible to post closures to BTA message queue
am: e5c78954fb

Change-Id: Ic90c6e430f33949695926f602ed9f801bebed6cc

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD
am: 2cb07a3468

Change-Id: Idfe3f56752b12a4961c692f2dd33117c0c02a77b

test_vendor: Build independently from the stack
am: 8f981328ad

Change-Id: I8635261da07e0ea20a3db3624e776c69d467ec6e

test_vendor: Fix style for UNUSED_ATTR, whitespace
am: 98dcd0babf

Change-Id: Idc44d0bc0f1c445d2b08bdd22436c47e062504e4

Make it possible to post closures to BTA message queue

Currently, every task executed on bta thread needs a separate event id,
structure holding its parameters, API method, and method that will be
executed in bta thread, which must accept generic tBTA_DM_MSG.

Because of that we end up with lots of boilerplate code, and possibility
to misinterpret bytes in tBTA_DM_MSG struct.

After this patch, we can post closures onto bta thread. Thanks to it we
don't need a struct to hold parameters, and the executed method can have
a meaningful parameters.

First experiments showed over 50% reduction in code size when rewriting
advertising code to new style, not to mention much better readability.

Bug: 30622771
Change-Id: I400839b920c04c67c893899969d07a02d9c1b2e6

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD

Removed unused log-related defines and typedefs

Change-Id: I794f07821a653f03e02cb5ad11790dfe2dfaea98

test_vendor: Build independently from the stack

Remove the dependency on bluetooth_* flags.
Enable more warnings.

(Based on the build flags in system/bt/Android.mk)

Change-Id: I5f480f15c037955404502a0216b61aa407670163
Signed-off-by: Myles Watson <mylesgw@google.com>

test_vendor: Fix style for UNUSED_ATTR, whitespace

Remove "/* args */" and use UNUSED_ATTR.  It's easier to find using grep.

Change-Id: Ic1a95e44b86d019647acafe982cfef5c69e1da1a
Signed-off-by: Myles Watson <mylesgw@google.com>

Fix a typo: __func_ -> __func__ am: 3e0f253c28
am: 3426fd6085

Change-Id: Iff5a8701c63ab68e280fd166d899aa58e41ea3d9

Fix a typo: __func_ -> __func__
am: 3e0f253c28

Change-Id: I4309954020a7630fa1dc4eca766fc12e982fbefc

Fix a typo: __func_ -> __func__

Change-Id: I8b6f0664a96932c35a804541d54a50370d09c022

A2DP-related naming refactoring and cleanup am: 7aaf7c6c42
am: 4e89dc5ac8

Change-Id: I40b313ac63ca8092a73901e64a5a80e78e9ffda5

A2DP-related naming refactoring and cleanup
am: 7aaf7c6c42

Change-Id: I2420388d1335ab2ff9edef957ed0934433e3e63e

A2DP-related naming refactoring and cleanup

* Renamed A2DP-related function names, types, constants and fields.
  Also, move some of those functions to a better location.
 - tAVDT_DATA_CBACK -> tAVDT_SINK_DATA_CBACK
 - tAVDT_MEDIA_CBACK -> tAVDT_SINK_MEDIA_CBACK
 - p_data_cback -> p_sink_data_cback
 - p_media_cback -> p_sink_media_cback
 - p_app_data_cback -> p_app_sink_data_cback
 - tBTA_AV_DATA_CBACK -> tBTA_AV_SINK_DATA_CBACK
 - BTA_AV_MEDIA_SINK_CFG_EVT -> BTA_AV_SINK_MEDIA_CFG_EVT
 - BTA_AV_MEDIA_DATA_EVT -> BTA_AV_SINK_MEDIA_DATA_EVT
 - bta_av_stream_data_cback() -> bta_av_sink_data_cback()
 - bte_av_media_callback() -> bte_av_sink_media_callback()
 - tBTA_AV_CODEC -> tA2D_CODEC
 - btif_a2dp_get_track_frequency() -> A2D_sbc_get_track_frequency()
 - btif_a2dp_get_track_channel_count() -> A2D_sbc_get_track_channel_count()
 - dump_codec_info() -> A2D_sbc_dump_codec_info()

* Misc. other cleanup:
  - log messages cleanup
  - normalize usage of "+=" in Android.mk files

Change-Id: Ida1528fb5d75cc322533f921daefb65be44562f1

test_vendor: Allow includes from bt_hci_defs am: a11e1c1b71
am: ee0c32d010

Change-Id: Ic41e9fc0f1ce86bcb7cd9dee5594bac9db664c60

Removed BTA_AV_INCLUDED conditional guard which is always true am: bde45bd653
am: fb8d7890a1

Change-Id: Ib05380ae4ecfa4150db9811af051cb754e710302

test_vendor: Allow includes from bt_hci_defs
am: a11e1c1b71

Change-Id: Ie3494281cda35263f0b0a259eef045541aba3585

Removed BTA_AV_INCLUDED conditional guard which is always true
am: bde45bd653

Change-Id: I3e3775b77c304fa5d440fc841bb6c76efbe3ef89

test_vendor: Allow includes from bt_hci_defs

Avoid duplicating header files for BT constants.

Update Android.mk to search $(BT_DIR)/include and
$(BOARD_BLUETOOTH_BDROID_BUILDCFG_INCLUDE_DIR)

Change-Id: I1403724c1cf9cda8156a5b65cf55da8178bd910f
Signed-off-by: Myles Watson <mylesgw@google.com>

Removed BTA_AV_INCLUDED conditional guard which is always true

Change-Id: Idf7063359600b30c007fbde89dde55ef987c0368