> -----Original Message----- > From: Tanmay Upadhyay [mailto:tanmay.upadh...@einfochips.com] > Sent: Friday, October 15, 2010 5:28 PM > To: Prafulla Wadaskar > Cc: u-boot@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.upadh...@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 . . _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot