On Sat, 6 Jul 2024 at 01:18, Tom Rini <tr...@konsulko.com> wrote:
>
> On Thu, Jul 04, 2024 at 01:05:12PM +0530, Sughosh Ganu wrote:
>
> > Add separate config symbols for enabling the LMB module for the SPL
> > phase. The LMB module implementation now relies on alloced list data
> > structure which requires heap area to be present. Add specific config
> > symbol for the SPL phase of U-Boot so that this can be enabled on
> > platforms which support a heap in SPL.
> >
> > Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org>
> > ---
> > Changes since V1: New patch
> >
> >  lib/Kconfig | 12 +++++++++++-
> >  1 file changed, 11 insertions(+), 1 deletion(-)
> >
> > diff --git a/lib/Kconfig b/lib/Kconfig
> > index 072ed0ecfa..7eea517b3b 100644
> > --- a/lib/Kconfig
> > +++ b/lib/Kconfig
> > @@ -1103,7 +1103,17 @@ config LMB
> >       default y if ARC || ARM || M68K || MICROBLAZE || MIPS || \
> >                    NIOS2 || PPC || RISCV || SANDBOX || SH || X86 || XTENSA
> >       help
> > -       Support the library logical memory blocks.
> > +       Support the library logical memory blocks. This will require
> > +       a malloc() implementation for defining the data structures
> > +       needed for maintaining the LMB memory map.
>
> Even today, LMB really should be def_bool y rather than an option, so
> this series should correct that. That said...

Okay

>
> > +config SPL_LMB
> > +     bool "Enable LMB module for SPL"
> > +     depends on SPL && SPL_FRAMEWORK && SPL_SYS_MALLOC
> > +     help
> > +       Enable support for Logical Memory Block library routines in
> > +       SPL. This will require a malloc() implementation for defining
> > +       the data structures needed for maintaining the LMB memory map.
>
> The question I guess becomes when do we need LMB in SPL, exactly? And I
> guess it's another case where it should be def_bool y (but still depends
> on what you have here) since we need to make sure we don't overwrite
> running SPL.

So this is a question even I had. Do we really need to enable LMB in
SPL ? The main reason for introducing the symbol was to have more
granularity to remove the LMB code from SPL, but should this really be
enabled in SPL is something that I am not too sure about.

-sughosh

>
> --
> Tom

Reply via email to