[PATCH v4 1/3] Makefile: Add size check for u-boot-with-spl.bin

[Ilias Apalodimas]

Hi Philip,

On Tue, 8 Jul 2025 at 13:40, Philip Oberfichtner <pro at denx.de> wrote:
>
> Like other images, u-boot-with-spl.bin may be subject to size
> restrictions. Extend CONFIG_SPL_SIZE_LIMIT to handle this case.
>
> Signed-off-by: Philip Oberfichtner <pro at denx.de>
> ---
>
> Notes:
>     Changes in v4: none
>
>     Changes in v3:
>         Reuse existing SPL_SIZE_LIMIT instead of implementing a new option
>
>     Changes in v2: none
>
>  Makefile           | 1 +
>  common/spl/Kconfig | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index fa9ce756083..791c00713ec 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1545,6 +1545,7 @@ OBJCOPYFLAGS_u-boot-with-spl.bin = -I binary -O binary \
>                                    --pad-to=$(CONFIG_SPL_PAD_TO)
>  u-boot-with-spl.bin: $(SPL_IMAGE) $(SPL_PAYLOAD) FORCE
>         $(call if_changed,pad_cat)
> +       $(SPL_SIZE_CHECK)

There's a rule in the Makefile
ifneq ($(CONFIG_SPL_SIZE_LIMIT),0x0)
SPL_SIZE_CHECK = @$(call size_check,$@,$$(tools/spl_size_limit))
else
SPL_SIZE_CHECK =
endif

and

isn't this running?

Thanks
/Ilias

>
>  ifeq ($(CONFIG_ARCH_LPC32XX)$(CONFIG_SPL),yy)
>  MKIMAGEFLAGS_lpc32xx-spl.img = -T lpc32xximage -a $(CONFIG_SPL_TEXT_BASE)
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index 9a17ccb2d3d..dcb25b593f1 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -43,7 +43,7 @@ config SPL_SIZE_LIMIT
>         default 0x30000 if ARCH_MVEBU && ARMADA_32BIT
>         default 0x0
>         help
> -         Specifies the maximum length of the U-Boot SPL image.
> +         Specifies the maximum length of U-Boot SPL images.
>           If this value is zero, it is ignored.
>
>  config SPL_SIZE_LIMIT_SUBTRACT_GD
> --
> 2.39.5
>