Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cannot enable MPU for nucleo_l552ze_q_ns #27809

Closed
NolwennV opened this issue Aug 26, 2020 · 2 comments · Fixed by #30226
Closed

Cannot enable MPU for nucleo_l552ze_q_ns #27809

NolwennV opened this issue Aug 26, 2020 · 2 comments · Fixed by #30226
Assignees
@erwango
Labels
area: Modules bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug

Comments

@NolwennV
Copy link

NolwennV commented Aug 26, 2020 •
edited
Loading

The current L5 TF-M implementation configures and enables both a secure and a non secure MPU. This is incompatible with an activation from Zephyr side (non secure binary).
This is a known issue tracked at : https://developer.trustedfirmware.org/T806

As a result, the MPU stay disabled for the target nucleo_l552ze_q_ns in the PR #27524.
@NolwennV NolwennV added the bug The issue is a bug, or the PR is fixing a bug label Aug 26, 2020
@NolwennV NolwennV changed the title MPU disabled for nucleo_l552ze_q_ns Cannot enable MPU for nucleo_l552ze_q_ns Aug 26, 2020
@NolwennV NolwennV mentioned this issue Aug 26, 2020
Merged
@erwango erwango self-assigned this Aug 27, 2020
@MaureenHelm MaureenHelm added the priority: low Low impact/importance bug label Sep 1, 2020
@github-actions
Copy link

github-actions bot commented Nov 1, 2020

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.

@github-actions github-actions bot added the Stale label Nov 1, 2020
@erwango erwango removed the Stale label Nov 2, 2020
@erwango
Copy link
Member

erwango commented Nov 2, 2020

This issue should be kept open for now.

erwango added a commit to erwango/zephyr that referenced this issue Nov 25, 2020
@erwango
boards: nulceo_l552ze_q: Enable MPU on ns target
8902a2a 
The bug on TFM that prevented to enable MPU on NS side is
fixed on TFM current version.
MPU can now be safely enabled on NS target.

Fixes: zephyrproject-rtos#27809

Signed-off-by: Erwan Gouriou <[email protected]>
@erwango erwango mentioned this issue Nov 25, 2020
Merged
ioannisg pushed a commit that referenced this issue Nov 25, 2020
@erwango @ioannisg
boards: nulceo_l552ze_q: Enable MPU on ns target
610fd27 
The bug on TFM that prevented to enable MPU on NS side is
fixed on TFM current version.
MPU can now be safely enabled on NS target.

Fixes: #27809

Signed-off-by: Erwan Gouriou <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erwango erwango

Labels
area: Modules bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

boards: nulceo_l552ze_q: Enable MPU on ns target erwango/zephyr
3 participants
@MaureenHelm @erwango @NolwennV