[U-Boot] [PATCH v3 3/4] kirkwood: implement kw_sdram_size_adjust
Holger Brunck
holger.brunck at keymile.com
Fri Sep 7 08:47:05 CEST 2012
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 at 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.brunck at keymile.com]
>>> Sent: 20 July 2012 18:04
>>> To: u-boot at 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.falauto at 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.falauto at keymile.com>
>>> Signed-off-by: Holger Brunck <holger.brunck at keymile.com>
>>> cc: Prafulla Wadaskar <prafulla at marvell.com>
>>> cc: Valentin Longchamp <valentin.longchamp at keymile.com>
>>> cc: Marek Vasut <marex at 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
Regards
Holger
More information about the U-Boot
mailing list