Merge branch 'v4l_for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media

Pull media updates from Mauro Carvalho Chehab:
 - removal of sn9c102.  This device driver was replaced a long time ago
   by gspca
 - solo6x10 and go7007 webcam drivers moved from staging into
   mainstream.  They were waiting for an API to allow setting the image
   detection matrix
 - SDR drivers moved from staging into mainstream: sdr-msi3101 (renamed
   as msi2500) and rtl2832
 - added SDR driver for airspy
 - added demux driver: si2165
 - rework at several RC subsystem, making the code for RC-5 SZ variant
   to be added at the standard RC5 decoder
 - added decoder for the XMP IR protocol
 - tuner driver moved from staging into mainstream: msi3101 (renamed as
   msi001)
 - added documentation for some additional SDR pixfmt
 - some device tree bindings documented
 - added support for exynos3250 at s5p-jpeg
 - remove the obsolete, unmaintained and broken mx1_camera driver
 - added support for remote controllers at au0828 driver
 - added a RC driver: sunxi-cir
 - several driver fixes, enhancements and cleanups.

* 'v4l_for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media: (455 commits)
  [media] cx23885: fix UNSET/TUNER_ABSENT confusion
  [media] coda: fix build error by making reset control optional
  [media] radio-miropcm20: fix sparse NULL pointer warning
  [media] MAINTAINERS: Update go7007 pattern
  [media] MAINTAINERS: Update solo6x10 patterns
  [media] media: atmel-isi: add primary DT support
  [media] media: atmel-isi: convert the pdata from pointer to structure
  [media] media: atmel-isi: add v4l2 async probe support
  [media] rcar_vin: add devicetree support
  [media] media: pxa_camera device-tree support
  [media] media: mt9m111: add device-tree suppport
  [media] soc_camera: add support for dt binding soc_camera drivers
  [media] media: soc_camera: pxa_camera documentation device-tree support
  [media] media: mt9m111: add device-tree documentation
  [media] s5p-mfc: remove unnecessary calling to function video_devdata()
  [media] s5p-jpeg: add chroma subsampling adjustment for Exynos3250
  [media] s5p-jpeg: Prevent erroneous downscaling for Exynos3250 SoC
  [media] s5p-jpeg: Assure proper crop rectangle initialization
  [media] s5p-jpeg: fix g_selection op
  [media] s5p-jpeg: Adjust jpeg_bound_align_image to Exynos3250 needs
  ...

1 files changed, 79 insertions, 46 deletions

diff --git a/drivers/media/v4l2-core/v4l2-subdev.c b/drivers/media/v4l2-core/v4l2-subdev.c
index 058c1a6e8392..b4d235c13fbf 100644
--- a/drivers/media/v4l2-core/v4l2-subdev.c
+++ b/drivers/media/v4l2-core/v4l2-subdev.c
@@ -126,6 +126,57 @@ static int subdev_close(struct file *file)
         return 0;
 }
 
+#if defined(CONFIG_VIDEO_V4L2_SUBDEV_API)
+static int check_format(struct v4l2_subdev *sd,
+                        struct v4l2_subdev_format *format)
+{
+        if (format->which != V4L2_SUBDEV_FORMAT_TRY &&
+            format->which != V4L2_SUBDEV_FORMAT_ACTIVE)
+                return -EINVAL;
+
+        if (format->pad >= sd->entity.num_pads)
+                return -EINVAL;
+
+        return 0;
+}
+
+static int check_crop(struct v4l2_subdev *sd, struct v4l2_subdev_crop *crop)
+{
+        if (crop->which != V4L2_SUBDEV_FORMAT_TRY &&
+            crop->which != V4L2_SUBDEV_FORMAT_ACTIVE)
+                return -EINVAL;
+
+        if (crop->pad >= sd->entity.num_pads)
+                return -EINVAL;
+
+        return 0;
+}
+
+static int check_selection(struct v4l2_subdev *sd,
+                           struct v4l2_subdev_selection *sel)
+{
+        if (sel->which != V4L2_SUBDEV_FORMAT_TRY &&
+            sel->which != V4L2_SUBDEV_FORMAT_ACTIVE)
+                return -EINVAL;
+
+        if (sel->pad >= sd->entity.num_pads)
+                return -EINVAL;
+
+        return 0;
+}
+
+static int check_edid(struct v4l2_subdev *sd, struct v4l2_subdev_edid *edid)
+{
+        if (edid->pad >= sd->entity.num_pads)
+                return -EINVAL;
+
+        if (edid->blocks && edid->edid == NULL)
+                return -EINVAL;
+
+        return 0;
+}
+#endif
+
 static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
 {
         struct video_device *vdev = video_devdata(file);
@@ -133,12 +184,16 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         struct v4l2_fh *vfh = file->private_data;
 #if defined(CONFIG_VIDEO_V4L2_SUBDEV_API)
         struct v4l2_subdev_fh *subdev_fh = to_v4l2_subdev_fh(vfh);
+        int rval;
 #endif
 
         switch (cmd) {
         case VIDIOC_QUERYCTRL:
                 return v4l2_queryctrl(vfh->ctrl_handler, arg);
 
+        case VIDIOC_QUERY_EXT_CTRL:
+                return v4l2_query_ext_ctrl(vfh->ctrl_handler, arg);
+
         case VIDIOC_QUERYMENU:
                 return v4l2_querymenu(vfh->ctrl_handler, arg);
 
@@ -203,12 +258,9 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_SUBDEV_G_FMT: {
                 struct v4l2_subdev_format *format = arg;
 
-                if (format->which != V4L2_SUBDEV_FORMAT_TRY &&
-                    format->which != V4L2_SUBDEV_FORMAT_ACTIVE)
-                        return -EINVAL;
-
-                if (format->pad >= sd->entity.num_pads)
-                        return -EINVAL;
+                rval = check_format(sd, format);
+                if (rval)
+                        return rval;
 
                 return v4l2_subdev_call(sd, pad, get_fmt, subdev_fh, format);
         }
@@ -216,12 +268,9 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_SUBDEV_S_FMT: {
                 struct v4l2_subdev_format *format = arg;
 
-                if (format->which != V4L2_SUBDEV_FORMAT_TRY &&
-                    format->which != V4L2_SUBDEV_FORMAT_ACTIVE)
-                        return -EINVAL;
-
-                if (format->pad >= sd->entity.num_pads)
-                        return -EINVAL;
+                rval = check_format(sd, format);
+                if (rval)
+                        return rval;
 
                 return v4l2_subdev_call(sd, pad, set_fmt, subdev_fh, format);
         }
@@ -229,14 +278,10 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_SUBDEV_G_CROP: {
                 struct v4l2_subdev_crop *crop = arg;
                 struct v4l2_subdev_selection sel;
-                int rval;
-
-                if (crop->which != V4L2_SUBDEV_FORMAT_TRY &&
-                    crop->which != V4L2_SUBDEV_FORMAT_ACTIVE)
-                        return -EINVAL;
 
-                if (crop->pad >= sd->entity.num_pads)
-                        return -EINVAL;
+                rval = check_crop(sd, crop);
+                if (rval)
+                        return rval;
 
                 rval = v4l2_subdev_call(sd, pad, get_crop, subdev_fh, crop);
                 if (rval != -ENOIOCTLCMD)
@@ -258,14 +303,10 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_SUBDEV_S_CROP: {
                 struct v4l2_subdev_crop *crop = arg;
                 struct v4l2_subdev_selection sel;
-                int rval;
-
-                if (crop->which != V4L2_SUBDEV_FORMAT_TRY &&
-                    crop->which != V4L2_SUBDEV_FORMAT_ACTIVE)
-                        return -EINVAL;
 
-                if (crop->pad >= sd->entity.num_pads)
-                        return -EINVAL;
+                rval = check_crop(sd, crop);
+                if (rval)
+                        return rval;
 
                 rval = v4l2_subdev_call(sd, pad, set_crop, subdev_fh, crop);
                 if (rval != -ENOIOCTLCMD)
@@ -336,12 +377,9 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_SUBDEV_G_SELECTION: {
                 struct v4l2_subdev_selection *sel = arg;
 
-                if (sel->which != V4L2_SUBDEV_FORMAT_TRY &&
-                    sel->which != V4L2_SUBDEV_FORMAT_ACTIVE)
-                        return -EINVAL;
-
-                if (sel->pad >= sd->entity.num_pads)
-                        return -EINVAL;
+                rval = check_selection(sd, sel);
+                if (rval)
+                        return rval;
 
                 return v4l2_subdev_call(
                         sd, pad, get_selection, subdev_fh, sel);
@@ -350,12 +388,9 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_SUBDEV_S_SELECTION: {
                 struct v4l2_subdev_selection *sel = arg;
 
-                if (sel->which != V4L2_SUBDEV_FORMAT_TRY &&
-                    sel->which != V4L2_SUBDEV_FORMAT_ACTIVE)
-                        return -EINVAL;
-
-                if (sel->pad >= sd->entity.num_pads)
-                        return -EINVAL;
+                rval = check_selection(sd, sel);
+                if (rval)
+                        return rval;
 
                 return v4l2_subdev_call(
                         sd, pad, set_selection, subdev_fh, sel);
@@ -364,10 +399,9 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_G_EDID: {
                 struct v4l2_subdev_edid *edid = arg;
 
-                if (edid->pad >= sd->entity.num_pads)
-                        return -EINVAL;
-                if (edid->blocks && edid->edid == NULL)
-                        return -EINVAL;
+                rval = check_edid(sd, edid);
+                if (rval)
+                        return rval;
 
                 return v4l2_subdev_call(sd, pad, get_edid, edid);
         }
@@ -375,10 +409,9 @@ static long subdev_do_ioctl(struct file *file, unsigned int cmd, void *arg)
         case VIDIOC_S_EDID: {
                 struct v4l2_subdev_edid *edid = arg;
 
-                if (edid->pad >= sd->entity.num_pads)
-                        return -EINVAL;
-                if (edid->blocks && edid->edid == NULL)
-                        return -EINVAL;
+                rval = check_edid(sd, edid);
+                if (rval)
+                        return rval;
 
                 return v4l2_subdev_call(sd, pad, set_edid, edid);
         }