On 20/11/2015 21:43, Marek Vasut wrote: > Using 50 MiB malloc pool in SPL is nonsense. Since the caches are not > enabled in SPL, it takes 2 seconds to init the pool and has no obvious > benefit. Reduce the size to 1 MiB.
You're absolutely right ! Nice catch ! > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Stefano Babic <sba...@denx.de> > Cc: Tim Harvey <thar...@gateworks.com> > --- > include/configs/imx6_spl.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/include/configs/imx6_spl.h b/include/configs/imx6_spl.h > index 1744f2c..43ce7fe 100644 > --- a/include/configs/imx6_spl.h > +++ b/include/configs/imx6_spl.h > @@ -63,15 +63,15 @@ > > #if defined(CONFIG_MX6SX) || defined(CONFIG_MX6UL) || defined(CONFIG_MX6SL) > #define CONFIG_SPL_BSS_START_ADDR 0x88200000 > -#define CONFIG_SPL_BSS_MAX_SIZE 0x100000 /* 1 MB */ > +#define CONFIG_SPL_BSS_MAX_SIZE 0x100000 /* 1 MB */ > #define CONFIG_SYS_SPL_MALLOC_START 0x88300000 > -#define CONFIG_SYS_SPL_MALLOC_SIZE 0x3200000 /* 50 MB */ > +#define CONFIG_SYS_SPL_MALLOC_SIZE 0x100000 /* 1 MB */ > #define CONFIG_SYS_TEXT_BASE 0x87800000 > #else > #define CONFIG_SPL_BSS_START_ADDR 0x18200000 > #define CONFIG_SPL_BSS_MAX_SIZE 0x100000 /* 1 MB */ > #define CONFIG_SYS_SPL_MALLOC_START 0x18300000 > -#define CONFIG_SYS_SPL_MALLOC_SIZE 0x3200000 /* 50 MB */ > +#define CONFIG_SYS_SPL_MALLOC_SIZE 0x100000 /* 1 MB */ > #define CONFIG_SYS_TEXT_BASE 0x17800000 > #endif > #endif > Tested-by: Stefano Babic <sba...@denx.de> Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
