[PATCH 1/2] spec: add MPU CTRL option to be usable on ARMV7M based BSPs
karel at functional.vision
Thu Mar 16 14:42:27 UTC 2023
On 3/16/23 14:34, Sebastian Huber wrote:
> On 16.03.23 14:28, Karel Gardas wrote:
>> +description: |
>> + Default value of the ARM MPU CTRL register
>> +- enabled-by:
>> + - arm/imxrt1052
>> + - arm/stm32h7
>> + - arm/nucleo-h743zi
>> + - arm/stm32h7b3i-dk
>> + - arm/stm32h747i-disco
>> + - arm/stm32h757i-eval
>> + value: (ARMV7M_MPU_CTRL_ENABLE | ARMV7M_MPU_CTRL_PRIVDEFENA)
>> +- enabled-by: true
>> + value: ARMV7M_MPU_CTRL_ENABLE
> The patch set looks good, but please make the current value
> (ARMV7M_MPU_CTRL_ENABLE | ARMV7M_MPU_CTRL_PRIVDEFENA) the default value.
Done. I also fixed imxrt by moving mpu opt before bspopts in order to
get define generated into the bspopts.h.
Thanks for the review and suggestion on this!
More information about the devel