[PATCH 1/2] spec: add MPU CTRL option to be usable on ARMV7M based BSPs

Karel Gardas 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
>> +default:
>> +- 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!
Karel



More information about the devel mailing list