[U-Boot] [PATCH 03/10] imx6: move generic imx6 options to mx6_common.h

Tom Rini trini at konsulko.com
Tue May 12 12:51:35 CEST 2015


On Tue, May 12, 2015 at 10:40:25AM +0200, Markus Niebel wrote:

> Hello Peter,
> 
> sorry for the noise, did not see v2 of your series. Just one thing:
> 
> Am 11.05.2015 um 18:22 schrieb Peter Robinson:
> > All boards define CONFIG_MX6, CONFIG_DISPLAY_BOARDINFO, CONFIG_DISPLAY_CPUINFO
> > and CONFIG_SYS_GENERIC_BOARD so define them in mx6_common
> > 
> > Signed-off-by: Peter Robinson <pbrobinson at gmail.com>
> > Reviewed-by: Tom Rini <trini at konsulko.com>
> 
> [...] 
> 
> >  
> > diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h
> > index 85bf039..fc19fbb 100644
> > --- a/include/configs/mx6_common.h
> > +++ b/include/configs/mx6_common.h
> > @@ -38,4 +38,9 @@
> >  #include <asm/imx-common/gpio.h>
> >  #include <config_cmd_default.h>
> >  
> > +#define CONFIG_MX6
> 
> Here we need an #if !defined(CONFIG_MX6) because secomx6quq7
> (and soon hopefully other boards) sets this via Kconfig under
> arch/arm/cpu/armv7/mx6

Oh good catch.  Would it be hard to just make everyone do this via
Kconfig instead?  I would expect not...

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150512/93da93b5/attachment.sig>


More information about the U-Boot mailing list