diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-03-31 18:25:30 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-04-01 11:59:31 -0400 |
commit | 91ad1203535da95bb13072bdb59e1dc3ca76ec5d (patch) | |
tree | 06d8a81796bf3a2a79d7ce0f0357ca27c215a0b4 | |
parent | eb8972b4407f81b07ea6fc71fd91f9fc7a35a81e (diff) |
fbdev: newport: newport_*wait() return 0 on timeout
With a postfix decrement t reaches -1 on timeout which results in a
return of 0.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Cc: Krzysztof Helt <krzysztof.h1@poczta.fm>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | include/video/newport.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/video/newport.h b/include/video/newport.h index 1f5ebeaa818f..001b935e71c4 100644 --- a/include/video/newport.h +++ b/include/video/newport.h | |||
@@ -453,7 +453,7 @@ static __inline__ int newport_wait(struct newport_regs *regs) | |||
453 | { | 453 | { |
454 | int t = BUSY_TIMEOUT; | 454 | int t = BUSY_TIMEOUT; |
455 | 455 | ||
456 | while (t--) | 456 | while (--t) |
457 | if (!(regs->cset.status & NPORT_STAT_GBUSY)) | 457 | if (!(regs->cset.status & NPORT_STAT_GBUSY)) |
458 | break; | 458 | break; |
459 | return !t; | 459 | return !t; |
@@ -463,7 +463,7 @@ static __inline__ int newport_bfwait(struct newport_regs *regs) | |||
463 | { | 463 | { |
464 | int t = BUSY_TIMEOUT; | 464 | int t = BUSY_TIMEOUT; |
465 | 465 | ||
466 | while (t--) | 466 | while (--t) |
467 | if(!(regs->cset.status & NPORT_STAT_BBUSY)) | 467 | if(!(regs->cset.status & NPORT_STAT_BBUSY)) |
468 | break; | 468 | break; |
469 | return !t; | 469 | return !t; |