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

Wolfgang Denk
2013-04-11T05:41:22Z

Dear Fabio Estevam,

In message <CAOMZO5D5W34_9O1KPMDZEuS=rOHPBFq2rvwAuVUGw+0CywVF5Q@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?