diff options
author | Devin Heitmueller <dheitmueller@kernellabs.com> | 2012-08-06 21:46:57 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-08-09 19:33:13 -0400 |
commit | 0a3dabb15ef9039188464647113398ed95400318 (patch) | |
tree | 42a47cc31b843ed8e1755587f867336ac07b37fb /drivers | |
parent | 7c287f1849cd904c210608a6a7bcec055bcb1152 (diff) |
[media] xc5000: properly report i2c write failures
The logic as written would *never* actually return an error condition,
since the loop would run until the counter hit zero but the check was
for a value less than zero.
Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/media/common/tuners/xc5000.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/common/tuners/xc5000.c b/drivers/media/common/tuners/xc5000.c index a3f775a2ccd7..7189e06fb35f 100644 --- a/drivers/media/common/tuners/xc5000.c +++ b/drivers/media/common/tuners/xc5000.c | |||
@@ -343,7 +343,7 @@ static int xc_write_reg(struct xc5000_priv *priv, u16 regAddr, u16 i2cData) | |||
343 | } | 343 | } |
344 | } | 344 | } |
345 | } | 345 | } |
346 | if (WatchDogTimer < 0) | 346 | if (WatchDogTimer <= 0) |
347 | result = XC_RESULT_I2C_WRITE_FAILURE; | 347 | result = XC_RESULT_I2C_WRITE_FAILURE; |
348 | 348 | ||
349 | return result; | 349 | return result; |