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.g...@gmx.de>
reviewed-by : Vikas Manocha <vikas.mano...@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; > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
