diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-06-17 19:26:02 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-06-18 16:03:41 -0400 |
commit | b4bd2ababd20b6ecdd49cf96e39c875fbedd53af (patch) | |
tree | cd44bcaca19a16e333975f09b7a7711311fcb3be /drivers/spi/spi_bfin5xx.c | |
parent | 349888ee7b2c1ffb44c806adf6f4289ca4a6fd42 (diff) |
spi_bfin5xx: limit reaches -1
bfin_spi_flush() returns limit, which reaches -1 upon timeout. but in
function bfin_spi_pump_transfers() it is compared with 0.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Acked-by: David Brownell <dbrownell@users.sourceforge.net>
Cc: Bryan Wu <cooloney@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/spi/spi_bfin5xx.c')
-rw-r--r-- | drivers/spi/spi_bfin5xx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c index 011c5bddba6a..2e5fd0977334 100644 --- a/drivers/spi/spi_bfin5xx.c +++ b/drivers/spi/spi_bfin5xx.c | |||
@@ -169,7 +169,7 @@ static int bfin_spi_flush(struct driver_data *drv_data) | |||
169 | unsigned long limit = loops_per_jiffy << 1; | 169 | unsigned long limit = loops_per_jiffy << 1; |
170 | 170 | ||
171 | /* wait for stop and clear stat */ | 171 | /* wait for stop and clear stat */ |
172 | while (!(read_STAT(drv_data) & BIT_STAT_SPIF) && limit--) | 172 | while (!(read_STAT(drv_data) & BIT_STAT_SPIF) && --limit) |
173 | cpu_relax(); | 173 | cpu_relax(); |
174 | 174 | ||
175 | write_STAT(drv_data, BIT_STAT_CLR); | 175 | write_STAT(drv_data, BIT_STAT_CLR); |