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

doc: mpsl: List changes related to nRF54h20 SoC #20892

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

ppryga-nordic
Copy link
Contributor

Added list of changes done to MPSL integration layer that were needed for nRF54h20 SoC support.

@ppryga-nordic ppryga-nordic requested a review from a team as a code owner March 12, 2025 13:33
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Mar 12, 2025
@ppryga-nordic ppryga-nordic force-pushed the mpsl-changelog-3-0-0-update branch from 47bb577 to b89468f Compare March 12, 2025 13:36
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 12, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 4

Inputs:

Sources:

sdk-nrf: PR head: add588661038e49772f3ebcd9d1e5d1be37698e9

more details

sdk-nrf:

PR head: add588661038e49772f3ebcd9d1e5d1be37698e9
merge base: 700816344f6640cf0fa96aff3fe79a6d4d157897
target head (main): 76259e03400cf80ff7da36698ffe44bdde1928ac
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1)
doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

Comment on lines 798 to 804
* Added integration with nrf2 clock control driver for nRF54H20 SoC.
* Added integration with Zephyr's System Power Management for nRF54H20 SoC.
* Added global domain HSFLL120 320MHz frequency request if MPSL is enabled.
The high frequency in global domain is required to ensure that instructions fetching from L2-cache and MRAM is as fast as possible.
It is needed for radio protocols to operate correctly.
* Added MRAM always-on request for scheduled radio events.
It is needed to avoid MRAM wake-up latency for radio protocols.
Copy link
Contributor

@peknis peknis Mar 12, 2025

Choose a reason for hiding this comment

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

Suggested change
* Added integration with nrf2 clock control driver for nRF54H20 SoC.
* Added integration with Zephyr's System Power Management for nRF54H20 SoC.
* Added global domain HSFLL120 320MHz frequency request if MPSL is enabled.
The high frequency in global domain is required to ensure that instructions fetching from L2-cache and MRAM is as fast as possible.
It is needed for radio protocols to operate correctly.
* Added MRAM always-on request for scheduled radio events.
It is needed to avoid MRAM wake-up latency for radio protocols.
* Added:
* Integration with the nrf2 clock control driver for the nRF54H20 SoC.
* Integration with Zephyr's System Power Management for the nRF54H20 SoC.
* Global domain HSFLL120 320MHz frequency request if MPSL is enabled.
The high frequency in global domain is required to ensure that fetching instructions from L2-cache and MRAM is as fast as possible.
It is needed for the radio protocols to operate correctly.
* MRAM always-on request for scheduled radio events.
It is needed to avoid MRAM wake-up latency for radio protocols.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed as suggested.

Copy link

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-20892/nrf/releases_and_maturity/releases/release-notes-changelog.html

@ppryga-nordic ppryga-nordic force-pushed the mpsl-changelog-3-0-0-update branch from b89468f to a6f5f91 Compare March 17, 2025 13:29
Added list of changes done to MPSL integration layer that
were needed for nRF54h20 SoC support.

Signed-off-by: Piotr Pryga <piotr.pryga@nordicsemi.no>
@ppryga-nordic ppryga-nordic force-pushed the mpsl-changelog-3-0-0-update branch from a6f5f91 to add5886 Compare March 18, 2025 06:02
@ppryga-nordic
Copy link
Contributor Author

@peknis could you review the recent change?

@nordicjm nordicjm merged commit a3f9adf into nrfconnect:main Mar 20, 2025
14 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