[PATCH] arm: mvebu: support for the new Armada XP evaluation board(DB-MV784MP-GP)

[Gregory CLEMENT]

On 01/31/2013 11:07 AM, Thomas Petazzoni wrote:
> Dear Andrew Lunn,
> 
> On Thu, 31 Jan 2013 10:50:25 +0100, Andrew Lunn wrote:
> 
>> We have the same issue with OpenBox AX3. All the devices we have in
>> our hands have 1GB soldered down, plus 2GB on expansion. The DT
>> currently has the full 3GB.
>>
>> However, there has been discussion of reducing this down to 1GB since
>> both Thomas and Gregory have unstable systems with the full 3GB, but
>> mine box is stable with the full 3GB. So i don't know how strong a
>> precedent that sets....
> 
> It seems that if we have CONFIG_ARM_ATAG_DTB_COMPAT, then the code in
> arch/arm/boot/compressed/atags_to_fdt.c will update the DT memory node
> according to the ATAG_MEM passed by the bootloader.
> 

>From my point of view these dts file are just given as reference, and the
bootloader should be able to modify them on the fly according the hardware
configuration of the board.

And for old bootloader then we can use the CONFIG_ARM_ATAG_DTB_COMPAT as
pointed by Thomas.

So I am not sure that the amount of memory is very important.

However I think that having a value by default is useful, so unless
there is a strong argument against it, I would like to keep it, maybe with
a comment in the dts to emphasize that it is just a default value and the
amount of memory can be changed.


> Best regards,
> 
> Thomas
> 


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com