Hi Wolfgang, On Tue, Dec 4, 2018 at 7:37 AM Wolfgang Denk <w...@denx.de> wrote:
> Is there any chance you mis-applied my patch? Ok, so I started again. 1. Applied the following patch: http://dark-code.bulix.org/tualst-517948 2. make mproper; make make pico-pi-imx7d_defconfig; make 3. Build fails: /bin/sh: 1: printf: ((768 - 69) * 1024): expected numeric value u-boot-nodtb.bin exceeds file size limit: limit: 0 bytes actual: 482952 bytes excess: 482952 bytes The reason for the failure is because there is an extra CONFIG_BOARD_SIZE_LIMIT check inside the main Makefile. Your patch only covers arch/arm/mach-imx/Makefile. If I remove the check from the main Makefile: --- a/Makefile +++ b/Makefile @@ -772,21 +772,6 @@ LDPPFLAGS += \ ######################################################################### ######################################################################### -ifneq ($(CONFIG_BOARD_SIZE_LIMIT),) -BOARD_SIZE_CHECK = \ - @actual=`wc -c $@ | awk '{print $$1}'`; \ - limit=`printf "%d" $(CONFIG_BOARD_SIZE_LIMIT)`; \ - if test $$actual -gt $$limit; then \ - echo "$@ exceeds file size limit:" >&2 ; \ - echo " limit: $$limit bytes" >&2 ; \ - echo " actual: $$actual bytes" >&2 ; \ - echo " excess: $$((actual - limit)) bytes" >&2; \ - exit 1; \ - fi -else -BOARD_SIZE_CHECK = -endif - # Statically apply RELA-style relocations (currently arm64 only) # This is useful for arm64 where static relocation needs to be performed on # the raw binary, but certain simulators only accept an ELF file (but don't Then I am able to successfully build it. It seems we need to avoid the double CONFIG_BOARD_SIZE_LIMIT check. Ideas? Thanks _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
