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

[Stefano Babic]

Hi Jagan,

On 04/10/2016 15:43, Jagan Teki wrote:
> 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?

Please try with -master. Anyway, I have pushed my rearranged -next (just
the order was changed due to issues in other patchset), and I see the
same problems.

Regards,
Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================