[U-Boot] U-Boot build report @ Sat Mar 31 05:30:15 CEST 2012
Hello guys,
we now have exactly these boards not building for -RC1, can you please look into it?:
arm/flea3 FAILED FOR: eldk4.2 eldk5.1 arm/hawkboard FAILED FOR: eldk4.2 arm/hawkboard_uart FAILED FOR: eldk4.2 arm/highbank FAILED FOR: eldk4.2 arm/mx35pdk FAILED FOR: eldk4.2 eldk5.1 arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1 powerpc/P2020DS_36BIT FAILED FOR: eldk4.2 powerpc/PMC440 FAILED FOR: eldk4.2 eldk5.1 powerpc/TQM8541 FAILED FOR: eldk4.2 powerpc/TQM8555 FAILED FOR: eldk4.2
If you need logs, please contact me, I'll provide them.
Best regards, Marek Vasut
Hi Marek,
Ccing Simon.
On Sat, 31 Mar 2012 14:59:06 +0200 Marek Vasut marex@denx.de wrote: ...
we now have exactly these boards not building for -RC1, can you please look into it?:
arm/flea3 FAILED FOR: eldk4.2 eldk5.1
This is fixed by patch http://patchwork.ozlabs.org/patch/149658/
...
arm/mx35pdk FAILED FOR: eldk4.2 eldk5.1
Same as for flea3 board.
arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1
These boards actually build without errors and warnings, but errors/warnings status is reported by MAKEALL because of the output:
DTC: dts->dtb on file "dt.dtb.tmp"
This could be fixed by
diff --git a/dts/Makefile b/dts/Makefile index 5792afd..d19a622 100644 --- a/dts/Makefile +++ b/dts/Makefile @@ -47,7 +47,7 @@ DT_BIN := $(obj)dt.dtb
$(DT_BIN): $(TOPDIR)/board/$(VENDOR)/dts/$(DEVICE_TREE).dts cat $< | $(CPP) -P $(DTS_CPPFLAGS) - >$@.tmp - $(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp + $(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp 2> /dev/null rm $@.tmp
process_lds = \
...
powerpc/PMC440 FAILED FOR: eldk4.2 eldk5.1
This could be fixed by http://patchwork.ozlabs.org/patch/147745 but maybe we should better move timer_get_boot_us() to common/bootstage.c to fix it.
Thanks for build testing and reporting!
Anatolij
On Sat, 31 Mar 2012 15:43:03 +0200 Anatolij Gustschin agust@denx.de wrote: ...
arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1
These boards actually build without errors and warnings, but errors/warnings status is reported by MAKEALL because of the output:
DTC: dts->dtb on file "dt.dtb.tmp"
This could be fixed by
diff --git a/dts/Makefile b/dts/Makefile index 5792afd..d19a622 100644 --- a/dts/Makefile +++ b/dts/Makefile @@ -47,7 +47,7 @@ DT_BIN := $(obj)dt.dtb
$(DT_BIN): $(TOPDIR)/board/$(VENDOR)/dts/$(DEVICE_TREE).dts cat $< | $(CPP) -P $(DTS_CPPFLAGS) - >$@.tmp
- $(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp
- $(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp 2> /dev/null rm $@.tmp
process_lds = \
Thinking more about this I see that it is wrong. dtc could die for other reasons and it should be handled as error. We should fix dtc instead not to output this info to stderr. Currently it does:
fprintf(stderr, "DTC: %s->%s on file "%s"\n", inform, outform, arg);
Thanks,
Anatolij
Dear Anatolij Gustschin,
On Sat, 31 Mar 2012 15:43:03 +0200 Anatolij Gustschin agust@denx.de wrote: ...
arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1
These boards actually build without errors and warnings, but errors/warnings status is reported by MAKEALL because of the output:
DTC: dts->dtb on file "dt.dtb.tmp"
This could be fixed by
diff --git a/dts/Makefile b/dts/Makefile index 5792afd..d19a622 100644 --- a/dts/Makefile +++ b/dts/Makefile @@ -47,7 +47,7 @@ DT_BIN := $(obj)dt.dtb
$(DT_BIN): $(TOPDIR)/board/$(VENDOR)/dts/$(DEVICE_TREE).dts
cat $< | $(CPP) -P $(DTS_CPPFLAGS) - >$@.tmp
- $(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp
$(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp 2> /dev/null
rm $@.tmp
process_lds = \
Thinking more about this I see that it is wrong. dtc could die for other reasons and it should be handled as error. We should fix dtc instead not to output this info to stderr. Currently it does:
fprintf(stderr, "DTC: %s->%s on file "%s"\n", inform, outform, arg);
Agreed! Doing stderr output of info text is plain wrong :( Will you send a patch mainline please?
Thanks,
Anatolij
Best regards, Marek Vasut
Hi,
On Mar 31, 2012 7:13 AM, "Marek Vasut" marex@denx.de wrote:
Dear Anatolij Gustschin,
On Sat, 31 Mar 2012 15:43:03 +0200 Anatolij Gustschin agust@denx.de wrote: ...
arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1
These boards actually build without errors and warnings, but errors/warnings status is reported by MAKEALL because of the output:
DTC: dts->dtb on file "dt.dtb.tmp"
This could be fixed by
diff --git a/dts/Makefile b/dts/Makefile index 5792afd..d19a622 100644 --- a/dts/Makefile +++ b/dts/Makefile @@ -47,7 +47,7 @@ DT_BIN := $(obj)dt.dtb
$(DT_BIN): $(TOPDIR)/board/$(VENDOR)/dts/$(DEVICE_TREE).dts
cat $< | $(CPP) -P $(DTS_CPPFLAGS) - >$@.tmp
- $(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp
$(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp 2> /dev/null
rm $@.tmp
process_lds = \
Thinking more about this I see that it is wrong. dtc could die for other reasons and it should be handled as error. We should fix dtc instead not to output this info to stderr. Currently it does:
fprintf(stderr, "DTC: %s->%s on file "%s"\n", inform, outform, arg);
Agreed! Doing stderr output of info text is plain wrong :( Will you send
a patch
mainline please?
I will do that if you like, it has bugged me for a while.
Regards, Simon
Thanks,
Anatolij
Best regards, Marek Vasut
Dear Simon Glass,
Hi,
On Mar 31, 2012 7:13 AM, "Marek Vasut" marex@denx.de wrote:
Dear Anatolij Gustschin,
On Sat, 31 Mar 2012 15:43:03 +0200 Anatolij Gustschin agust@denx.de wrote: ...
arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1
These boards actually build without errors and warnings, but errors/warnings status is reported by MAKEALL because of the output:
DTC: dts->dtb on file "dt.dtb.tmp"
This could be fixed by
diff --git a/dts/Makefile b/dts/Makefile index 5792afd..d19a622 100644 --- a/dts/Makefile +++ b/dts/Makefile @@ -47,7 +47,7 @@ DT_BIN := $(obj)dt.dtb
$(DT_BIN): $(TOPDIR)/board/$(VENDOR)/dts/$(DEVICE_TREE).dts
cat $< | $(CPP) -P $(DTS_CPPFLAGS) - >$@.tmp
- $(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp
$(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp 2> /dev/null
rm $@.tmp
process_lds = \
Thinking more about this I see that it is wrong. dtc could die for other reasons and it should be handled as error. We should fix dtc instead not to output this info to stderr. Currently it does:
fprintf(stderr, "DTC: %s->%s on file "%s"\n",
inform, outform, arg);Agreed! Doing stderr output of info text is plain wrong :( Will you send
a patch
mainline please?
I will do that if you like, it has bugged me for a while.
Thank you very much, I owe you :)
Regards, Simon
Thanks,
Anatolij
Best regards, Marek Vasut
Best regards, Marek Vasut
Hi Marek,
On Mar 31, 2012 5:59 AM, "Marek Vasut" marex@denx.de wrote:
Hello guys,
we now have exactly these boards not building for -RC1, can you please
look into
it?:
Do you build all archs including m68k, nds32, etc.?
Regards, Simon
arm/flea3 FAILED FOR: eldk4.2 eldk5.1 arm/hawkboard FAILED FOR: eldk4.2 arm/hawkboard_uart FAILED FOR: eldk4.2 arm/highbank FAILED FOR: eldk4.2 arm/mx35pdk FAILED FOR: eldk4.2 eldk5.1 arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1 powerpc/P2020DS_36BIT FAILED FOR: eldk4.2 powerpc/PMC440 FAILED FOR: eldk4.2 eldk5.1 powerpc/TQM8541 FAILED FOR: eldk4.2 powerpc/TQM8555 FAILED FOR: eldk4.2
If you need logs, please contact me, I'll provide them.
Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Dear Simon Glass,
Hi Marek,
On Mar 31, 2012 5:59 AM, "Marek Vasut" marex@denx.de wrote:
Hello guys,
we now have exactly these boards not building for -RC1, can you please
look into
it?:
Do you build all archs including m68k, nds32, etc.?
Only ARM and PPC @ denx.
I build ARMEL, ARMHF, PPC, MIPS, MIPSEL, X86 and planning to build sandbox @ dorms. I also do DUTS testing on qemu/x86, planning to extend it further though.
Regards, Simon
arm/flea3 FAILED FOR: eldk4.2 eldk5.1 arm/hawkboard FAILED FOR: eldk4.2 arm/hawkboard_uart FAILED FOR: eldk4.2 arm/highbank FAILED FOR: eldk4.2 arm/mx35pdk FAILED FOR: eldk4.2 eldk5.1 arm/seaboard FAILED FOR: eldk4.2 eldk5.1 arm/ventana FAILED FOR: eldk4.2 eldk5.1 powerpc/P2020DS_36BIT FAILED FOR: eldk4.2 powerpc/PMC440 FAILED FOR: eldk4.2 eldk5.1 powerpc/TQM8541 FAILED FOR: eldk4.2 powerpc/TQM8555 FAILED FOR: eldk4.2
If you need logs, please contact me, I'll provide them.
Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Best regards, Marek Vasut
On Sat, Mar 31, 2012 at 02:59:06PM +0200, Marek Vasut wrote:
Hello guys,
we now have exactly these boards not building for -RC1, can you please look into it?:
[snip]
arm/hawkboard FAILED FOR: eldk4.2 arm/hawkboard_uart FAILED FOR: eldk4.2
SPL link failure. Newer toolchains give us a built-in udelay, and we provide __udelay but not udelay on these boards since we don't use CONFIG_SPL_LIBGENERIC_SUPPORT and adding that in might be a fatal size bloat. Not sure what we want to do about this, and the last thread or two about this was unresolved.
Dear Tom Rini,
On Sat, Mar 31, 2012 at 02:59:06PM +0200, Marek Vasut wrote:
Hello guys,
we now have exactly these boards not building for -RC1, can you please look into
it?:
[snip]
arm/hawkboard FAILED FOR: eldk4.2 arm/hawkboard_uart FAILED FOR: eldk4.2
SPL link failure. Newer toolchains give us a built-in udelay, and we provide __udelay but not udelay on these boards since we don't use CONFIG_SPL_LIBGENERIC_SUPPORT and adding that in might be a fatal size bloat. Not sure what we want to do about this, and the last thread or two about this was unresolved.
But we have --gc-sections and stuff like that so unneeded bloat should be removed.
And it's not only this, I see also problem with lldiv.
Configuring for hawkboard board... arch/arm/cpu/arm926ejs/davinci/libdavinci.o: In function `lldiv': /workspace/compiler/eldk4.2/device/arm___hawkboard/include/div64.h:45: undefined reference to `__div64_32' drivers/mtd/nand/libnand.o: In function `nand_command': /workspace/compiler/eldk4.2/device/arm___hawkboard/drivers/mtd/nand/nand_base.c:591: undefined reference to `udelay' /workspace/compiler/eldk4.2/device/arm___hawkboard/drivers/mtd/nand/nand_base.c:607: undefined reference to `udelay' /workspace/compiler/eldk4.2/device/arm___hawkboard/drivers/mtd/nand/nand_base.c:613: undefined reference to `udelay' make[1]: *** [/workspace/compiler/eldk4.2/device/arm___hawkboard/spl/u-boot-spl] Error 1 make: *** [spl/u-boot-spl.bin] Error 2 text data bss dec hex filename 191820 6388 54764 252972 3dc2c ./u-boot
--------------------- SUMMARY ---------------------------- Boards compiled: 1 Boards with warnings or errors: 1 ( hawkboard ) ----------------------------------------------------------
Best regards, Marek Vasut
On Mon, Apr 02, 2012 at 06:10:18PM +0200, Marek Vasut wrote:
Dear Tom Rini,
On Sat, Mar 31, 2012 at 02:59:06PM +0200, Marek Vasut wrote:
Hello guys,
we now have exactly these boards not building for -RC1, can you please look into
it?:
[snip]
arm/hawkboard FAILED FOR: eldk4.2 arm/hawkboard_uart FAILED FOR: eldk4.2
SPL link failure. Newer toolchains give us a built-in udelay, and we provide __udelay but not udelay on these boards since we don't use CONFIG_SPL_LIBGENERIC_SUPPORT and adding that in might be a fatal size bloat. Not sure what we want to do about this, and the last thread or two about this was unresolved.
But we have --gc-sections and stuff like that so unneeded bloat should be removed.
OK, in this case --gc-sections and how our code is structured work together and a quick for each symbol in spl/libgeneric.o check of spl/u-boot-spl shows nothing unneeded added with ELDK4.2 and with newer toolchains nothing at all added.
And it's not only this, I see also problem with lldiv.
Same thing, built-in vs not.
Dear Tom Rini,
On Mon, Apr 02, 2012 at 06:10:18PM +0200, Marek Vasut wrote:
Dear Tom Rini,
On Sat, Mar 31, 2012 at 02:59:06PM +0200, Marek Vasut wrote:
Hello guys,
we now have exactly these boards not building for -RC1, can you please look into
it?:
[snip]
arm/hawkboard FAILED FOR: eldk4.2 arm/hawkboard_uart FAILED FOR: eldk4.2
SPL link failure. Newer toolchains give us a built-in udelay, and we provide __udelay but not udelay on these boards since we don't use CONFIG_SPL_LIBGENERIC_SUPPORT and adding that in might be a fatal size bloat. Not sure what we want to do about this, and the last thread or two about this was unresolved.
But we have --gc-sections and stuff like that so unneeded bloat should be removed.
OK, in this case --gc-sections and how our code is structured work together and a quick for each symbol in spl/libgeneric.o check of spl/u-boot-spl shows nothing unneeded added with ELDK4.2 and with newer toolchains nothing at all added.
And it's not only this, I see also problem with lldiv.
Same thing, built-in vs not.
So I can expect a patch?
Best regards, Marek Vasut
participants (4)
-
Anatolij Gustschin -
Marek Vasut -
Simon Glass -
Tom Rini