Page MenuHomePhabricator
Create Task
Maniphest T651

tfm_spm_hal_post_init functionality is broken
Closed, ResolvedPublic
Actions

Description

The original idea of adding tfm_spm_hal_post_init was to let tfm do hardware (or any other) initialization that cannot be done in SystemInit() in asm startup code. One of the user-case scenarios is an initialization that creates variables and data structures that are used later in platform drivers. C library init wipes off these variables.

Please see the following changes for reference.
https://review.trustedfirmware.org/c/trusted-firmware-m/+/1973
https://review.trustedfirmware.org/c/trusted-firmware-m/+/2211
https://review.trustedfirmware.org/c/trusted-firmware-m/+/2212

The patch that was merged recently (https://review.trustedfirmware.org/c/trusted-firmware-m/+/3040) messes things up: it moves isolation protection above the post init call. Instead of moving tfm_spm_hal_init_isolation_hw() above tfm_spm_hal_post_init, it had to take UART init out and move it below tfm_spm_hal_init_isolation_hw().

Essentially it brakes psoc64 target. We came up with some workarounds, but hopefully this issue can be addressed properly.

Event Timeline

ainh created this task.Jan 16 2020, 10:07 PM
KenLSoft added a comment.Jan 19 2020, 5:43 AM

Hi Andrei,

Looks like this caused the UART not working - are you using UART in the core? Just collecting feedback since we are check if we could remove UART logging from core (convert this to another logging mechanism).

And the prefix 'post' is going to be removed - The SystemInit is a CMSIS involved init which is out of core scope, for core, there is only one init for it.

I assume providing a new solution would need weeks (discussions and feedbacks), please create workarounds if you want to make it work asap before the new mechanism is decided.

/Ken

ainh added a comment.Jan 22 2020, 1:15 AM

Hi Ken,
yes, we are using UART in core to print some info like protection status and boot progress.
For the hw initialization, SystemInit called from startup code is not sufficient enough as it cannot pass status and data to the rest of the platform code written in C. This was the main reason for adding post init function.

KenLSoft added a comment.Oct 14 2020, 9:49 AM

Close it as no follow-ups, assume it has been fixed.

KenLSoft closed this task as Resolved.Oct 14 2020, 9:50 AM