spi: loopback-test: don't skip comparing the first byte of rx_buf
authorAkinobu Mita <akinobu.mita@gmail.com>
Fri, 17 Mar 2017 18:17:27 +0000 (03:17 +0900)
committerMark Brown <broonie@kernel.org>
Fri, 17 Mar 2017 21:53:56 +0000 (21:53 +0000)
When the loopback parameter is set, rx_buf are compared with tx_buf
after the spi_message is executed.  But the first byte of buffer is
not checked.

Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-loopback-test.c

index 99422f3a15e07ddaa4e40b85a74ee1d781324b50..85c0c4e391d9726726a702fffde21b21727a6c2d 100644 (file)
@@ -507,7 +507,7 @@ static int spi_test_check_loopback_result(struct spi_device *spi,
                        continue;
                /* so depending on tx_buf we need to handle things */
                if (xfer->tx_buf) {
-                       for (i = 1; i < xfer->len; i++) {
+                       for (i = 0; i < xfer->len; i++) {
                                txb = ((u8 *)xfer->tx_buf)[i];
                                rxb = ((u8 *)xfer->rx_buf)[i];
                                if (txb != rxb)