[PATCH] spi_bitbang: txrx_bufs zero status fix

[PATCH] spi_bitbang: txrx_bufs zero status fix

[Jan Nikitenko]

[-- Attachment #1: Type: text/plain, Size: 417 bytes --]

SPI controller drivers return number of bytes actually transfered from
bitbang->txrx_bufs() method. Returned zero number of bytes transfered
when non-zero number of bytes to transfer was requested should be
recognized as error.
The patch fixes that.

Signed-off-by: Jan Nikitenko <jan.nikitenko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

---

 spi_bitbang.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

[-- Attachment #2: spi_bitbang-txrx_bufs-zero-status-fix.patch --]
[-- Type: text/plain, Size: 497 bytes --]

diff --git a/drivers/spi/spi_bitbang.c b/drivers/spi/spi_bitbang.c
index f7f8580..83c911b 100644
--- a/drivers/spi/spi_bitbang.c
+++ b/drivers/spi/spi_bitbang.c
@@ -345,7 +345,9 @@ static void bitbang_work(struct work_struct *work)
 				status = bitbang->txrx_bufs(spi, t);
 			}
 			if (status != t->len) {
-				if (status > 0)
+				/* zero status, meaning zero number of bytes
+				 * transfered, is also invalid in this case */
+				if (status >= 0)
 					status = -EMSGSIZE;
 				break;
 			}

[-- Attachment #3: Type: text/plain, Size: 228 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

[-- Attachment #4: Type: text/plain, Size: 210 bytes --]

_______________________________________________
spi-devel-general mailing list
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/spi-devel-general