On Tue, 20 May 2025 09:11:26 -0300
Fabio Estevam <feste...@gmail.com> wrote:
> Hi Dario and Michael,
>
> On Tue, May 20, 2025 at 5:55 AM Dario Binacchi
> <dario.binac...@amarulasolutions.com> wrote:
> >
> > The commit dda454e933c6 ("serial: mxc: Support bulk enabling clocks")
> > breaks the booting of the BSH SMM S2 board. The analysis of the issue
> > revealed memory allocation failures during the registration of UART4
> > clocks as well as other peripherals. Increasing SYS_MALLOC_F_LEN to
> > 0x10000 fixed the issue.
> >
> > Dropping this option allows it to be set to the default value of
> > CONFIG_SYS_MALLOC_F_LEN, which is set by default to 0x10000 on
> > i.MX8M platforms.
> >
> > Co-developed-by: Michael Trimarchi <mich...@amarulasolutions.com>
> > Signed-off-by: Michael Trimarchi <mich...@amarulasolutions.com>
> > Signed-off-by: Dario Binacchi <dario.binac...@amarulasolutions.com>
> > ---
> >
> > (no changes since v1)
> >
> > configs/imx8mn_bsh_smm_s2_defconfig | 1 -
> > configs/imx8mn_bsh_smm_s2pro_defconfig | 1 -
> > 2 files changed, 2 deletions(-)
> >
> > diff --git a/configs/imx8mn_bsh_smm_s2_defconfig
> > b/configs/imx8mn_bsh_smm_s2_defconfig
> > index 1c11d4705419..206d216394db 100644
> > --- a/configs/imx8mn_bsh_smm_s2_defconfig
> > +++ b/configs/imx8mn_bsh_smm_s2_defconfig
> > @@ -15,7 +15,6 @@ CONFIG_SYS_MONITOR_LEN=524288
> > CONFIG_SPL_SERIAL=y
> > CONFIG_SPL_DRIVERS_MISC=y
> > CONFIG_SPL_STACK=0x980000
> > -CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000
>
> Thanks for investigating this issue and getting to the root cause.
>
> Reviewed-by: Fabio Estevam <feste...@gmail.com>
>
> I'm adding Hugo to the Cc as the imx8mn_var_som_defconfig target also
> uses CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000.
>
> Hugo, please consider doing the same change on imx8mn_var_som_defconfig.
Hi Fabio,
sorry for the late response.
Yes, I also ran into the same problem and manually set
CONFIG_SPL_SYS_MALLOC_F_LEN to 128kB, as was done in commit
52472504e9c4 ("rockchip: rk3568: Fix alloc space exhausted in SPL").
I have now tested with the default value of 64kB (0x10000) by
removing CONFIG_SPL_SYS_MALLOC_F_LEN and it also works fine.
I will submit a patch for this.
Thank you,
Hugo.
