public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Add bootstrap command
Date: Thu, 16 Jul 2009 20:25:37 +0200	[thread overview]
Message-ID: <200907162025.37727.sr@denx.de> (raw)
In-Reply-To: <200907161700.00555.matthias.fuchs@esd.eu>

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 'setup bootstrap eeprom'.
> >
> > If you already forgot what it's supposed to mean, then it definitely is
> > not a good name. Better a bit longer and more descriptive.
>
> I like 'sbe' - it short and I even gave it a meaning :-)

But unfortunately not specific enough for a "common" command.

> > > 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.
>
> I was a little inspired by the sequoia code that also comes with a
> bootstrap command. So please don't ask me to merge it into common code. But
> your finger on your own nose.

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.

> > Does you command support more features? What's the main
> > difference?
>
> Much simpler. Just call sbe with a descriptive argument like a CPU
> frequency or something like '667-66' on a 440EPx target with 66Mhz PCI
> clock or 'sr-test-only' for something you will remove later :-). This has
> two advantages over just using
> numbers: You can remove configurations without making the following configs
> in the table moving to the front and its a little more secure meaning you
> have to type a couple of valid character to  reconfigure the clocking. Just
> using "bootstrap 5" is error-prone.

Ack.

> Well, I like my syntax and behavior, but I do not want to totally dismiss
> Dirk's idea as long as I can keep my sbe command :-)

Seems that "your" command is not so bad. ;) I'll take a look at it tomorrow. 
Perhaps we can use some of your ideas in such a new common (PPC4xx) 
implementation. :)

Thanks.

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
=====================================================================

  reply	other threads:[~2009-07-16 18:25 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-15 13:48 [U-Boot] [PATCH] Add bootstrap command Dirk Eibach
2009-07-15 14:46 ` Mike Frysinger
2009-07-16 13:21   ` Stefan Roese
2009-07-16 13:47     ` Matthias Fuchs
2009-07-16 14:06       ` Stefan Roese
2009-07-16 14:16         ` Felix Radensky
2009-07-16 18:04           ` Stefan Roese
2009-07-16 15:00         ` Matthias Fuchs
2009-07-16 18:25           ` Stefan Roese [this message]
2009-07-16 20:08             ` Matthias Fuchs
2009-07-16 22:05               ` Wolfgang Denk
2009-07-17  4:49               ` Stefan Roese
2009-07-16 19:36       ` Wolfgang Denk
2009-07-16 20:14         ` Dave Mitchell
2009-07-16 22:11           ` Wolfgang Denk
2009-07-17  0:12             ` Dave Mitchell
2009-07-17  4:38               ` Stefan Roese
2009-07-17 21:41               ` Wolfgang Denk
2009-07-16 20:24         ` Matthias Fuchs
2009-07-16 22:18           ` Wolfgang Denk
2009-07-16 13:55 ` Matthias Fuchs

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200907162025.37727.sr@denx.de \
    --to=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox