[PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN

ZHIZHIKIN Andrey andrey.zhizhikin at leica-geosystems.com
Mon Dec 21 23:04:08 CET 2020


Hello Fabio,

> -----Original Message-----
> From: Fabio Estevam <festevam at gmail.com>
> Sent: Monday, December 21, 2020 7:41 PM
> To: sbabic at denx.de
> Cc: peng.fan at nxp.com; u-boot at lists.denx.de; ZHIZHIKIN Andrey
> <andrey.zhizhikin at leica-geosystems.com>; uboot-imx at nxp.com; Fabio Estevam
> <festevam at gmail.com>
> Subject: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
> 
> 
> When booting imx8mp-evk the following allocation error message is seen:
> 
> U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300)
> 
> alloc space exhausted
> 
> Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on
> other i.MX8MM/8MN boards.

It would be really great to get a review from Peng here, as his commit 28fff3fa1c
("imx: imx8mp_evk: fix boot issue") changed exactly this option from 0x10000 to 0x8000.

I've tested my local build with your patch and it boots OK, hence I cannot seem to find a
reasoning why the early malloc area size has been halved with above commit.

> 
> Reported-by: Andrey Zhizhikin <andrey.zhizhikin at leica-geosystems.com>
> Signed-off-by: Fabio Estevam <festevam at gmail.com>
> ---
>  configs/imx8mp_evk_defconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configs/imx8mp_evk_defconfig b/configs/imx8mp_evk_defconfig
> index cd5724e811ab..c34a10c2d906 100644
> --- a/configs/imx8mp_evk_defconfig
> +++ b/configs/imx8mp_evk_defconfig
> @@ -4,7 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x40200000
> CONFIG_SPL_GPIO_SUPPORT=y  CONFIG_SPL_LIBCOMMON_SUPPORT=y
> CONFIG_SPL_LIBGENERIC_SUPPORT=y
> -CONFIG_SYS_MALLOC_F_LEN=0x8000
> +CONFIG_SYS_MALLOC_F_LEN=0x10000
>  CONFIG_ENV_SIZE=0x1000
>  CONFIG_ENV_OFFSET=0x400000
>  CONFIG_SYS_I2C_MXC_I2C1=y
> --
> 2.17.1

Tested-by: Andrey Zhizhikin <andrey.zhizhikin at leica-geosystems.com>



More information about the U-Boot mailing list