Skip to content

Conversation

lylezhu2012
Copy link
Contributor

@lylezhu2012 lylezhu2012 commented Oct 11, 2025

Add documents to show how to use the Classic L2CAP, HFP, RFCOMM, and GOEP shell commands.

makeshi
makeshi previously approved these changes Oct 11, 2025
Add a document to show how to use the Classic L2CAP shell commands.

Signed-off-by: Lyle Zhu <[email protected]>
Add a document to show how to use the Classic HFP shell commands.

Signed-off-by: Lyle Zhu <[email protected]>
Add a document to show how to use the Classic RFCOMM shell commands.

Signed-off-by: Lyle Zhu <[email protected]>
@lylezhu2012 lylezhu2012 force-pushed the classic_doc_shell_classic_l2cap_hfp branch from cdca983 to 94dd630 Compare October 11, 2025 11:36
@lylezhu2012 lylezhu2012 changed the title doc: Bluetooth: shell: Add document for Classic L2CAP and HFP doc: Bluetooth: shell: Add document for Classic L2CAP, HFP, RFCOMM, and GOEP Oct 11, 2025
@lylezhu2012 lylezhu2012 requested a review from makeshi October 11, 2025 11:37
@Thalley Thalley removed their request for review October 12, 2025 18:32
makeshi
makeshi previously approved these changes Oct 13, 2025
Add a document to show how to use the Classic GOEP shell commands.

Signed-off-by: Lyle Zhu <[email protected]>
Copy link

@lylezhu2012 lylezhu2012 requested a review from makeshi October 15, 2025 03:09
@cvinayak cvinayak removed their request for review October 15, 2025 04:19
Copy link
Member

@jhedberg jhedberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok. My only concern is that this creates a lot of documentation to be updated every time you make some change to the code that impacts the commands or their output in the shell. That means that there's a substantial risk that this will go out of sync over time.

@henrikbrixandersen henrikbrixandersen merged commit fa61e87 into zephyrproject-rtos:main Oct 17, 2025
20 checks passed
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.

7 participants