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 /Documentation/volatile-considered-harmful.txt | |
| parent | 0c168ceb9e1898a7f2895e80ce9915835b083bd3 (diff) | |
| download | lwn-94d78e180c0323422854bc1718e657ac2d0cac1b.tar.gz lwn-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 'Documentation/volatile-considered-harmful.txt')
0 files changed, 0 insertions, 0 deletions