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

Update MPSL and SoftDevice Controller #1088

Conversation

NordicBuilder
Copy link
Contributor

Created from: latest/sub/test-fw-nrfconnect-ble/master #29640.

Signed-off-by: Sean Madigan [email protected]

CHANGELOG.rst contains the list of changes.

Signed-off-by: Sean Madigan <[email protected]>
CHANGELOG.rst contains the list of changes.

Signed-off-by: Sean Madigan <[email protected]>
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Oct 10, 2023
@sean-madigan sean-madigan added this to the 2.5.0 milestone Oct 10, 2023
* @return Returns value between 0x01-0xFF in case of error.
* See Vol 2, Part D, Error for a list of error codes and descriptions.
*/
uint8_t sdc_hci_cmd_vs_set_power_control_request_params(const sdc_hci_cmd_vs_set_power_control_request_params_t * p_params);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this something we should mention in the changelog?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@seko-nordic , it was intentional not mention anything now, right?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes this was on purpose. Reason being that it is not completely implemented, but once it is, it will replace the two existing VS commands sdc_hci_cmd_vs_set_auto_power_control_request_param and sdc_hci_cmd_vs_set_power_control_apr_handling. We want to minimize confusion with this, as customers cannot use it yet. (Answering as Sergey is in vacation at the moment.)

@rlubos rlubos merged commit dd75e0d into nrfconnect:main Oct 11, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants