This reverts commit 29b0d287.

Reason for revert: Change breaks multiple tests, see b/257340301

Change-Id: I4a5d7ac69b32a6dc1b782e5d4643ca4529108c22

The command, 'advertise on', start an advertising sets with default
parameters and data. This patch allows this command to be executed
more than one time to start multiple advertising sets.
A potential deadlock between btclient command handler and D-Bus
callback while stopping multiple advertising sets is also fixed
in this patch.

Bug: 233128828
Tag: #floss
Test: build.py --target test
Test: manual test with btclient
Change-Id: I68c50670e9101ce0618516a14f75e7d10e8a301a

* changes:
  RootCanal: Remove unecessary std::move in DualModeController
  RootCanal: Misc improvements

 - Add StorageModule to LeScanningManager
 - Add logic for removal

Bug: 236301249
Test: system/gd/cert/run --clean
Test: system/gd/cert/run --clean --sl4a LeAdvancedScanningTest
Test: system/gd/cert/run --clean --sl4a_sl4a LeIrkScanningTest
Tag: #refactor
Change-Id: I9c8bbb7aa6ef0b19eecf33f832301bd5ca494603

- Add definition of SetEventMaskPage2 in hci_packets.pdl
- Use generated class for SetResolvablePrivateAddressTimeout
- Remove unecessary validation of LeIsoCommand views

Test: m root-canal
Change-Id: I223c9f56839a734f6b11e37a7ae3417ef9c25153

- Remove some declared but un-implemented functions
  in DualModeController
- Move suggested_max_tx_octets and suggested_max_tx_time
  to the link layer controller
- Change the type of extended inquiry response
  to std::array<uint8_t, 240>, which accurately
  reflects the length of the response as passed
  from the HCI command

Test: m root-canal
Change-Id: I8837333f9c080454d987a9b7877d8c55e6d0c324

Bug: 191555414
Tag: #refactor
Test: gd/cert/run

Change-Id: Ie011725544accafc9c4cf2ad2b9f4ce6b1e61bac

* changes:
  unittest: Enum-ify tBTA_DM_STATE and add bta_dm_state_text
  unittest: Enum-ify tBTA_DM_EVT and add bta::bta_dm_event_text

* changes:
  Revert^2 "gatt: Remove reduntant `in_use` flag"
  Revert^2 "gatt: Improve send queued request from tGATT_TCB"
  Revert^2 "gatt: Improve queueing of GATT messages"

Update the sink audio location after loaded from the storage

Bug: 252935428
Tag: #refactor
Test: atest BluetoothInstrumentationTests
Test: BT off/on, and check the battery information
Change-Id: I820b43acb552b776b0322112ee3070ed0930b431

Fix obvious mistake.
Error can be visible if multiple BLE devices is in used, some of the
GATT commands are queued and then first disconnected device has no queue
GATT commands.

Bug: 257046947
Test: manual
 - Connect BLE devices X, Y
 - Queue multiple commands on device X
 - Disconnect device Y

Tag: #feature
Change-Id: Ic0877bf7322dfa5bf0103bb6eab18cadac51244b

This patch uses enum AdvertisingStatus to represent the status of
advertising callbacks, which aligns with the implementation in
libbluetooth.

Bug: 257215274
Tag: #floss
Test: build.py --target test
Test: manual test on volteer with btclient
Change-Id: I47786e242c5b75b1331b14fbe66c0a363c0f732b

75cc3cb7

Change-Id: I8a6669ee978c87edf6752a1c7719797df346bfa0

dd9e6591

Change-Id: Id5711aae12b1f9c5922c28fc7c58f26c6aa68266

267a4807

Change-Id: I8e0ee9fa832207de6cd8e8c5c7f7b5a45a6f282a

* changes:
  Revert "gatt: Improve queueing of GATT messages"
  Revert "gatt: Improve send queued request from tGATT_TCB"
  Revert "gatt: Remove reduntant `in_use` flag"

Revert submission 2268950

Bug: 257046947
Bug: 249431973
Test: reverting changes that broke earlier build
Reason for revert: b/257046947 infinite loop
Reverted Changes:
If19de77a8:gatt: Improve queueing of GATT messages
I1bfb4c546:gatt: Remove reduntant `in_use` flag
I1d3351ebc:gatt: Improve send queued request from tGATT_TCB

Change-Id: Ie898cf668476edfc09b8d94218bab16116268810

Revert submission 2268950

Bug: 257046947
Bug: 249431973
Test: reverting changes that broke earlier build
Reason for revert: b/257046947 infinite loop
Reverted Changes:
If19de77a8:gatt: Improve queueing of GATT messages
I1bfb4c546:gatt: Remove reduntant `in_use` flag
I1d3351ebc:gatt: Improve send queued request from tGATT_TCB

Change-Id: I2c925d07981aca37a36986e4214d70167b9e1cf6

Revert submission 2268950

Bug: 257046947
Bug: 249431973
Test: reverting changes that broke earlier build
Reason for revert: b/257046947 infinite loop
Reverted Changes:
If19de77a8:gatt: Improve queueing of GATT messages
I1bfb4c546:gatt: Remove reduntant `in_use` flag
I1d3351ebc:gatt: Improve send queued request from tGATT_TCB

Change-Id: Ief553d755b4002030a4ae99e16d9f5ebba325235