subsys: partition_manager: Fix RAM size for vpr_launcher #23026

nordic-segl · 2025-06-27T12:42:15Z

Fix size of RAM avaiable to vpr_launcher (cpuapp). Hardcoded value of 0x28000 is correct for nRF54L15 only.

Hardcode values for:

nRF54L09,
nRF54LV10A,
nRF54L20,
nRF54LM20A,
nRF7120.

nordic-segl · 2025-06-27T12:43:04Z

subsys/partition_manager/Kconfig

0x67c00 taken from https://github.com/nrfconnect/sdk-nrf/blob/main/snippets/nordic-flpr/soc/nrf54lm20a_cpuapp.overlay#L18C31-L18C40

nordic-segl · 2025-06-27T12:43:26Z

subsys/partition_manager/Kconfig

0x6fc00 taken from https://github.com/nrfconnect/sdk-zephyr/blob/main/snippets/nordic-flpr/soc/nrf54l20_cpuapp.overlay#L18C34-L18C42

nordic-segl · 2025-06-27T12:43:50Z

subsys/partition_manager/Kconfig

0x1fc00 taken from https://github.com/nrfconnect/sdk-nrf/blob/main/snippets/nordic-flpr/soc/nrf54lv10a_cpuapp.overlay#L17C37-L17C42

nordic-segl · 2025-06-27T12:44:19Z

subsys/partition_manager/Kconfig

0x23c00 taken from https://github.com/nrfconnect/sdk-zephyr/blob/main/snippets/nordic-flpr/soc/nrf54l09_cpuapp.overlay#L17C37-L17C42

NordicBuilder · 2025-06-27T12:45:08Z

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: ceb7c0b3b59fba6056070ec79750c978142be9ee

more details

sdk-nrf:

PR head: ceb7c0b3b59fba6056070ec79750c978142be9ee
merge base: e914cbfc928d2869fd652f8d8a406e6df393e6ad
target head (main): e914cbfc928d2869fd652f8d8a406e6df393e6ad
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)

subsys
│  ├── partition_manager
│  │  ├── Kconfig
│  │  │ pm.yml.vpr_launcher

Outputs:

Toolchain

Version: 3ae5dc3c63
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:3ae5dc3c63_776d264d2e

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

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
✅ Integration tests
- ✅ test-sdk-audio
- ✅ desktop52_verification
- ✅ test-fw-nrfconnect-boot
- ✅ test-fw-nrfconnect-apps
- ✅ test_ble_nrf_config
- ✅ test-fw-nrfconnect-ble_mesh
- ✅ test-fw-nrfconnect-ble_samples
- ✅ test-fw-nrfconnect-chip
- ✅ test-fw-nrfconnect-nfc
- ✅ test-fw-nrfconnect-nrf-iot_cloud
- ✅ test-fw-nrfconnect-nrf-iot_libmodem-nrf
- ✅ test-fw-nrfconnect-nrf-iot_serial_lte_modem
- ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- ✅ test-fw-nrfconnect-nrf-iot_samples
- ✅ test-fw-nrfconnect-nrf-iot_lwm2m
- ✅ doc-internal
- ✅ test-fw-nrfconnect-nrf-iot_thingy91
- ✅ test-fw-nrfconnect-nrf_crypto
- ✅ test-fw-nrfconnect-rpc
- ✅ test-fw-nrfconnect-rs
- ✅ test-fw-nrfconnect-fem
- ✅ test-fw-nrfconnect-tfm
- ✅ test-fw-nrfconnect-thread-main
- ✅ test-sdk-find-my
- ✅ test-fw-nrfconnect-nrf_lrcs_mosh
- ✅ test-fw-nrfconnect-nrf_lrcs_positioning
- ✅ test-sdk-wifi
- ✅ test-low-level
- ✅ test-sdk-pmic-samples
- ✅ test-sdk-mcuboot
- ✅ test-fw-nrfconnect-ps-main
- ✅ test-secdom-samples-public
- ⚠️ test-fw-nrfconnect-fw-update

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

nordic-segl · 2025-06-27T12:45:44Z

subsys/partition_manager/Kconfig

0x28000 for L15 is set here https://github.com/nrfconnect/sdk-zephyr/blob/main/snippets/nordic-flpr/soc/nrf54l15_cpuapp.overlay#L17

I don't know what is the correct value for nRF7120.

Fix size of RAM avaiable to vpr_launcher (cpuapp). Hardcoded value of 0x28000 is correct for nRF54L15 only. Hardcode values for: - nRF54L09, - nRF54LV10A, - nRF54L20, - nRF54LM20A, - nRF7120. Signed-off-by: Sebastian Głąb <[email protected]>

nordic-piks · 2025-06-27T13:05:24Z

Helps at CI (at least one test)

NordicBuilder · 2025-06-27T13:06:42Z

The following west manifest projects have changed revision in this Pull Request:

Name	Old Revision	New Revision	Diff

✅ All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

nordic-segl · 2025-06-27T13:21:58Z

I applied changes to counter_basic_api as described in NRFX-7902 (see commit DNM just for testing).
I'm unable to reproduce the mcause 2 issue.

15:15:20  INFO    - 1/1 [email protected]/nrf54lv10a/cpuflpr nrf.extended.drivers.counter.basic_api  PASSED (device: 1051628602, 22.972s <zephyr>)

github-actions · 2025-06-27T13:46:21Z

You can find the documentation preview for this PR here.

nordic-krch · 2025-06-27T13:56:06Z

subsys/partition_manager/Kconfig

+config PM_RAM_SIZE_VPR_LAUNCHER
+	hex
+	depends on SOC_NRF54L15_CPUFLPR || SOC_NRF54L09_ENGA_CPUFLPR || SOC_NRF54LV10A_ENGA_CPUFLPR || SOC_NRF54L20_ENGA_CPUFLPR || SOC_NRF54LM20A_ENGA_CPUFLPR || SOC_NRF7120_ENGA_CPUFLPR
+	default 0x28000 if SOC_NRF54L15_CPUFLPR || SOC_NRF7120_ENGA_CPUFLPR


Those values are redundant, already in DT. I was wondering if you could use something like:

default $(int2hex,$(sub,$(dt_nodelabel_reg_addr_int,cpuflpr_sram),536870912/*0x20000000*/))

We are in flpr Kconfig so cpuapp_sram cannot be used. Note that currently there is no int2hex function available but i've tried to add it and it is quite easy.

it's already in upstream: zephyrproject-rtos/zephyr#91552

Currently, I get build error.

Traceback (most recent call last): File "/home/segl/workspace/ncs/zephyr/scripts/kconfig/kconfiglib.py", line 2786, in _expand_macro res += args[int(new_args[0])] ^^^^^^^^^^^^^^^^ ValueError: invalid literal for int() with base 10: 'sub'

Let's use hardcoded values for now and improve it later.

NordicBuilder · 2025-06-27T14:17:52Z

Memory footprint analysis revealed the following potential issues

applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12430[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9178[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 9090[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 5846[B] - link (cc: @nrfconnect/ncs-ll-ursus)

Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-23026/4)

nordic-segl requested review from a team, nordicjm and tejlmand as code owners June 27, 2025 12:42

github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jun 27, 2025

nordic-segl requested review from nordic-krch and nordic-piks June 27, 2025 12:42

nordic-segl commented Jun 27, 2025

View reviewed changes

nordic-piks approved these changes Jun 27, 2025

View reviewed changes

nordic-segl force-pushed the NRFX-NONE_Fix-vpr_launcher-RAM-size-for-cpuapp branch from 9a8f943 to ceb7c0b Compare June 27, 2025 12:47

kl-cruz approved these changes Jun 27, 2025

View reviewed changes

nordic-segl requested review from a team as code owners June 27, 2025 13:05

github-actions bot added the manifest label Jun 27, 2025

NordicBuilder added manifest-zephyr DNM labels Jun 27, 2025

nordic-piks approved these changes Jun 27, 2025

View reviewed changes

nordic-segl force-pushed the NRFX-NONE_Fix-vpr_launcher-RAM-size-for-cpuapp branch from d33a75d to ceb7c0b Compare June 27, 2025 13:22

github-actions bot removed the manifest label Jun 27, 2025

NordicBuilder removed manifest-zephyr DNM labels Jun 27, 2025

tejlmand approved these changes Jun 27, 2025

View reviewed changes

de-nordic approved these changes Jun 27, 2025

View reviewed changes

nordic-krch reviewed Jun 27, 2025

View reviewed changes

nordic-piks merged commit a96ee0f into nrfconnect:main Jun 30, 2025
22 of 25 checks passed

nordic-segl deleted the NRFX-NONE_Fix-vpr_launcher-RAM-size-for-cpuapp branch June 30, 2025 06:01

subsys: partition_manager: Fix RAM size for vpr_launcher #23026

subsys: partition_manager: Fix RAM size for vpr_launcher #23026

Uh oh!

Conversation

nordic-segl commented Jun 27, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

NordicBuilder commented Jun 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

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

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nordic-piks commented Jun 27, 2025

Uh oh!

NordicBuilder commented Jun 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nordic-segl commented Jun 27, 2025

Uh oh!

github-actions bot commented Jun 27, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

NordicBuilder commented Jun 27, 2025

Memory footprint analysis revealed the following potential issues

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

NordicBuilder commented Jun 27, 2025 •

edited

Loading

NordicBuilder commented Jun 27, 2025 •

edited

Loading