| Message ID | 20250520085436.925901-5-dario.binacchi@amarulasolutions.com |
|---|---|
| State | New |
| Headers | show |
| Series |
|
| Related | show |
Hi Dario and Michael, On Tue, May 20, 2025 at 5:55 AM Dario Binacchi <dario.binacchi@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 <michael@amarulasolutions.com> > Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com> > Signed-off-by: Dario Binacchi <dario.binacchi@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 <festevam@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. To unsubscribe from this group and stop receiving emails from it, send an email to linux-amarula+unsubscribe@amarulasolutions.com.
On Tue, 20 May 2025 09:11:26 -0300 Fabio Estevam <festevam@gmail.com> wrote: > Hi Dario and Michael, > > On Tue, May 20, 2025 at 5:55 AM Dario Binacchi > <dario.binacchi@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 <michael@amarulasolutions.com> > > Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com> > > Signed-off-by: Dario Binacchi <dario.binacchi@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 <festevam@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. To unsubscribe from this group and stop receiving emails from it, send an email to linux-amarula+unsubscribe@amarulasolutions.com.
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 CONFIG_SPL_TEXT_BASE=0x912000 CONFIG_SPL_HAS_BSS_LINKER_SECTION=y CONFIG_SPL_BSS_START_ADDR=0x950000 diff --git a/configs/imx8mn_bsh_smm_s2pro_defconfig b/configs/imx8mn_bsh_smm_s2pro_defconfig index d2010d13b8cb..ef486f51eeeb 100644 --- a/configs/imx8mn_bsh_smm_s2pro_defconfig +++ b/configs/imx8mn_bsh_smm_s2pro_defconfig @@ -16,7 +16,6 @@ CONFIG_SPL_MMC=y CONFIG_SPL_SERIAL=y CONFIG_SPL_DRIVERS_MISC=y CONFIG_SPL_STACK=0x980000 -CONFIG_SPL_SYS_MALLOC_F_LEN=0x2000 CONFIG_SPL_TEXT_BASE=0x912000 CONFIG_SPL_HAS_BSS_LINKER_SECTION=y CONFIG_SPL_BSS_START_ADDR=0x950000
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 <michael@amarulasolutions.com> Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com> Signed-off-by: Dario Binacchi <dario.binacchi@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(-)