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 | |
| 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>
| -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); |
