diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2011-02-23 12:38:19 +0100 |
---|---|---|
committer | Ben Dooks <ben-linux@fluff.org> | 2011-03-21 10:23:55 +0100 |
commit | 93c92cfdec8c3f8764894b37606c174f68fd9338 (patch) | |
tree | 131888459d95f1139685e8c5b3f8ac30c7215d47 | |
parent | i2c-pxa2xx: add support for shared IRQ handler (diff) | |
download | linux-93c92cfdec8c3f8764894b37606c174f68fd9338.tar.xz linux-93c92cfdec8c3f8764894b37606c174f68fd9338.zip |
i2c-pxa2xx: check timeout correctly
timeout here maybe 0 if the event occured and a task with a higher
priority stole the cpu and we were sleeping longer than the timeout
value we specified.
In case of a real timeout I changed the error code to I2C_RETRY so we
retry the transfer.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Dirk Brandewie <dirk.brandewie@gmail.com>
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
-rw-r--r-- | drivers/i2c/busses/i2c-pxa.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c index fa8dd2c634d9..86128363a57f 100644 --- a/drivers/i2c/busses/i2c-pxa.c +++ b/drivers/i2c/busses/i2c-pxa.c @@ -770,8 +770,10 @@ static int i2c_pxa_do_xfer(struct pxa_i2c *i2c, struct i2c_msg *msg, int num) */ ret = i2c->msg_idx; - if (timeout == 0) + if (!timeout && i2c->msg_num) { i2c_pxa_scream_blue_murder(i2c, "timeout"); + ret = I2C_RETRY; + } out: return ret; |