[PATCH v2 1/2] spl: Add size check for u-boot-with-spl.bin

[Philip Oberfichtner]

Hi Tom,

On Tue, May 27, 2025 at 07:49:06AM -0600, Tom Rini wrote:
> On Tue, May 27, 2025 at 02:10:48PM +0200, Philip Oberfichtner wrote:
> 
> > Introduce another SIZE_CHECK macro for u-boot-with-spl.bin.
> > 
> > Signed-off-by: Philip Oberfichtner <pro at denx.de>
> > ---
> > 
> > Notes:
> >     Changes in v2: none
> > 
> >  Makefile           | 7 +++++++
> >  common/spl/Kconfig | 7 +++++++
> >  2 files changed, 14 insertions(+)
> 
> So, for u-boot-with-spl.imx we use BOARD_SIZE_CHECK and
> CONFIG_BOARD_SIZE_LIMIT. I think for consistency it would be better to
> re-use that option here and expand the help text on
> CONFIG_BOARD_SIZE_LIMIT to note which build targets make use of it.

Thanks for the feedback! I've tried that, but I have the following
issue:
	- 402K for u-boot-nodtb.bin
	- 152K for u-boot-lzma.img
	- 191K for u-boot-with-spl.bin

The end result, u-boot-with-spl.bin, meets the size restriction of 192K.
Nevertheless, an intermediary (uncompressed) artifact, u-boot-nodtb.bin,
causes the build to fail.


Instead of introducing a new Kconfig option, it should be fine to reuse 
CONFIG_SPL_SIZE_LIMIT for u-boot-with-spl.bin. I'm not sure which way is
better. What do you think?

Best regards,
Philip


> 
> -- 
> Tom



-- 
=====================================================================
DENX Software Engineering GmbH,        Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-22  Fax: +49-8142-66989-80   Email: pro at denx.de
=====================================================================