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
