[media] cx231xx: fix tuner compliance issues
authorHans Verkuil <hans.verkuil@cisco.com>
Thu, 13 Sep 2012 15:54:36 +0000 (12:54 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 5 Mar 2013 17:25:58 +0000 (14:25 -0300)
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>
drivers/media/usb/cx231xx/cx231xx-avcore.c
drivers/media/usb/cx231xx/cx231xx-video.c

index 7222079137406313589bb4119ca5e01b6fe1946f..4706ed35029323812750ef3f0a931d39558621a7 100644 (file)
@@ -2133,7 +2133,7 @@ int cx231xx_tuner_post_channel_change(struct cx231xx *dev)
 
        status = vid_blk_write_word(dev, DIF_AGC_IF_REF, dwval);
 
-       return status;
+       return status == sizeof(dwval) ? 0 : -EIO;
 }
 
 /******************************************************************************
index a31f7daef3dd2c6c5c04e7c7bb9f92f1982be4ef..2b7b0944b7b4645e662fb7d9d248eb2004954f23 100644 (file)
@@ -1322,6 +1322,7 @@ static int vidioc_g_tuner(struct file *file, void *priv, struct v4l2_tuner *t)
        t->capability = V4L2_TUNER_CAP_NORM;
        t->rangehigh = 0xffffffffUL;
        t->signal = 0xffff;     /* LOCKED */
+       call_all(dev, tuner, g_tuner, t);
 
        return 0;
 }
@@ -1350,6 +1351,9 @@ static int vidioc_g_frequency(struct file *file, void *priv,
        struct cx231xx_fh *fh = priv;
        struct cx231xx *dev = fh->dev;
 
+       if (f->tuner)
+               return -EINVAL;
+
        f->type = fh->radio ? V4L2_TUNER_RADIO : V4L2_TUNER_ANALOG_TV;
        f->frequency = dev->ctl_freq;