[U-Boot] [PATCH v5 00/21] imx6: Add Engicam i.CoreM6 QDL support

[Jagan Teki]

On Tue, Oct 4, 2016 at 6:58 PM, Stefano Babic <sbabic at denx.de> wrote:
> Hi Jagan,
>
> On 30/09/2016 08:44, Jagan Teki wrote:
>
>>
>> Can you pick this series.
>
> Series cannot be applied: I see several issue. The series breaks for
> different reason a lot of boards. I get 3 boards broken and 15 with
> warnings. Here the resons:
>
> CONFIG_DEFAULT_FDT_FILE was moved to Kconfig, but without updating the
> boards that have already defined. This generated a duplication, for
> example :
>
> w+include/configs/mx53ard.h:84:0: warning: "CONFIG_DEFAULT_FDT_FILE"
> redefined [enabled by default]
> w+include/generated/autoconf.h:13:0: note: this is the location of the
> previous definition
>        arm:  +   dms-ba16-1g
> +In file included from include/config.h:5:0,
>
> You have to move at the same time all CONFIG_DEFAULT_FDT_FILE from the
> board's config file to defconfig.
>
> I add at the same time build errors by compiling the new boards, for
> example:
>
>        arm:  +   imx6qdl_icore_mmc
> +arch/arm/imx-common/cpu.c: At top level:
> +arch/arm/cpu/armv7/built-in.o: In function `lldiv':
> +include/div64.h:45: undefined reference to `__div64_32'
> +arch/arm/cpu/armv7/built-in.o: In function `lpddr2_rl':
> +arch/arm/cpu/armv7/mx6/ddr.c:943: undefined reference to `hang'
> +arch/arm/cpu/armv7/built-in.o: In function `mx6_lpddr2_cfg':
> +arch/arm/cpu/armv7/mx6/ddr.c:1175: undefined reference to `mdelay'
> +arch/arm/cpu/armv7/built-in.o: In function `mx6_ddr3_cfg':
> +arch/arm/cpu/armv7/mx6/ddr.c:1281: undefined reference to `hang'
> +arch/arm/cpu/armv7/mx6/ddr.c:1480: undefined reference to `mdelay'
> +arch/arm/cpu/armv7/built-in.o: In function `mx6_dram_cfg':
> +arch/arm/cpu/armv7/mx6/ddr.c:1493: undefined reference to `hang'
>
> It looks strange, it is maybe due to the fact that I am rearranging the
> order how I apply the patches because I see issues in other patchset.
> But can you check it, please ?

May be because of u-boot-imx/master is head, shall I try it on master
instead of next?

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.