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.

Sign up for GitHub

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

Jump to bottom

ISO Broadcaster fails silently on unsupported RTN/SDU_Interval combination #46186

Closed
GatCode opened this issue Jun 2, 2022 · 3 comments · Fixed by #49394
Closed

ISO Broadcaster fails silently on unsupported RTN/SDU_Interval combination #46186

GatCode opened this issue Jun 2, 2022 · 3 comments · Fixed by #49394
Assignees
@cvinayak
Labels
area: Bluetooth Controller area: Bluetooth ISO Bluetooth LE Isochronous Channels bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Milestone
v3.2.0

Comments

@GatCode
Copy link
Contributor

GatCode commented Jun 2, 2022 •
edited

When a user selects a specific RTN/SDU_Interval combination in the ISO_BROADCAST sample, broacaster fails without an error message. The user doesn't know what has happened.

As @Thalley suggested, a short error message would be nice to have.

Note: tested on two nRF52840DK boards with the code of the following draft PR: #46187

Screenshot 2022-06-02 at 12 18 32 PM
@GatCode GatCode added the Enhancement Changes/Updates/Additions to existing features label Jun 2, 2022
@GatCode GatCode mentioned this issue Jun 2, 2022
Closed
@Thalley Thalley added area: Bluetooth Controller area: Bluetooth ISO Bluetooth LE Isochronous Channels labels Jun 2, 2022
@Thalley
Copy link
Collaborator

Thalley commented Jun 2, 2022

The error seen is that no ISO data is seemingly received on the sync receiver, right @GatCode ?

@Thalley Thalley added bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug and removed Enhancement Changes/Updates/Additions to existing features labels Jun 2, 2022
@GatCode
Copy link
Contributor Author

GatCode commented Jun 2, 2022

Exactly, only the BigInfo is printed on the receiver side.
Additionally the output on the sender side is exactly the screenshot above but it should print Sending value ... every second.

@Thalley
Copy link
Collaborator

Thalley commented Jun 2, 2022

Exactly, only the BigInfo is printed on the receiver side. Additionally the output on the sender side is exactly the screenshot above but it should print Sending value ... every second.

So at least the periodic advertisement seems to be fine. From your log, it seems that just the BIG sync never completes.

@github-actions github-actions bot added the Stale label Aug 2, 2022
@zephyrproject-rtos zephyrproject-rtos deleted a comment from github-actions bot Aug 2, 2022
@cvinayak cvinayak removed the Stale label Aug 2, 2022
@cvinayak cvinayak mentioned this issue Aug 23, 2022
Merged
@carlescufi carlescufi added this to the v3.2.0 milestone Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cvinayak cvinayak

Labels
area: Bluetooth Controller area: Bluetooth ISO Bluetooth LE Isochronous Channels bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
v3.2.0
Development

Successfully merging a pull request may close this issue.

Bluetooth: Controller: Fix BIG Create silently fail on RTN/SDU_interval cvinayak/zephyr
4 participants
@Thalley @cvinayak @carlescufi @GatCode