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 <pbrobin...@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

[...]

Regards

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

Reply via email to