Skip to content

Fix bluetooth shell issue #90065

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

CanWang001
Copy link

@CanWang001 CanWang001 commented May 16, 2025

This PR contains three patches:

  1. Remove redundant spaces in string.
  2. Fix issue that BR security level cannot be set to 4 by shell command
  3. Fix issue that BR connection is not selected when multiple connections exist and then one of connections is disconnected.

This string contains two consecutive spaces. Remove one of them.

Signed-off-by: Can Wang <[email protected]>
Host stack supports to set BR security level to 4 but the security level
cannot be set to 4 by the shell command.

Update the code to support BR security level 4.

Signed-off-by: Can Wang <[email protected]>
@CanWang001 CanWang001 marked this pull request as ready for review May 17, 2025 14:53
@github-actions github-actions bot added area: Bluetooth area: Bluetooth Host Bluetooth Host (excluding BR/EDR) labels May 17, 2025
LE and BR connection have already been established, after that, LE
disconnection occurs, BR connection will not be selected as the next
default connection.

Fix this issue by searching for both BR and LE after disconnection
occurs.

Signed-off-by: Can Wang <[email protected]>
@CanWang001 CanWang001 force-pushed the fix_bt_shell_issue branch from 5b26acd to ab43e18 Compare May 19, 2025 11:42
@CanWang001 CanWang001 requested a review from Thalley May 19, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants