On Friday 11 November 2011 08:35:14 Loïc Minier wrote:
> On Thu, Nov 10, 2011, Mike Frysinger wrote:
> > >  CONFIG_IMX_CONFIG is set in boards.cfg.
> > 
> > ugly undocumented CONFIG's are awesome
> > 
> > this really should be in a subdir rather than the top level.  we want
> > to keep arch/soc-specific cruft out of the top level Makefile if
> > possible.  isn't there a sub-config.mk somewhere you could add the
> > line unconditionally ?
> 
>  Can't see one, but Aneesh proposes the OMAP approach of
>  arch/arm/cpu/**/config.mk files.
> 
>  The CONFIG_IMX_CONFIG actually changes per-board, so the boards.cfg
>  approach to set it seems ok; however we could try defining the
>  u-boot.imx rule in a common i.MX makefile snippet somewhere.
> 
>  Perhaps arch/arm/cpu/armv7/config.mk or
>  arch/arm/cpu/armv7/mx5/Makefile, or a new file; I don't have a strong
>  opinion.

if there's a config option to key off of, and it affects multiple arm 
arches/socs, then arch/arm/config.mk would probably be the best place to add 
the lines.

maybe Albert has an opinion on the arch/arm/ layout.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to