> -----Original Message----- > From: Holger Brunck [mailto:holger.bru...@keymile.com] > Sent: 06 September 2012 23:47 > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; Gerlando Falauto; Valentin Longchamp; Marek > Vasut > Subject: Re: [PATCH v3 3/4] kirkwood: implement kw_sdram_size_adjust > > On 09/07/2012 01:20 AM, Prafulla Wadaskar wrote: > > > > > >> -----Original Message----- > >> From: Prafulla Wadaskar > >> Sent: 20 July 2012 12:10 > >> To: 'Holger Brunck'; u-boot@lists.denx.de > >> Cc: Gerlando Falauto; Valentin Longchamp; Marek Vasut > >> Subject: RE: [PATCH v3 3/4] kirkwood: implement > kw_sdram_size_adjust > >> > >> > >> > >>> -----Original Message----- > >>> From: Holger Brunck [mailto:holger.bru...@keymile.com] > >>> Sent: 20 July 2012 18:04 > >>> To: u-boot@lists.denx.de > >>> Cc: Gerlando Falauto; Holger Brunck; Prafulla Wadaskar; Valentin > >>> Longchamp; Marek Vasut > >>> Subject: [PATCH v3 3/4] kirkwood: implement kw_sdram_size_adjust > >>> > >>> From: Gerlando Falauto <gerlando.fala...@keymile.com> > >>> > >>> Size of the SDRAM chips might differ between any two (otherwise > >>> identical) instances of the same board. So add a function which > >> reads > >>> out the current ram size and set them in the SDRAM size register > of > >>> kirkwood. > >>> > >>> Signed-off-by: Gerlando Falauto <gerlando.fala...@keymile.com> > >>> Signed-off-by: Holger Brunck <holger.bru...@keymile.com> > >>> cc: Prafulla Wadaskar <prafu...@marvell.com> > >>> cc: Valentin Longchamp <valentin.longch...@keymile.com> > >>> cc: Marek Vasut <ma...@denx.de> > >>> --- > >>> chages for v3: > >>> - rename dram_size_fixup to kw_sdramsize_adjust and move them to > >>> kirkwood/dram.c > >>> > >>> changes for v2: > >>> - rebase to current u-boot-marvell.git master branch > >>> > >>> arch/arm/cpu/arm926ejs/kirkwood/dram.c | 11 +++++++++++ > >>> arch/arm/include/asm/arch-kirkwood/cpu.h | 1 + > >>> 2 files changed, 12 insertions(+), 0 deletions(-) > >>> > >>> diff --git a/arch/arm/cpu/arm926ejs/kirkwood/dram.c > >>> b/arch/arm/cpu/arm926ejs/kirkwood/dram.c > >>> index 5e2f9d8..9f97964 100644 > >>> --- a/arch/arm/cpu/arm926ejs/kirkwood/dram.c > >>> +++ b/arch/arm/cpu/arm926ejs/kirkwood/dram.c > >>> @@ -97,6 +97,17 @@ u32 kw_sdram_bs(enum memory_bank bank) > >>> return result; > >>> } > >>> > >>> +void kw_sdram_size_adjust(void) > >> > >> I think you must pass bank_id to this function so that one can tune > >> any/all of the four available DRAM banks. > > > > Hi Hogler > > This has not yet closed hence not pulled this patch series. > > > > Do you have any comments on this? > > > > updates concerning these change request were send long time ago. The > references are: > > http://lists.denx.de/pipermail/u-boot/2012-July/129116.html > http://lists.denx.de/pipermail/u-boot/2012-July/129117.html
Thanks Holger, I will pull them soon. Regards... Prafulla . . . _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot