All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jerry Van Baren <gerald.vanbaren@ge.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC][PATCH v2] bootm: Add sub commands
Date: Tue, 23 Sep 2008 07:46:22 -0400	[thread overview]
Message-ID: <48D8D70E.1010700@ge.com> (raw)
In-Reply-To: <20080923113444.EA82E24844@gemini.denx.de>

Wolfgang Denk wrote:
> Dear Kumar Gala,
> 
> In message <1222151236-13920-1-git-send-email-galak@kernel.crashing.org> you wrote:
>> This version:
>> * cleans ups issues pointed out by Jerry
>> * adds a state machine to the command processing
>> * adds bd_t and cmdline process on linux for ppc
> 
> Thanks a lot.
> 
> I'm still missing the possibility to use longer sub-command names, and
> code to parse these in a sane way.

[snip]

> Anyway, as mentioned before, I do not like to be restrricted  to  one
> letter  subcommand  names - we would rather sooner than later come up
> with names that nobody can remember.

But They All Do It That Way (typed with a whine).

> Maybe we can extend find_cmd() (in  "common/command.c")  to  take  an
> alternative  command  table  (for  the  sub-commands)  and  reuse the
> existing code?
 >
> [And then probably do this also for some other commands that take
> subcommands..]

BINGO, now I understand your objection.  :-)

> Best regards,
> 
> Wolfgang Denk

Best regards,
gvb

      reply	other threads:[~2008-09-23 11:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-23  6:27 [U-Boot] [RFC][PATCH v2] bootm: Add sub commands Kumar Gala
2008-09-23 11:34 ` Wolfgang Denk
2008-09-23 11:46   ` Jerry Van Baren [this message]

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=48D8D70E.1010700@ge.com \
    --to=gerald.vanbaren@ge.com \
    --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.