From mboxrd@z Thu Jan  1 00:00:00 1970
From: marex@denx.de (Marek Vasut)
Date: Mon, 14 Dec 2015 15:12:24 +0100
Subject: [PATCH V4 2/2] ARM: dts: imx6q: add Novena board
In-Reply-To: <566EC9E6.6060402@cogentembedded.com>
References: <20151211110700.GD11252@tiger> <201512140456.08114.marex@denx.de>
 <566EC9E6.6060402@cogentembedded.com>
Message-ID: <201512141512.24281.marex@denx.de>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Monday, December 14, 2015 at 02:53:42 PM, Sergei Shtylyov wrote:
> Hello.
> 
> On 12/14/2015 6:56 AM, Marek Vasut wrote:
> >>> From: Sean Cross <xobs@kosagi.com>
> >>> 
> >>> Novena is an open-hardware laptop/desktop/bare board.
> >>> 
> >>> See http://www.kosagi.com/w/index.php?title=Novena_Main_Page
> >>> 
> >>> Signed-off-by: Sean Cross <xobs@kosagi.com>
> >>> Signed-off-by: Marek Vasut <marex@denx.de>
> >>> Cc: Fabio Estevam <fabio.estevam@freescale.com>
> >>> Cc: Lucas Stach <l.stach@pengutronix.de>
> >>> Cc: Sean Cross <xobs@kosagi.com>
> >>> Cc: Shawn Guo <shawnguo@kernel.org>
> >> 
> >> It looks pretty good to me now, except a small question below.
> > 
> > Thanks
> > 
> >>> ---
> >>> 
> >>>   arch/arm/boot/dts/Makefile         |   1 +
> >>>   arch/arm/boot/dts/imx6q-novena.dts | 789
> >>>   +++++++++++++++++++++++++++++++++++++ 2 files changed, 790
> >>>   insertions(+)
> >>>   create mode 100644 arch/arm/boot/dts/imx6q-novena.dts
> >> 
> >> <snip>
> >> 
> >>> +/ {
> >>> +	model = "Kosagi Novena Dual/Quad";
> >>> +	compatible = "kosagi,imx6q-novena", "fsl,imx6q";
> >>> +
> >>> +	memory {
> >>> +		reg = <0x10000000 0xefffffff>;
> >> 
> >> What's the trick of having memory size being 0xefffffff?
> > 
> > That is a typo, should obviously be 0xffffffff ,
> 
>     You didn't get it. :-) It' the _size_, not limit.

So it was correct from the get-go ? Uh ...

Thanks for clarification, I will remember this :)

Best regards,
Marek Vasut