All of lore.kernel.org
 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 15:21:29 +0200	[thread overview]
Message-ID: <200907161521.29197.sr@denx.de> (raw)
In-Reply-To: <200907151046.13717.vapier@gentoo.org>

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"? Any other good ideas 
about this naming welcome. :)

Other than this, patch looks good. 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 13:21 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 [this message]
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
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=200907161521.29197.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.