Thanks Simon,

-----Original Message----- From: Simon Glass [mailto:sjg@google.com] On Behalf Of Simon Glass Sent: Saturday, July 01, 2017 10:06 AM To: U-Boot Mailing List u-boot@lists.denx.de Cc: Simon Glass sjg@chromium.org; Albert Aribaud albert.u.boot@aribaud.net; Toshifumi NISHINAGA tnishinaga.dev@gmail.com; Vikas MANOCHA vikas.manocha@st.com Subject: [PATCH] stm32: Correct positioning of declaration

The current code gives a warning:

arch/arm/mach-stm32/stm32f7/soc.c: In function 'arch_cpu_init': arch/arm/mach-stm32/stm32f7/soc.c:38:2: error: 'for' loop initial declarations are only allowed in C99 or C11 mode for (int i = 0; i < ARRAY_SIZE(stm32_region_config); i++) ^ arch/arm/mach-stm32/stm32f7/soc.c:38:2: note: use option -std=c99, -std=gnu99, -std=c11 or -std=gnu11 to compile your code

Fix it by moving the declaration to the top of the function.

Reviewed-by : Vikas MANOCHA vikas.manocha@st.com

Cheers, Vikas

Signed-off-by: Simon Glass sjg@chromium.org Series-cc trini

arch/arm/mach-stm32/stm32f7/soc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mach-stm32/stm32f7/soc.c b/arch/arm/mach-stm32/stm32f7/soc.c index 74a9350a31..a960cc1cbf 100644 --- a/arch/arm/mach-stm32/stm32f7/soc.c +++ b/arch/arm/mach-stm32/stm32f7/soc.c @@ -17,6 +17,8 @@ u32 get_cpu_rev(void)

int arch_cpu_init(void) {

• int i;
• struct mpu_region_config stm32_region_config[] = { { 0x00000000, REGION_0, XN_DIS, PRIV_RW_USR_RW, O_I_WB_RD_WR_ALLOC, REGION_4GB },

@@ -35,7 +37,7 @@ int arch_cpu_init(void) };

disable_mpu();

• for (int i = 0; i < ARRAY_SIZE(stm32_region_config); i++)

• for (i = 0; i < ARRAY_SIZE(stm32_region_config); i++) mpu_config(&stm32_region_config[i]); enable_mpu();

-- 2.13.2.725.g09c95d1e9-goog