From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerry Van Baren Date: Tue, 23 Sep 2008 07:46:22 -0400 Subject: [U-Boot] [RFC][PATCH v2] bootm: Add sub commands In-Reply-To: <20080923113444.EA82E24844@gemini.denx.de> References: <1222151236-13920-1-git-send-email-galak@kernel.crashing.org> <20080923113444.EA82E24844@gemini.denx.de> Message-ID: <48D8D70E.1010700@ge.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.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