[U-Boot] [PATCH v3 7/9] arc: add AXS101 board support

Wolfgang Denk wd at denx.de
Mon Feb 3 23:44:45 CET 2014


Dear Alexey,

In message <1391457382.2357.46.camel at abrodkin-8560l.internal.synopsys.com> you wrote:
> 
> > > +#define CONFIG_SYS_MAXARGS	16
> > 
> > You define this pretty much inconsistentlyu across your boards.  Is
> > this intentionally?
>
> Not clear what do you mean.
> All 3 boards have CONFIG_SYS_MAXARGS = 16.

Indeed. Sorry, it seems I mixed this up with a patch for a broadcom board
which had 64.

> > Uh, oh.  Environment in EEPROM is about the worst thing you can do.
> > You should reconsider this design decision.
>
> Well I may agree on this point.
> But due to the fact that Linux driver is still not ready for our custom
> NAND controller we don't rely on NAND for whatever storage. And then the
> only persistent storage that remains is I2C EEPROM.

I see.  Except that I would hesitate to use "I2C EEPROM" and
"persistent" in the same sentence ;-)

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
"More software projects have gone awry for lack of calendar time than
for all other causes combined."
                         - Fred Brooks, Jr., _The Mythical Man Month_


More information about the U-Boot mailing list