[PATCH] ls10xxx: Use a sane SPL_SYS_MALLOC_F_LEN default

[Tom Rini]

On Thu, Apr 28, 2022 at 12:31:08PM -0400, Sean Anderson wrote:
> Hi Fabio,
> 
> On 4/28/22 12:23 PM, Fabio Estevam wrote:
> > Hi Sean,
> > 
> > On Thu, Apr 28, 2022 at 1:02 PM Sean Anderson <sean.anderson at seco.com> wrote:
> >>
> >> SPL_SYS_MALLOC_F_LEN defaults to SYS_MALLOC_F_LEN. 0x10000 (the new
> >> default) is 64 KiB, or around half of the total OCRAM size. Revert to
> >> the previous default of 0x2000. This fixes SPL boot.
> >>
> >> Fixes: 545eceb520 ("imx8/ls10xx: Use a sane SYS_MALLOC_F_LEN default")
> >> Signed-off-by: Sean Anderson <sean.anderson at seco.com>
> > 
> > Thanks for the fix.
> > 
> > Would it be better to fix it like this instead?
> > 
> > --- a/Kconfig
> > +++ b/Kconfig
> > @@ -257,9 +257,7 @@ config SYS_MALLOC_F_LEN
> >         default 0x4000 if SANDBOX || RISCV || ARCH_APPLE || ROCKCHIP_RK3368 || \
> >                           ROCKCHIP_RK3399
> >         default 0x8000 if RCAR_GEN3
> > -       default 0x10000 if ARCH_IMX8 || (ARCH_IMX8M && !IMX8MQ) || \
> > -                          ARCH_LS1012A || ARCH_LS1021A || ARCH_LS1043A || \
> > -                          ARCH_LS1046A
> > +       default 0x10000 if ARCH_IMX8 || (ARCH_IMX8M && !IMX8MQ)
> >         default 0x2000
> >         help
> >           Before relocation, memory is very limited on many platforms. Still,
> > 
> 
> That would also work. The primary problem is that between v2 and v3 the default
> got raised from 0x2000 to 0x10000. Do you remember why you added layerscape
> stuff in the first place? The only non-default value for this property for this
> family was to set it to 0x6000 for some ls2 (ls3?) SoCs (e.g. ls1088).

So yes, I asked since I _think_ before layerscape was using a
smaller-than-0x2000 value and we have had a number of problems with
that.  Sorry for the noise!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220428/b7c01ad8/attachment.sig>