Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PW_SID:918026] Bluetooth: hci: Remove deadcode #2035

Closed
wants to merge 2 commits into from
Closed

Conversation

tedd-an
Copy link
Owner

@tedd-an tedd-an commented Dec 16, 2024

From: "Dr. David Alan Gilbert" [email protected]

hci_bdaddr_list_del_with_flags() was added in 2020's
commit 8baaa40 ("Bluetooth: Add bdaddr_list_with_flags for classic
whitelist")
but has remained unused.

hci_remove_ext_adv_instance() was added in 2020's
commit eca0ae4 ("Bluetooth: Add initial implementation of BIS
connections")
but has remained unused.

Remove them.

Signed-off-by: Dr. David Alan Gilbert [email protected]

include/net/bluetooth/hci_core.h | 2 --
include/net/bluetooth/hci_sync.h | 1 -
net/bluetooth/hci_core.c | 20 --------------------
net/bluetooth/hci_sync.c | 24 ------------------------
4 files changed, 47 deletions(-)

tedd-an and others added 2 commits December 12, 2024 21:24
This patch adds workflow files for ci:

[schedule_work.yml]
 - The workflow file for scheduled work
 - Sync the repo with upstream repo and rebase the workflow branch
 - Review the patches in the patchwork and creates the PR if needed

[ci.yml]
 - The workflow file for CI tasks
 - Run CI tests when PR is created
hci_bdaddr_list_del_with_flags() was added in 2020's
commit 8baaa40 ("Bluetooth: Add bdaddr_list_with_flags for classic
whitelist")
but has remained unused.

hci_remove_ext_adv_instance() was added in 2020's
commit eca0ae4 ("Bluetooth: Add initial implementation of BIS
connections")
but has remained unused.

Remove them.

Signed-off-by: Dr. David Alan Gilbert <[email protected]>
@tedd-an tedd-an closed this Jan 6, 2025
@tedd-an tedd-an deleted the 918026 branch January 6, 2025 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant