[U-Boot] [PATCH v4] pico-imx7d: Increase the CONFIG_ENV_OFFSET size

Fabio Estevam festevam at gmail.com
Tue Dec 4 10:41:01 UTC 2018


Hi Wolfgang,

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


More information about the U-Boot mailing list