On Mon, May 11, 2015 at 02:13:00PM +0100, Peter Robinson wrote: > On Mon, May 11, 2015 at 1:32 PM, Tom Rini <[email protected]> wrote: > > On Mon, May 11, 2015 at 12:23:21PM +0100, Peter Robinson wrote: > > > >> The linux/sizes.h, asm/arch/imx-regs.h, asm/imx-common/gpio.h, > >> config_cmd_default.h includes are used fairly universally across imx6 > >> boards > >> so include them in mx6_common.h by default. > > [snip] > >> diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h > >> index e0528ce..edaa4f6 100644 > >> --- a/include/configs/mx6_common.h > >> +++ b/include/configs/mx6_common.h > >> @@ -31,4 +31,9 @@ > >> #define CONFIG_MP > >> #define CONFIG_MXC_GPT_HCLK > >> > >> +#include <linux/sizes.h> > >> +#include <asm/arch/imx-regs.h> > >> +#include <asm/imx-common/gpio.h> > >> +#include <config_cmd_default.h> > > > > This and then 7/10 aren't quite right. We want to make sure that prior > > to including config_cmd_default.h we set CONFIG_SYS_NO_FLASH so that we > > don't have to undef CONFIG_CMD_FLASH / CONFIG_CMD_IMLS > > OK, so is it enough to reorder them in the mx6_common.h?
Yeah. They aren't right in most of the board configs today so we an just get it right here in the end. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
