diff options
author | Akinobu Mita <akinobu.mita@gmail.com> | 2017-03-17 14:17:27 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-03-17 17:53:56 -0400 |
commit | 8494801db1fc21867f587dae465236100bf228cc (patch) | |
tree | 6a3b1e32fc6af1bdf6d2e731b694703c0bc640eb /drivers/spi/spi-loopback-test.c | |
parent | c4e121aeb754f1ac5f2bcc791aa4177dcfb3b401 (diff) |
spi: loopback-test: don't skip comparing the first byte of rx_buf
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>
Diffstat (limited to 'drivers/spi/spi-loopback-test.c')
-rw-r--r-- | drivers/spi/spi-loopback-test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/spi/spi-loopback-test.c b/drivers/spi/spi-loopback-test.c index 99422f3a15e0..85c0c4e391d9 100644 --- a/drivers/spi/spi-loopback-test.c +++ b/drivers/spi/spi-loopback-test.c | |||
@@ -507,7 +507,7 @@ static int spi_test_check_loopback_result(struct spi_device *spi, | |||
507 | continue; | 507 | continue; |
508 | /* so depending on tx_buf we need to handle things */ | 508 | /* so depending on tx_buf we need to handle things */ |
509 | if (xfer->tx_buf) { | 509 | if (xfer->tx_buf) { |
510 | for (i = 1; i < xfer->len; i++) { | 510 | for (i = 0; i < xfer->len; i++) { |
511 | txb = ((u8 *)xfer->tx_buf)[i]; | 511 | txb = ((u8 *)xfer->tx_buf)[i]; |
512 | rxb = ((u8 *)xfer->rx_buf)[i]; | 512 | rxb = ((u8 *)xfer->rx_buf)[i]; |
513 | if (txb != rxb) | 513 | if (txb != rxb) |