CMSIS_5 icon indicating copy to clipboard operation
CMSIS_5 copied to clipboard

Update core_cm33.h

Open AlbertHuang-CPU opened this issue 4 years ago • 2 comments

Add SFSR and SFAR in SCB_Type.

AlbertHuang-CPU avatar Jul 21 '21 09:07 AlbertHuang-CPU

Hi @AlbertHuang-CPU,

We checked the header for these registers and it looks like they are already part of the SAU_Type. This is aligned with the Arm®v8-M Architecture Reference Manual.

Is it a fair statement that is should be enough to have the registers defined there? Or are there any specific requirements to have the registers being part of the SCB_Type in addition?

Thanks, Jonatan

JonatanAntoni avatar Jul 21 '21 10:07 JonatanAntoni

Hi Jonatan,

In Arm v8m definition, the address is just in the SCB scope. @.***D77E62.79588C60] image

In practical, SAU can be configured out, and be taken over by implementation defined IDAU. That means although there is no SAU, the secure fault still possible happens and the SFSR still need to be accessed.

Regards, Albert

From: Jonatan Antoni @.> Sent: Wednesday, July 21, 2021 6:18 PM To: ARM-software/CMSIS_5 @.> Cc: Albert Huang @.>; Mention @.> Subject: Re: [ARM-software/CMSIS_5] Update core_cm33.h (#1255)

Hi @AlbertHuang-CPUhttps://github.com/AlbertHuang-CPU,

We checked the header for these registers and it looks like they are already part of the SAU_Type. This is aligned with the Arm®v8-M Architecture Reference Manual.

Is it a fair statement that is should be enough to have the registers defined there? Or are there any specific requirements to have the registers being part of the SCB_Type in addition?

Thanks, Jonatan

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ARM-software/CMSIS_5/pull/1255#issuecomment-884072819, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AN45N2XAYEIXY3NA7M3LMMLTY2NFFANCNFSM5AXTPCWQ.

AlbertHuang-CPU avatar Jul 21 '21 10:07 AlbertHuang-CPU