On Tue, Apr 24, 2018 at 06:24:43PM +0530, Lokesh Vutla wrote: > The Cortex-R* processors are a mid-range CPUs for use in deeply-embedded, > real-time systems. It implements the ARMv7-R architecture, and includes > Thumb-2 technology for optimum code density and processing throughput. > > Except for MPU(Memory Protection Unit) and few CP15 registers, most of the > features are compatible with v7 architecture. This series adds minimal > support for v7-R architecture by reusing the v7 support. Also adding > support for MPU.
There's some whitespace issues that I'm sure you'll fix for the next version. One naming thing that I think needs a little work is that we say CONFIG_ARM_MPU (which is OK'ish, can we use something a little more specific? It's a V7 architectural thing, yes?) but then v7m_mpu.h, even tho we use it on both v7r and v7m as there's just a few differences. That should get consolidated, off the top of my head, to CONFIG_CPU_V7_HAS_MPU (following other things like virt) and maybe v7_mpu.h ? -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot