diff options
| author | Roel Kluin <roel.kluin@gmail.com> | 2009-02-04 23:23:22 +0300 |
|---|---|---|
| committer | Samuel Ortiz <samuel@sortiz.org> | 2009-02-17 11:00:13 +0300 |
| commit | 29c6a2e6f88225ae2673aabd2de0fa2126653231 (patch) | |
| tree | 1f068b2de59b5c036732b7c0262f4bea6f7b4327 | |
| parent | a39a021fd73ce06aad8d1081ac711a36930e6cb8 (diff) | |
| download | linux-29c6a2e6f88225ae2673aabd2de0fa2126653231.tar.xz | |
mfd: wm8350 tries reaches -1
With a postfix decrement tries will reach -1 rather than 0,
so the warning will not be issued even upon timeout.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Samuel Ortiz <sameo@openedhand.com>
| -rw-r--r-- | drivers/mfd/wm8350-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mfd/wm8350-core.c b/drivers/mfd/wm8350-core.c index e5e82c7cc523..ea3801833176 100644 --- a/drivers/mfd/wm8350-core.c +++ b/drivers/mfd/wm8350-core.c @@ -1111,7 +1111,7 @@ int wm8350_read_auxadc(struct wm8350 *wm8350, int channel, int scale, int vref) do { schedule_timeout_interruptible(1); reg = wm8350_reg_read(wm8350, WM8350_DIGITISER_CONTROL_1); - } while (tries-- && (reg & WM8350_AUXADC_POLL)); + } while (--tries && (reg & WM8350_AUXADC_POLL)); if (!tries) dev_err(wm8350->dev, "adc chn %d read timeout\n", channel); |