[U-Boot] [PATCH] mpc512x: Add esd gmbh mecp5123 board support
Stefan Roese
sr at denx.de
Tue Jun 9 09:24:13 CEST 2009
Hi Wolfgang,
On Monday 08 June 2009 20:14:33 Wolfgang Denk wrote:
> Dear Stefan,
>
> In message <200906081937.02354.sr at denx.de> you wrote:
> > > > + out_be32(&im->mddrc.lut_table4_alternate_upper,
> > > > CONFIG_SYS_MDDRCGRP_LUT4_AU);
> > > > + out_be32(&im->mddrc.lut_table4_alternate_lower,
> > > > CONFIG_SYS_MDDRCGRP_LUT4_AL);
> > >
> > > Lines too long.
> >
> > OK. Some is true for aria and mcp5121ads btw. But I'll change it in the
> > next patch version.
>
> If you argument that the too long lines are easier to read then the
> wrapped ones, I will not fight about this :-)
Yes, sometimes I really prefer these "too long lines" because it's just better
readable. I'll keep this code part as is then. :)
> > > How long does it to boot with such an environment in EEPROM?
> >
> > Quite long. About 10 seconds to the U-Boot prompt.
>
> He. And then try "setenv baudrate 115200 ; saveenv" and measure again.
About 6 seconds.
> Does your customer really want to release such a product to his
> customers? Other boards boot in a fraction of that time.
I know. I asked again. Please accept the EEPROM decision for now.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
More information about the U-Boot
mailing list