V4L/DVB (12948): v4l1-compat: fix VIDIOC_G_STD handling
authorHans Verkuil <hverkuil@xs4all.nl>
Tue, 15 Sep 2009 11:08:20 +0000 (08:08 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 19 Sep 2009 03:15:25 +0000 (00:15 -0300)
The VIDIOC_G_STD ioctl may not be present in the case of radio receivers.
In that case G_STD will return an error. The v4l1-compat layer should not
attempt to propagate that error to the caller, instead it should be
ignored.

Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/v4l1-compat.c

index 761fbd64db5850adfeba11a8cc3b32ba0f7f75a4..0c2105ca611e023e3059d2629aa9e43ec02ea3d1 100644 (file)
@@ -564,10 +564,9 @@ static noinline long v4l1_compat_get_input_info(
                break;
        }
        chan->norm = 0;
-       err = drv(file, VIDIOC_G_STD, &sid);
-       if (err < 0)
-               dprintk("VIDIOCGCHAN / VIDIOC_G_STD: %ld\n", err);
-       if (err == 0) {
+       /* Note: G_STD might not be present for radio receivers,
+        * so we should ignore any errors. */
+       if (drv(file, VIDIOC_G_STD, &sid) == 0) {
                if (sid & V4L2_STD_PAL)
                        chan->norm = VIDEO_MODE_PAL;
                if (sid & V4L2_STD_NTSC)
@@ -776,10 +775,9 @@ static noinline long v4l1_compat_get_tuner(
                        tun->flags |= VIDEO_TUNER_SECAM;
        }
 
-       err = drv(file, VIDIOC_G_STD, &sid);
-       if (err < 0)
-               dprintk("VIDIOCGTUNER / VIDIOC_G_STD: %ld\n", err);
-       if (err == 0) {
+       /* Note: G_STD might not be present for radio receivers,
+        * so we should ignore any errors. */
+       if (drv(file, VIDIOC_G_STD, &sid) == 0) {
                if (sid & V4L2_STD_PAL)
                        tun->mode = VIDEO_MODE_PAL;
                if (sid & V4L2_STD_NTSC)