> -----Original Message-----
> From: u-boot-boun...@lists.denx.de [mailto:u-boot-
> boun...@lists.denx.de] On Behalf Of Gerlando Falauto
> Sent: 06 July 2012 17:20
> To: u-boot@lists.denx.de
> Cc: Valentin Longchamp; Holger Brunck; Gerlando Falauto
> Subject: [U-Boot] [PATCH v2 2/2] km_arm: enable SDRAM window size
> fixup
> 
> Size of the SDRAM chips might differ between any two (otherwise
> identical) instances of the same board.
> 
> So change board_early_init_f() to use km_sdram_bs_set(), in order to
> fixup the window size register at runtime, according to the detected
> SDRAM size.
> 
> Signed-off-by: Gerlando Falauto <gerlando.fala...@keymile.com>
> Cc: Valentin Longchamp <valentin.longch...@keymile.com>
> Cc: Holger Brunck <holger.bru...@keymile.com>
> ---
> changes for v2:
> - rebase to current u-boot-marvell.git master branch
> 
>  board/keymile/km_arm/km_arm.c |   12 ++++++++++++
>  1 files changed, 12 insertions(+), 0 deletions(-)
> 
> diff --git a/board/keymile/km_arm/km_arm.c
> b/board/keymile/km_arm/km_arm.c
> index 2b2ca39..24cf6c7 100644
> --- a/board/keymile/km_arm/km_arm.c
> +++ b/board/keymile/km_arm/km_arm.c
> @@ -241,6 +241,17 @@ int misc_init_r(void)
>       return 0;
>  }
> 
> +static void dram_size_fixup(void)

The name "fixup" doesn't sound good.

> +{
> +     u32 size;
> +
> +     /* probe currently equipped RAM size */
> +     size = get_ram_size((void *)kw_sdram_bar(0), kw_sdram_bs(0));
> +
> +     /* fixup SDRAM window size accordingly */
> +     kw_sdram_bs_set(0, size);
> +}
> +

This is useful API for other boards too.
I would like to suggest to add this change in dram_init(), the window size can 
be corrected right after sdram size detection. And that will be applicable for 
all supported bank.

So may you please push in to dram.c?

Regards...
Prafulla . . .
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to