On Thu, 23 Apr 2009 16:27:39 +0200, Roel Kluin wrote:
> Ok, here's for drivers/i2c/busses/i2c-pxa.c. Note that I found another,
> the last hunk.
> --------------------------->8-------------8<------------------------------
> With `while (timeout--)' timeout reaches -1 after the loop, so the tests
> below are off by one.
>
> Signed-off-by: Roel Kluin <[email protected]>
> ---
Ben, Wolfram, I'll let you handle this one as it's an arm driver.
> diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
> index c1405c8..acc7143 100644
> --- a/drivers/i2c/busses/i2c-pxa.c
> +++ b/drivers/i2c/busses/i2c-pxa.c
> @@ -265,10 +265,10 @@ static int i2c_pxa_wait_bus_not_busy(struct pxa_i2c
> *i2c)
> show_state(i2c);
> }
>
> - if (timeout <= 0)
> + if (timeout < 0)
> show_state(i2c);
>
> - return timeout <= 0 ? I2C_RETRY : 0;
> + return timeout < 0 ? I2C_RETRY : 0;
> }
>
> static int i2c_pxa_wait_master(struct pxa_i2c *i2c)
> @@ -612,7 +612,7 @@ static int i2c_pxa_pio_set_master(struct pxa_i2c *i2c)
> show_state(i2c);
> }
>
> - if (timeout <= 0) {
> + if (timeout < 0) {
> show_state(i2c);
> dev_err(&i2c->adap.dev,
> "i2c_pxa: timeout waiting for bus free\n");
>
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
