Hi, On 11/24/2016 11:10 AM, Michael Kurz wrote: > Fix base address of fmc sdram controller. > The base address is defined twice, once in fmc.h and once in stm32.h. > Fix wrong definition in stm32.h. > Remove the definiton in fmc.h and use the definition in stm32.h > > Signed-off-by: Michael Kurz <michi.k...@gmail.com>
Nitpicking...re-word the commit message, it seems repeating itself in 2nd & 4th line. Otherwise, Acked-by: Vikas Manocha <vikas.mano...@st.com> Cheers, Vikas > > Series-changes 3: > - Split cleanup patch > > --- > > Changes in v3: None > Changes in v2: None > > arch/arm/include/asm/arch-stm32f7/fmc.h | 3 +-- > arch/arm/include/asm/arch-stm32f7/stm32.h | 2 +- > 2 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/include/asm/arch-stm32f7/fmc.h > b/arch/arm/include/asm/arch-stm32f7/fmc.h > index d61a86f..4741e5a 100644 > --- a/arch/arm/include/asm/arch-stm32f7/fmc.h > +++ b/arch/arm/include/asm/arch-stm32f7/fmc.h > @@ -24,8 +24,7 @@ struct stm32_fmc_regs { > /* > * FMC registers base > */ > -#define STM32_SDRAM_FMC_BASE 0xA0000140 > -#define STM32_SDRAM_FMC ((struct stm32_fmc_regs > *)STM32_SDRAM_FMC_BASE) > +#define STM32_SDRAM_FMC ((struct stm32_fmc_regs > *)SDRAM_FMC_BASE) > > /* Control register SDCR */ > #define FMC_SDCR_RPIPE_SHIFT 13 /* RPIPE bit shift */ > diff --git a/arch/arm/include/asm/arch-stm32f7/stm32.h > b/arch/arm/include/asm/arch-stm32f7/stm32.h > index 3a4ae89..92a868f 100644 > --- a/arch/arm/include/asm/arch-stm32f7/stm32.h > +++ b/arch/arm/include/asm/arch-stm32f7/stm32.h > @@ -49,7 +49,7 @@ > #define FLASH_CNTL_BASE (AHB1_PERIPH_BASE + 0x3C00) > > > -#define SDRAM_FMC_BASE (AHB3_PERIPH_BASE + 0x4A0000140) > +#define SDRAM_FMC_BASE (AHB3_PERIPH_BASE + 0x40000140) > > static const u32 sect_sz_kb[CONFIG_SYS_MAX_FLASH_SECT] = { > [0 ... 3] = 32 * 1024, > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot