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:917446] [v2] Bluetooth: btusb: add sysfs attribute to control USB alt setting #2618

Open
wants to merge 1 commit into
base: workflow
Choose a base branch
from

Conversation

BluezTestBot
Copy link
Owner

When a Bluetooth raw socket is open, the HCI event related to SCO
connection changes are not dispatched to the hci_event module, and
the underlying Bluetooth controller's USB Interface 1 will not be
updated accordingly.

This patch adds isoc_alt sysfs attribute, allowing user space
to update the alternate setting of the USB interface alternate
setting as needed.

Signed-off-by: Ying Hsu [email protected]

This commit has been tested on a chromebook with AX211.

Changes in v2:

  • Add check for the existence of a valid USB interface 1.

drivers/bluetooth/btusb.c | 34 +++++++++++++++++++++++++++++++++-
1 file changed, 33 insertions(+), 1 deletion(-)

When a Bluetooth raw socket is open, the HCI event related to SCO
connection changes are not dispatched to the hci_event module, and
the underlying Bluetooth controller's USB Interface 1 will not be
updated accordingly.

This patch adds `isoc_alt` sysfs attribute, allowing user space
to update the alternate setting of the USB interface alternate
setting as needed.

Signed-off-by: Ying Hsu <[email protected]>
Copy link

CheckPatch
Desc: Run checkpatch.pl script
Duration: 0.24 seconds
Result: PENDING

Copy link

GitLint
Desc: Run gitlint
Duration: 0.24 seconds
Result: PENDING

Copy link

SubjectPrefix
Desc: Check subject contains "Bluetooth" prefix
Duration: 0.09 seconds
Result: PASS

Copy link

BuildKernel
Desc: Build Kernel for Bluetooth
Duration: 24.52 seconds
Result: PASS

Copy link

CheckAllWarning
Desc: Run linux kernel with all warning enabled
Duration: 28.65 seconds
Result: PASS

Copy link

CheckSparse
Desc: Run sparse tool with linux kernel
Duration: 30.25 seconds
Result: PASS

Copy link

BuildKernel32
Desc: Build 32bit Kernel for Bluetooth
Duration: 24.18 seconds
Result: PASS

Copy link

TestRunnerSetup
Desc: Setup kernel and bluez for test-runner
Duration: 430.44 seconds
Result: PASS

Copy link

TestRunner_l2cap-tester
Desc: Run l2cap-tester with test-runner
Duration: 20.19 seconds
Result: PASS

Copy link

TestRunner_iso-tester
Desc: Run iso-tester with test-runner
Duration: 29.42 seconds
Result: PASS

Copy link

TestRunner_bnep-tester
Desc: Run bnep-tester with test-runner
Duration: 4.76 seconds
Result: PASS

Copy link

TestRunner_mgmt-tester
Desc: Run mgmt-tester with test-runner
Duration: 119.16 seconds
Result: FAIL
Output:

Total: 490, Passed: 484 (98.8%), Failed: 2, Not Run: 4

Failed Test Cases
LL Privacy - Start Discovery 2 (Disable RL)          Failed       0.190 seconds
LL Privacy - Set Device Flag 1 (Device Privacy)      Failed       0.153 seconds

Copy link

TestRunner_rfcomm-tester
Desc: Run rfcomm-tester with test-runner
Duration: 7.52 seconds
Result: PASS

Copy link

TestRunner_sco-tester
Desc: Run sco-tester with test-runner
Duration: 9.32 seconds
Result: PASS

Copy link

TestRunner_ioctl-tester
Desc: Run ioctl-tester with test-runner
Duration: 8.02 seconds
Result: PASS

Copy link

TestRunner_mesh-tester
Desc: Run mesh-tester with test-runner
Duration: 5.86 seconds
Result: PASS

Copy link

TestRunner_smp-tester
Desc: Run smp-tester with test-runner
Duration: 6.91 seconds
Result: PASS

Copy link

TestRunner_userchan-tester
Desc: Run userchan-tester with test-runner
Duration: 5.02 seconds
Result: PASS

Copy link

IncrementalBuild
Desc: Incremental build with the patches in the series
Duration: 0.61 seconds
Result: PENDING

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