-
Notifications
You must be signed in to change notification settings - Fork 1.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
applications: nrf_desktop: Use high GD frequency on nRF54H20 #20469
base: main
Are you sure you want to change the base?
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 399aeca0e88cdc04070023662a7f0ec82bd1bf15 more detailssdk-nrf:
Github labels
List of changed files detected by CI (3)
Outputs:ToolchainVersion: 4cff34261a Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
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.
thanks, @ppryga-nordic .
one question - if this is required why this is not a default option?
You can find the documentation preview for this PR at this link. Note: This comment is automatically posted by the Documentation Publish GitHub Action. |
applications/nrf_desktop/configuration/nrf54h20dk_nrf54h20_cpurad/images/ipc_radio/prj.conf
Outdated
Show resolved
Hide resolved
...tions/nrf_desktop/configuration/nrf54h20dk_nrf54h20_cpurad/images/ipc_radio/prj_release.conf
Outdated
Show resolved
Hide resolved
0f84885
to
3533f4a
Compare
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
73ab86e
to
0f3ffe4
Compare
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
0f3ffe4
to
62e00aa
Compare
After documentation is built, you will find the preview for this PR here. Preview links for modified nRF Connect SDK documents: |
That is more complex. In essence that should not be used at all, I'm going to add a runtime request to MPSL. The option, as I was told, was provided as optimization for some (don't know which) application, to avoid immediate request to high frequency during boot process and then releasing it back. |
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.
Approved with a minor nit.
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
e916f23
to
d1c1ae1
Compare
In nRF Desktop application, the DVFS module running on the application core selects clock control and enables automatic low GD frequency request. Update IPC radio configuration to ensure high GD frequency, which is required to ensure stability of the Bluetooth stack with SoftDevice Link Layer. Jira: NCSDK-31824 Signed-off-by: Marek Pieta <[email protected]> Signed-off-by: Piotr Pryga <[email protected]> Signed-off-by: Pekka Niskanen <[email protected]>
d1c1ae1
to
399aeca
Compare
In nRF Desktop application, the DVFS module running on the application core selects clock control and enables automatic low GD frequency request. Update IPC radio configuration to ensure high GD frequency, which is required to ensure stability of the Bluetooth stack with SoftDevice Link Layer.
Jira: NCSDK-31824