-
Notifications
You must be signed in to change notification settings - Fork 2k
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.
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
cpu/stm32: disable MPU for cortex-m0+ (stm32g0 and stm32l052t8) #14823
Conversation
MPU is broken on cortex-m0+ in the current state
cf805e0
to
fd71e09
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack, similar to #14806
|
can we keep it open until this is really fixed ? |
Only if you change it to "MPU doesn't work on Cortex-m0+" :) |
Done |
We have a lot more Cortex-m0+ than sam32g0 - should we disable the MPU there too? |
I think so yes, they are probably as broken as the stm32g0 |
There's only one stm32l0 cpu model left for cortex-m0+. There's also saml1x but it's cortex-m23 and I don't know the state for those ones. |
Fine with me!
The saml1x is already disabled with #14806. (and yeah, cortex-m23 is also broken.) |
I tried i-nucleo-lrwan1 (the stm32l0) and it's broken the same. |
Wait, this time I forgot to update Kconfig ;) |
ea152cc
to
b4aa2da
Compare
Contribution description
This PR remove the MPU feature from the list of features provided by the stm32g0 family and stm32l052t8 model since MPU is broken on cortex-m0+ and this results in nearly usable state for stm32g0 family and the related stm32l0 model.
Testing procedure
All applications are now working on nucleo-g070rb and i-nucleo-lrwan1
Issues/PRs references
This was reported in #14822 (Let's keep it open until a real fix is proposed and the feature can be re-enabled)