> -----Original Message-----
> From: Fabio Estevam [mailto:feste...@gmail.com]
> Sent: 2019年5月4日 1:05
> To: sba...@denx.de
> Cc: dl-uboot-imx <uboot-...@nxp.com>; u-boot@lists.denx.de;
> pjtex...@koncepto.io; lukas.a...@aisec.fraunhofer.de;
> offougajo...@gmail.com; tr...@konsulko.com; Fabio Estevam
> <feste...@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 <pjtex...@koncepto.io>
> Suggested-by: Lukas Auer <lukas.a...@aisec.fraunhofer.de>
> Signed-off-by: Fabio Estevam <feste...@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....@nxp.com>

> --
> 2.17.1

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to