-
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: Add tx power control to cordio HCI driver #13040
Conversation
@paul-szczepanek-arm, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks good to me. It would be good to have review from an ST maintainer.
@ARMmbed/mbed-os-maintainers Can you help with that ?
CI started |
Test run: SUCCESSSummary: 6 of 6 test jobs passed |
@Mergifyio refresh |
Command |
I merged another related PR, this is now in conflict with master. |
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
const int8_t conversion[] = { | ||
-40, -21 - 10 - 19, -18, -16, -15, -14, -13, -12, -11, -10, -9, -8, | ||
-7, -6, -5, -4, -3, -2, -2, -1, -1, 0, 0, 0, 1, 2, 3, 4, 5, 6, | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it missing commas between -21, -10 and -19? I tested using both ways and with commas the level_db for 6dbm is higher than without comma.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for catching that, that's some copy paste catastrophe, now fixed and double checked against STM32WBx5 application note
81151de
to
fe98309
Compare
@felipeLeast @0xc0170 are you guys happy with this now ? |
Yes! |
CI started |
Test run: SUCCESSSummary: 6 of 6 test jobs passed |
Summary of changes
Adds a new optional call to the CordioHCIDriver. This exposes controller support for setting TX power. This may or may not be implemented by each board. An error code for unimplemented feature will be returned if it's not present.
Impact of changes
Migration actions required
Documentation
none
Pull request type
Test results
Reviewers
@pan-