Hi Wolfgang,
On Tue, Dec 4, 2018 at 7:37 AM Wolfgang Denk wd@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