[PATCH v3 4/5] board: bsh: imx8mn_bsh_smm_s2/s2pro: enlarge CONFIG_SPL_SYS_MALLOC_F_LEN

[Fabio Estevam]

Hi Dario and Michael,

On Tue, May 20, 2025 at 5:55 AM Dario Binacchi
<dario.binacchi at 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 at amarulasolutions.com>
> Signed-off-by: Michael Trimarchi <michael at amarulasolutions.com>
> Signed-off-by: Dario Binacchi <dario.binacchi at 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 at 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.