-
Notifications
You must be signed in to change notification settings - Fork 3k
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
BLE: fix advertising set termination event #14747
Conversation
@paul-szczepanek-arm, thank you for your changes. |
@paul-szczepanek-arm, thank you for your changes. |
If you add WIP: as a prefix to the headline, it will append do not merge label automatically. I'll do it manually now |
b0dac0d
to
b4be302
Compare
b4be302
to
cc0f81f
Compare
The neatness of the commits belies the violent history of this PR. |
0072938
to
78e620a
Compare
the hci handle is different from host handle, this is how it's done in conn open event, we carry the conn id in the hdr.param
f64eb66
to
afeb696
Compare
@chrisswinchatt-arm , since you're doing a rebase, this PR includes a new cordio fix - you'll have to include that when it's merged. |
note to self: merge ARMmbed/mbed-os-example-ble#380 after this is merged |
This pull request has automatically been marked as stale because it has had no recent activity. @pan-, @chrisswinchatt-arm, @ARMmbed/mbed-os-maintainers, please complete review of the changes to move the PR forward. Thank you for your contributions. |
Come on guys, please review. |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Approved to be in the release candidate we are about to produce soon. |
Summary of changes
This fixes the wrong handle and status being reported in the advertising stop event. To make the event consistent (and correct) a workaround at CORDIO level is needed.
Now the terminate event has a correctly translated status with the connected field set based on whether the event has a valid connection id. The connection id is now translated from HCI id. Added documentation to clarify the behaviour.
Impact of changes
Migration actions required
Documentation
none
Pull request type
Test results
Reviewers