aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoel Kluin <roel.kluin@gmail.com>2009-04-06 22:00:54 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2009-04-07 11:31:07 -0400
commitc8fc657e6a114fadf78fdf8103e289a169c91c5d (patch)
tree1520389a952ea5f9d624f06d054b7d68f5802b2f
parent93b61bddc13d9acf1fe341b39d826e80f3182d1e (diff)
spi: limit reaches -1, tested 0
With a postfix decrement limit will reach -1 rather than 0, so the warning will not be issued. Also, add a cpu_relax() into the busy-wait loop. Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Cc: Mariusz Ceier <mceier@gmail.com> Acked-by: David Brownell <dbrownell@users.sourceforge.net> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--drivers/spi/spi_imx.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/spi/spi_imx.c b/drivers/spi/spi_imx.c
index 0480d8bb19d3..014af69fc1bb 100644
--- a/drivers/spi/spi_imx.c
+++ b/drivers/spi/spi_imx.c
@@ -779,7 +779,8 @@ static irqreturn_t interrupt_transfer(struct driver_data *drv_data)
779 779
780 /* Read trailing bytes */ 780 /* Read trailing bytes */
781 limit = loops_per_jiffy << 1; 781 limit = loops_per_jiffy << 1;
782 while ((read(drv_data) == 0) && limit--); 782 while ((read(drv_data) == 0) && --limit)
783 cpu_relax();
783 784
784 if (limit == 0) 785 if (limit == 0)
785 dev_err(&drv_data->pdev->dev, 786 dev_err(&drv_data->pdev->dev,