[U-Boot] [PATCH] Add bootstrap command

Wolfgang Denk wd at denx.de
Thu Jul 16 21:36:25 CEST 2009


Dear Matthias Fuchs,

In message <200907161547.17433.matthias.fuchs at esd.eu> you wrote:
>
> > Hi Dirk,
> > 
> > On Wednesday 15 July 2009 16:46:12 Mike Frysinger wrote:
> > > On Wednesday 15 July 2009 09:48:00 Dirk Eibach wrote:
> > > > This adds a generic command for programming I2C bootstrap eeproms.
> > > > Implementation for Canyonlands board is included.
> > >
> > > "bootstrap" is pretty generic.  can you pick something a little more
> > > descriptive/specific ?
> > 
> > I agree with Mike. Please change the command name and the files to something 
> > more specific. How about "4xxbootstrap"? Or "4xxstrap"? 
> To long for my taste. 
> 
> > Any other good ideas  
> > about this naming welcome. :)
> We called this command "sbe" on our PMC440 (440EPx) and upcoming 
> PMC405DE (405EP) board. I must admit that I forget its meaning.
> Probably something like 'setup bootstrap eeprom'. But it works

Indeed. "sbe" is not an acceptable name as nobody will know what it's
suppost to mean.

> a little mit different that the bootstrap command.

Actually the name "bootstrap" command itself is something that I
really dislike.

Please find a better description (avoiding "bootstrap"), and then
chose a descriptive name.

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
A well-written program is its own heaven; a poorly-written program is
its own hell.             -- Geoffrey James, "The Tao of Programming"


More information about the U-Boot mailing list