[media] meye: convert to the control framework

Convert the meye driver to the control framework. Some private controls
have been replaced with standardized controls (SHARPNESS and JPEGQUAL).
The AGC control looks like it can be replaced by the AUTOGAIN control, but
it isn't a boolean so I do not know how to interpret it.
The FRAMERATE control looks like it can be replaced by S_PARM, but again,
without knowing how to interpret it I decided to leave it alone.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

4 files changed, 99 insertions, 194 deletions

diff --git a/drivers/media/pci/meye/meye.c b/drivers/media/pci/meye/meye.c
index 3b39deaa8f6b..7859c43479d7 100644
--- a/drivers/media/pci/meye/meye.c
+++ b/drivers/media/pci/meye/meye.c
@@ -35,6 +35,8 @@
 #include <media/v4l2-common.h>
 #include <media/v4l2-device.h>
 #include <media/v4l2-ioctl.h>
+#include <media/v4l2-fh.h>
+#include <media/v4l2-event.h>
 #include <asm/uaccess.h>
 #include <asm/io.h>
 #include <linux/delay.h>
@@ -865,7 +867,7 @@ static int meye_open(struct file *file)
                 meye.grab_buffer[i].state = MEYE_BUF_UNUSED;
         kfifo_reset(&meye.grabq);
         kfifo_reset(&meye.doneq);
-        return 0;
+        return v4l2_fh_open(file);
 }
 
 static int meye_release(struct file *file)
@@ -873,7 +875,7 @@ static int meye_release(struct file *file)
         mchip_hic_stop();
         mchip_dma_free();
         clear_bit(0, &meye.in_use);
-        return 0;
+        return v4l2_fh_release(file);
 }
 
 static int meyeioc_g_params(struct meye_params *p)
@@ -1032,8 +1034,9 @@ static int vidioc_querycap(struct file *file, void *fh,
         cap->version = (MEYE_DRIVER_MAJORVERSION << 8) +
                        MEYE_DRIVER_MINORVERSION;
 
-        cap->capabilities = V4L2_CAP_VIDEO_CAPTURE |
+        cap->device_caps = V4L2_CAP_VIDEO_CAPTURE |
                             V4L2_CAP_STREAMING;
+        cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
 
         return 0;
 }
@@ -1063,191 +1066,50 @@ static int vidioc_s_input(struct file *file, void *fh, unsigned int i)
         return 0;
 }
 
-static int vidioc_queryctrl(struct file *file, void *fh,
-                                struct v4l2_queryctrl *c)
-{
-        switch (c->id) {
-
-        case V4L2_CID_BRIGHTNESS:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Brightness");
-                c->minimum = 0;
-                c->maximum = 63;
-                c->step = 1;
-                c->default_value = 32;
-                c->flags = 0;
-                break;
-        case V4L2_CID_HUE:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Hue");
-                c->minimum = 0;
-                c->maximum = 63;
-                c->step = 1;
-                c->default_value = 32;
-                c->flags = 0;
-                break;
-        case V4L2_CID_CONTRAST:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Contrast");
-                c->minimum = 0;
-                c->maximum = 63;
-                c->step = 1;
-                c->default_value = 32;
-                c->flags = 0;
-                break;
-        case V4L2_CID_SATURATION:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Saturation");
-                c->minimum = 0;
-                c->maximum = 63;
-                c->step = 1;
-                c->default_value = 32;
-                c->flags = 0;
-                break;
-        case V4L2_CID_AGC:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Agc");
-                c->minimum = 0;
-                c->maximum = 63;
-                c->step = 1;
-                c->default_value = 48;
-                c->flags = 0;
-                break;
-        case V4L2_CID_MEYE_SHARPNESS:
-        case V4L2_CID_SHARPNESS:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Sharpness");
-                c->minimum = 0;
-                c->maximum = 63;
-                c->step = 1;
-                c->default_value = 32;
-
-                /* Continue to report legacy private SHARPNESS ctrl but
-                 * say it is disabled in preference to ctrl in the spec
-                 */
-                c->flags = (c->id == V4L2_CID_SHARPNESS) ? 0 :
-                                                V4L2_CTRL_FLAG_DISABLED;
-                break;
-        case V4L2_CID_PICTURE:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Picture");
-                c->minimum = 0;
-                c->maximum = 63;
-                c->step = 1;
-                c->default_value = 0;
-                c->flags = 0;
-                break;
-        case V4L2_CID_JPEGQUAL:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "JPEG quality");
-                c->minimum = 0;
-                c->maximum = 10;
-                c->step = 1;
-                c->default_value = 8;
-                c->flags = 0;
-                break;
-        case V4L2_CID_FRAMERATE:
-                c->type = V4L2_CTRL_TYPE_INTEGER;
-                strcpy(c->name, "Framerate");
-                c->minimum = 0;
-                c->maximum = 31;
-                c->step = 1;
-                c->default_value = 0;
-                c->flags = 0;
-                break;
-        default:
-                return -EINVAL;
-        }
-
-        return 0;
-}
-
-static int vidioc_s_ctrl(struct file *file, void *fh, struct v4l2_control *c)
+static int meye_s_ctrl(struct v4l2_ctrl *ctrl)
 {
         mutex_lock(&meye.lock);
-        switch (c->id) {
+        switch (ctrl->id) {
         case V4L2_CID_BRIGHTNESS:
                 sony_pic_camera_command(
-                        SONY_PIC_COMMAND_SETCAMERABRIGHTNESS, c->value);
-                meye.brightness = c->value << 10;
+                        SONY_PIC_COMMAND_SETCAMERABRIGHTNESS, ctrl->val);
+                meye.brightness = ctrl->val << 10;
                 break;
         case V4L2_CID_HUE:
                 sony_pic_camera_command(
-                        SONY_PIC_COMMAND_SETCAMERAHUE, c->value);
-                meye.hue = c->value << 10;
+                        SONY_PIC_COMMAND_SETCAMERAHUE, ctrl->val);
+                meye.hue = ctrl->val << 10;
                 break;
         case V4L2_CID_CONTRAST:
                 sony_pic_camera_command(
-                        SONY_PIC_COMMAND_SETCAMERACONTRAST, c->value);
-                meye.contrast = c->value << 10;
+                        SONY_PIC_COMMAND_SETCAMERACONTRAST, ctrl->val);
+                meye.contrast = ctrl->val << 10;