V4L/DVB (8788): v4l: replace video_get_drvdata(video_devdata(filp)) with video_drvdata(filp)

Use the new video_drvdata(filp) function where it is safe to do so.

Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

11 files changed, 81 insertions, 170 deletions

diff --git a/drivers/media/video/cpia2/cpia2_v4l.c b/drivers/media/video/cpia2/cpia2_v4l.c
index eb9f15cd4c45..897e8d1a5c3c 100644
--- a/drivers/media/video/cpia2/cpia2_v4l.c
+++ b/drivers/media/video/cpia2/cpia2_v4l.c
@@ -241,8 +241,7 @@ static struct v4l2_queryctrl controls[] = {
  *****************************************************************************/
 static int cpia2_open(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct camera_data *cam = video_get_drvdata(dev);
+        struct camera_data *cam = video_drvdata(file);
         int retval = 0;
 
         if (!cam) {
@@ -357,8 +356,7 @@ static int cpia2_close(struct inode *inode, struct file *file)
 static ssize_t cpia2_v4l_read(struct file *file, char __user *buf, size_t count,
                               loff_t *off)
 {
-        struct video_device *dev = video_devdata(file);
-        struct camera_data *cam = video_get_drvdata(dev);
+        struct camera_data *cam = video_drvdata(file);
         int noblock = file->f_flags&O_NONBLOCK;
 
         struct cpia2_fh *fh = file->private_data;
@@ -382,9 +380,7 @@ static ssize_t cpia2_v4l_read(struct file *file, char __user *buf, size_t count,
  *****************************************************************************/
 static unsigned int cpia2_v4l_poll(struct file *filp, struct poll_table_struct *wait)
 {
-        struct video_device *dev = video_devdata(filp);
-        struct camera_data *cam = video_get_drvdata(dev);
-
+        struct camera_data *cam = video_drvdata(filp);
         struct cpia2_fh *fh = filp->private_data;
 
         if(!cam)
@@ -1579,8 +1575,7 @@ static int ioctl_dqbuf(void *arg,struct camera_data *cam, struct file *file)
 static int cpia2_do_ioctl(struct inode *inode, struct file *file,
                           unsigned int ioctl_nr, void *arg)
 {
-        struct video_device *dev = video_devdata(file);
-        struct camera_data *cam = video_get_drvdata(dev);
+        struct camera_data *cam = video_drvdata(file);
         int retval = 0;
 
         if (!cam)
@@ -1860,9 +1855,8 @@ static int cpia2_ioctl(struct inode *inode, struct file *file,
  *****************************************************************************/
 static int cpia2_mmap(struct file *file, struct vm_area_struct *area)
 {
+        struct camera_data *cam = video_drvdata(file);
         int retval;
-        struct video_device *dev = video_devdata(file);
-        struct camera_data *cam = video_get_drvdata(dev);
 
         /* Priority check */
         struct cpia2_fh *fh = file->private_data;
diff --git a/drivers/media/video/et61x251/et61x251_core.c b/drivers/media/video/et61x251/et61x251_core.c
index 8db2a05bf9c5..7a85c41b0eea 100644
--- a/drivers/media/video/et61x251/et61x251_core.c
+++ b/drivers/media/video/et61x251/et61x251_core.c
@@ -1214,7 +1214,7 @@ static int et61x251_open(struct inode* inode, struct file* filp)
         if (!down_read_trylock(&et61x251_dev_lock))
                 return -ERESTARTSYS;
 
-        cam = video_get_drvdata(video_devdata(filp));
+        cam = video_drvdata(filp);
 
         if (wait_for_completion_interruptible(&cam->probe)) {
                 up_read(&et61x251_dev_lock);
@@ -1297,7 +1297,7 @@ static int et61x251_release(struct inode* inode, struct file* filp)
 
         down_write(&et61x251_dev_lock);
 
-        cam = video_get_drvdata(video_devdata(filp));
+        cam = video_drvdata(filp);
 
         et61x251_stop_transfer(cam);
         et61x251_release_buffers(cam);
@@ -1318,7 +1318,7 @@ static ssize_t
 et61x251_read(struct file* filp, char __user * buf,
               size_t count, loff_t* f_pos)
 {
-        struct et61x251_device* cam = video_get_drvdata(video_devdata(filp));
+        struct et61x251_device *cam = video_drvdata(filp);
         struct et61x251_frame_t* f, * i;
         unsigned long lock_flags;
         long timeout;
@@ -1426,7 +1426,7 @@ exit:
 
 static unsigned int et61x251_poll(struct file *filp, poll_table *wait)
 {
-        struct et61x251_device* cam = video_get_drvdata(video_devdata(filp));
+        struct et61x251_device *cam = video_drvdata(filp);
         struct et61x251_frame_t* f;
         unsigned long lock_flags;
         unsigned int mask = 0;
@@ -1502,7 +1502,7 @@ static struct vm_operations_struct et61x251_vm_ops = {
 
 static int et61x251_mmap(struct file* filp, struct vm_area_struct *vma)
 {
-        struct et61x251_device* cam = video_get_drvdata(video_devdata(filp));
+        struct et61x251_device *cam = video_drvdata(filp);
         unsigned long size = vma->vm_end - vma->vm_start,
                       start = vma->vm_start;
         void *pos;
@@ -2395,7 +2395,7 @@ et61x251_vidioc_s_parm(struct et61x251_device* cam, void __user * arg)
 static int et61x251_ioctl_v4l2(struct inode* inode, struct file* filp,
                                unsigned int cmd, void __user * arg)
 {
-        struct et61x251_device* cam = video_get_drvdata(video_devdata(filp));
+        struct et61x251_device *cam = video_drvdata(filp);
 
         switch (cmd) {
 
@@ -2490,7 +2490,7 @@ static int et61x251_ioctl_v4l2(struct inode* inode, struct file* filp,
 static int et61x251_ioctl(struct inode* inode, struct file* filp,
                          unsigned int cmd, unsigned long arg)
 {
-        struct et61x251_device* cam = video_get_drvdata(video_devdata(filp));
+        struct et61x251_device *cam = video_drvdata(filp);
         int err = 0;
 
         if (mutex_lock_interruptible(&cam->fileop_mutex))
diff --git a/drivers/media/video/saa5246a.c b/drivers/media/video/saa5246a.c
index 0178b49da0c1..ff81049e936d 100644
--- a/drivers/media/video/saa5246a.c
+++ b/drivers/media/video/saa5246a.c
@@ -149,6 +149,7 @@ static int saa5246a_probe(struct i2c_adapter *adap)
 static int saa5246a_detach(struct i2c_client *client)
 {
         struct video_device *vd = i2c_get_clientdata(client);
+
         i2c_detach_client(client);
         video_unregister_device(vd);
         kfree(video_get_drvdata(vd));
@@ -581,8 +582,7 @@ static inline int saa5246a_stop_dau(struct saa5246a_device *t,
 static int do_saa5246a_ioctl(struct inode *inode, struct file *file,
                             unsigned int cmd, void *arg)
 {
-        struct video_device *vd = video_devdata(file);
-        struct saa5246a_device *t = video_get_drvdata(vd);
+        struct saa5246a_device *t = video_drvdata(file);
 
         switch(cmd)
         {
@@ -723,8 +723,7 @@ static inline unsigned int vtx_fix_command(unsigned int cmd)
 static int saa5246a_ioctl(struct inode *inode, struct file *file,
                          unsigned int cmd, unsigned long arg)
 {
-        struct video_device *vd = video_devdata(file);
-        struct saa5246a_device *t = video_get_drvdata(vd);
+        struct saa5246a_device *t = video_drvdata(file);
         int err;
 
         cmd = vtx_fix_command(cmd);
@@ -736,8 +735,7 @@ static int saa5246a_ioctl(struct inode *inode, struct file *file,
 
 static int saa5246a_open(struct inode *inode, struct file *file)
 {
-        struct video_device *vd = video_devdata(file);
-        struct saa5246a_device *t = video_get_drvdata(vd);
+        struct saa5246a_device *t = video_drvdata(file);
 
         if (t->client == NULL)
                 return -ENODEV;
@@ -779,8 +777,7 @@ static int saa5246a_open(struct inode *inode, struct file *file)
 
 static int saa5246a_release(struct inode *inode, struct file *file)
 {
-        struct video_device *vd = video_devdata(file);
-        struct saa5246a_device *t = video_get_drvdata(vd);
+        struct saa5246a_device *t = video_drvdata(file);
 
         /* Stop all acquisition circuits. */
         i2c_senddata(t, SAA5246A_REGISTER_R1,
diff --git a/drivers/media/video/saa5249.c b/drivers/media/video/saa5249.c
index c784715a0b04..8517aa4f0681 100644
--- a/drivers/media/video/saa5249.c
+++ b/drivers/media/video/saa5249.c
@@ -319,8 +319,7 @@ static int do_saa5249_ioctl(struct inode *inode, struct file *file,
                             unsigned int cmd, void *arg)
 {
         static int virtual_mode = false;
-        struct video_device *vd = video_devdata(file);
-        struct saa5249_device *t = video_get_drvdata(vd);
+        struct saa5249_device *t = video_drvdata(file);
 
         switch(cmd)
         {
@@ -618,8 +617,7 @@ static inline unsigned int vtx_fix_command(unsigned int cmd)
 static int saa5249_ioctl(struct inode *inode, struct file *file,
                          unsigned int cmd, unsigned long arg)
 {
-        struct video_device *vd = video_devdata(file);
-        struct saa5249_device *t = video_get_drvdata(vd);
+        struct saa5249_device *t = video_drvdata(file);
         int err;
 
         cmd = vtx_fix_command(cmd);
@@ -631,8 +629,7 @@ static int saa5249_ioctl(struct inode *inode, struct file *file,
 
 static int saa5249_open(struct inode *inode, struct file *file)
 {
-        struct video_device *vd = video_devdata(file);
-        struct saa5249_device *t = video_get_drvdata(vd);
+        struct saa5249_device *t = video_drvdata(file);
         int pgbuf;
 
         if (t->client == NULL)
@@ -669,8 +666,7 @@ static int saa5249_open(struct inode *inode, struct file *file)
 
 static int saa5249_release(struct inode *inode, struct file *file)
 {
-        struct video_device *vd = video_devdata(file);
-        struct saa5249_device *t = video_get_drvdata(vd);
+        struct saa5249_device *t = video_drvdata(file);
 
         i2c_senddata(t, 1, 0x20, -1);           /* Turn off CCT */
         i2c_senddata(t, 5, 3, 3, -1);           /* Turn off TV-display */
diff --git a/drivers/media/video/sn9c102/sn9c102_core.c b/drivers/media/video/sn9c102/sn9c102_core.c
index 2da6938718f2..4b76c45c148c 100644
--- a/drivers/media/video/sn9c102/sn9c102_core.c
+++ b/drivers/media/video/sn9c102/sn9c102_core.c
@@ -1746,7 +1746,7 @@ static int sn9c102_open(struct inode* inode, struct file* filp)
         if (!down_read_trylock(&sn9c102_dev_lock))
                 return -ERESTARTSYS;
 
-        cam = video_get_drvdata(video_devdata(filp));
+        cam = video_drvdata(filp);
 
         if (wait_for_completion_interruptible(&cam->probe)) {
                 up_read(&sn9c102_dev_lock);
@@ -1843,7 +1843,7 @@ static int sn9c102_release(struct inode* inode, struct file* filp)
 
         down_write(&sn9c102_dev_lock);
 
-        cam = video_get_drvdata(video_devdata(filp));
+        cam = video_drvdata(filp);
 
         sn9c102_stop_transfer(cam);
         sn9c102_release_buffers(cam);
@@ -1863,7 +1863,7 @@ static int sn9c102_release(struct inode* inode, struct file* filp)
 static ssize_t
 sn9c102_read(struct file* filp, char __user * buf, size_t count, loff_t* f_pos)
 {
-        struct sn9c102_device* cam = video_get_drvdata(video_devdata(filp));
+        struct sn9c102_device *cam = video_drvdata(filp);
         struct sn9c102_frame_t* f, * i;
         unsigned long lock_flags;
         long timeout;
@@ -1987,7 +1987,7 @@ exit:
 
 static unsigned int sn9c102_poll(struct file *filp, poll_table *wait)
 {
-        struct sn9c102_device* cam = video_get_drvdata(video_devdata(filp));
+        struct sn9c102_device *cam = video_drvdata(filp);
         struct sn9c102_frame_t* f;
         unsigned long lock_flags;
         unsigned int mask = 0;
@@ -2063,7 +2063,7 @@ static struct vm_operations_struct sn9c102_vm_ops = {
 
 static int sn9c102_mmap(struct file* filp, struct vm_area_struct *vma)
 {
-        struct sn9c102_device* cam = video_get_drvdata(video_devdata(filp));
+        struct sn9c102_device *cam = video_drvdata(filp);
         unsigned long size = vma->vm_end - vma->vm_start,
                       start = vma->vm_start;
         void *pos;
@@ -3075,7 +3075,7 @@ sn9c102_vidioc_s_audio(struct sn9c102_device* cam, void __user * arg)
 static int sn9c102_ioctl_v4l2(struct inode* inode, struct file* filp,
                               unsigned int cmd, void __user * arg)
 {
-        struct sn9c102_device* cam = video_get_drvdata(video_devdata(filp));
+        struct sn9c102_device *cam = video_drvdata(filp);
 
         switch (cmd) {
 
@@ -3179,7 +3179,7 @@ static int sn9c102_ioctl_v4l2(struct inode* inode, struct file* filp,
 static int sn9c102_ioctl(struct inode* inode, struct file* filp,
                          unsigned int cmd, unsigned long arg)
 {
-        struct sn9c102_device* cam = video_get_drvdata(video_devdata(filp));
+        struct sn9c102_device *cam = video_drvdata(filp);
         int err = 0;
 
         if (mutex_lock_interruptible(&cam->fileop_mutex))
diff --git a/drivers/media/video/usbvideo/vicam.c b/drivers/media/video/usbvideo/vicam.c
index 1ffcc393fcbb..7a127d6bfdee 100644
--- a/drivers/media/video/usbvideo/vicam.c
+++ b/drivers/media/video/usbvideo/vicam.c
@@ -472,8 +472,7 @@ vicam_ioctl(struct inode *inode, struct file *file, unsigned int ioctlnr, unsign
 static int
 vicam_open(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct vicam_camera *cam = video_get_drvdata(dev);
+        struct vicam_camera *cam = video_drvdata(file);
 
         DBG("open\n");
 
diff --git a/drivers/media/video/usbvision/usbvision-video.c b/drivers/media/video/usbvision/usbvision-video.c
index b76295a5be8b..782ee643601c 100644
--- a/drivers/media/video/usbvision/usbvision-video.c
+++ b/drivers/media/video/usbvision/usbvision-video.c
@@ -360,9 +360,7 @@ static void usbvision_remove_sysfs(struct video_device *vdev)
  */
 static int usbvision_v4l2_open(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int errCode = 0;
 
         PDEBUG(DBG_IO, "open");
@@ -439,9 +437,7 @@ static int usbvision_v4l2_open(struct inode *inode, struct file *file)
  */
 static int usbvision_v4l2_close(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         PDEBUG(DBG_IO, "close");
         mutex_lock(&usbvision->lock);
@@ -486,9 +482,7 @@ static int usbvision_v4l2_close(struct inode *inode, struct file *file)
 static int vidioc_g_register (struct file *file, void *priv,
                                 struct v4l2_register *reg)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int errCode;
 
         if (!v4l2_chip_match_host(reg->match_type, reg->match_chip))
@@ -507,9 +501,7 @@ static int vidioc_g_register (struct file *file, void *priv,
 static int vidioc_s_register (struct file *file, void *priv,
                                 struct v4l2_register *reg)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int errCode;
 
         if (!v4l2_chip_match_host(reg->match_type, reg->match_chip))
@@ -528,9 +520,7 @@ static int vidioc_s_register (struct file *file, void *priv,
 static int vidioc_querycap (struct file *file, void  *priv,
                                         struct v4l2_capability *vc)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         strlcpy(vc->driver, "USBVision", sizeof(vc->driver));
         strlcpy(vc->card,
@@ -550,9 +540,7 @@ static int vidioc_querycap (struct file *file, void  *priv,
 static int vidioc_enum_input (struct file *file, void *priv,
                                 struct v4l2_input *vi)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int chan;
 
         if ((vi->index >= usbvision->video_inputs) || (vi->index < 0) )
@@ -605,9 +593,7 @@ static int vidioc_enum_input (struct file *file, void *priv,
 
 static int vidioc_g_input (struct file *file, void *priv, unsigned int *input)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         *input = usbvision->ctl_input;
         return 0;
@@ -615,9 +601,7 @@ static int vidioc_g_input (struct file *file, void *priv, unsigned int *input)
 
 static int vidioc_s_input (struct file *file, void *priv, unsigned int input)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         if ((input >= usbvision->video_inputs) || (input < 0) )
                 return -EINVAL;
@@ -634,9 +618,8 @@ static int vidioc_s_input (struct file *file, void *priv, unsigned int input)
 
 static int vidioc_s_std (struct file *file, void *priv, v4l2_std_id *id)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
+
         usbvision->tvnormId=*id;
 
         mutex_lock(&usbvision->lock);
@@ -652,9 +635,7 @@ static int vidioc_s_std (struct file *file, void *priv, v4l2_std_id *id)
 static int vidioc_g_tuner (struct file *file, void *priv,
                                 struct v4l2_tuner *vt)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         if (!usbvision->have_tuner || vt->index)        // Only tuner 0
                 return -EINVAL;
@@ -673,9 +654,7 @@ static int vidioc_g_tuner (struct file *file, void *priv,
 static int vidioc_s_tuner (struct file *file, void *priv,
                                 struct v4l2_tuner *vt)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         // Only no or one tuner for now
         if (!usbvision->have_tuner || vt->index)
@@ -689,9 +668,7 @@ static int vidioc_s_tuner (struct file *file, void *priv,
 static int vidioc_g_frequency (struct file *file, void *priv,
                                 struct v4l2_frequency *freq)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         freq->tuner = 0; // Only one tuner
         if(usbvision->radio) {
@@ -707,9 +684,7 @@ static int vidioc_g_frequency (struct file *file, void *priv,
 static int vidioc_s_frequency (struct file *file, void *priv,
                                 struct v4l2_frequency *freq)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         // Only no or one tuner for now
         if (!usbvision->have_tuner || freq->tuner)
@@ -723,9 +698,7 @@ static int vidioc_s_frequency (struct file *file, void *priv,
 
 static int vidioc_g_audio (struct file *file, void *priv, struct v4l2_audio *a)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         memset(a,0,sizeof(*a));
         if(usbvision->radio) {
@@ -750,9 +723,7 @@ static int vidioc_s_audio (struct file *file, void *fh,
 static int vidioc_queryctrl (struct file *file, void *priv,
                             struct v4l2_queryctrl *ctrl)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int id=ctrl->id;
 
         memset(ctrl,0,sizeof(*ctrl));
@@ -769,9 +740,7 @@ static int vidioc_queryctrl (struct file *file, void *priv,
 static int vidioc_g_ctrl (struct file *file, void *priv,
                                 struct v4l2_control *ctrl)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         call_i2c_clients(usbvision, VIDIOC_G_CTRL, ctrl);
 
         return 0;
@@ -780,9 +749,7 @@ static int vidioc_g_ctrl (struct file *file, void *priv,
 static int vidioc_s_ctrl (struct file *file, void *priv,
                                 struct v4l2_control *ctrl)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         call_i2c_clients(usbvision, VIDIOC_S_CTRL, ctrl);
 
         return 0;
@@ -791,9 +758,7 @@ static int vidioc_s_ctrl (struct file *file, void *priv,
 static int vidioc_reqbufs (struct file *file,
                            void *priv, struct v4l2_requestbuffers *vr)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int ret;
 
         RESTRICT_TO_RANGE(vr->count,1,USBVISION_NUMFRAMES);
@@ -821,9 +786,7 @@ static int vidioc_reqbufs (struct file *file,
 static int vidioc_querybuf (struct file *file,
                             void *priv, struct v4l2_buffer *vb)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         struct usbvision_frame *frame;
 
         /* FIXME : must control
@@ -859,9 +822,7 @@ static int vidioc_querybuf (struct file *file,
 
 static int vidioc_qbuf (struct file *file, void *priv, struct v4l2_buffer *vb)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         struct usbvision_frame *frame;
         unsigned long lock_flags;
 
@@ -898,9 +859,7 @@ static int vidioc_qbuf (struct file *file, void *priv, struct v4l2_buffer *vb)
 
 static int vidioc_dqbuf (struct file *file, void *priv, struct v4l2_buffer *vb)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int ret;
         struct usbvision_frame *f;
         unsigned long lock_flags;
@@ -941,9 +900,7 @@ static int vidioc_dqbuf (struct file *file, void *priv, struct v4l2_buffer *vb)
 
 static int vidioc_streamon(struct file *file, void *priv, enum v4l2_buf_type i)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int b=V4L2_BUF_TYPE_VIDEO_CAPTURE;
 
         usbvision->streaming = Stream_On;
@@ -955,9 +912,7 @@ static int vidioc_streamon(struct file *file, void *priv, enum v4l2_buf_type i)
 static int vidioc_streamoff(struct file *file,
                             void *priv, enum v4l2_buf_type type)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int b=V4L2_BUF_TYPE_VIDEO_CAPTURE;
 
         if (type != V4L2_BUF_TYPE_VIDEO_CAPTURE)
@@ -990,9 +945,7 @@ static int vidioc_enum_fmt_vid_cap (struct file *file, void  *priv,
 static int vidioc_g_fmt_vid_cap (struct file *file, void *priv,
                                         struct v4l2_format *vf)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         vf->fmt.pix.width = usbvision->curwidth;
         vf->fmt.pix.height = usbvision->curheight;
         vf->fmt.pix.pixelformat = usbvision->palette.format;
@@ -1008,9 +961,7 @@ static int vidioc_g_fmt_vid_cap (struct file *file, void *priv,
 static int vidioc_try_fmt_vid_cap (struct file *file, void *priv,
                                struct v4l2_format *vf)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int formatIdx;
 
         /* Find requested format in available ones */
@@ -1038,9 +989,7 @@ static int vidioc_try_fmt_vid_cap (struct file *file, void *priv,
 static int vidioc_s_fmt_vid_cap(struct file *file, void *priv,
                                struct v4l2_format *vf)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int ret;
 
         if( 0 != (ret=vidioc_try_fmt_vid_cap (file, priv, vf)) ) {
@@ -1068,9 +1017,7 @@ static int vidioc_s_fmt_vid_cap(struct file *file, void *priv,
 static ssize_t usbvision_v4l2_read(struct file *file, char __user *buf,
                       size_t count, loff_t *ppos)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int noblock = file->f_flags & O_NONBLOCK;
         unsigned long lock_flags;
 
@@ -1179,10 +1126,7 @@ static int usbvision_v4l2_mmap(struct file *file, struct vm_area_struct *vma)
                 start = vma->vm_start;
         void *pos;
         u32 i;
-
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
 
         PDEBUG(DBG_MMAP, "mmap");
 
@@ -1239,9 +1183,7 @@ static int usbvision_v4l2_mmap(struct file *file, struct vm_area_struct *vma)
  */
 static int usbvision_radio_open(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int errCode = 0;
 
         PDEBUG(DBG_IO, "%s:", __func__);
@@ -1291,9 +1233,7 @@ out:
 
 static int usbvision_radio_close(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct usb_usbvision *usbvision =
-                (struct usb_usbvision *) video_get_drvdata(dev);
+        struct usb_usbvision *usbvision = video_drvdata(file);
         int errCode = 0;
 
         PDEBUG(DBG_IO, "");
diff --git a/drivers/media/video/uvc/uvc_v4l2.c b/drivers/media/video/uvc/uvc_v4l2.c
index d7bd71be40a9..d4758c8e13ad 100644
--- a/drivers/media/video/uvc/uvc_v4l2.c
+++ b/drivers/media/video/uvc/uvc_v4l2.c
@@ -400,15 +400,13 @@ static int uvc_has_privileges(struct uvc_fh *handle)
 
 static int uvc_v4l2_open(struct inode *inode, struct file *file)
 {
-        struct video_device *vdev;
         struct uvc_video_device *video;
         struct uvc_fh *handle;
         int ret = 0;
 
         uvc_trace(UVC_TRACE_CALLS, "uvc_v4l2_open\n");
         mutex_lock(&uvc_driver.open_mutex);
-        vdev = video_devdata(file);
-        video = video_get_drvdata(vdev);
+        video = video_drvdata(file);
 
         if (video->dev->state & UVC_DEV_DISCONNECTED) {
                 ret = -ENODEV;
@@ -440,8 +438,7 @@ done:
 
 static int uvc_v4l2_release(struct inode *inode, struct file *file)
 {
-        struct video_device *vdev = video_devdata(file);
-        struct uvc_video_device *video = video_get_drvdata(vdev);
+        struct uvc_video_device *video = video_drvdata(file);
         struct uvc_fh *handle = (struct uvc_fh *)file->private_data;
 
         uvc_trace(UVC_TRACE_CALLS, "uvc_v4l2_release\n");
@@ -1031,8 +1028,7 @@ static struct vm_operations_struct uvc_vm_ops = {
 
 static int uvc_v4l2_mmap(struct file *file, struct vm_area_struct *vma)
 {
-        struct video_device *vdev = video_devdata(file);
-        struct uvc_video_device *video = video_get_drvdata(vdev);
+        struct uvc_video_device *video = video_drvdata(file);
         struct uvc_buffer *uninitialized_var(buffer);
         struct page *page;
         unsigned long addr, start, size;
@@ -1085,8 +1081,7 @@ done:
 
 static unsigned int uvc_v4l2_poll(struct file *file, poll_table *wait)
 {
-        struct video_device *vdev = video_devdata(file);
-        struct uvc_video_device *video = video_get_drvdata(vdev);
+        struct uvc_video_device *video = video_drvdata(file);
 
         uvc_trace(UVC_TRACE_CALLS, "uvc_v4l2_poll\n");
 
diff --git a/drivers/media/video/vino.c b/drivers/media/video/vino.c
index ca421cc84f8f..8ec57df1904f 100644
--- a/drivers/media/video/vino.c
+++ b/drivers/media/video/vino.c
@@ -4024,8 +4024,7 @@ out:
 
 static int vino_open(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct vino_channel_settings *vcs = video_get_drvdata(dev);
+        struct vino_channel_settings *vcs = video_drvdata(file);
         int ret = 0;
         dprintk("open(): channel = %c\n",
                (vcs->channel == VINO_CHANNEL_A) ? 'A' : 'B');
@@ -4056,8 +4055,7 @@ static int vino_open(struct inode *inode, struct file *file)
 
 static int vino_close(struct inode *inode, struct file *file)
 {
-        struct video_device *dev = video_devdata(file);
-        struct vino_channel_settings *vcs = video_get_drvdata(dev);
+        struct vino_channel_settings *vcs = video_drvdata(file);
         dprintk("close():\n");
 
         mutex_lock(&vcs->mutex);
@@ -4100,8 +4098,7 @@ static struct vm_operations_struct vino_vm_ops = {
 
 static int vino_mmap(struct file *file, struct vm_area_struct *vma)
 {
-        struct video_device *dev = video_devdata(file);
-        struct vino_channel_settings *vcs = video_get_drvdata(dev);
+        struct vino_channel_settings *vcs = video_drvdata(file);
 
         unsigned long start = vma->vm_start;
         unsigned long size = vma->vm_end - vma->vm_start;
@@ -4206,8 +4203,7 @@ out:
 
 static unsigned int vino_poll(struct file *file, poll_table *pt)
 {
-        struct video_device *dev = video_devdata(file);
-        struct vino_channel_settings *vcs = video_get_drvdata(dev);
+        struct vino_channel_settings *vcs = video_drvdata(file);
         unsigned int outgoing;
         unsigned int ret = 0;
 
@@ -4247,8 +4243,7 @@ error:
 static int vino_do_ioctl(struct inode *inode, struct file *file,
                       unsigned int cmd, void *arg)
 {
-        struct video_device *dev = video_devdata(file);
-        struct vino_channel_settings *vcs = video_get_drvdata(dev);
+        struct vino_channel_settings *vcs = video_drvdata(file);
 
 #ifdef VINO_DEBUG
         switch (_IOC_TYPE(cmd)) {
@@ -4355,8 +4350,7 @@ static int vino_do_ioctl(struct inode *inode, struct file *file,
 static int vino_ioctl(struct inode *inode, struct file *file,
                       unsigned int cmd, unsigned long arg)
 {
-        struct video_device *dev = video_devdata(file);
-        struct vino_channel_settings *vcs = video_get_drvdata(dev);
+        struct vino_channel_settings *vcs = video_drvdata(file);
         int ret;
 
         if (mutex_lock_interruptible(&vcs->mutex))
diff --git a/drivers/media/video/w9966.c b/drivers/media/video/w9966.c
index 14ebb15837fb..b2dbe48a92bb 100644
--- a/drivers/media/video/w9966.c
+++ b/drivers/media/video/w9966.c
@@ -187,16 +187,14 @@ static ssize_t w9966_v4l_read(struct file *file, char __user *buf,
 
 static int w9966_exclusive_open(struct inode *inode, struct file *file)
 {
-        struct video_device *vdev = video_devdata(file);
-        struct w9966_dev *cam = video_get_drvdata(vdev);
+        struct w9966_dev *cam = video_drvdata(file);
 
         return test_and_set_bit(0, &cam->in_use) ? -EBUSY : 0;
 }
 
 static int w9966_exclusive_release(struct inode *inode, struct file *file)
 {
-        struct video_device *vdev = video_devdata(file);
-        struct w9966_dev *cam = video_get_drvdata(vdev);
+        struct w9966_dev *cam = video_drvdata(file);
 
         clear_bit(0, &cam->in_use);
         return 0;
@@ -732,8 +730,7 @@ static int w9966_wReg_i2c(struct w9966_dev* cam, int reg, int data)
 static int w9966_v4l_do_ioctl(struct inode *inode, struct file *file,
                               unsigned int cmd, void *arg)
 {
-        struct video_device *vdev = video_devdata(file);
-        struct w9966_dev *cam = video_get_drvdata(vdev);
+        struct w9966_dev *cam = video_drvdata(file);
 
         switch(cmd)
         {
@@ -891,8 +888,7 @@ static int w9966_v4l_ioctl(struct inode *inode, struct file *file,
 static ssize_t w9966_v4l_read(struct file *file, char  __user *buf,
                               size_t count, loff_t *ppos)
 {
-        struct video_device *vdev = video_devdata(file);
-        struct w9966_dev *cam = video_get_drvdata(vdev);
+        struct w9966_dev *cam = video_drvdata(file);
         unsigned char addr = 0xa0;      // ECP, read, CCD-transfer, 00000
         unsigned char __user *dest = (unsigned char __user *)buf;
         unsigned long dleft = count;
diff --git a/drivers/media/video/zc0301/zc0301_core.c b/drivers/media/video/zc0301/zc0301_core.c
index 0c3287734c93..6a0902bcba6b 100644
--- a/drivers/media/video/zc0301/zc0301_core.c
+++ b/drivers/media/video/zc0301/zc0301_core.c
@@ -657,7 +657,7 @@ static int zc0301_open(struct inode* inode, struct file* filp)
         if (!down_read_trylock(&zc0301_dev_lock))
                 return -EAGAIN;
 
-        cam = video_get_drvdata(video_devdata(filp));
+        cam = video_drvdata(filp);
 
         if (wait_for_completion_interruptible(&cam->probe)) {
                 up_read(&zc0301_dev_lock);
@@ -739,7 +739,7 @@ static int zc0301_release(struct inode* inode, struct file* filp)
 
         down_write(&zc0301_dev_lock);
 
-        cam = video_get_drvdata(video_devdata(filp));
+        cam = video_drvdata(filp);
 
         zc0301_stop_transfer(cam);
         zc0301_release_buffers(cam);
@@ -759,7 +759,7 @@ static int zc0301_release(struct inode* inode, struct file* filp)
 static ssize_t
 zc0301_read(struct file* filp, char __user * buf, size_t count, loff_t* f_pos)
 {
-        struct zc0301_device* cam = video_get_drvdata(video_devdata(filp));
+        struct zc0301_device *cam = video_drvdata(filp);
         struct zc0301_frame_t* f, * i;
         unsigned long lock_flags;
         long timeout;
@@ -866,7 +866,7 @@ exit:
 
 static unsigned int zc0301_poll(struct file *filp, poll_table *wait)
 {
-        struct zc0301_device* cam = video_get_drvdata(video_devdata(filp));
+        struct zc0301_device *cam = video_drvdata(filp);
         struct zc0301_frame_t* f;
         unsigned long lock_flags;
         unsigned int mask = 0;
@@ -941,7 +941,7 @@ static struct vm_operations_struct zc0301_vm_ops = {
 
 static int zc0301_mmap(struct file* filp, struct vm_area_struct *vma)
 {
-        struct zc0301_device* cam = video_get_drvdata(video_devdata(filp));
+        struct zc0301_device *cam = video_drvdata(filp);
         unsigned long size = vma->vm_end - vma->vm_start,
                       start = vma->vm_start;
         void *pos;
@@ -1796,7 +1796,7 @@ zc0301_vidioc_s_parm(struct zc0301_device* cam, void __user * arg)
 static int zc0301_ioctl_v4l2(struct inode* inode, struct file* filp,
                              unsigned int cmd, void __user * arg)
 {
-        struct zc0301_device* cam = video_get_drvdata(video_devdata(filp));
+        struct zc0301_device *cam = video_drvdata(filp);
 
         switch (cmd) {
 
@@ -1891,7 +1891,7 @@ static int zc0301_ioctl_v4l2(struct inode* inode, struct file* filp,
 static int zc0301_ioctl(struct inode* inode, struct file* filp,
                         unsigned int cmd, unsigned long arg)
 {
-        struct zc0301_device* cam = video_get_drvdata(video_devdata(filp));
+        struct zc0301_device *cam = video_drvdata(filp);
         int err = 0;
 
         if (mutex_lock_interruptible(&cam->fileop_mutex))