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

Paulraj, Sandeep
2009-09-08T18:56:41Z

> -----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>
>
> Best regards,
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> Work 8 hours, sleep 8 hours; but not the same 8 hours.