From mboxrd@z Thu Jan 1 00:00:00 1970 From: Masahiro Yamada Date: Thu, 14 Sep 2017 11:08:46 +0900 Subject: [U-Boot] [PATCH 0/6] Sync and consolidate Linux-derived printk, BUILD_BUG, BUG, WARN, etc. In-Reply-To: <1505303106-9780-1-git-send-email-yamada.masahiro@socionext.com> References: <1505303106-9780-1-git-send-email-yamada.masahiro@socionext.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 2017-09-13 20:45 GMT+09:00 Masahiro Yamada : > 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 +: Warning (unit_address_vs_reg): Node /images/uboot at 1 has a unit name, but no reg property +: Warning (unit_address_vs_reg): Node /images/atf at 1 has a unit name, but no reg property +: Warning (unit_address_vs_reg): Node /images/fdt at 1 has a unit name, but no reg property +: Warning (unit_address_vs_reg): Node /configurations/config at 1 has a unit name, but no reg property +: Warning (unit_address_vs_reg): Node /images/fdt at 2 has a unit name, but no reg property +: Warning (unit_address_vs_reg): Node /configurations/config at 2 has a unit name, but no reg property + writel((unsigned int) ep->dma_buf, ®->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 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 06: bug.h: sync BUILD_BUG stuff with Linux 4.13 07: bug.h: move runtime BUG/WARN macros into -../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