BLE: RF tester commands (and Cordio unhandled command complete) #13055
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@paul-szczepanek-arm, thank you for your changes. |
0xc0170
changed the title
0xc0170
previously requested changes
Jun 3, 2020
…ck was incorrectly picking a define that wasn't even the last in the table. It now uses the size of the table.
…ete events for all commands that it doesn't have a specific handler for. This adds a catch-all handler that allows the user application to handle any command complete that isn't already handled by the stack. This involves adding a new type of event and routing the event through the stack to the device where it's forwarded to the existing event handling in mbed-os.
…allow you to send the HCI commands HCI_LE_Receiver_Test, HCI_LE_Transmitter_Test and HCI_LE_Test_End. The results of the test are obtained by the command complete command for HCI_LE_Test_End and passed to the user by the callback register in the test start calls.
paul-szczepanek-arm
force-pushed
the
cordio-rf-tester
branch
from
30885c5
to
fbe9312
Compare
|
As requested I have expanded the commit messages. |
pan-
approved these changes
Jun 4, 2020
0xc0170
approved these changes
Jun 9, 2020
|
CI started |
Test run: SUCCESSSummary: 6 of 6 test jobs passed |
|
All green but the incorrect status reported from mergify makes it red. I'll merge as it can't be overwritten. |
adbridge
added
Release-pending
and removed
release-type: feature
Release-pending
labels
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of changes
Added a new optional call in the cordio hci driver that allows users to issues rf tester commands for the ble chip.
New event in the cordio stack for command complete. This allows mbed to handle command complete commands not handled by cordio.
This is needed by the rf tester commands in this PR to work since they need the results returned to them.
Impact of changes
Migration actions required
Documentation
none
Pull request type
Test results
Reviewers
@pan-