Skip to content

platform: nordic_nrf: Add mramc service for nrf7120#249

Merged
degjorva merged 4 commits into
nrfconnect:mainfrom
travis3630:WZN-9267-nrf7120-nrf-mramc-service-tfm
May 18, 2026
Merged

platform: nordic_nrf: Add mramc service for nrf7120#249
degjorva merged 4 commits into
nrfconnect:mainfrom
travis3630:WZN-9267-nrf7120-nrf-mramc-service-tfm

Conversation

@travis3630
Copy link
Copy Markdown
Contributor

nrf7120 zephyr mramc driver needs to access some configuration registers in mramc which is secure only, nrf_mramc service is created to access MRAMC initialise and change write config of mramc. MRAMC service functions is added under iotcl service type.

@travis3630
Copy link
Copy Markdown
Contributor Author

Hi, I am creating nrf_mramc service under ioctl, is that okay? PR is open here and will go upstream later if you all generally agree on the changes? @Vge0rge @degjorva @AntonZma @frkv @Hi-Im-David @rob-robinson-14

@travis3630
Copy link
Copy Markdown
Contributor Author

Changes in sdk-nrf: nrfconnect/sdk-nrf#28399
soc-flash-nrf-mramc update in zephyr: https://github.com/zephyrproject-rtos/zephyr/pull/107181/commits: [tests: driver: flash: Add tfm-mramc service support to nrf_mramc driver]

@Vge0rge
Copy link
Copy Markdown
Contributor

Vge0rge commented May 5, 2026

Added support for NRFX KMU driver for Nordic targets
to exchange LIB_KMU.

Change-Id: I0e8e7e40611374f1fe5a8c183cf40200e2aec999
Signed-off-by: Anton Zyma <anton.zyma@nordicsemi.no>
(cherry picked from commit 8eef667)
@travis3630 travis3630 force-pushed the WZN-9267-nrf7120-nrf-mramc-service-tfm branch 2 times, most recently from c92e700 to d721535 Compare May 11, 2026 11:47
This reverts commit 5880bf3.

Signed-off-by: Travis Lam <travis.lam@nordicsemi.no>
nrf7120 zephyr mramc driver needs to access some configuration
registers in mramc which is secure only, nrf_mramc service is
created to access MRAMC initialise and change write config of
mramc. MRAMC service functions is added under iotcl service type.

Change-Id: I93e411a0a51c8d96f1c5239efa006afab304c72e
Signed-off-by: Travis Lam <travis.lam@nordicsemi.no>
(cherry picked from commit 66503e7c7c9d644f39a9ca35550d6c5af4792cd5)
@travis3630 travis3630 force-pushed the WZN-9267-nrf7120-nrf-mramc-service-tfm branch 2 times, most recently from b967567 to 9cb2847 Compare May 11, 2026 12:02
Add the tfm_platform_system_off APIs in a similar manner
as the existing tfm_platform_system_reset. This API
should enable implementations to allow setting the
TF-M to the lowest power mode using their own
HAL APIs.

Right now this will work for isolation level 1 (SFN mode).
In the IPC mode there is a need for better TF-M support for
this. There is a discussion with the TF-M owners to add logic
to TF-M so that it can inform all the partitions in order
to make sure that it is safe to go to system off mode.

Signed-off-by: Georgios Vasilakis <georgios.vasilakis@nordicsemi.no>
@travis3630
Copy link
Copy Markdown
Contributor Author

There is an error: pathspec 'auto-manifest-sdk-trusted-firmware-m-249' did not match any file(s) known to git

Seems to be the auto-manifest PR in Nordic Builder is not created, any help from anyone?

@tomi-font
Copy link
Copy Markdown
Collaborator

@Vge0rge @nordic-piks

@nordic-piks
Copy link
Copy Markdown

@jangalda-nsc can you help with automanifest update?

@jangalda-nsc
Copy link
Copy Markdown

@jangalda-nsc can you help with automanifest update?

If PR in sdk-nrf was not opened on PR created event, automanifest workflow can't recover and open manifest PR on push events.
I created a branch in Nordicbuilder's sdk-nrf fork, which made workflow green, but manifest PR is still not open.
@travis3630 is it ok for you handle manifest PR on your own?

@travis3630
Copy link
Copy Markdown
Contributor Author

@jangalda-nsc can you help with automanifest update?

If PR in sdk-nrf was not opened on PR created event, automanifest workflow can't recover and open manifest PR on push events. I created a branch in Nordicbuilder's sdk-nrf fork, which made workflow green, but manifest PR is still not open. @travis3630 is it ok for you handle manifest PR on your own?

Sure, I have the manifest PR here: https://github.com/nrfconnect/sdk-nrf/pull/28399/commits , can anyone review again and get it merged

Copy link
Copy Markdown
Contributor

@frkv frkv left a comment

Choose a reason for hiding this comment

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

LGTM

@degjorva degjorva merged commit e45f5f8 into nrfconnect:main May 18, 2026
1 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants