Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-17 Thread Wolfgang Denk
Dear Dave Mitchell, In message <4a5fc1fd.1090...@gmail.com> you wrote: > > > Yes, I know. But then, bootstrapping [1] is really a name with a > > _very_ fuzzy meaning. It can be anything. The fact thay it was chosen > > in the UM documentation does't make it better - in U-Boot context I > >

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Stefan Roese
On Thursday 16 July 2009 22:08:27 Matthias Fuchs wrote: > > OK. But if your "sbe" command is "better" than the current bootstrap one, > > then let's see if it makes sense to use your command as the common one. > > Dirk's approach is very generic. Putting nothing but the EEPROM data and a > descript

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Stefan Roese
On Friday 17 July 2009 02:12:45 Dave Mitchell wrote: > >> Keep in mind the processor UM documentation section that covers these > >> boot methods is labeled "Bootstrap Operations". It refers the registers > >> that the EEPROM values populate as "bootstrap registers", etc. > > > > Yes, I know. But t

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Dave Mitchell
On 7/16/2009 5:11 PM, Wolfgang Denk wrote: > Dear Dave Mitchell, > > In message<4a5f8a1d.4030...@gmail.com> you wrote: >>> Please find a better description (avoiding "bootstrap"), and then >>> chose a descriptive name. >> Keep in mind the processor UM documentation section that covers these >> boo

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Wolfgang Denk
Dear Matthias Fuchs, In message <200907162224.23377.matthias.fu...@esd-electronics.com> you wrote: > > > Please find a better description (avoiding "bootstrap"), and then > > chose a descriptive name. > ccc - config chip|cpu configuration > scc - setup chip|cpu configuration > wcc - write chip|cpu

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Wolfgang Denk
Dear Dave Mitchell, In message <4a5f8a1d.4030...@gmail.com> you wrote: > > > Please find a better description (avoiding "bootstrap"), and then > > chose a descriptive name. > > Keep in mind the processor UM documentation section that covers these > boot methods is labeled "Bootstrap Operations".

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Wolfgang Denk
Dear Matthias Fuchs, In message <200907162208.28398.matthias.fu...@esd-electronics.com> you wrote: > > > But unfortunately not specific enough for a "common" command. > I think we are talking about a generic 4xx command to write data into > the EEPROM that is read by the bootstrap controller. But

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Matthias Fuchs
> Dear Matthias Fuchs, > > In message <200907161547.17433.matthias.fu...@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 I

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Dave Mitchell
On 7/16/2009 2:36 PM, Wolfgang Denk wrote: > Dear Matthias Fuchs, > > In message<200907161547.17433.matthias.fu...@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 c

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Matthias Fuchs
> On Thursday 16 July 2009 17:00:00 Matthias Fuchs wrote: > > > > > 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

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Wolfgang Denk
Dear Matthias Fuchs, In message <200907161547.17433.matthias.fu...@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 eepro

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Stefan Roese
On Thursday 16 July 2009 17:00:00 Matthias Fuchs wrote: > > > > 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

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Stefan Roese
On Thursday 16 July 2009 16:16:36 Felix Radensky wrote: > >> But it works > >> a little mit different that the bootstrap command. > > > > It would be great if you could merge/consolidate such 4xx custom commands > > into this common one. Does you command support more features? What's the > > main d

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Matthias Fuchs
On Thursday 16 July 2009 16:06, Stefan Roese wrote: > On Thursday 16 July 2009 15:47:17 Matthias Fuchs wrote: > > > > "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 >

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Felix Radensky
Hi, Stefan Roese wrote: > On Thursday 16 July 2009 15:47:17 Matthias Fuchs wrote: "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 "4x

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Stefan Roese
On Thursday 16 July 2009 15:47:17 Matthias Fuchs wrote: > > > "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 "4xxst

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Matthias Fuchs
Please see my comments below. On Wednesday 15 July 2009 15:48, Dirk Eibach wrote: ... > diff --git a/common/cmd_bootstrap.c b/common/cmd_bootstrap.c > new file mode 100644 > index 000..6fceb23 > --- /dev/null > +++ b/common/cmd_bootstrap.c > @@ -0,0 +1,75 @@ > +/* > + * (C) Copyright 2009 >

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Matthias Fuchs
On Thursday 16 July 2009 15:21, Stefan Roese 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

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-16 Thread Stefan Roese
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 so

Re: [U-Boot] [PATCH] Add bootstrap command

2009-07-15 Thread Mike Frysinger
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 ? -mike signature.asc Descript

[U-Boot] [PATCH] Add bootstrap command

2009-07-15 Thread Dirk Eibach
This adds a generic command for programming I2C bootstrap eeproms. Implementation for Canyonlands board is included. Signed-off-by: Dirk Eibach --- board/amcc/canyonlands/Makefile|5 +- board/amcc/canyonlands/bootstrap.c | 177 ++-- common/Makefile