Skip to content

boards: nordic: nrf54h20dk: Allocate 64KB more MRAM to cpurad #88662

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

olivier-le-sage
Copy link
Contributor

Allocate more MRAM to cpurad by reducing the size of the cpuapp DFU partition.

This is motivated by the amount of NVM needed for important advanced BLE features on this platform, like channel sounding.

Allocate more MRAM to cpurad by reducing the size of the cpuapp DFU
partition.

This is motivated by the amount of NVM needed for important advanced
BLE features on this platform, like channel sounding.

Signed-off-by: Olivier Lesage <[email protected]>
@olivier-le-sage olivier-le-sage marked this pull request as ready for review April 16, 2025 05:59
@github-actions github-actions bot added the platform: nRF Nordic nRFx label Apr 16, 2025
@hubertmis
Copy link
Member

hubertmis commented Apr 23, 2025

How many samples (what percentage) use the advanced Bluetooth features that require more memory?

If the number is relatively low, I would suggest adding device tree overlays in these few samples instead of reducing the app's (or DFU) space in the majority of them. Reducing the default app's space gives a bad first impression for nRF54H20 evaluators, as they see they have little room for their application (or updates).

@olivier-le-sage
Copy link
Contributor Author

How many samples (what percentage) use the advanced Bluetooth features that require more memory?

If the number is relatively low, I would suggest adding device tree overlays in these few samples instead of reducing the app's (or DFU) space in the majority of them. Reducing the default app's space gives a bad first impression for nRF54H20 evaluators, as they see they have little room for their application (or updates).

I may be misunderstanding, but isn't a 900kB DFU partition already very, very large? To a point that reducing it to ~850kB has little impact?

It's difficult for me to comment on what percentage of 54h20 customers would need this extra space when using BLE since we only really know this at link time, and I don't know all their use cases. But considering that, I would have thought it would be better to allocate a bit too much than too little, since it avoids creating possible pitfalls and a need for complicated device tree manipulation on the user side

@hubertmis
Copy link
Member

The DFU partition was temporarily expanded to work around missing DFU features like the update of a single CPU at a time. It is intended to be shrunk back, and the Application's CPU code should utilize the remaining size.

I see that you propose utilizing some of this area for Radio. It would result in a smaller area available for the Application.

The default partition sizes we create in the board files for DKs should be fitting well a 'typical application'. So that if someone evaluates the SoC, they don't need to start repartitioning anytime soon. We already have a few samples and applications that don't fit these default partition sizes, and they repartition in the sample-specific .overlay files.

So, if we have only one sample (channel sounding) that requires more memory in the Radio CPU than allocated by default, I would recommend following the pattern and creating an overlay file for this sample. If creating this overlay starts becoming a rule because multiple samples don't fit in the default memory map, then we should consider changing the default memory map.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: nRF Nordic nRFx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants