From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Tue, 23 Sep 2014 11:47:55 +0200
Subject: [U-Boot] [PATCH V3] ARM: mx6: Add support for Kosagi Novena
In-Reply-To: <54206945.3020300@mail.bg>
References: <1411307086-12105-1-git-send-email-marex@denx.de>
	<54206945.3020300@mail.bg>
Message-ID: <201409231147.55859.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Monday, September 22, 2014 at 08:24:05 PM, Nikolay Dimitrov wrote:
> Hi Marek,
> 
> On 09/21/2014 04:44 PM, Marek Vasut wrote:
> > diff --git a/board/kosagi/novena/Makefile b/board/kosagi/novena/Makefile
> > new file mode 100644
> > index 0000000..f67bbc9
> > --- /dev/null
> > +++ b/board/kosagi/novena/Makefile
> > @@ -0,0 +1,11 @@
> > +#
> > +# Copyright (C) 2014 Marek Vasut<marex@denx.de>
> > +#
> > +# SPDX-License-Identifier:	GPL-2.0+
> > +#
> > +
> > +ifndef CONFIG_SPL_BUILD
> > +obj-y	:= novena.o
> > +else
> > +obj-y	:= novena_spl.o
> > +endif
> 
> What do you say about using only "positive" descriptions instead of the
> negation, like this:
> 
> ifdef CONFIG_SPL_BUILD
> obj-y := novena_spl.o
> else
> obj-y := novena.o
> endif
> 
> In long term people tend to misunderstand the negations, and sometimes
> (unfortunately) start to build additional negations on top, so it's hard
> to follow the logic in the end.

OK

> One more comment - isn't the file novena.o used also for the TPL, when
> building for SPL?

No, there's no TPL involved on MX6.

Best regards,
Marek Vasut