[U-Boot] [PATCH] imx27lite: add support for imx27lite board from LogicPD
Jean-Christophe PLAGNIOL-VILLARD
plagnioj at jcrosoft.com
Mon Aug 3 14:17:48 CEST 2009
On 10:19 Mon 03 Aug , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
>
> In message <20090803053218.GA15665 at game.jcrosoft.org> you wrote:
> >
> ...
> > > +/* malloc() len */
> > > +#define CONFIG_SYS_MALLOC_LEN (0x10000 + 256 * 1024)
> > please do not mix hexa and decimal
I've as there is other think to fix otherwise I'll not have ask
>
> Please be reasonable. This is a perfectly valid and easy to read
> notation. I see no reason to change this - anything else would
> probably be much harder to read.
>
> > > +#define CONFIG_LOADADDR 0xa0800000 /* loadaddr env var */
> > > +#define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
> > why 2?
>
> Because they do different things.
I've take a look they do nearly the same think it will be better to avoid have
2 configs specialy when there are the same
btw I've found nothing about it in the README
Best Regards,
J.
More information about the U-Boot
mailing list