Re: [PATCH 07/10] arm64: dts: add the Marvell Armada 3700 family and a development board

From: Gregory CLEMENT
Date: Wed Feb 03 2016 - 05:45:44 EST


Hi Mark,

On mar., fÃvr. 02 2016, Mark Rutland <mark.rutland@xxxxxxx> wrote:

>> + memory {
>> + device_type = "memory";
>> + /* use only 256 MB on the 512 MB available */
>> + reg = <0x00000000 0x00000000 0x00000000 0x10000000>;
>> + };
>
> It would be good to comment as to why we can't use 256M of the memory.

Actually the comment is wrong and the size too, we do use the 512MB.

I will fix it

Thanks,

Gregory

>
> Otherwise this looks fine.
>
> Mark.

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