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

Pierre-Jean Texier pjtexier at koncepto.io
Fri May 3 17:19:15 UTC 2019


Hi Fabio,

Le 03/05/2019 à 19:05, Fabio Estevam a écrit :
> 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,


Tested-by: Pierre-Jean Texier <pjtexier at koncepto.io>


Thanks

Pierre-Jean



More information about the U-Boot mailing list