-
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
Add cpu_load module from Zephyr and align it with existing cpu_load module #21098
base: main
Are you sure you want to change the base?
Conversation
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 project with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 8e99030e143d312c588a894384dabd6ad28a2fb3 more detailssdk-nrf:
zephyr:
Github labels
List of changed files detected by CI (40)
Outputs:ToolchainVersion: 4ffa2202d5 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR here. Preview links for modified nRF Connect SDK documents: https://ncsdoc.z6.web.core.windows.net/PR-21098/nrf/libraries/debug/cpu_load.html |
doc/nrf/releases_and_maturity/releases/release-notes-3.0.0-preview1.rst
Outdated
Show resolved
Hide resolved
0cc616c
to
187c6b1
Compare
Zephyr is getting generic CPU load module and it is using same Kconfig name. NCS CPU load module is Nordic specific because it is using POWER SLEEPENTER,SLEEPEXIT events to track idle state and CPU load. It is also not support on all targets (nrf54h20, nRF54L Series). Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
Improve PPI handling to use GPPI where possible and DPPI directly only in case of sharing DPPI channels. Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
Add note about added prefix to cpu_load Kconfig options. Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
Add cpu_load module from upstream. Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
187c6b1
to
8e99030
Compare
|
@@ -625,7 +625,9 @@ Common Application Framework | |||
Debug libraries | |||
--------------- | |||
|
|||
|no_changes_yet_note| | |||
* :ref:`cpu_load` library: |
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.
BTW, this entry should most likely go to https://github.com/nrfconnect/sdk-nrf/blob/main/doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Fetching cpu_load module from Zephyr upstream. This module is generic and does not require PPI or POWER->SLEEPENTER/SLEEPEXIT events so it can be used on nrf54h20 (app/rad) and nrf54l (app, flpr). Upstream module uses the same Kconfig name so Kconfig options for existing NCS cpu_load module got
NRF_
prefix. API name is not modified.