[U-Boot] [PATCH 09/17] ARM: SPL: Only call mem_malloc_init if configured

[Stefano Babic]

On 15/08/2012 23:30, Tom Rini wrote:
> We can only attempt to setup a malloc pool if
> CONFIG_SYS_SPL_MALLOC_START is defined, and not all boards require it.
> Make the call depend on the define.
> 
> Signed-off-by: Tom Rini <trini at ti.com>
> ---
>  arch/arm/lib/spl.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/lib/spl.c b/arch/arm/lib/spl.c
> index 4d33f99..71a467e 100644
> --- a/arch/arm/lib/spl.c
> +++ b/arch/arm/lib/spl.c
> @@ -149,8 +149,10 @@ void board_init_r(gd_t *id, ulong dummy)
>  	u32 boot_device;
>  	debug(">>spl:board_init_r()\n");
>  
> +#ifdef CONFIG_SYS_SPL_MALLOC_START
>  	mem_malloc_init(CONFIG_SYS_SPL_MALLOC_START,
>  			CONFIG_SYS_SPL_MALLOC_SIZE);
> +#endif
>  
>  #ifdef CONFIG_SPL_BOARD_INIT
>  	spl_board_init();
> 

Agree, mem_malloc_init should be not always called. However, how are the
dependencies with other SPL configuration ? I think (but I can be wrong)
that we need it if CONFIG_SPL_LIBDISK_SUPPORT or CONFIG_SPL_FAT_SUPPORT
are set. Should we add them to your #ifdef case ?

Regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================