Hi Chris, On Thu, Mar 26, 2015 at 02:53:58PM -0700, Chris Kuethe wrote: > On Thu, Mar 26, 2015 at 4:49 AM, <and...@inversepath.com> wrote: > > From: Andrej Rosano <and...@inversepath.com> > > > > Hello, > > > > this series adds support for USB armory board. The patches are > > prepared against imx tree as it uses the new arch/board approach. > > > > The first patch add introduces the arch/board approach for mx5, > > as already done for mx6. > > Yep, that works. > > > The second patch adds basic support for USB armory board and > > is an updated version (v4) with the modifications described bellow. > > If I test this against HEAD as of > f643d9294f45487f22e8f33d6572530f17eff4e9, I get an error reated to > IMX_CONFIG.
Please note that the patch is prepared against imx tree and not the mainline one, where the "arch/board approach" is not yet merged. Cheers, Andrej > > "board/inversepath/usbarmory/Kconfig:3:warning: config symbol defined > without type" > > Here's a diff to fix that > > diff --git a/board/inversepath/usbarmory/Kconfig > b/board/inversepath/usbarmory/Kconfig > index c2cd544..4776b1d 100644 > --- a/board/inversepath/usbarmory/Kconfig > +++ b/board/inversepath/usbarmory/Kconfig > @@ -1,8 +1,5 @@ > if TARGET_USBARMORY > > -config IMX_CONFIG > - default "board/inversepath/usbarmory/imximage.cfg" > - > config SYS_BOARD > default "usbarmory" > > diff --git a/configs/usbarmory_defconfig b/configs/usbarmory_defconfig > index 10bdd9e..237b1ca 100644 > --- a/configs/usbarmory_defconfig > +++ b/configs/usbarmory_defconfig > @@ -1,3 +1,4 @@ > CONFIG_ARM=y > CONFIG_ARCH_MX5=y > CONFIG_TARGET_USBARMORY=y > +CONFIG_SYS_EXTRA_OPTIONS="IMX_CONFIG=board/inversepath/usbarmory/imximage.cfg" > > > > Changes for v4: > > - Remove ifdef for CONFIG_FSL_ESDHC as it must always be present > > - Remove explicit GPIO input setup (all GPIO are input after reset) > > - Remove dummy board_late_init() and CONFIG_BOARD_LATE_INIT > > - Remove CONFIG_BOOTARGS > > - Fix compiling errors due to config_distro_bootcmd.h inclusion position > > Tested-by: Chris Kuethe <chris.kue...@gmail.com> > > > Changes for v3: > > - Add config_distro support > > > > Changes for v2: > > - Fix double print_cpuinfo() call > > - Fix CONFIG_BOOTCOMMAND typo > > - Fix CONFIG_SYS_FSL_ESDHC_NUM to 1 > > > > Andrej Rosano (2): > > ARM: mx5: move to a standard arch/board approach > > ARM: mx5: add support for USB armory board > > > > > > Andrej Rosano (2): > > ARM: mx5: move to a standard arch/board approach > > ARM: mx5: add support for USB armory board > > > > -- > GDB has a 'break' feature; why doesn't it have 'fix' too? -- Andrej Rosano Inverse Path Srl <and...@inversepath.com> http://www.inversepath.com 0x01939B21 5BB8 574E 68E8 D841 E18F D5E9 CEAD E0CF 0193 9B21 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot