ARM: SoC MPU definition cleanup

[ioannisg]

A clean-up patch.

We move the definition of the fixed (Flash and SRAM) MPU region definition for ARM Cortex-M platforms in a common place, and remove all duplicate definitions in each SoC / SoC series.

For backwards compatibility we keep the ability for SoCs to define their own fixed regions, with a Kconfig defined for this purpose. We exercise this for the Kinetis SoCs with the NXP MPU.

[zephyrbot]

All checks passed.

checkpatch (informational only, not a failure)

-:450: WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations
#450: FILE: soc/arm/common/cortex_m/arm_mpu_regions.c:18:
+			 REGION_FLASH_ATTR(CONFIG_FLASH_BASE_ADDRESS, \

-:460: WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations
#460: FILE: soc/arm/common/cortex_m/arm_mpu_regions.c:27:
+			 REGION_RAM_ATTR(CONFIG_SRAM_BASE_ADDRESS, \

- total: 0 errors, 2 warnings, 218 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Your patch has style problems, please review.

NOTE: Ignored message types: AVOID_EXTERNS BRACES CONFIG_EXPERIMENTAL CONST_STRUCT DATE_TIME FILE_PATH_CHANGES MINMAX NETWORKING_BLOCK_COMMENT_STYLE PRINTK_WITHOUT_KERN_LEVEL SPLIT_STRING VOLATILE

NOTE: If any of the errors are false positives, please report
      them to the maintainers.

Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

[agansari]

Looks okay, removes redundant code.

CPU_HAS_CUSTOM_FIXED_SOC_MPU_REGIONS may be used in the future to split multi-core and secure/non-secure partitioning, not just kinetis devices.

[henrikbrixandersen]

Not sure about the Kconfig name (CONFIG_CPU_HAS_CUSTOM_FIXED_SOC_MPU_REGIONS seems a bit too long and complex to understand), but the change looks good to me. Thanks for working on this.