From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Date: Tue, 08 Sep 2009 14:09:43 -0500 Subject: [U-Boot] [PATCH] TI DaVinci: Remove references to SZ_xx In-Reply-To: <0554BEF07D437848AF01B9C9B5F0BC5D925680C6@dlee01.ent.ti.com> References: <1252434246-19517-1-git-send-email-s-paulraj@ti.com> <20090908185057.CDC1A832E8DE@gemini.denx.de> <0554BEF07D437848AF01B9C9B5F0BC5D925680C6@dlee01.ent.ti.com> Message-ID: <4AA6ABF7.5060709@windriver.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >>> >>> 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 >> 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 >> Then this change needs to be a separate patch. Nak Tom