[U-Boot] [PATCH] Kirkwood: Add support for Ka-Ro TK71

[Valentin Longchamp <valentin.longchamp@keymile.com>]

Hi Marek and Prafulla,

On 06/01/2012 03:03 PM, Marek Vasut wrote:
> Dear Prafulla Wadaskar,
> 
>>> -----Original Message-----
>>> From: Marek Vasut [mailto:marex at denx.de]
>>> Sent: 31 May 2012 16:37
>>> To: u-boot at lists.denx.de
>>> Cc: Marek Vasut; Prafulla Wadaskar; Wolfgang Denk
>>> Subject: [PATCH] Kirkwood: Add support for Ka-Ro TK71
>>>
>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>> Cc: Prafulla Wadaskar <prafulla@marvell.com>
>>> Cc: Wolfgang Denk <wd@denx.de>
>>> ---
>>>
>>>  board/karo/tk71/Makefile         |   45 ++++++++++
>>>  board/karo/tk71/kwbimage-256.cfg |  174
>>>
>>> ++++++++++++++++++++++++++++++++++++++
>>>
>>>  board/karo/tk71/kwbimage-512.cfg |  174
>>>
>>> ++++++++++++++++++++++++++++++++++++++
>>
>> Dear Marek
>> Just for DRAM size change do not add one more cfg file, configure by
>> default 256MB of RAM in default kwbimg.cfg file and in function
>> board_early_init_f() tune it to 512 for your other board version.
> 
> There's only one single bit flipped between those two kwb configs. Do you think 
> it'd work if we just configured the system for 512MB RAM and ran get_ram_size() 
> to see if it has only 256MB? That'd eliminate two board entries for this tk71.
> 

I would like to have your advice on this as well Prafulla.

We have tested this on km_arm (we will have the same boards with 1/2 the RAM
size) and with the above get_ram_size() it works as expected.

We still should, however, at some point (board_early_init_f() is a good
candidate) then reduce the corresponding RAM CS size (reg @1504 for CS0 so that
the window is the same size as what was detected by get_ram_size). What do you
guys think ?

Valentin