diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-08-10 04:25:03 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-08-12 12:21:06 -0400 |
commit | ffd491fd6f4129c201926edbbd63e2088d82d2fa (patch) | |
tree | bbf57b6bac26f1063dd51c69d145bbfc4bac866e /drivers/media/common/tuners | |
parent | e0e52d4e9f5bce7ea887027c127473eb654a5a04 (diff) |
[media] qt1010: signedness bug in qt1010_init_meas1()
qt1010_init_meas2() returns zero on success and negative error codes on
failure so the return type should be int instead of u8.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/common/tuners')
-rw-r--r-- | drivers/media/common/tuners/qt1010.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/common/tuners/qt1010.c b/drivers/media/common/tuners/qt1010.c index 2d79b1f5d5eb..bdc39e11030e 100644 --- a/drivers/media/common/tuners/qt1010.c +++ b/drivers/media/common/tuners/qt1010.c | |||
@@ -288,7 +288,7 @@ static int qt1010_init_meas1(struct qt1010_priv *priv, | |||
288 | return qt1010_writereg(priv, 0x1e, 0x00); | 288 | return qt1010_writereg(priv, 0x1e, 0x00); |
289 | } | 289 | } |
290 | 290 | ||
291 | static u8 qt1010_init_meas2(struct qt1010_priv *priv, | 291 | static int qt1010_init_meas2(struct qt1010_priv *priv, |
292 | u8 reg_init_val, u8 *retval) | 292 | u8 reg_init_val, u8 *retval) |
293 | { | 293 | { |
294 | u8 i, val; | 294 | u8 i, val; |