[U-Boot] [PATCH v2] imx: Use a convenient default value for SYS_MALLOC_F_LEN

[Joris OFFOUGA]

Hi Fabio,

Tested-by Joris Offouga <offougajoris at gmail.com>

BR,

Joris


Le ven. 3 mai 2019 à 19:20, Pierre-Jean Texier <pjtexier at koncepto.io> a
écrit :

> Hi Fabio,
>
> Le 03/05/2019 à 19:05, Fabio Estevam a écrit :
> > Commit 3a7c45f6a772 ("simple-bus: add DM_FLAG_PRE_RELOC flag to
> > simple-bus driver") causes some i.MX boards that were converted
> > to DM, such as warp7, to fail to boot.
> >
> > As explained by Lukas Auer:
> >
> > "With the patch, U-Boot probes the drivers for devices under simple-bus
> > device tree nodes in the pre-relocation device model. The default value
> > of CONFIG_SYS_MALLOC_F_LEN (0x400) leaves U-Boot with not enough memory
> to
> > do this, causing it to hang."
> >
> > Fix this problem by providing a convenient default value for
> > CONFIG_SYS_MALLOC_F_LEN.
> >
> > Reported-by: Pierre-Jean Texier <pjtexier at koncepto.io>
> > Suggested-by: Lukas Auer <lukas.auer at aisec.fraunhofer.de>
> > Signed-off-by: Fabio Estevam <festevam at gmail.com>
> > ---
> > Changes since v1:
> > - Move the default setting to the main Kconfig and make it depend
> > on i.MX
> >
> >   Kconfig | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/Kconfig b/Kconfig
> > index 7a5491bd67..fd4ff42c17 100644
> > --- a/Kconfig
> > +++ b/Kconfig
> > @@ -138,6 +138,8 @@ config SYS_MALLOC_F_LEN
> >       depends on SYS_MALLOC_F
> >       default 0x1000 if AM33XX
> >       default 0x2800 if SANDBOX
> > +     default 0x2000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_MX7 || \
> > +                        ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5)
> >       default 0x400
> >       help
> >         Before relocation, memory is very limited on many platforms.
> Still,
>
>
> Tested-by: Pierre-Jean Texier <pjtexier at koncepto.io>
>
>
> Thanks
>
> Pierre-Jean
>
>