davids5 commented on a change in pull request #4702:
URL: https://github.com/apache/incubator-nuttx/pull/4702#discussion_r734539769
##########
File path: arch/arm/src/stm32/stm32_mpuinit.c
##########
@@ -78,6 +78,12 @@ void stm32_mpuinitialize(void)
mpu_showtype();
+#if defined(CONFIG_MPU_RESET)
+ /* Reset MPU if enabled */
+
+ mpu_reset();
Review comment:
@xiaoxiang781216 Are you suggesting a "once" mechanism or something
different (code example will make it clear)?
I prefer the compile time simplicity and besides the `once` mechanism will
not work. Because the early reset is done before bss and data is initialized.
The static 'once' will get wiped.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
