diff options
author | Marek Vasut <marex@denx.de> | 2012-09-03 22:40:16 -0400 |
---|---|---|
committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-09-05 19:42:11 -0400 |
commit | 204e706fa2ac1a62ff6423039945eb567c6c7efc (patch) | |
tree | e314ad7d3ae45cd1302e4fb57c34944f3367d270 /drivers/spi | |
parent | 010b481834b2b60f7d8543263a63e69396019f7b (diff) |
mxs/spi: Increment the transfer length only if transfer succeeded
The transfer function incremented (struct spi_message)->actual_length
unconditionally, even if the transfer failed. Rectify this by incrementing
this only if transfer succeeded.
Signed-off-by: Marek Vasut <marex@denx.de>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'drivers/spi')
-rw-r--r-- | drivers/spi/spi-mxs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/spi/spi-mxs.c b/drivers/spi/spi-mxs.c index bcba098e97c5..138c8523dbba 100644 --- a/drivers/spi/spi-mxs.c +++ b/drivers/spi/spi-mxs.c | |||
@@ -473,12 +473,12 @@ static int mxs_spi_transfer_one(struct spi_master *master, | |||
473 | &first, &last, 0); | 473 | &first, &last, 0); |
474 | } | 474 | } |
475 | 475 | ||
476 | m->actual_length += t->len; | ||
477 | if (status) { | 476 | if (status) { |
478 | stmp_reset_block(ssp->base); | 477 | stmp_reset_block(ssp->base); |
479 | break; | 478 | break; |
480 | } | 479 | } |
481 | 480 | ||
481 | m->actual_length += t->len; | ||
482 | first = last = 0; | 482 | first = last = 0; |
483 | } | 483 | } |
484 | 484 | ||