Messages in this thread Patch in this message | | | From | David Brownell <> | Subject | [patch 2.6.28-rc9] spi: spi_write_then_read() regression fix | Date | Sat, 20 Dec 2008 23:32:35 -0800 |
| |
From: David Brownell <dbrownell@users.sourceforge.net>
The recent "simplify spi_write_then_read()" patch included a small regression from the 2.6.27 behavior with its performance win.
All SPI transfers are full duplex, and are packaged as half duplex by either discarding the data that's read ("write only"), or else by writing zeroes ("read only"). That patch wasn't ensuring that zeroes were getting written out during the "half duplex read" part of the transaction; instead, old RX bits were getting sent.
The fix is trivial: zero the buffer before using it.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net> Cc: Vernon Sauder <vernoninhand@gmail.com>
--- a/drivers/spi/spi.c +++ b/drivers/spi/spi.c @@ -677,11 +677,13 @@ int spi_write_then_read(struct spi_device *spi, /* ... unless someone else is using the pre-allocated buffer */ if (!mutex_trylock(&lock)) { - local_buf = kmalloc(SPI_BUFSIZ, GFP_KERNEL); + local_buf = kzalloc(SPI_BUFSIZ, GFP_KERNEL); if (!local_buf) return -ENOMEM; - } else + } else { local_buf = buf; + memset(local_buf, 0, x.len); + } memcpy(local_buf, txbuf, n_tx); x.tx_buf = local_buf;
| |