[U-Boot] [PATCH 0/6] Sync and consolidate Linux-derived printk, BUILD_BUG, BUG, WARN, etc.

Masahiro Yamada yamada.masahiro at socionext.com
Thu Sep 14 02:08:46 UTC 2017 

2017-09-13 20:45 GMT+09:00 Masahiro Yamada <yamada.masahiro at socionext.com>:
> buildman is 60% done, and no regression so far.
>
> I will report the full-stat when buildman is complete.
>
>

Finished buildman test.

No regression from the point of C grammar.

But, image size exceeded its platform limit
for some boards.
(ex. colibri_vf_defconfig)


This is because all pr_*() except pr_debug()
are now enabled.
So, more are more log message strings are compiled now.

Hmm, I need to implement
CONFIG_LOGLEVEL and CONFIG_SPL_LOGLEVEL now
to shut up lower priority log functions
such as pr_info(), pr_notice() ??

I will take a look.


The full-stat of buildman is as follows:




01: ARM: DRA72x: Add support for detection of DRA71x SR 2.1
   aarch64:  +   nanopi_neo2 bananapi_m64 orangepi_prime orangepi_pc2
pine64_plus orangepi_win a64-olinuxino evb-rk3328 nanopi_a64
sopine_baseboard orangepi_zero_plus2
       x86:  +   qemu-x86_64 qemu-x86_efi_payload64 chromebook_link64
       arm:  +   am43xx_hs_evm am57xx_hs_evm imx6qdl_icore_rqs
am335x_hs_evm ethernut5 am43xx_evm imx6qdl_icore_nand
imx6qdl_icore_mmc dra7xx_hs_evm am43xx_evm_usbhost_boot db-88f6720
am335x_evm am57xx_evm dra7xx_evm
     nds32:  +   adp-ag101p adp-ae3xx
+       src, &length)) {
+            ^
+In file included from ../common/spl/spl_fit.c:8:0:
+../include/common.h:627:5: note: expected 'long unsigned int *' but
argument is of type 'size_t * {aka unsigned int *}'
+ int gunzip(void *, int, unsigned char *, unsigned long *);
+     ^~~~~~
+   if (gunzip((void *)load_addr, CONFIG_SYS_BOOTM_LEN,
+       ^
+../include/common.h:627:5: note: expected 'long unsigned int *' but
argument is of type 'size_t *'
+     ^
+../arch/nds32/cpu/n1213/start.S: Assembler messages:
+../arch/nds32/cpu/n1213/start.S:109: Error: unknown pseudo-op: `.relax_hint'
+../arch/nds32/cpu/n1213/start.S:111: Error: unknown pseudo-op: `.relax_hint'
+../arch/nds32/cpu/n1213/start.S:113: Error: unrecognized form of
instruction `add5.pc $gp'
+make[2]: *** [../scripts/Makefile.build:316:
arch/nds32/cpu/n1213/start.o] Error 1
+make[1]: *** [Makefile:1279: arch/nds32/cpu/n1213] Error 2
+make: *** [Makefile:150: sub-make] Error 2
+arch/arm/dts/ethernut5.dtb: Warning (reg_format): "reg" property in
/i2c-gpio-0/pcf8563 at 50 has invalid length (4 bytes) (#address-cells ==
2, #size-cells == 1)
+arch/arm/dts/ethernut5.dtb: Warning (avoid_default_addr_size):
Relying on default #address-cells value for /i2c-gpio-0/pcf8563 at 50
+arch/arm/dts/ethernut5.dtb: Warning (avoid_default_addr_size):
Relying on default #size-cells value for /i2c-gpio-0/pcf8563 at 50
+WARNING: BL31 file bl31.bin NOT found, resulting binary is non-functional
+Please read the section on ARM Trusted Firmware (ATF) in
board/sunxi/README.sunxi64
+<stdout>: Warning (unit_address_vs_reg): Node /images/uboot at 1 has a
unit name, but no reg property
+<stdout>: Warning (unit_address_vs_reg): Node /images/atf at 1 has a
unit name, but no reg property
+<stdout>: Warning (unit_address_vs_reg): Node /images/fdt at 1 has a
unit name, but no reg property
+<stdout>: Warning (unit_address_vs_reg): Node
/configurations/config at 1 has a unit name, but no reg property
+<stdout>: Warning (unit_address_vs_reg): Node /images/fdt at 2 has a
unit name, but no reg property
+<stdout>: Warning (unit_address_vs_reg): Node
/configurations/config at 2 has a unit name, but no reg property
+  writel((unsigned int) ep->dma_buf, &reg->out_endp[ep_num].doepdma);
+         ^
+../arch/arm/include/asm/io.h:146:34: note: in definition of macro 'writel'
+ #define writel(v,c) ({ u32 __v = v; __iowmb(); __arch_putl(__v,c); __v; })
+                                  ^
+   port->priv->mdio_base = (void *)mdio_addr;
+                           ^
+../arch/x86/lib/crt0_x86_64_efi.S: Assembler messages:
+../arch/x86/lib/crt0_x86_64_efi.S:16: Error: bad register name `%rsp'
+../arch/x86/lib/crt0_x86_64_efi.S:17: Error: bad register name `%rcx'
+../arch/x86/lib/crt0_x86_64_efi.S:18: Error: bad register name `%rdx'
+../arch/x86/lib/crt0_x86_64_efi.S:21: Error: bad register name `%rip)'
+../arch/x86/lib/crt0_x86_64_efi.S:22: Error: bad register name `%rip)'
+../arch/x86/lib/crt0_x86_64_efi.S:24: Error: bad register name `%rcx'
+../arch/x86/lib/crt0_x86_64_efi.S:25: Error: bad register name `%rdx'
+../arch/x86/lib/crt0_x86_64_efi.S:26: Error: bad register name `%rcx'
+../arch/x86/lib/crt0_x86_64_efi.S:27: Error: bad register name `%rdx'
+../arch/x86/lib/crt0_x86_64_efi.S:30: Error: bad register name `%rdi'
+../arch/x86/lib/crt0_x86_64_efi.S:31: Error: bad register name `%rsi'
+../arch/x86/lib/crt0_x86_64_efi.S:34: Error: bad register name `%rsp'
+../arch/x86/lib/crt0_x86_64_efi.S:47: Warning: setting incorrect
section attributes for .reloc
+make[2]: *** [../scripts/Makefile.build:316:
arch/x86/lib/crt0_x86_64_efi.o] Error 1
+make[1]: *** [Makefile:1279: arch/x86/lib] Error 2
w+../common/spl/spl_fit.c: In function 'spl_load_fit_image':
w+../common/spl/spl_fit.c:201:12: warning: passing argument 4 of
'gunzip' from incompatible pointer type [-Wincompatible-pointer-types]
w+../common/spl/spl_fit.c:200:7: warning: passing argument 4 of
'gunzip' from incompatible pointer type
w+../drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c: In function 'setdma_rx':
w+../drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c:116:9: warning: cast
from pointer to integer of different size [-Wpointer-to-int-cast]
w+../drivers/net/mvpp2.c: In function 'phy_info_parse':
w+../drivers/net/mvpp2.c:4751:27: warning: cast to pointer from
integer of different size [-Wint-to-pointer-cast]
02: printk: collect printk stuff into <linux/printk.h>
       arm:  +   colibri_vf openrd_base vf610twr_nand openrd_ultimate
vf610twr openrd_client
-../include/common.h:627:5: note: expected 'long unsigned int *' but
argument is of type 'size_t * {aka unsigned int *}'
-../include/common.h:627:5: note: expected 'long unsigned int *' but
argument is of type 'size_t *'
+../include/common.h:623:5: note: expected 'long unsigned int *' but
argument is of type 'size_t * {aka unsigned int *}'
+../include/common.h:623:5: note: expected 'long unsigned int *' but
argument is of type 'size_t *'
+u-boot-nodtb.bin exceeds file size limit:
+  limit:  393216 bytes
+  actual: 434976 bytes
+  excess: 41760 bytes
+make[1]: *** [Makefile:945: u-boot-nodtb.bin] Error 1
+  limit:  524288 bytes
+  actual: 541568 bytes
+  excess: 17280 bytes
+  actual: 435000 bytes
+  excess: 41784 bytes
+  actual: 541520 bytes
+  excess: 17232 bytes
+  actual: 435024 bytes
+  excess: 41808 bytes
+  actual: 543952 bytes
+  excess: 19664 bytes
03: treewide: replace with error() with pr_err()
-  actual: 434976 bytes
-  excess: 41760 bytes
-  actual: 541568 bytes
-  excess: 17280 bytes
-  actual: 435000 bytes
-  excess: 41784 bytes
-  actual: 541520 bytes
-  excess: 17232 bytes
-  actual: 435024 bytes
-  excess: 41808 bytes
-  actual: 543952 bytes
-  excess: 19664 bytes
+  actual: 434624 bytes
+  excess: 41408 bytes
+  actual: 541184 bytes
+  excess: 16896 bytes
+  actual: 434656 bytes
+  excess: 41440 bytes
+  actual: 541136 bytes
+  excess: 16848 bytes
+  actual: 434672 bytes
+  excess: 41456 bytes
+  actual: 542608 bytes
+  excess: 18320 bytes
04: common.h: remove error()
-../include/common.h:623:5: note: expected 'long unsigned int *' but
argument is of type 'size_t * {aka unsigned int *}'
-../include/common.h:623:5: note: expected 'long unsigned int *' but
argument is of type 'size_t *'
+../include/common.h:618:5: note: expected 'long unsigned int *' but
argument is of type 'size_t * {aka unsigned int *}'
+../include/common.h:618:5: note: expected 'long unsigned int *' but
argument is of type 'size_t *'
05: vsprintf.h: include <linux/types.h>
06: bug.h: sync BUILD_BUG stuff with Linux 4.13
07: bug.h: move runtime BUG/WARN macros into <linux/bug.h>
-../include/common.h:618:5: note: expected 'long unsigned int *' but
argument is of type 'size_t * {aka unsigned int *}'
-../include/common.h:618:5: note: expected 'long unsigned int *' but
argument is of type 'size_t *'
-  actual: 434624 bytes
-  excess: 41408 bytes
-  actual: 541184 bytes
-  excess: 16896 bytes
-  actual: 434656 bytes
-  excess: 41440 bytes
-  actual: 541136 bytes
-  excess: 16848 bytes
-  actual: 434672 bytes
-  excess: 41456 bytes
-  actual: 542608 bytes
-  excess: 18320 bytes
+../include/common.h:611:5: note: expected 'long unsigned int *' but
argument is of type 'size_t * {aka unsigned int *}'
+../include/common.h:611:5: note: expected 'long unsigned int *' but
argument is of type 'size_t *'
+  actual: 434760 bytes
+  excess: 41544 bytes
+  actual: 541312 bytes
+  excess: 17024 bytes
+  actual: 434784 bytes
+  excess: 41568 bytes
+  actual: 541272 bytes
+  excess: 16984 bytes
+  actual: 434800 bytes
+  excess: 41584 bytes
+  actual: 542840 bytes
+  excess: 18552 bytes







-- 
Best Regards
Masahiro Yamada

More information about the U-Boot mailing list