[U-Boot] [PATCH] imx: Add titanium board support (i.MX6 based)

Wolfgang Denk wd at denx.de
Thu Apr 11 07:41:22 CEST 2013


Dear Fabio Estevam,

In message <CAOMZO5D5W34_9O1KPMDZEuS=rOHPBFq2rvwAuVUGw+0CywVF5Q at mail.gmail.com> you wrote:
>
> > +/* Size of malloc() pool */
> > +#define CONFIG_SYS_MALLOC_LEN          (2 * 1024 * 1024)
> 
> What about SZ_2M ?
...
> > +#define PHYS_SDRAM_SIZE                        (1u * 512 * 1024 * 1024)
> 
> What about using SZ_512M instead? Same for other (x << y) sizes passed
> in other places.

No.  The use of the SZ_* macros is strongly discouraged.  Please see
the list archives for previous discussion and explanations why.  One
day, when I find enough free time, I will prepare a patch to get rid
of all of these.  [But I don't see when this will actually happen,
yet.]

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
If ignorance is bliss, why aren't there more happy people?


More information about the U-Boot mailing list