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

Tom
2009-09-08T19:09:43Z

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@ti.com> you wrote:
>>> From: Sandeep Paulraj <s-paulraj@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@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@denx.de>
>>

Then this change needs to be a separate patch.

Nak Tom