search

raspberrypi / pico-sdk

BSD 3-Clause "New" or "Revised" License
3.69k stars 917 forks source link

Wrong CMSIS mpu definitions? #1027

Open WestfW opened 2 years ago

WestfW commented 2 years ago

as part of the CMSIS add, there exists a file: .../src/rp2_common/cmsis/stub/CMSIS/Core/Include/mpu_armv7.h containing the memory protection unit definitions for "ARM v7m" cpus.

But the rp2040 is a Cortex M0, which is a v6m cpu.

(Nothing currently uses this file, as far as I can tell, and I don't know if there even exists an mpu_armv6.h)

lurch commented 2 years ago

I don't know if there even exists an mpu_armv6.h

Even the upstream CMSIS "only" has mpu_armv7.h and mpu_armv8.h.

But there does appear to be a bunch of MPU defines inside core_cm0plus.h ?

kilograham commented 1 year ago

We just package what they have; keeping this around though as we look at more CMSIS integration