[PATCH] Kconfig: Fix SYS_MALLOC_F_LEN for i.MX8MQ

[Tom Rini]

On Tue, Jul 05, 2022 at 12:19:54PM +0200, Heiko Thiery wrote:
> Hi all,
> 
> Am Mo., 11. Apr. 2022 um 14:29 Uhr schrieb Tom Rini <trini at konsulko.com>:
> >
> > On Mon, Apr 11, 2022 at 05:07:04PM +0800, Peng Fan (OSS) wrote:
> >
> > > From: Peng Fan <peng.fan at nxp.com>
> > >
> > > i.MX8MQ use SYS_MALLOC_F_LEN 0x2000, not 0x10000. The OCRAM_S only has
> > > 32KB memory, use 0x10000 will make SPL not bootable.
> > >
> > > Signed-off-by: Peng Fan <peng.fan at nxp.com>
> > > Reviewed-by: Michael Trimarchi <michael at amarulasolutions.com>
> > > Reviewed-by: Fabio Estevam <festevam at denx.de>
> > > ---
> > >  Kconfig | 6 ++++--
> > >  1 file changed, 4 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/Kconfig b/Kconfig
> > > index 6b1ddfc549e..3e95ec9d7b9 100644
> > > --- a/Kconfig
> > > +++ b/Kconfig
> > > @@ -251,8 +251,10 @@ config SYS_MALLOC_F_LEN
> > >       default 0x4000 if SANDBOX || RISCV
> > >       default 0x2000 if (ARCH_MX7 || ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5 || \
> > >                          ARCH_QEMU || ARCH_SUNXI || ARCH_OWL)
> > > -     default 0x10000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_LS1012A || \
> > > -                         ARCH_LS1021A || ARCH_LS1043A || ARCH_LS1046A)
> > > +     default 0x10000 if (ARCH_IMX8 || (ARCH_IMX8M && !IMX8MQ) || \
> > > +                         ARCH_LS1012A || ARCH_LS1021A || ARCH_LS1043A || \
> > > +                         ARCH_LS1046A)
> > > +     default 0x2000 if IMX8MQ
> > >       default 0x400
> > >       help
> > >         Before relocation, memory is very limited on many platforms. Still,
> >
> > Note that we can put this with the rest of the 0x2000 default instances.
> > And as an aside, imx8mq_phanbell_defconfig sets this to 0x4000.  That
> > said, I've applied this to master (with the location change I noted), thanks!
> >
> > --
> > Tom
> 
> I just tested v2022.07-rc6 and noticed that by reducing the value the
> kontron pitx-imx8m (imx8mq) board does not start anymore. I locally
> increased the value to 0xc000 and the board works again.
> 
> So should I increase the value just for this board or should I
> increase the default value for the IMX8MQ?

I would _really_ like to see defaults per SoC.  Even if that means
saying "default 0xc000 if IMX8Q && SPL_FEAT_X" as I know with tight
constraints on memory AND the need to have some of the bigger feature
knobs enabled it can be tricky to get usable defaults.  But what I
really want to avoid are both the problem of everyone copy/pasting a
value that's too big or too small and also avoid the problem of N boards
needing the same fix because they all failed due to the same problem.

-- 
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/20220705/c066d4e5/attachment.sig>