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

[Backport v3.5-branch] Bluetooth: Controller: Fix missing ext adv terminate event #64317

Merged
merged 2 commits into from
Nov 10, 2023

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Oct 24, 2023

Backport f64d123~2..f64d123 from #64275.

Fixes #65056.

Fix periodic advertising sync window calculation to include
the scheduling resolution margin, i.e. be double as with
the event jitter value.

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
(cherry picked from commit 05c85dd)
Fix missing Extended Advertising terminate event and
advertising scheduling not being stopped.

Under race conditions, auxiliary event is aborted without
the generation  of done extra event which is suppose to
stop the scheduling when max events count is used.

The fix now generates the done extra event.

Signed-off-by: Vinayak Kariappa Chettimada <[email protected]>
(cherry picked from commit f64d123)
@jhedberg
Copy link
Member

@cvinayak there needs to be a matching issue (bug report) before this backport can be accepted

@jhedberg
Copy link
Member

jhedberg commented Nov 1, 2023

@cvinayak there needs to be a matching issue (bug report) before this backport can be accepted

@cvinayak @carlescufi ping

@jhedberg
Copy link
Member

jhedberg commented Nov 7, 2023

@cvinayak there needs to be a matching issue (bug report) before this backport can be accepted

@cvinayak @carlescufi ping

@cvinayak @carlescufi ping

@jhedberg jhedberg merged commit 086b5b9 into v3.5-branch Nov 10, 2023
24 checks passed
@nashif nashif deleted the backport-64275-to-v3.5-branch branch February 13, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Controller area: Bluetooth Backport Backport PR and backport failure issues
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants