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

Mike Frysinger vapier at gentoo.org
Thu Aug 26 04:04:11 CEST 2010


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20100825/0c095ac0/attachment.pgp 


More information about the U-Boot mailing list