[U-Boot] [PATCH v2] imx: Use a convenient default value for SYS_MALLOC_F_LEN

Peng Fan peng.fan at nxp.com
Sun May 5 04:12:40 UTC 2019



> -----Original Message-----
> From: Fabio Estevam [mailto:festevam at gmail.com]
> Sent: 2019年5月4日 1:05
> To: sbabic at denx.de
> Cc: dl-uboot-imx <uboot-imx at nxp.com>; u-boot at lists.denx.de;
> pjtexier at koncepto.io; lukas.auer at aisec.fraunhofer.de;
> offougajoris at gmail.com; trini at konsulko.com; Fabio Estevam
> <festevam at gmail.com>
> Subject: [PATCH v2] imx: Use a convenient default value for
> SYS_MALLOC_F_LEN
> 
> Commit 3a7c45f6a772 ("simple-bus: add DM_FLAG_PRE_RELOC flag to
> simple-bus driver") causes some i.MX boards that were converted to DM, such
> as warp7, to fail to boot.
> 
> As explained by Lukas Auer:
> 
> "With the patch, U-Boot probes the drivers for devices under simple-bus
> device tree nodes in the pre-relocation device model. The default value of
> CONFIG_SYS_MALLOC_F_LEN (0x400) leaves U-Boot with not enough
> memory to do this, causing it to hang."
> 
> Fix this problem by providing a convenient default value for
> CONFIG_SYS_MALLOC_F_LEN.
> 
> Reported-by: Pierre-Jean Texier <pjtexier at koncepto.io>
> Suggested-by: Lukas Auer <lukas.auer at aisec.fraunhofer.de>
> Signed-off-by: Fabio Estevam <festevam at gmail.com>
> ---
> Changes since v1:
> - Move the default setting to the main Kconfig and make it depend on i.MX
> 
>  Kconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Kconfig b/Kconfig
> index 7a5491bd67..fd4ff42c17 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -138,6 +138,8 @@ config SYS_MALLOC_F_LEN
>  	depends on SYS_MALLOC_F
>  	default 0x1000 if AM33XX
>  	default 0x2800 if SANDBOX
> +	default 0x2000 if (ARCH_IMX8 || ARCH_IMX8M || ARCH_MX7 || \
> +			   ARCH_MX7ULP || ARCH_MX6 || ARCH_MX5)
>  	default 0x400
>  	help
>  	  Before relocation, memory is very limited on many platforms. Still,

Reviewed-by: Peng Fan <peng.fan at nxp.com>

> --
> 2.17.1



More information about the U-Boot mailing list