Subject: [PATCH V4 2/2] ARM: dts: imx6q: add Novena board

From: [email-masked] (Marek Vasut)
Date: Mon, 14 Dec 2015 04:56:07 +0100

On Monday, December 14, 2015 at 02:38:45 AM, Shawn Guo wrote:
> On Sat, Dec 12, 2015 at 04:27:13AM +0100, Marek Vasut wrote:
> > From: Sean Cross <[email-masked]>
> > 
> > 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 <[email-masked]>
> > Signed-off-by: Marek Vasut <[email-masked]>
> > Cc: Fabio Estevam <[email-masked]>
> > Cc: Lucas Stach <[email-masked]>
> > Cc: Sean Cross <[email-masked]>
> > Cc: Shawn Guo <[email-masked]>
> 
> 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 , I just tested it on the board.
Thanks for spotting it, do you want me to repost a V5 or can you fix it ?

Best regards,
Marek Vasut