V4L/DVB (8783): v4l: add all missing video_device release callbacks

All drivers that call video_device_register where checked to see if they
set the release callback of struct video_device. Where that callback was
missing it was added.

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

24 files changed, 36 insertions, 17 deletions

diff --git a/drivers/media/radio/radio-aimslab.c b/drivers/media/radio/radio-aimslab.c
index b657c7bb47cc..59fdaf85e41b 100644
--- a/drivers/media/radio/radio-aimslab.c
+++ b/drivers/media/radio/radio-aimslab.c
@@ -420,6 +420,7 @@ static struct video_device rtrack_radio = {
         .name           = "RadioTrack radio",
         .fops           = &rtrack_fops,
         .ioctl_ops      = &rtrack_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __init rtrack_init(void)
diff --git a/drivers/media/radio/radio-aztech.c b/drivers/media/radio/radio-aztech.c
index ae18a7cf0d59..ddd96295a9fe 100644
--- a/drivers/media/radio/radio-aztech.c
+++ b/drivers/media/radio/radio-aztech.c
@@ -381,9 +381,10 @@ static const struct v4l2_ioctl_ops aztech_ioctl_ops = {
 };
 
 static struct video_device aztech_radio = {
-        .name               = "Aztech radio",
-        .fops               = &aztech_fops,
-        .ioctl_ops          = &aztech_ioctl_ops,
+        .name           = "Aztech radio",
+        .fops           = &aztech_fops,
+        .ioctl_ops      = &aztech_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 module_param_named(debug,aztech_radio.debug, int, 0644);
diff --git a/drivers/media/radio/radio-cadet.c b/drivers/media/radio/radio-cadet.c
index 04c3698d32e4..0490a1fa999d 100644
--- a/drivers/media/radio/radio-cadet.c
+++ b/drivers/media/radio/radio-cadet.c
@@ -589,6 +589,7 @@ static struct video_device cadet_radio = {
         .name           = "Cadet radio",
         .fops           = &cadet_fops,
         .ioctl_ops      = &cadet_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 #ifdef CONFIG_PNP
diff --git a/drivers/media/radio/radio-gemtek-pci.c b/drivers/media/radio/radio-gemtek-pci.c
index a7b9a5df4ca7..d65fd287bde1 100644
--- a/drivers/media/radio/radio-gemtek-pci.c
+++ b/drivers/media/radio/radio-gemtek-pci.c
@@ -401,9 +401,10 @@ static const struct v4l2_ioctl_ops gemtek_pci_ioctl_ops = {
 };
 
 static struct video_device vdev_template = {
-        .name          = "Gemtek PCI Radio",
-        .fops          = &gemtek_pci_fops,
-        .ioctl_ops     = &gemtek_pci_ioctl_ops,
+        .name           = "Gemtek PCI Radio",
+        .fops           = &gemtek_pci_fops,
+        .ioctl_ops      = &gemtek_pci_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __devinit gemtek_pci_probe( struct pci_dev *pci_dev, const struct pci_device_id *pci_id )
diff --git a/drivers/media/radio/radio-gemtek.c b/drivers/media/radio/radio-gemtek.c
index cc90329026f2..2d991ae2f205 100644
--- a/drivers/media/radio/radio-gemtek.c
+++ b/drivers/media/radio/radio-gemtek.c
@@ -581,9 +581,10 @@ static const struct v4l2_ioctl_ops gemtek_ioctl_ops = {
 };
 
 static struct video_device gemtek_radio = {
-        .name                   = "GemTek Radio card",
-        .fops                   = &gemtek_fops,
-        .ioctl_ops              = &gemtek_ioctl_ops,
+        .name           = "GemTek Radio card",
+        .fops           = &gemtek_fops,
+        .ioctl_ops      = &gemtek_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 /*
diff --git a/drivers/media/radio/radio-maestro.c b/drivers/media/radio/radio-maestro.c
index 03c20948eef2..182b433981ca 100644
--- a/drivers/media/radio/radio-maestro.c
+++ b/drivers/media/radio/radio-maestro.c
@@ -388,6 +388,7 @@ static struct video_device maestro_radio = {
         .name           = "Maestro radio",
         .fops           = &maestro_fops,
         .ioctl_ops      = &maestro_ioctl_ops,
+        .release        = video_device_release,
 };
 
 static int __devinit maestro_probe(struct pci_dev *pdev,
diff --git a/drivers/media/radio/radio-maxiradio.c b/drivers/media/radio/radio-maxiradio.c
index 263ef8b02fd8..4e2e8a3d9b5f 100644
--- a/drivers/media/radio/radio-maxiradio.c
+++ b/drivers/media/radio/radio-maxiradio.c
@@ -402,9 +402,10 @@ static const struct v4l2_ioctl_ops maxiradio_ioctl_ops = {
 };
 
 static struct video_device maxiradio_radio = {
-        .name               = "Maxi Radio FM2000 radio",
-        .fops               = &maxiradio_fops,
-        .ioctl_ops          = &maxiradio_ioctl_ops,
+        .name           = "Maxi Radio FM2000 radio",
+        .fops           = &maxiradio_fops,
+        .ioctl_ops      = &maxiradio_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __devinit maxiradio_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
diff --git a/drivers/media/radio/radio-rtrack2.c b/drivers/media/radio/radio-rtrack2.c
index 5b93b47be714..8fe820da73e8 100644
--- a/drivers/media/radio/radio-rtrack2.c
+++ b/drivers/media/radio/radio-rtrack2.c
@@ -326,6 +326,7 @@ static struct video_device rtrack2_radio = {
         .name           = "RadioTrack II radio",
         .fops           = &rtrack2_fops,
         .ioctl_ops      = &rtrack2_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __init rtrack2_init(void)
diff --git a/drivers/media/radio/radio-sf16fmi.c b/drivers/media/radio/radio-sf16fmi.c
index 55f8334513f3..6d865a7a91ec 100644
--- a/drivers/media/radio/radio-sf16fmi.c
+++ b/drivers/media/radio/radio-sf16fmi.c
@@ -326,6 +326,7 @@ static struct video_device fmi_radio = {
         .name           = "SF16FMx radio",
         .fops           = &fmi_fops,
         .ioctl_ops      = &fmi_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 /* ladis: this is my card. does any other types exist? */
diff --git a/drivers/media/radio/radio-sf16fmr2.c b/drivers/media/radio/radio-sf16fmr2.c
index 14153ed939d0..9625d1f0194e 100644
--- a/drivers/media/radio/radio-sf16fmr2.c
+++ b/drivers/media/radio/radio-sf16fmr2.c
@@ -442,6 +442,7 @@ static struct video_device fmr2_radio = {
         .name           = "SF16FMR2 radio",
         .fops           = &fmr2_fops,
         .ioctl_ops      = &fmr2_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __init fmr2_init(void)
diff --git a/drivers/media/radio/radio-terratec.c b/drivers/media/radio/radio-terratec.c
index eed14972a940..de13ec81ce49 100644
--- a/drivers/media/radio/radio-terratec.c
+++ b/drivers/media/radio/radio-terratec.c
@@ -398,6 +398,7 @@ static struct video_device terratec_radio = {
         .name           = "TerraTec ActiveRadio",
         .fops           = &terratec_fops,
         .ioctl_ops      = &terratec_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __init terratec_init(void)
diff --git a/drivers/media/radio/radio-trust.c b/drivers/media/radio/radio-trust.c
index dd0c425ef3b8..e7b111fcd105 100644
--- a/drivers/media/radio/radio-trust.c
+++ b/drivers/media/radio/radio-trust.c
@@ -378,6 +378,7 @@ static struct video_device trust_radio = {
         .name           = "Trust FM Radio",
         .fops           = &trust_fops,
         .ioctl_ops      = &trust_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __init trust_init(void)
diff --git a/drivers/media/radio/radio-typhoon.c b/drivers/media/radio/radio-typhoon.c
index 878468507237..c2eedb7a73a5 100644
--- a/drivers/media/radio/radio-typhoon.c
+++ b/drivers/media/radio/radio-typhoon.c
@@ -375,6 +375,7 @@ static struct video_device typhoon_radio = {
         .name           = "Typhoon Radio",
         .fops           = &typhoon_fops,
         .ioctl_ops      = &typhoon_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 #ifdef CONFIG_RADIO_TYPHOON_PROC_FS
diff --git a/drivers/media/radio/radio-zoltrix.c b/drivers/media/radio/radio-zoltrix.c
index 4d76e710194b..0d43e8401eb5 100644
--- a/drivers/media/radio/radio-zoltrix.c
+++ b/drivers/media/radio/radio-zoltrix.c
@@ -439,6 +439,7 @@ static struct video_device zoltrix_radio = {
         .name           = "Zoltrix Radio Plus",
         .fops           = &zoltrix_fops,
         .ioctl_ops      = &zoltrix_ioctl_ops,
+        .release        = video_device_release_empty,
 };
 
 static int __init zoltrix_init(void)
diff --git a/drivers/media/video/bw-qcam.c b/drivers/media/video/bw-qcam.c
index e8e1bb122b59..ace4ff9ea023 100644
--- a/drivers/media/video/bw-qcam.c
+++ b/drivers/media/video/bw-qcam.c
@@ -926,6 +926,7 @@ static struct video_device qcam_template=
 {
         .name           = "Connectix Quickcam",
         .fops           = &qcam_fops,
+        .release        = video_device_release_empty,
 };
 
 #define MAX_CAMS 4
diff --git a/drivers/media/video/c-qcam.c b/drivers/media/video/c-qcam.c
index 2196c1dc5aee..17aa0adb3467 100644
--- a/drivers/media/video/c-qcam.c
+++ b/drivers/media/video/c-qcam.c
@@ -722,6 +722,7 @@ static struct video_device qcam_template=
 {
         .name           = "Colour QuickCam",
         .fops           = &qcam_fops,
+        .release        = video_device_release_empty,
 };
 
 /* Initialize the QuickCam driver control structure. */
diff --git a/drivers/media/video/cpia.c b/drivers/media/video/cpia.c
index a661800b0e69..c0600fdbfc6f 100644
--- a/drivers/media/video/cpia.c
+++ b/drivers/media/video/cpia.c
@@ -3801,6 +3801,7 @@ static const struct file_operations cpia_fops = {
 static struct video_device cpia_template = {
         .name           = "CPiA Camera",
         .fops           = &cpia_fops,
+        .release        = video_device_release_empty,
 };
 
 /* initialise cam_data structure  */
diff --git a/drivers/media/video/pms.c b/drivers/media/video/pms.c
index b39794f0baf5..994807818aa2 100644
--- a/drivers/media/video/pms.c
+++ b/drivers/media/video/pms.c
@@ -915,6 +915,7 @@ static struct video_device pms_template=
 {
         .name           = "Mediavision PMS",
         .fops           = &pms_fops,
+        .release        = video_device_release_empty,
 };
 
 static struct pms_device pms_device;
diff --git a/drivers/media/video/saa5249.c b/drivers/media/video/saa5249.c
index dfd8a9338c55..225117c789ca 100644
--- a/drivers/media/video/saa5249.c
+++ b/drivers/media/video/saa5249.c
@@ -708,6 +708,7 @@ static struct video_device saa_template =
 {
         .name           = IF_NAME,
         .fops           = &saa_fops,
+        .release        = video_device_release,
 };
 
 MODULE_LICENSE("GPL");
diff --git a/drivers/media/video/se401.c b/drivers/media/video/se401.c
index d16bbf0d2783..b01f25d72922 100644
--- a/drivers/media/video/se401.c
+++ b/drivers/media/video/se401.c
@@ -1236,6 +1236,7 @@ static const struct file_operations se401_fops = {
 static struct video_device se401_template = {
         .name =         "se401 USB camera",
         .fops =         &se401_fops,
+        .release = video_device_release_empty,
 };
 
 
diff --git a/drivers/media/video/stradis.c b/drivers/media/video/stradis.c
index a3cbe9be3c15..bbad54f85c83 100644
--- a/drivers/media/video/stradis.c
+++ b/drivers/media/video/stradis.c
@@ -1925,6 +1925,7 @@ static struct video_device saa_template = {
         .name = "SAA7146A",
         .fops = &saa_fops,
         .minor = -1,
+        .release = video_device_release_empty,
 };
 
 static int __devinit configure_saa7146(struct pci_dev *pdev, int num)
diff --git a/drivers/media/video/usbvideo/usbvideo.c b/drivers/media/video/usbvideo/usbvideo.c
index 6b1b2003a65c..9569e8ced4b4 100644
--- a/drivers/media/video/usbvideo/usbvideo.c
+++ b/drivers/media/video/usbvideo/usbvideo.c
@@ -1006,10 +1006,6 @@ allocate_done:
 
 EXPORT_SYMBOL(usbvideo_AllocateDevice);
 
-static void usbvideo_dummy_release(struct video_device *vfd)
-{
-}
-
 int usbvideo_RegisterVideoDevice(struct uvd *uvd)
 {
         char tmp1[20], tmp2[20];        /* Buffers for printing */
@@ -1043,7 +1039,7 @@ int usbvideo_RegisterVideoDevice(struct uvd *uvd)
                 return -EINVAL;
         }
         uvd->vdev.parent = &uvd->dev->dev;
-        uvd->vdev.release = usbvideo_dummy_release;
+        uvd->vdev.release = video_device_release_empty;
         if (video_register_device(&uvd->vdev, VFL_TYPE_GRABBER, video_nr) < 0) {
                 err("%s: video_register_device failed", __func__);
                 return -EPIPE;
diff --git a/drivers/media/video/usbvideo/vicam.c b/drivers/media/video/usbvideo/vicam.c
index efb878a7402e..d7728256f295 100644
--- a/drivers/media/video/usbvideo/vicam.c
+++ b/drivers/media/video/usbvideo/vicam.c
@@ -801,6 +801,7 @@ static struct video_device vicam_template = {
         .name           = "ViCam-based USB Camera",
         .fops           = &vicam_fops,
         .minor          = -1,
+        .release        = video_device_release_empty,
 };
 
 /* table of devices that work with this driver */
diff --git a/drivers/media/video/w9966.c b/drivers/media/video/w9966.c
index c635cffb1fb8..e3580119640f 100644
--- a/drivers/media/video/w9966.c
+++ b/drivers/media/video/w9966.c
@@ -216,6 +216,7 @@ static const struct file_operations w9966_fops = {
 static struct video_device w9966_template = {
         .name           = W9966_DRIVERNAME,
         .fops           = &w9966_fops,
+        .release        = video_device_release_empty,
 };
 
 /*