[U-Boot] [PATCH 045/126] spl: Add a size check for TPL

[Simon Glass]

Hi Simon,

On Thu, 26 Sep 2019 at 06:23, Simon Goldschmidt
<simon.k.r.goldschmidt at gmail.com> wrote:
>
> Hi Simon,
>
> On Wed, Sep 25, 2019 at 5:36 PM Simon Glass <sjg at chromium.org> wrote:
> >
> > We have the ability to enforce a maximum size for SPL but not yet for TPL.
> > Add a new option for this.
> >
> > Document the size check macro while we are here.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> >  Makefile           | 7 +++++++
> >  common/spl/Kconfig | 8 ++++++++
> >  2 files changed, 15 insertions(+)
> >
> > diff --git a/Makefile b/Makefile
> > index a7a48b6aef3..43961af590f 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -806,6 +806,12 @@ else
> >  SPL_SIZE_CHECK =
> >  endif
> >
> > +ifneq ($(CONFIG_TPL_SIZE_LIMIT),0)
> > +TPL_SIZE_CHECK = @$(call size_check,$@,$(CONFIG_TPL_SIZE_LIMIT))
>
> Using a constant here is not what SPL does by now: some months ago, I
> implemented a more sophisticated size check for SPL that also integrates
> 'gd', stack and heap usage into the size check. Wouldn't that be required for
> TPL as well (at least on platforms where everything must fit into SRAM)?

Yes I see, but in this case we just need the image to be below a
certain size. The actual memory used is somewhere else.

Regards,
SImon