[U-Boot] [PATCH] TI DaVinci: Remove references to SZ_xx

Tom Tom.Rix at windriver.com
Tue Sep 8 21:09:43 CEST 2009


Paulraj, Sandeep wrote:
> 
>> -----Original Message-----
>> From: Wolfgang Denk [mailto:wd at denx.de]
>> Sent: Tuesday, September 08, 2009 2:51 PM
>> To: Paulraj, Sandeep
>> Cc: u-boot at lists.denx.de
>> Subject: Re: [U-Boot] [PATCH] TI DaVinci: Remove references to SZ_xx
>>
>> Dear s-paulraj at ti.com,
>>
>> In message <1252434246-19517-1-git-send-email-s-paulraj at ti.com> you wrote:
>>> From: Sandeep Paulraj <s-paulraj at ti.com>
>>>
>>> This patch removes the asm/sizes.h header file from being
>>> included in the DaVinci SOC configs.
>>> References to SZ_xx have been replaced by appropriate
>>> bit shifted values.
>>>
>>> Signed-off-by: Sandeep Paulraj <s-paulraj at ti.com>
>> Note that this patch actually changes the configuration:
>>
>>> --- a/include/configs/davinci_dm355evm.h
>>> +++ b/include/configs/davinci_dm355evm.h
>> ...
>>>  /* U-Boot memory configuration */
>>> -#define CONFIG_STACKSIZE		SZ_256K		/* regular stack */
>>> -#define CONFIG_SYS_MALLOC_LEN		SZ_512K		/* malloc()
>> arena */
>>> +#define CONFIG_STACKSIZE		(256 << 10)	/* 256 KiB */
>>> +#define CONFIG_SYS_MALLOC_LEN		(1 << 20)	/* 1 MiB*/
>> as it increases the malloc() arena size form 512 to 1024 KiB.
>>
>> Assuming this was done intentionally:
> [Sandeep] Yes this was intentional because for NANDs with large block sizes(256 KiB), if this is not increased we will get "out of memory" errors.
>> Acked-by: Wolfgang Denk <wd at denx.de>
>>

Then this change needs to be a separate patch.

Nak Tom


More information about the U-Boot mailing list