diff options
| author |   Roel Kluin <roel.kluin@gmail.com> | 2009-03-28 21:34:42 +0100 |
|---|---|---|
| committer |   Jean Delvare <khali@linux-fr.org> | 2009-03-28 21:34:42 +0100 |
| commit | 94d78e180c0323422854bc1718e657ac2d0cac1b (patch) | |
| tree | 7db1c35614e593f80ee17c10915dda76d2ee2836 /drivers/i2c/busses/i2c-pxa.c | |
| parent | i2c-algo-pcf: Style cleanups (diff) | |
| download | linux-dev-94d78e180c0323422854bc1718e657ac2d0cac1b.tar.xz linux-dev-94d78e180c0323422854bc1718e657ac2d0cac1b.zip | |
i2c-algo-pcf: Handle timeout correctly
With a postfix decrement these timeouts reach -1 rather than 0, but after the
loop it is tested whether they have become 0.
As pointed out by Jean Delvare, the msg_num should be tested before the timeout.
With the current order, you could exit with a timeout error while all the
messages were successfully transferred.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Eric Brower <ebrower@gmail.com>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions