History log of /system/bt/stack/btm/ble_advertiser_hci_interface.cc
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
be8d3f542517246c9cc8b3b02f2519baf300c53f 28-Aug-2017 Jakub Pawlowski <jpawlowski@google.com> Add Suspend/Resume for advertising

This is needed for resolving list handling.

Bug: 64846264
Test: updated unit tests
Change-Id: I3d9c7b90d3b69d459d33c4ca7a9849ca3a7abc40
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
9e030fde05352ec4385d7baf6cc2af89e95e039c 25-Jun-2017 Jakub Pawlowski <jpawlowski@google.com> Rename bt_bdaddr_t into RawAddress (3/3)

Test: compilation test
Change-Id: I4571721a0c6165a66450ee157a53d8d15bfc45d0
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
b8a477e9ac51898b05cb328eda897d95f1ef3d02 17-Jun-2017 Jakub Pawlowski <jpawlowski@google.com> Get rid of BD_ADDR

Test: compilation test
Change-Id: I8493485873a81f897fbaef1504076ef18274a8a5
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
c2276b06572ab6fc1f900fbb1f41087e77d47e2a 10-Jun-2017 Jakub Pawlowski <jpawlowski@google.com> Use bt_bdaddr_t instead of BD_ADDR

Test: compilation test, sl4a GattReadTest
Change-Id: I8d1bd6914aec55bb53495b1d0d5e3d37b86865e6
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
a85cb8ea676601e430e0bbc4dc534129d1e0303f 01-Jun-2017 Sunny Kapdi <sunnyk@codeaurora.org> Fix params of SetPeriodicAdvertisingParameters

Fix the sequence of SetPeriodicAdvertisingParameters
in the hci_cmd

Bug: 62268713
Change-Id: Ibdcd362458e9d7ee2f95106a2aefa76e1c264103
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
1f62c122e908573497a8c69ccd7bd829ce02a0b9 04-Apr-2017 Jakub Pawlowski <jpawlowski@google.com> Advertising manager improvements

* Keep track wether advertiser is enabled.
* Make sure that random address for connectable advertisers is updated
when advertising is disabled, as per ESR11-E7716.
* Make sure that the local variable holding the address is properly
updated after the address is pushed to the controller.
* Use "LE Remove Advertising Set" command to free advertiser after use.

Bug: 35935853
Bug: 30622771
Test: manual
Change-Id: I1415f7272dd99e5e81ce1e2b7ef2bf98f7229cf9
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
256afc4d84cc0b05b96881ade6bcbfbab8636c51 17-Mar-2017 Jakub Pawlowski <jpawlowski@google.com> Report back TX Power value

For LE Extended Advertising report the value returned by controller.
For VSC Advertising return the value send to controller.
For Legacy Advertising use 0.

Test: manual
Bug: 30622771
Change-Id: Ifc49af86e7f064c91ed2d20e6f87532159d1bffa
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
10661946ccaf2e542d24ca0c8d837aa97fc3b8c1 20-Mar-2017 Jakub Pawlowski <jpawlowski@google.com> Do not use Multi-Adv instance 0 in VSC

Bug: 36365653
Test: manual
Change-Id: If44b2f0206bab91fa77f2a1ff68d83e1593f28e2
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
bb858fbf93ed8c3b72dd8bcd8ad3fbcb65e5bb7a 12-Mar-2017 Jakub Pawlowski <jpawlowski@google.com> Add "LE Remove Advertising Set Command"

Bug: 30622771
Test: manual
Change-Id: Ie0df73475359c69af8df672454fe0c0374523c88
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
8e774d1c533dde1b0775aa61933b24eef2213de4 13-Mar-2017 Lakshmipathi K <lakshmipathi.k@intel.com> LE Periodic Advertising - advertiser interface

Bug: 30622771
Test: manual
Change-Id: I358b872c9ffb52f007db4f111520c05b9521aa7f
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
cd0d18937e85cd55bcee6355a6688858eba63b16 24-Feb-2017 Jakub Pawlowski <jpawlowski@google.com> Fix initial advertising command order

Set the address during SetParameters, for VSC implementation, and right
after setting parameters for BT5 implementation.

Test: sl4a ConcurrentBleAdvertisingTest
Bug: 33922058
Bug: 34962013
Change-Id: Idcc60a36884607de191066d8be1ee3bc459e86fc
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
9bb5a643906130367d3a1bb57141ef2ebd6c7855 23-Feb-2017 Jakub Pawlowski <jpawlowski@google.com> Log which advertising HCI interface is used

This is useful for debugging advertising issues.

Test: manual
Change-Id: Ia51eccb1ff7922a5864572a89e77404299965862
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
6d75adcc03975cf6c9cae95349233aa6c3a19da2 23-Jan-2017 Jakub Pawlowski <jpawlowski@google.com> Add advertising handle zero quirk

Some controllers are strict about VSC handle mapping, and don't allow
using zero value.

Test: manual test
Change-Id: Ia2df17af16e0e4da375d6e7d5b5f882ced508ce9
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
dabf8d36c6a6896953fff4a12365716d438f173a 28-Dec-2016 Jakub Pawlowski <jpawlowski@google.com> HCI interface improvements

This patch improves btu_hcif_send_cmd_with_cb, which will accept
base::Callback instead of function pointer as argument. It will also
carry information about where was packet send from, which can be
displayed when command status is received, greatly improving logs.

The improved method is also used for handling VSC advertising.

Bug: 30622771
Test: sl4a ConcurrentBleAdvertiserTest
Change-Id: I55c3e32f9231c00bfa85f971265809d6f1fecdfb
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
699888256f683c3b14305e834d1e622f66241e29 02-Dec-2016 Jakub Pawlowski <jpawlowski@google.com> Update BleAdvertiserHciInterface

This patch updates BleAdvertiserHciInterface to be on pair with
Bluetooth 5.0 LE Advertising Extension feature.

Bug: 30622771
Test: net_test_stack_multi_adv native test
Change-Id: Ie0ec8c7f8ed0d7a874e392123b3486c767121140
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
751381c64fee63590f0888c8eac5c8a474f1e25b 30-Nov-2016 Jakub Pawlowski <jpawlowski@google.com> Handle Advertising Set Terminated event

Make BleAdvertiserHciExtendedImpl report when advertising set is
disabled because of new connection.

Bug: 30622771
Test: sl4a ConcurrentBleAdvertisingTest
Change-Id: Ic13a31fe4bb92f121a29d540274d13893775a450
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
5af96d3fda59989740f694cf7683d55f66057cf4 29-Nov-2016 Jakub Pawlowski <jpawlowski@google.com> Advertising Extension Implementation

This patch implements current BleAdvertiserHciInterface using LE
Advertising extension HCI commands.

Bug: 30622771
Test: sl4a ConcurrentBleAdvertisingTest
Change-Id: Icaaf24d6d02d8a887b3bb57d5b858b929f11c413
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
911d1ae03efec2d54c3b1b605589d790d1745488 29-Nov-2016 Myles Watson <mylesgw@google.com> Apply clang-format to the rest of the tree

find * | grep "\.[ch]" | xargs clang-format --style=file -i

Test: mma -j32
Change-Id: I6fcc9862bb7bc07c2a367ca58fef2b3cd27a6f05
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
06325e3d5ddb863230f57e60bebac71ef7cb7ccc 14-Nov-2016 Jakub Pawlowski <jpawlowski@google.com> Reenable advertising instance after an LE advertising state change

Ensure BleAdvertisingManager::OnAdvertisingStateChanged() is called
after an advertising state change to re-enable advertising after
a connection.

This patch also fixes the registration test, which was broken since
one more advertising instance was freed for general usage.

Test: net_test_stack_multi_adv native test
Change-Id: I76887d98cb3ec386ebc0f7fad95170b8c18b9116
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
d964cf944e23470d696177fb2ddf8df3684290b8 03-Nov-2016 Jakub Pawlowski <jpawlowski@google.com> Unify advertising API (1/3)

This patch unifies advertising API. On startup, if VSC multi advertising
is avaliable, it will be used, otherwise
BleAdvertiserLegacyHciInterfaceImpl will be used, which provides one
advertising instance.

Test: sl4a tests are passing, all proper tests updated accordingly
Change-Id: I73753962e3a33a980e8a9a1174c8d88611a47e6b
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
6c0ca7abd9452473bcf6cc467923202d093dac59 20-Oct-2016 Jakub Pawlowski <jpawlowski@google.com> Various small warning/build fixes for Linux

Test: build with ninja
Change-Id: Ic0f4d23e4a56a42be6e660f2afec83400ea6daa6
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
1dd77510884826e6996264aa40db4f495c3ec67a 10-Oct-2016 Jakub Pawlowski <jpawlowski@google.com> Make advertising instance count part of the HCI interface

The number of advertising instances will ultimately depend on the HCI
interface used. BleAdvertisingManagerImpl should not depend on a global
function for that.

Test: Covered by BleAdvertiseApiTest sl4a test
Bug: 30622771
Change-Id: I1399de3f4289708f7218eae9c00ac7372e4246db
/system/bt/stack/btm/ble_advertiser_hci_interface.cc
e47b769e71fa88f207db747e62974662e43aa07c 28-Sep-2016 Jakub Pawlowski <jpawlowski@google.com> Make Bluetooth LE multi-advertising testable

This patch turns the code that manages LE multi-advertising into the
BleAdvertisingManager class. All HCI calls made by this code were
abstracted into BleAdvertiserHciInterface.

Thanks to those changes, we can mock BleAdvertiserHciInterface and test
the advertising manager.

The BleAdvertisingManagerTest, still has some dependencies on other
code from the "stack" module. These have to be implemented on top of the
test file. In the future, this dependencies should be replaced by mockable
classes.

Test: Tests covering the change are added in this patch.
Bug: 30622771
Change-Id: I4a363678815c235ee01255f462db2a671ef89610
/system/bt/stack/btm/ble_advertiser_hci_interface.cc