[U-Boot] [PATCH 2/5] arch: arm: omap: Declare size of ddr very early

Lukasz Majewski lukma at denx.de
Tue Apr 18 13:33:12 UTC 2017


Hi Ravi,

> From: Lokesh Vutla <lokeshvutla at ti.com>
> 
> Declare the size of ddr very early in spl, so that this can be
> used to enable cache.
> 
> Signed-off-by: Lokesh Vutla <lokeshvutla at ti.com>
> Signed-off-by: Ravi Babu <ravibabu at ti.com>
> ---
>  arch/arm/mach-omap2/am33xx/board.c  | 4 ++++
>  arch/arm/mach-omap2/hwinit-common.c | 1 +
>  2 files changed, 5 insertions(+)
> 
> diff --git a/arch/arm/mach-omap2/am33xx/board.c
> b/arch/arm/mach-omap2/am33xx/board.c index 2bfc864..568f36f 100644
> --- a/arch/arm/mach-omap2/am33xx/board.c
> +++ b/arch/arm/mach-omap2/am33xx/board.c
> @@ -327,6 +327,10 @@ void board_init_f(ulong dummy)
>  	early_system_init();
>  	board_early_init_f();
>  	sdram_init();
> +	/* dram_init must store complete ramsize in gd->ram_size */
> +	gd->ram_size = get_ram_size(
> +			(void *)CONFIG_SYS_SDRAM_BASE,
> +			CONFIG_MAX_RAM_BANK_SIZE);

The above call is a bit strange, since the sdram_init() function
sets the DDR (EMIF) controller to use the whole SDRAM available.

Why one cannot take this info directly from EMIF controller?

>  }
>  #endif
>  
> diff --git a/arch/arm/mach-omap2/hwinit-common.c
> b/arch/arm/mach-omap2/hwinit-common.c index f317293..cac3274 100644
> --- a/arch/arm/mach-omap2/hwinit-common.c
> +++ b/arch/arm/mach-omap2/hwinit-common.c
> @@ -171,6 +171,7 @@ void board_init_f(ulong dummy)
>  #endif
>  	/* For regular u-boot sdram_init() is called from
> dram_init() */ sdram_init();
> +	gd->ram_size = omap_sdram_size();
>  }
>  #endif
>  




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de


More information about the U-Boot mailing list