Clean-up stale/unused hci_request.c code
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v7 Latest9 patches 1 replyIn review 8 of 9 applied
- v610 patches 1 replySuperseded
- v510 patches 1 replySuperseded
Earlier versions · 4
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v4 | 4 patches | 14 replies | Superseded | |
| v3 | 4 patches | 1 reply | Superseded | |
| v2 | 3 patches | 1 reply | Applied 3/3 | |
| v1 | 3 patches | 1 reply | Superseded |
Patches in v7 8 of 9 applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/9 | Bluetooth: Convert le_scan_disable timeout to hci_syncnet/ net/ | +74 | 1 reply | Applied |
| 2/9 | Bluetooth: Rework le_scan_restart for hci_syncnet/ net/ | +75 | 0 replies | Applied |
| 3/9 | Bluetooth: Delete unused hci_req_stop_discovery()net/ net/ | +0 | 0 replies | Applied |
| 4/9 | Bluetooth: Convert SCO configure_datapath to hci_syncnet/ net/ net/ | +75 | 0 replies | Applied |
| 5/9 | Bluetooth: Move Adv Instance timer to hci_syncinclude/ net/ net/ +2 more | +125 | 0 replies | Applied |
| 6/9 | Bluetooth: Delete unreferenced hci_request codenet/ net/ | +2 | 0 replies | Applied |
| 7/9 | Bluetooth: move hci_get_random_address() to hci_syncinclude/ net/ net/ +1 more | +170 | 0 replies | Applied |
| 8/9 | Bluetooth: convert hci_update_adv_data to hci_syncinclude/ net/ net/ +3 more | +23 | 0 replies | Applied |
| 9/9 | Bluetooth: Convert hci_abort_conn to hci_syncinclude/ net/ net/ +3 more | +135 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v7 yet.
Cover letter [PATCH v7 0/9] Clean-up stale/unused hci_request.c code
This will be a growing patch-set of conversions and dead-code removal towards the goal of retiring hci_request.c
The patch sets will be split amoung the work queues and delayed work queues as initialized in hci_request_setup(), with the ultimate goal of eliminating hci_request.c entirely.
v2: Published
v3: Continuing work. This does include one conversion (SCO configure_datapath) that has been tested with mgmt-tester and sco-tester, but has not been tested with a controller with an off-loadable codec.
v4: Clean-up checkpatch warnings.
v5: Remove remaining procedures from hci_request.c/h. Not that other entities (notably amp.c, hci_event.c and hci_core.c) are still using the hci_request mechanism, however, the mgmt-tester unit test only currently execute opcode 0x1408 - HCI_OP_READ_ENC_KEY_SIZE during normal testing.
v6: Fix CI reported errors, and recoded hci_abort_conn for optional sync queue usage to fix some L2CAP test cases.
v7: Delete and rebase to remove conflicting interleave scan clean-up. Will be done using work Luiz has been working on.
Brian Gix (9):
Bluetooth: Convert le_scan_disable timeout to hci_sync
Bluetooth: Rework le_scan_restart for hci_sync
Bluetooth: Delete unused hci_req_stop_discovery()
Bluetooth: Convert SCO configure_datapath to hci_sync
Bluetooth: Move Adv Instance timer to hci_sync
Bluetooth: Delete unreferenced hci_request code
Bluetooth: move hci_get_random_address() to hci_sync
Bluetooth: convert hci_update_adv_data to hci_sync
Bluetooth: Convert hci_abort_conn to hci_sync
include/net/bluetooth/hci_sync.h | 10 +- net/bluetooth/hci_conn.c | 92 +- net/bluetooth/hci_core.c | 2 +- net/bluetooth/hci_event.c | 2 +- net/bluetooth/hci_request.c | 1650 +++--------------------------- net/bluetooth/hci_request.h | 53 - net/bluetooth/hci_sync.c | 500 ++++++++- net/bluetooth/mgmt.c | 7 +- 8 files changed, 733 insertions(+), 1583 deletions(-)
Signature · 1 line
2.37.1