Skip to content

nrfx: fix errata7 workaround in nrfx_nvmc_uicr_word_read/write#146

Open
campbellyoung wants to merge 1 commit into
nordicsemi:masterfrom
campbellyoung:nrfx-uicr-tfm-fix
Open

nrfx: fix errata7 workaround in nrfx_nvmc_uicr_word_read/write#146
campbellyoung wants to merge 1 commit into
nordicsemi:masterfrom
campbellyoung:nrfx-uicr-tfm-fix

Conversation

@campbellyoung

Copy link
Copy Markdown

The errata 7 workaround is broken in the current state, and it causes issues with NRF Connect SDK 3.2.1.

Symptom: if I read UICR via TF-M, the actual read works fine, but the next time I generate a panic, it will HardFault instead of being handled properly.

I'm not sure specifically WHY this happens, but this patch fixes it, and seems more logically consistent with the actual errata.

Root issue:

NRF_ERRATA_DYNAMIC_CHECK does a flash read in nrf91_errata_7 ( when it checks NRF_FICR_S) which happens before the actual memory barrier executes.

Pull requests are a little annoying with all the different repos, so I've done the PR here and added a working sample to reproduce the issue below, hope that's fine.

https://github.com/campbellyoung/sdk-nrf/blob/tfm-uicr-panic/samples/crypto/tfm_uicr_panic/

@CLAassistant

CLAassistant commented Apr 8, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@nika-nordic

Copy link
Copy Markdown
Contributor

hi @campbellyoung ,

thanks for reporting this. We will look into the issue internally and come back to you

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.

3 participants