diff options
author | Hans Verkuil <hans.verkuil@cisco.com> | 2012-09-13 11:54:36 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-03-05 12:25:58 -0500 |
commit | b251f95767e1b13b9b32b207b53f5be1c491899e (patch) | |
tree | 775392f7e2d90ee3112d8909107caa20dea99fc9 /drivers/media/usb/cx231xx/cx231xx-video.c | |
parent | 06c46003f7077cce7f87a75d327635e45c6d2b64 (diff) |
[media] cx231xx: fix tuner compliance issues
The g_tuner call wasn't passed on to the subdevices, g_frequency didn't
check for invalid tuners and a low-level function that was expected to
return 0 or a negative error returned a positive number instead, causing
s_frequency to return bogus errors.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/usb/cx231xx/cx231xx-video.c')
-rw-r--r-- | drivers/media/usb/cx231xx/cx231xx-video.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/media/usb/cx231xx/cx231xx-video.c b/drivers/media/usb/cx231xx/cx231xx-video.c index a31f7daef3dd..2b7b0944b7b4 100644 --- a/drivers/media/usb/cx231xx/cx231xx-video.c +++ b/drivers/media/usb/cx231xx/cx231xx-video.c | |||
@@ -1322,6 +1322,7 @@ static int vidioc_g_tuner(struct file *file, void *priv, struct v4l2_tuner *t) | |||
1322 | t->capability = V4L2_TUNER_CAP_NORM; | 1322 | t->capability = V4L2_TUNER_CAP_NORM; |
1323 | t->rangehigh = 0xffffffffUL; | 1323 | t->rangehigh = 0xffffffffUL; |
1324 | t->signal = 0xffff; /* LOCKED */ | 1324 | t->signal = 0xffff; /* LOCKED */ |
1325 | call_all(dev, tuner, g_tuner, t); | ||
1325 | 1326 | ||
1326 | return 0; | 1327 | return 0; |
1327 | } | 1328 | } |
@@ -1350,6 +1351,9 @@ static int vidioc_g_frequency(struct file *file, void *priv, | |||
1350 | struct cx231xx_fh *fh = priv; | 1351 | struct cx231xx_fh *fh = priv; |
1351 | struct cx231xx *dev = fh->dev; | 1352 | struct cx231xx *dev = fh->dev; |
1352 | 1353 | ||
1354 | if (f->tuner) | ||
1355 | return -EINVAL; | ||
1356 | |||
1353 | f->type = fh->radio ? V4L2_TUNER_RADIO : V4L2_TUNER_ANALOG_TV; | 1357 | f->type = fh->radio ? V4L2_TUNER_RADIO : V4L2_TUNER_ANALOG_TV; |
1354 | f->frequency = dev->ctl_freq; | 1358 | f->frequency = dev->ctl_freq; |
1355 | 1359 | ||