media: v4l2-compat-ioctl32.c: drop pr_info for unknown buffer type
authorHans Verkuil <hans.verkuil@cisco.com>
Wed, 14 Feb 2018 11:44:31 +0000 (12:44 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 16 Feb 2018 19:23:02 +0000 (20:23 +0100)
commit 169f24ca68bf0f247d111aef07af00dd3a02ae88 upstream.

There is nothing wrong with using an unknown buffer type. So
stop spamming the kernel log whenever this happens. The kernel
will just return -EINVAL to signal this.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/media/v4l2-core/v4l2-compat-ioctl32.c

index de3e99d..bef9c99 100644 (file)
@@ -179,8 +179,6 @@ static int __get_v4l2_format32(struct v4l2_format *kp, struct v4l2_format32 __us
                return copy_from_user(&kp->fmt.meta, &up->fmt.meta,
                                      sizeof(kp->fmt.meta)) ? -EFAULT : 0;
        default:
-               pr_info("compat_ioctl32: unexpected VIDIOC_FMT type %d\n",
-                       kp->type);
                return -EINVAL;
        }
 }
@@ -233,8 +231,6 @@ static int __put_v4l2_format32(struct v4l2_format *kp, struct v4l2_format32 __us
                return copy_to_user(&up->fmt.meta, &kp->fmt.meta,
                                    sizeof(kp->fmt.meta)) ? -EFAULT : 0;
        default:
-               pr_info("compat_ioctl32: unexpected VIDIOC_FMT type %d\n",
-                       kp->type);
                return -EINVAL;
        }
 }