On 06/21/2011 05:22 PM, Fabio Estevam wrote:
> On Tue, Jun 21, 2011 at 6:33 PM, Rob Herring <[email protected]> wrote:
>> From: Rob Herring <[email protected]>
>>
>> Add basic support for Calxeda Highbank platform. Only minimal support to boot
>> is included.
>>
>> Signed-off-by: Jason Hobbs <[email protected]>
>> Signed-off-by: Rob Herring <[email protected]>
>> Cc: Albert ARIBAUD <[email protected]>
>> ---
>>  arch/arm/cpu/armv7/highbank/Makefile  |   46 ++++++++++++
>>  arch/arm/cpu/armv7/highbank/config.mk |    4 +
>>  arch/arm/cpu/armv7/highbank/timer.c   |  124 
>> +++++++++++++++++++++++++++++++++
>>  board/highbank/Makefile               |   49 +++++++++++++
>>  board/highbank/highbank.c             |   49 +++++++++++++
>>  boards.cfg                            |    1 +
>>  include/configs/highbank.h            |  101 +++++++++++++++++++++++++++
> 
> You should add an entry to MAINTAINERS file.
> ...
> 
>> +int dram_init(void)
>> +{
>> +       gd->ram_size = SZ_512M;
> 
> You could use
>  gd->ram_size = get_ram_size((volatile void *)PHYS_SDRAM_1,  
> PHYS_SDRAM_1_SIZE);

Except that does not work on qemu, and qemu can't model all of ram. So I
need to limit the ram size, but have the same image work on h/w and qemu.

Rob

_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to