[U-Boot] [RFC][PATCH v2] bootm: Add sub commands

Jerry Van Baren gerald.vanbaren at ge.com
Tue Sep 23 13:46:22 CEST 2008


Wolfgang Denk wrote:
> Dear Kumar Gala,
> 
> In message <1222151236-13920-1-git-send-email-galak at 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


More information about the U-Boot mailing list