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

Cherry pick zbus and host fixes #1375

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

alexsven
Copy link
Contributor

@alexsven alexsven commented Nov 6, 2023

Needed changes for fixing a looping issue when removing a PA sync and adding message subscribers to zbus

rodrigopex and others added 3 commits November 6, 2023 08:15
Besides the changed channel reference, the message subscribers receive a
copy of the message during the VDED execution. ZBus guarantees message
delivery for `MSG_SUBSCRIBERS`.

Signed-off-by: Rodrigo Peixoto <[email protected]>
(cherry picked from commit 0b0aa43)
Signed-off-by: Alexander Svensen <[email protected]>
…e PA Sync

When calling bt_bap_broadcast_sink_delete, the broadcast sink
should not attempt to terminate the PA Sync. The PA sync can live
on without the broadcast sink, just as the broadcast sink can live
on without the PA sync (which is why the PA sync check was completely
removed).

Signed-off-by: Emil Gydesen <[email protected]>
(cherry picked from commit 7339849)
Signed-off-by: Alexander Svensen <[email protected]>
…rminated

If the PA gets terminated, we clear the pa_sync field of the
corresponding broadcast sink object.

Signed-off-by: Emil Gydesen <[email protected]>
(cherry picked from commit c47b7f7)
Signed-off-by: Alexander Svensen <[email protected]>
@alexsven
Copy link
Contributor Author

alexsven commented Nov 6, 2023

Tested in nrfconnect/sdk-nrf#12968

@rlubos rlubos merged commit 7a9d7da into nrfconnect:main Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants