Skip to content

Reset and release CRC before any nBoot API call - #64

Open
alamfarjadf wants to merge 1 commit into
OpenDevicePartnership:mcxa-secure-bootfrom
alamfarjadf:crc0-reset
Open

alamfarjadf wants to merge 1 commit into
OpenDevicePartnership:mcxa-secure-bootfrom
alamfarjadf:crc0-reset

Conversation

@alamfarjadf

Copy link
Copy Markdown

As recently discovered, MCXA5xx MCUs with ROM version greater than 3 causes a hard fault because ROM does not release CRC from reset on boot.

So, when nBoot APIs are called to authenticate the image, CRC0 is still held in reset and causes a hard fault.

This isn't described in the current version of the RM.

NXP clarified that making sure CRC is clocked and released is now a requirement going forward.

Added enable_and_reset call to CRC0 right after Embassy is initialized in board.rs.

@felipebalbi

Copy link
Copy Markdown
Contributor

Before we accept this, let's get an actual Erratum for this fault. Or at least updated RM or official recommendation. Has anybody done any power impact analysis for this?

@alamfarjadf

Copy link
Copy Markdown
Author

Before we accept this, let's get an actual Erratum for this fault. Or at least updated RM or official recommendation. Has anybody done any power impact analysis for this?

Agreed, we will get more clarity soon.

As for power analysis, no. The change is kind of non-optional.

@felipebalbi

Copy link
Copy Markdown
Contributor

Before we accept this, let's get an actual Erratum for this fault. Or at least updated RM or official recommendation. Has anybody done any power impact analysis for this?

Agreed, we will get more clarity soon.

As for power analysis, no. The change is kind of non-optional.

Understood, however we don't want a bug being filed in a few weeks saying we regressed power targets. So it needs to be documented, IMO :-)

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.

2 participants