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 <roel.kluin@xxxxxxxxx> > --- 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html