V4L/DVB (10884): radio-gemtek: convert to v4l2_device.
authorHans Verkuil <hverkuil@xs4all.nl>
Fri, 6 Mar 2009 16:50:42 +0000 (13:50 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 30 Mar 2009 15:43:12 +0000 (12:43 -0300)
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/radio/radio-gemtek.c

index 2b68be7..91448c4 100644 (file)
 #include <linux/init.h>                /* Initdata                     */
 #include <linux/ioport.h>      /* request_region               */
 #include <linux/delay.h>       /* udelay                       */
-#include <asm/io.h>            /* outb, outb_p                 */
-#include <asm/uaccess.h>       /* copy to/from user            */
 #include <linux/videodev2.h>   /* kernel radio structs         */
+#include <linux/version.h>     /* for KERNEL_VERSION MACRO     */
+#include <linux/mutex.h>
+#include <linux/io.h>          /* outb, outb_p                 */
+#include <linux/uaccess.h>     /* copy to/from user            */
 #include <media/v4l2-ioctl.h>
-#include <media/v4l2-common.h>
-#include <linux/spinlock.h>
+#include <media/v4l2-device.h>
 
-#include <linux/version.h>     /* for KERNEL_VERSION MACRO     */
-#define RADIO_VERSION KERNEL_VERSION(0,0,3)
-#define RADIO_BANNER "GemTek Radio card driver: v0.0.3"
+#define RADIO_VERSION KERNEL_VERSION(0, 0, 3)
 
 /*
  * Module info.
@@ -57,7 +56,6 @@ static int shutdown   = 1;
 static int keepmuted   = 1;
 static int initmute    = 1;
 static int radio_nr    = -1;
-static unsigned long in_use;
 
 module_param(io, int, 0444);
 MODULE_PARM_DESC(io, "Force I/O port for the GemTek Radio card if automatic "
@@ -112,12 +110,19 @@ module_param(radio_nr, int, 0444);
 #define SHORT_DELAY 5          /* usec */
 #define LONG_DELAY 75          /* usec */
 
-struct gemtek_device {
+struct gemtek {
+       struct v4l2_device v4l2_dev;
+       struct video_device vdev;
+       struct mutex lock;
        unsigned long lastfreq;
        int muted;
+       int verified;
+       int io;
        u32 bu2614data;
 };
 
+static struct gemtek gemtek_card;
+
 #define BU2614_FREQ_BITS       16 /* D0..D15, Frequency data           */
 #define BU2614_PORT_BITS       3 /* P0..P2, Output port control data   */
 #define BU2614_VOID_BITS       4 /* unused                             */
@@ -153,10 +158,6 @@ struct gemtek_device {
 #define BU2614_FMUN_MASK       MKMASK(FMUN)
 #define BU2614_TEST_MASK       MKMASK(TEST)
 
-static struct gemtek_device gemtek_unit;
-
-static spinlock_t lock;
-
 /*
  * Set data which will be sent to BU2614FS.
  */
@@ -166,33 +167,33 @@ static spinlock_t lock;
 /*
  * Transmit settings to BU2614FS over GemTek IC.
  */
-static void gemtek_bu2614_transmit(struct gemtek_device *dev)
+static void gemtek_bu2614_transmit(struct gemtek *gt)
 {
        int i, bit, q, mute;
 
-       spin_lock(&lock);
+       mutex_lock(&gt->lock);
 
-       mute = dev->muted ? GEMTEK_MT : 0x00;
+       mute = gt->muted ? GEMTEK_MT : 0x00;
 
-       outb_p(mute | GEMTEK_DA | GEMTEK_CK, io);
+       outb_p(mute | GEMTEK_DA | GEMTEK_CK, gt->io);
        udelay(SHORT_DELAY);
-       outb_p(mute | GEMTEK_CE | GEMTEK_DA | GEMTEK_CK, io);
+       outb_p(mute | GEMTEK_CE | GEMTEK_DA | GEMTEK_CK, gt->io);
        udelay(LONG_DELAY);
 
-       for (i = 0, q = dev->bu2614data; i < 32; i++, q >>= 1) {
-           bit = (q & 1) ? GEMTEK_DA : 0;
-           outb_p(mute | GEMTEK_CE | bit, io);
-           udelay(SHORT_DELAY);
-           outb_p(mute | GEMTEK_CE | bit | GEMTEK_CK, io);
-           udelay(SHORT_DELAY);
+       for (i = 0, q = gt->bu2614data; i < 32; i++, q >>= 1) {
+               bit = (q & 1) ? GEMTEK_DA : 0;
+               outb_p(mute | GEMTEK_CE | bit, gt->io);
+               udelay(SHORT_DELAY);
+               outb_p(mute | GEMTEK_CE | bit | GEMTEK_CK, gt->io);
+               udelay(SHORT_DELAY);
        }
 
-       outb_p(mute | GEMTEK_DA | GEMTEK_CK, io);
+       outb_p(mute | GEMTEK_DA | GEMTEK_CK, gt->io);
        udelay(SHORT_DELAY);
-       outb_p(mute | GEMTEK_CE | GEMTEK_DA | GEMTEK_CK, io);
+       outb_p(mute | GEMTEK_CE | GEMTEK_DA | GEMTEK_CK, gt->io);
        udelay(LONG_DELAY);
 
-       spin_unlock(&lock);
+       mutex_unlock(&gt->lock);
 }
 
 /*
@@ -206,107 +207,109 @@ static unsigned long gemtek_convfreq(unsigned long freq)
 /*
  * Set FM-frequency.
  */
-static void gemtek_setfreq(struct gemtek_device *dev, unsigned long freq)
+static void gemtek_setfreq(struct gemtek *gt, unsigned long freq)
 {
-
-       if (keepmuted && hardmute && dev->muted)
+       if (keepmuted && hardmute && gt->muted)
                return;
 
-       if (freq < GEMTEK_LOWFREQ)
-               freq = GEMTEK_LOWFREQ;
-       else if (freq > GEMTEK_HIGHFREQ)
-               freq = GEMTEK_HIGHFREQ;
+       freq = clamp_val(freq, GEMTEK_LOWFREQ, GEMTEK_HIGHFREQ);
 
-       dev->lastfreq = freq;
-       dev->muted = 0;
+       gt->lastfreq = freq;
+       gt->muted = 0;
 
-       gemtek_bu2614_set(dev, BU2614_PORT, 0);
-       gemtek_bu2614_set(dev, BU2614_FMES, 0);
-       gemtek_bu2614_set(dev, BU2614_SWIN, 0); /* FM-mode      */
-       gemtek_bu2614_set(dev, BU2614_SWAL, 0);
-       gemtek_bu2614_set(dev, BU2614_FMUN, 1); /* GT bit set   */
-       gemtek_bu2614_set(dev, BU2614_TEST, 0);
+       gemtek_bu2614_set(gt, BU2614_PORT, 0);
+       gemtek_bu2614_set(gt, BU2614_FMES, 0);
+       gemtek_bu2614_set(gt, BU2614_SWIN, 0);  /* FM-mode      */
+       gemtek_bu2614_set(gt, BU2614_SWAL, 0);
+       gemtek_bu2614_set(gt, BU2614_FMUN, 1);  /* GT bit set   */
+       gemtek_bu2614_set(gt, BU2614_TEST, 0);
 
-       gemtek_bu2614_set(dev, BU2614_STDF, GEMTEK_STDF_3_125_KHZ);
-       gemtek_bu2614_set(dev, BU2614_FREQ, gemtek_convfreq(freq));
+       gemtek_bu2614_set(gt, BU2614_STDF, GEMTEK_STDF_3_125_KHZ);
+       gemtek_bu2614_set(gt, BU2614_FREQ, gemtek_convfreq(freq));
 
-       gemtek_bu2614_transmit(dev);
+       gemtek_bu2614_transmit(gt);
 }
 
 /*
  * Set mute flag.
  */
-static void gemtek_mute(struct gemtek_device *dev)
+static void gemtek_mute(struct gemtek *gt)
 {
        int i;
-       dev->muted = 1;
+
+       gt->muted = 1;
 
        if (hardmute) {
                /* Turn off PLL, disable data output */
-               gemtek_bu2614_set(dev, BU2614_PORT, 0);
-               gemtek_bu2614_set(dev, BU2614_FMES, 0); /* CT bit off   */
-               gemtek_bu2614_set(dev, BU2614_SWIN, 0); /* FM-mode      */
-               gemtek_bu2614_set(dev, BU2614_SWAL, 0);
-               gemtek_bu2614_set(dev, BU2614_FMUN, 0); /* GT bit off   */
-               gemtek_bu2614_set(dev, BU2614_TEST, 0);
-               gemtek_bu2614_set(dev, BU2614_STDF, GEMTEK_PLL_OFF);
-               gemtek_bu2614_set(dev, BU2614_FREQ, 0);
-               gemtek_bu2614_transmit(dev);
-       } else {
-               spin_lock(&lock);
+               gemtek_bu2614_set(gt, BU2614_PORT, 0);
+               gemtek_bu2614_set(gt, BU2614_FMES, 0);  /* CT bit off   */
+               gemtek_bu2614_set(gt, BU2614_SWIN, 0);  /* FM-mode      */
+               gemtek_bu2614_set(gt, BU2614_SWAL, 0);
+               gemtek_bu2614_set(gt, BU2614_FMUN, 0);  /* GT bit off   */
+               gemtek_bu2614_set(gt, BU2614_TEST, 0);
+               gemtek_bu2614_set(gt, BU2614_STDF, GEMTEK_PLL_OFF);
+               gemtek_bu2614_set(gt, BU2614_FREQ, 0);
+               gemtek_bu2614_transmit(gt);
+               return;
+       }
 
-               /* Read bus contents (CE, CK and DA). */
-               i = inb_p(io);
-               /* Write it back with mute flag set. */
-               outb_p((i >> 5) | GEMTEK_MT, io);
-               udelay(SHORT_DELAY);
+       mutex_lock(&gt->lock);
 
-               spin_unlock(&lock);
-       }
+       /* Read bus contents (CE, CK and DA). */
+       i = inb_p(gt->io);
+       /* Write it back with mute flag set. */
+       outb_p((i >> 5) | GEMTEK_MT, gt->io);
+       udelay(SHORT_DELAY);
+
+       mutex_unlock(&gt->lock);
 }
 
 /*
  * Unset mute flag.
  */
-static void gemtek_unmute(struct gemtek_device *dev)
+static void gemtek_unmute(struct gemtek *gt)
 {
        int i;
-       dev->muted = 0;
 
+       gt->muted = 0;
        if (hardmute) {
                /* Turn PLL back on. */
-               gemtek_setfreq(dev, dev->lastfreq);
-       } else {
-               spin_lock(&lock);
+               gemtek_setfreq(gt, gt->lastfreq);
+               return;
+       }
+       mutex_lock(&gt->lock);
 
-               i = inb_p(io);
-               outb_p(i >> 5, io);
-               udelay(SHORT_DELAY);
+       i = inb_p(gt->io);
+       outb_p(i >> 5, gt->io);
+       udelay(SHORT_DELAY);
 
-               spin_unlock(&lock);
-       }
+       mutex_unlock(&gt->lock);
 }
 
 /*
  * Get signal strength (= stereo status).
  */
-static inline int gemtek_getsigstr(void)
+static inline int gemtek_getsigstr(struct gemtek *gt)
 {
-       return inb_p(io) & GEMTEK_NS ? 0 : 1;
+       int sig;
+
+       mutex_lock(&gt->lock);
+       sig = inb_p(gt->io) & GEMTEK_NS ? 0 : 1;
+       mutex_unlock(&gt->lock);
+       return sig;
 }
 
 /*
  * Check if requested card acts like GemTek Radio card.
  */
-static int gemtek_verify(int port)
+static int gemtek_verify(struct gemtek *gt, int port)
 {
-       static int verified = -1;
        int i, q;
 
-       if (verified == port)
+       if (gt->verified == port)
                return 1;
 
-       spin_lock(&lock);
+       mutex_lock(&gt->lock);
 
        q = inb_p(port);        /* Read bus contents before probing. */
        /* Try to turn on CE, CK and DA respectively and check if card responds
@@ -316,15 +319,15 @@ static int gemtek_verify(int port)
                udelay(SHORT_DELAY);
 
                if ((inb_p(port) & (~GEMTEK_NS)) != (0x17 | (1 << (i + 5)))) {
-                       spin_unlock(&lock);
+                       mutex_unlock(&gt->lock);
                        return 0;
                }
        }
        outb_p(q >> 5, port);   /* Write bus contents back. */
        udelay(SHORT_DELAY);
 
-       spin_unlock(&lock);
-       verified = port;
+       mutex_unlock(&gt->lock);
+       gt->verified = port;
 
        return 1;
 }
@@ -332,83 +335,61 @@ static int gemtek_verify(int port)
 /*
  * Automatic probing for card.
  */
-static int gemtek_probe(void)
+static int gemtek_probe(struct gemtek *gt)
 {
+       struct v4l2_device *v4l2_dev = &gt->v4l2_dev;
        int ioports[] = { 0x20c, 0x30c, 0x24c, 0x34c, 0x248, 0x28c };
        int i;
 
        if (!probe) {
-               printk(KERN_INFO "Automatic device probing disabled.\n");
+               v4l2_info(v4l2_dev, "Automatic device probing disabled.\n");
                return -1;
        }
 
-       printk(KERN_INFO "Automatic device probing enabled.\n");
+       v4l2_info(v4l2_dev, "Automatic device probing enabled.\n");
 
        for (i = 0; i < ARRAY_SIZE(ioports); ++i) {
-               printk(KERN_INFO "Trying I/O port 0x%x...\n", ioports[i]);
+               v4l2_info(v4l2_dev, "Trying I/O port 0x%x...\n", ioports[i]);
 
                if (!request_region(ioports[i], 1, "gemtek-probe")) {
-                       printk(KERN_WARNING "I/O port 0x%x busy!\n",
+                       v4l2_warn(v4l2_dev, "I/O port 0x%x busy!\n",
                               ioports[i]);
                        continue;
                }
 
-               if (gemtek_verify(ioports[i])) {
-                       printk(KERN_INFO "Card found from I/O port "
+               if (gemtek_verify(gt, ioports[i])) {
+                       v4l2_info(v4l2_dev, "Card found from I/O port "
                               "0x%x!\n", ioports[i]);
 
                        release_region(ioports[i], 1);
-
-                       io = ioports[i];
-                       return io;
+                       gt->io = ioports[i];
+                       return gt->io;
                }
 
                release_region(ioports[i], 1);
        }
 
-       printk(KERN_ERR "Automatic probing failed!\n");
-
+       v4l2_err(v4l2_dev, "Automatic probing failed!\n");
        return -1;
 }
 
 /*
  * Video 4 Linux stuff.
  */
-
-static struct v4l2_queryctrl radio_qctrl[] = {
-       {
-               .id = V4L2_CID_AUDIO_MUTE,
-               .name = "Mute",
-               .minimum = 0,
-               .maximum = 1,
-               .default_value = 1,
-               .type = V4L2_CTRL_TYPE_BOOLEAN,
-       }, {
-               .id = V4L2_CID_AUDIO_VOLUME,
-               .name = "Volume",
-               .minimum = 0,
-               .maximum = 65535,
-               .step = 65535,
-               .default_value = 0xff,
-               .type = V4L2_CTRL_TYPE_INTEGER,
-       }
-};
-
-static int gemtek_exclusive_open(struct file *file)
+static int gemtek_open(struct file *file)
 {
-       return test_and_set_bit(0, &in_use) ? -EBUSY : 0;
+       return 0;
 }
 
-static int gemtek_exclusive_release(struct file *file)
+static int gemtek_release(struct file *file)
 {
-       clear_bit(0, &in_use);
        return 0;
 }
 
 static const struct v4l2_file_operations gemtek_fops = {
        .owner          = THIS_MODULE,
-       .open           = gemtek_exclusive_open,
-       .release        = gemtek_exclusive_release,
+       .open           = gemtek_open,
+       .release        = gemtek_release,
        .ioctl          = video_ioctl2,
 };
 
@@ -417,23 +398,25 @@ static int vidioc_querycap(struct file *file, void *priv,
 {
        strlcpy(v->driver, "radio-gemtek", sizeof(v->driver));
        strlcpy(v->card, "GemTek", sizeof(v->card));
-       sprintf(v->bus_info, "ISA");
+       strlcpy(v->bus_info, "ISA", sizeof(v->bus_info));
        v->version = RADIO_VERSION;
-       v->capabilities = V4L2_CAP_TUNER;
+       v->capabilities = V4L2_CAP_TUNER | V4L2_CAP_RADIO;
        return 0;
 }
 
 static int vidioc_g_tuner(struct file *file, void *priv, struct v4l2_tuner *v)
 {
+       struct gemtek *gt = video_drvdata(file);
+
        if (v->index > 0)
                return -EINVAL;
 
-       strcpy(v->name, "FM");
+       strlcpy(v->name, "FM", sizeof(v->name));
        v->type = V4L2_TUNER_RADIO;
        v->rangelow = GEMTEK_LOWFREQ;
        v->rangehigh = GEMTEK_HIGHFREQ;
        v->capability = V4L2_TUNER_CAP_LOW | V4L2_TUNER_CAP_STEREO;
-       v->signal = 0xffff * gemtek_getsigstr();
+       v->signal = 0xffff * gemtek_getsigstr(gt);
        if (v->signal) {
                v->audmode = V4L2_TUNER_MODE_STEREO;
                v->rxsubchans = V4L2_TUNER_SUB_STEREO;
@@ -441,65 +424,56 @@ static int vidioc_g_tuner(struct file *file, void *priv, struct v4l2_tuner *v)
                v->audmode = V4L2_TUNER_MODE_MONO;
                v->rxsubchans = V4L2_TUNER_SUB_MONO;
        }
-
        return 0;
 }
 
 static int vidioc_s_tuner(struct file *file, void *priv, struct v4l2_tuner *v)
 {
-       if (v->index > 0)
-               return -EINVAL;
-       return 0;
+       return (v->index != 0) ? -EINVAL : 0;
 }
 
-static int vidioc_s_frequency(struct file *file, void *priv,
+static int vidioc_g_frequency(struct file *file, void *priv,
                              struct v4l2_frequency *f)
 {
-       struct gemtek_device *rt = video_drvdata(file);
-
-       gemtek_setfreq(rt, f->frequency);
+       struct gemtek *gt = video_drvdata(file);
 
+       if (f->tuner != 0)
+               return -EINVAL;
+       f->type = V4L2_TUNER_RADIO;
+       f->frequency = gt->lastfreq;
        return 0;
 }
 
-static int vidioc_g_frequency(struct file *file, void *priv,
+static int vidioc_s_frequency(struct file *file, void *priv,
                              struct v4l2_frequency *f)
 {
-       struct gemtek_device *rt = video_drvdata(file);
+       struct gemtek *gt = video_drvdata(file);
 
-       f->type = V4L2_TUNER_RADIO;
-       f->frequency = rt->lastfreq;
+       if (f->tuner != 0 || f->type != V4L2_TUNER_RADIO)
+               return -EINVAL;
+       gemtek_setfreq(gt, f->frequency);
        return 0;
 }
 
 static int vidioc_queryctrl(struct file *file, void *priv,
                            struct v4l2_queryctrl *qc)
 {
-       int i;
-
-       for (i = 0; i < ARRAY_SIZE(radio_qctrl); ++i) {
-               if (qc->id && qc->id == radio_qctrl[i].id) {
-                       memcpy(qc, &(radio_qctrl[i]), sizeof(*qc));
-                       return 0;
-               }
+       switch (qc->id) {
+       case V4L2_CID_AUDIO_MUTE:
+               return v4l2_ctrl_query_fill(qc, 0, 1, 1, 0);
+       default:
+               return -EINVAL;
        }
-       return -EINVAL;
 }
 
 static int vidioc_g_ctrl(struct file *file, void *priv,
                         struct v4l2_control *ctrl)
 {
-       struct gemtek_device *rt = video_drvdata(file);
+       struct gemtek *gt = video_drvdata(file);
 
        switch (ctrl->id) {
        case V4L2_CID_AUDIO_MUTE:
-               ctrl->value = rt->muted;
-               return 0;
-       case V4L2_CID_AUDIO_VOLUME:
-               if (rt->muted)
-                       ctrl->value = 0;
-               else
-                       ctrl->value = 65535;
+               ctrl->value = gt->muted;
                return 0;
        }
        return -EINVAL;
@@ -508,35 +482,19 @@ 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 gemtek_device *rt = video_drvdata(file);
+       struct gemtek *gt = video_drvdata(file);
 
        switch (ctrl->id) {
        case V4L2_CID_AUDIO_MUTE:
                if (ctrl->value)
-                       gemtek_mute(rt);
-               else
-                       gemtek_unmute(rt);
-               return 0;
-       case V4L2_CID_AUDIO_VOLUME:
-               if (ctrl->value)
-                       gemtek_unmute(rt);
+                       gemtek_mute(gt);
                else
-                       gemtek_mute(rt);
+                       gemtek_unmute(gt);
                return 0;
        }
        return -EINVAL;
 }
 
-static int vidioc_g_audio(struct file *file, void *priv, struct v4l2_audio *a)
-{
-       if (a->index > 1)
-               return -EINVAL;
-
-       strcpy(a->name, "Radio");
-       a->capability = V4L2_AUDCAP_STEREO;
-       return 0;
-}
-
 static int vidioc_g_input(struct file *filp, void *priv, unsigned int *i)
 {
        *i = 0;
@@ -545,16 +503,20 @@ static int vidioc_g_input(struct file *filp, void *priv, unsigned int *i)
 
 static int vidioc_s_input(struct file *filp, void *priv, unsigned int i)
 {
-       if (i != 0)
-               return -EINVAL;
+       return (i != 0) ? -EINVAL : 0;
+}
+
+static int vidioc_g_audio(struct file *file, void *priv, struct v4l2_audio *a)
+{
+       a->index = 0;
+       strlcpy(a->name, "Radio", sizeof(a->name));
+       a->capability = V4L2_AUDCAP_STEREO;
        return 0;
 }
 
 static int vidioc_s_audio(struct file *file, void *priv, struct v4l2_audio *a)
 {
-       if (a->index != 0)
-               return -EINVAL;
-       return 0;
+       return (a->index != 0) ? -EINVAL : 0;
 }
 
 static const struct v4l2_ioctl_ops gemtek_ioctl_ops = {
@@ -572,62 +534,73 @@ static const struct v4l2_ioctl_ops gemtek_ioctl_ops = {
        .vidioc_s_ctrl          = vidioc_s_ctrl
 };
 
-static struct video_device gemtek_radio = {
-       .name           = "GemTek Radio card",
-       .fops           = &gemtek_fops,
-       .ioctl_ops      = &gemtek_ioctl_ops,
-       .release        = video_device_release_empty,
-};
-
 /*
  * Initialization / cleanup related stuff.
  */
 
-/*
- * Initilize card.
- */
 static int __init gemtek_init(void)
 {
-       printk(KERN_INFO RADIO_BANNER "\n");
+       struct gemtek *gt = &gemtek_card;
+       struct v4l2_device *v4l2_dev = &gt->v4l2_dev;
+       int res;
 
-       spin_lock_init(&lock);
+       strlcpy(v4l2_dev->name, "gemtek", sizeof(v4l2_dev->name));
 
-       gemtek_probe();
-       if (io) {
-               if (!request_region(io, 1, "gemtek")) {
-                       printk(KERN_ERR "I/O port 0x%x already in use.\n", io);
+       v4l2_info(v4l2_dev, "GemTek Radio card driver: v0.0.3\n");
+
+       mutex_init(&gt->lock);
+
+       gt->verified = -1;
+       gt->io = io;
+       gemtek_probe(gt);
+       if (gt->io) {
+               if (!request_region(gt->io, 1, "gemtek")) {
+                       v4l2_err(v4l2_dev, "I/O port 0x%x already in use.\n", gt->io);
                        return -EBUSY;
                }
 
-               if (!gemtek_verify(io))
-                       printk(KERN_WARNING "Card at I/O port 0x%x does not "
+               if (!gemtek_verify(gt, gt->io))
+                       v4l2_warn(v4l2_dev, "Card at I/O port 0x%x does not "
                               "respond properly, check your "
-                              "configuration.\n", io);
+                              "configuration.\n", gt->io);
                else
-                       printk(KERN_INFO "Using I/O port 0x%x.\n", io);
+                       v4l2_info(v4l2_dev, "Using I/O port 0x%x.\n", gt->io);
        } else if (probe) {
-               printk(KERN_ERR "Automatic probing failed and no "
+               v4l2_err(v4l2_dev, "Automatic probing failed and no "
                       "fixed I/O port defined.\n");
                return -ENODEV;
        } else {
-               printk(KERN_ERR "Automatic probing disabled but no fixed "
+               v4l2_err(v4l2_dev, "Automatic probing disabled but no fixed "
                       "I/O port defined.");
                return -EINVAL;
        }
 
-       video_set_drvdata(&gemtek_radio, &gemtek_unit);
+       res = v4l2_device_register(NULL, v4l2_dev);
+       if (res < 0) {
+               v4l2_err(v4l2_dev, "Could not register v4l2_device\n");
+               release_region(gt->io, 1);
+               return res;
+       }
 
-       if (video_register_device(&gemtek_radio, VFL_TYPE_RADIO, radio_nr) < 0) {
-               release_region(io, 1);
+       strlcpy(gt->vdev.name, v4l2_dev->name, sizeof(gt->vdev.name));
+       gt->vdev.v4l2_dev = v4l2_dev;
+       gt->vdev.fops = &gemtek_fops;
+       gt->vdev.ioctl_ops = &gemtek_ioctl_ops;
+       gt->vdev.release = video_device_release_empty;
+       video_set_drvdata(&gt->vdev, gt);
+
+       if (video_register_device(&gt->vdev, VFL_TYPE_RADIO, radio_nr) < 0) {
+               v4l2_device_unregister(v4l2_dev);
+               release_region(gt->io, 1);
                return -EBUSY;
        }
 
        /* Set defaults */
-       gemtek_unit.lastfreq = GEMTEK_LOWFREQ;
-       gemtek_unit.bu2614data = 0;
+       gt->lastfreq = GEMTEK_LOWFREQ;
+       gt->bu2614data = 0;
 
        if (initmute)
-               gemtek_mute(&gemtek_unit);
+               gemtek_mute(gt);
 
        return 0;
 }
@@ -637,15 +610,19 @@ static int __init gemtek_init(void)
  */
 static void __exit gemtek_exit(void)
 {
+       struct gemtek *gt = &gemtek_card;
+       struct v4l2_device *v4l2_dev = &gt->v4l2_dev;
+
        if (shutdown) {
                hardmute = 1;   /* Turn off PLL */
-               gemtek_mute(&gemtek_unit);
+               gemtek_mute(gt);
        } else {
-               printk(KERN_INFO "Module unloaded but card not muted!\n");
+               v4l2_info(v4l2_dev, "Module unloaded but card not muted!\n");
        }
 
-       video_unregister_device(&gemtek_radio);
-       release_region(io, 1);
+       video_unregister_device(&gt->vdev);
+       v4l2_device_unregister(&gt->v4l2_dev);
+       release_region(gt->io, 1);
 }
 
 module_init(gemtek_init);