Hi,

On 07/30/2017 11:34 AM, Heinrich Schuchardt wrote:

For DEVICE_NON_SHARED the newly assigned value of attr is overwritten due to a missing break.

The problem was indicated by cppcheck.

Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de

reviewed-by : Vikas Manocha vikas.manocha@st.com

Cheers, Vikas

---

arch/arm/cpu/armv7m/mpu.c | 1 + 1 file changed, 1 insertion(+)

diff --git a/arch/arm/cpu/armv7m/mpu.c b/arch/arm/cpu/armv7m/mpu.c index 31a243b49a..4622aa4826 100644 --- a/arch/arm/cpu/armv7m/mpu.c +++ b/arch/arm/cpu/armv7m/mpu.c @@ -68,6 +68,7 @@ void mpu_config(struct mpu_region_config *reg_config) break; case DEVICE_NON_SHARED: attr = (2 << TEX_SHIFT) | BUFFERABLE;

• break;
  

  default: attr = 0; /* strongly ordered */ break;