[U-Boot] [PATCH v2] Kirkwood: bugfix: DRAM size initialization

Prafulla Wadaskar prafulla at marvell.com
Thu Oct 28 12:23:22 CEST 2010



> -----Original Message-----
> From: Tanmay Upadhyay [mailto:tanmay.upadhyay at einfochips.com]
> Sent: Friday, October 15, 2010 5:28 PM
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; Tanmay Upadhyay
> Subject: [PATCH v2] Kirkwood: bugfix: DRAM size initialization
> 
> If start of any DRAM bank is greater than total DDR size, remaining
> DDR banks' start address & size were left un-initialized in dram_init
> function. This could break other functions who uses array
> 'gd->bd->bi_dram'. Kirkwood network driver is one example. This also
> stops Linux kernel from booting.
> 
> v2 - Set start address also to 0. Without this Linux kernel couldn't
>      boot up
> 
> Signed-off-by: Tanmay Upadhyay <tanmay.upadhyay at einfochips.com>
> ---
>  arch/arm/cpu/arm926ejs/kirkwood/dram.c |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/cpu/arm926ejs/kirkwood/dram.c
> b/arch/arm/cpu/arm926ejs/kirkwood/dram.c
> index 7439c87..a4344b8 100644
> --- a/arch/arm/cpu/arm926ejs/kirkwood/dram.c
> +++ b/arch/arm/cpu/arm926ejs/kirkwood/dram.c
> @@ -81,6 +81,16 @@ int dram_init(void)
>  		gd->ram_size += gd->bd->bi_dram[i].size;
> 
>  	}
> +
> +	for (; i < CONFIG_NR_DRAM_BANKS; i++) {
> +		/* If above loop terminated prematurely, we need to set
> +		 * remaining banks' start address & size as 0. Otherwise other
> +		 * u-boot functions and Linux kernel gets wrong values which
> +		 * could result in crash */
> +		gd->bd->bi_dram[i].start = 0;
> +		gd->bd->bi_dram[i].size = 0;
> +	}
> +
>  	return 0;
>  }
> 


Applied to u-boot-marvell.git master branch
Regards..
Prafulla . .


More information about the U-Boot mailing list