Re: [U-Boot] [PATCH] SPI: cmd_spi.c: add option to specify bus and mode

On Tuesday, August 24, 2010 06:53:49 Reinhard Meyer wrote:

...

this usage string no longer makes sense. how about: "[<bus>:]<cs>[.<mode>] <bit_len> <dout> - Send <bit_len> bits from <dout> out the SPI\n"

The problem is, that without blowing up the parser even more, only 2 forms of the command are valid:

the classic form

<cs> <bit_len> <dout>

or

<bus>:<cs>[.<mode>] <bit_len> <dout>

how so ? the bus:cs.mode field is always going to be one argv, and your patch has it so that argv[1] is always parsed against this form. i dont see where this limitation is that you're seeing ... -mike