[U-Boot] [PATCH] command "sspi": add write-only flag '.w' (discard read data)

2011-07-16 Thread Andreas Pretzsch
The sspi command writes the given data out on SPI and prints the data it reads to the console. For write-only slaves (i.e. a SPI-connected latch used as output expander), this is pointless and clutters the console. When called as "sspi.w", this output is omitted. The flag is optional and backwards

Re: [U-Boot] [PATCH] command "sspi": add write-only flag '.w' (discard read data)

2011-07-18 Thread Mike Frysinger
On Sat, Jul 16, 2011 at 13:32, Andreas Pretzsch wrote: > The sspi command writes the given data out on SPI and prints the data it > reads to the console. For write-only slaves (i.e. a SPI-connected latch > used as output expander), this is pointless and clutters the console. > When called as "sspi.

Re: [U-Boot] [PATCH] command "sspi": add write-only flag '.w' (discard read data)

2011-07-20 Thread Andreas Pretzsch
Am Montag, den 18.07.2011, 13:49 -0400 schrieb Mike Frysinger: > On Sat, Jul 16, 2011 at 13:32, Andreas Pretzsch wrote: > > The sspi command writes the given data out on SPI and prints the data it > > reads to the console. For write-only slaves (i.e. a SPI-connected latch > > used as output expande

Re: [U-Boot] [PATCH] command "sspi": add write-only flag '.w' (discard read data)

2011-07-20 Thread Mike Frysinger
On Wednesday, July 20, 2011 13:04:47 Andreas Pretzsch wrote: > I also thought about passing NULL for the read buffer, but a quick > browse through the code showed that most, but not all SPI drivers are > prepared for that. And as there is already a static rx buffer in the spi > command code, I pref

Re: [U-Boot] [PATCH] command "sspi": add write-only flag '.w' (discard read data)

2011-07-21 Thread Detlev Zundel
Hi Mike, [...] >> Following the common U-Boot way to do this, I'd suggest >> sspi [:][.] [din_env_var] >> and either never print the read result to the console (my favorite) or >> only if no env variable to store is passed. To be defined, comments >> welcome. > > is this the common u-boot

Re: [U-Boot] [PATCH] command "sspi": add write-only flag '.w' (discard read data)

2011-07-22 Thread Andreas Pretzsch
Am Donnerstag, den 21.07.2011, 11:01 +0200 schrieb Detlev Zundel: > Hi Mike, > > [...] > > >> Following the common U-Boot way to do this, I'd suggest > >>sspi [:][.] [din_env_var] > >> and either never print the read result to the console (my favorite) or > >> only if no env variable to sto

Re: [U-Boot] [PATCH] command "sspi": add write-only flag '.w' (discard read data)

2011-07-22 Thread Mike Frysinger
On Fri, Jul 22, 2011 at 14:02, Andreas Pretzsch wrote: > So how to proceed with the sspi command ? i think any proposals pending for the sspi command could be handled with a better hush implementation, so i'd prefer to go that route as the "bigger picture" > Also, given any rework of the sspi com