Skip to content

Conversation

@FrancescoSer
Copy link
Contributor

Updated nRF54H20 PM optimization guide.

Copilot AI review requested due to automatic review settings November 6, 2025 08:51
@NordicBuilder NordicBuilder added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 6, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the nRF54H20 power management optimization guide by adding comprehensive technical documentation about the device's power management architecture and optimization strategies.

Key changes:

  • Added detailed explanation of logical domains, power domains, and the locality principle
  • Documented peripheral access strategies, DMA constraints, and system-off entry prerequisites
  • Included practical integration guidelines and optimization checklists

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 6, 2025

CI Information

To view the history of this post, click the 'edited' button above
Build number: 10

Inputs:

Sources:

sdk-nrf: PR head: 724649aa4ab6ebefe15fe16c2fd584bfa12ede6a

more details

sdk-nrf:

PR head: 724649aa4ab6ebefe15fe16c2fd584bfa12ede6a
merge base: 619d1660fa866e88fe4c5c302f5af099b85a093c
target head (main): 0c79406a493836eff01af3bf4a75b49a5a47ca16
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 (2)
doc
│  ├── nrf
│  │  ├── app_dev
│  │  │  ├── device_guides
│  │  │  │  ├── nrf54h
│  │  │  │  │  │ ug_nrf54h20_pm_optimization.rst
│  │  ├── 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
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-wifi
    • test-secdom-samples-public

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

@github-actions
Copy link

github-actions bot commented Nov 6, 2025

Copilot AI review requested due to automatic review settings November 20, 2025 08:59
@FrancescoSer FrancescoSer marked this pull request as ready for review November 20, 2025 08:59
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@FrancescoSer FrancescoSer added this to the 3.2.0 milestone Nov 25, 2025
Copilot AI review requested due to automatic review settings November 25, 2025 17:24
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -353,32 +353,62 @@ When choosing between single-core and dual-core architectures (using either the
return 0;
}

Deep-sleep policy
=================
Disabling power management
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Splitting this into a new PR

Copy link

@pizi-nordic pizi-nordic left a comment

Choose a reason for hiding this comment

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

This looks good. I have mostly pointed minor things, which IMHO makes text more clear.
Part about DMA and MRAM interaction needs to be changed, as suggested usage of MRAM in DMA transaction sounds strange. See relevant comment for details.

The :c:func:`pm_policy_latency_request_add`, :c:func:`pm_policy_latency_request_update`, and :c:func:`pm_policy_latency_request_remove` functions allow you to set a maximum latency requirement.
The system will not enter any power state that would cause the wake-up latency to exceed the specified value.

Setting a latency of ``0`` microseconds effectively disables all power management, as no power state can meet a zero-latency requirement.
Copy link
Contributor

Choose a reason for hiding this comment

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

nrfconnect/sdk-zephyr#3542 adds functionality that allows locking all power states in a much faster way. To lock all power states pm_policy_state_all_lock_get() is used and pm_policy_state_all_lock_put() to unlock. It has reference counter so lock is released when last user releases the lock.
Locking all states with this API overcomes a PM specific locking so if full lock is taken unlocking a specific state will keep it locked until full lock is released.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I'll clarify this later on, for now I'm separating PM disabling edits from the rest of the PR

Copilot AI review requested due to automatic review settings November 27, 2025 12:44
@NordicBuilder NordicBuilder requested a review from a team November 27, 2025 12:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings November 27, 2025 14:45
@FrancescoSer FrancescoSer requested a review from a team as a code owner November 27, 2025 14:45
@NordicBuilder NordicBuilder removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 27, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Updated nRF54H20 PM optimization guide.

Signed-off-by: Francesco Domenico Servidio <[email protected]>
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.

5 participants