Merge branch 'omap-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tmlind...
[pandora-kernel.git] / drivers / media / video / w9966.c
index 25364b8..635420d 100644 (file)
@@ -1,7 +1,7 @@
 /*
        Winbond w9966cf Webcam parport driver.
 
-       Version 0.32
+       Version 0.33
 
        Copyright (C) 2001 Jakob Kemi <jakob.kemi@post.utfors.se>
 
 #include <linux/module.h>
 #include <linux/init.h>
 #include <linux/delay.h>
-#include <linux/videodev.h>
+#include <linux/version.h>
+#include <linux/videodev2.h>
 #include <linux/slab.h>
 #include <media/v4l2-common.h>
 #include <media/v4l2-ioctl.h>
+#include <media/v4l2-device.h>
 #include <linux/parport.h>
 
 /*#define DEBUG*/                              /* Undef me for production */
 #define W9966_I2C_W_DATA       0x02
 #define W9966_I2C_W_CLOCK      0x01
 
-struct w9966_dev {
+struct w9966 {
+       struct v4l2_device v4l2_dev;
        unsigned char dev_state;
        unsigned char i2c_state;
        unsigned short ppmode;
@@ -114,7 +117,7 @@ struct w9966_dev {
        signed char contrast;
        signed char color;
        signed char hue;
-       unsigned long in_use;
+       struct mutex lock;
 };
 
 /*
@@ -133,9 +136,9 @@ static const char *pardev[] = {[0 ... W9966_MAXCAMS] = "aggressive"};
 #endif
 module_param_array(pardev, charp, NULL, 0);
 MODULE_PARM_DESC(pardev, "pardev: where to search for\n"
-                        "\teach camera. 'aggressive' means brute-force search.\n"
-                        "\tEg: >pardev=parport3,aggressive,parport2,parport1< would assign\n"
-                        "\tcam 1 to parport3 and search every parport for cam 2 etc...");
+               "\teach camera. 'aggressive' means brute-force search.\n"
+               "\tEg: >pardev=parport3,aggressive,parport2,parport1< would assign\n"
+               "\tcam 1 to parport3 and search every parport for cam 2 etc...");
 
 static int parmode;
 module_param(parmode, int, 0);
@@ -144,75 +147,7 @@ MODULE_PARM_DESC(parmode, "parmode: transfer mode (0=auto, 1=ecp, 2=epp");
 static int video_nr = -1;
 module_param(video_nr, int, 0);
 
-/*
- *     Private data
- */
-
-static struct w9966_dev w9966_cams[W9966_MAXCAMS];
-
-/*
- *     Private function declares
- */
-
-static inline void w9966_setState(struct w9966_dev *cam, int mask, int val);
-static inline int  w9966_getState(struct w9966_dev *cam, int mask, int val);
-static inline void w9966_pdev_claim(struct w9966_dev *vdev);
-static inline void w9966_pdev_release(struct w9966_dev *vdev);
-
-static int w9966_rReg(struct w9966_dev *cam, int reg);
-static int w9966_wReg(struct w9966_dev *cam, int reg, int data);
-#if 0
-static int w9966_rReg_i2c(struct w9966_dev *cam, int reg);
-#endif
-static int w9966_wReg_i2c(struct w9966_dev *cam, int reg, int data);
-static int w9966_findlen(int near, int size, int maxlen);
-static int w9966_calcscale(int size, int min, int max, int *beg, int *end, unsigned char *factor);
-static int w9966_setup(struct w9966_dev *cam, int x1, int y1, int x2, int y2, int w, int h);
-
-static int  w9966_init(struct w9966_dev *cam, struct parport* port);
-static void w9966_term(struct w9966_dev *cam);
-
-static inline void w9966_i2c_setsda(struct w9966_dev *cam, int state);
-static inline int  w9966_i2c_setscl(struct w9966_dev *cam, int state);
-static inline int  w9966_i2c_getsda(struct w9966_dev *cam);
-static inline int  w9966_i2c_getscl(struct w9966_dev *cam);
-static int w9966_i2c_wbyte(struct w9966_dev *cam, int data);
-#if 0
-static int w9966_i2c_rbyte(struct w9966_dev *cam);
-#endif
-
-static long w9966_v4l_ioctl(struct file *file,
-                          unsigned int cmd, unsigned long arg);
-static ssize_t w9966_v4l_read(struct file *file, char __user *buf,
-                             size_t count, loff_t *ppos);
-
-static int w9966_exclusive_open(struct file *file)
-{
-       struct w9966_dev *cam = video_drvdata(file);
-
-       return test_and_set_bit(0, &cam->in_use) ? -EBUSY : 0;
-}
-
-static int w9966_exclusive_release(struct file *file)
-{
-       struct w9966_dev *cam = video_drvdata(file);
-
-       clear_bit(0, &cam->in_use);
-       return 0;
-}
-
-static const struct v4l2_file_operations w9966_fops = {
-       .owner          = THIS_MODULE,
-       .open           = w9966_exclusive_open,
-       .release        = w9966_exclusive_release,
-       .ioctl          = w9966_v4l_ioctl,
-       .read           = w9966_v4l_read,
-};
-static struct video_device w9966_template = {
-       .name           = W9966_DRIVERNAME,
-       .fops           = &w9966_fops,
-       .release        = video_device_release_empty,
-};
+static struct w9966 w9966_cams[W9966_MAXCAMS];
 
 /*
  *     Private function defines
@@ -220,39 +155,39 @@ static struct video_device w9966_template = {
 
 
 /* Set camera phase flags, so we know what to uninit when terminating */
-static inline void w9966_setState(struct w9966_dev *cam, int mask, int val)
+static inline void w9966_set_state(struct w9966 *cam, int mask, int val)
 {
        cam->dev_state = (cam->dev_state & ~mask) ^ val;
 }
 
 /* Get camera phase flags */
-static inline int w9966_getState(struct w9966_dev *cam, int mask, int val)
+static inline int w9966_get_state(struct w9966 *cam, int mask, int val)
 {
        return ((cam->dev_state & mask) == val);
 }
 
 /* Claim parport for ourself */
-static inline void w9966_pdev_claim(struct w9966_dev *cam)
+static void w9966_pdev_claim(struct w9966 *cam)
 {
-       if (w9966_getState(cam, W9966_STATE_CLAIMED, W9966_STATE_CLAIMED))
+       if (w9966_get_state(cam, W9966_STATE_CLAIMED, W9966_STATE_CLAIMED))
                return;
        parport_claim_or_block(cam->pdev);
-       w9966_setState(cam, W9966_STATE_CLAIMED, W9966_STATE_CLAIMED);
+       w9966_set_state(cam, W9966_STATE_CLAIMED, W9966_STATE_CLAIMED);
 }
 
 /* Release parport for others to use */
-static inline void w9966_pdev_release(struct w9966_dev *cam)
+static void w9966_pdev_release(struct w9966 *cam)
 {
-       if (w9966_getState(cam, W9966_STATE_CLAIMED, 0))
+       if (w9966_get_state(cam, W9966_STATE_CLAIMED, 0))
                return;
        parport_release(cam->pdev);
-       w9966_setState(cam, W9966_STATE_CLAIMED, 0);
+       w9966_set_state(cam, W9966_STATE_CLAIMED, 0);
 }
 
 /* Read register from W9966 interface-chip
    Expects a claimed pdev
    -1 on error, else register data (byte) */
-static int w9966_rReg(struct w9966_dev *cam, int reg)
+static int w9966_read_reg(struct w9966 *cam, int reg)
 {
        /* ECP, read, regtransfer, REG, REG, REG, REG, REG */
        const unsigned char addr = 0x80 | (reg & 0x1f);
@@ -273,7 +208,7 @@ static int w9966_rReg(struct w9966_dev *cam, int reg)
 /* Write register to W9966 interface-chip
    Expects a claimed pdev
    -1 on error */
-static int w9966_wReg(struct w9966_dev *cam, int reg, int data)
+static int w9966_write_reg(struct w9966 *cam, int reg, int data)
 {
        /* ECP, write, regtransfer, REG, REG, REG, REG, REG */
        const unsigned char addr = 0xc0 | (reg & 0x1f);
@@ -291,97 +226,168 @@ static int w9966_wReg(struct w9966_dev *cam, int reg, int data)
        return 0;
 }
 
-/* Initialize camera device. Setup all internal flags, set a
-   default video mode, setup ccd-chip, register v4l device etc..
-   Also used for 'probing' of hardware.
-   -1 on error */
-static int w9966_init(struct w9966_dev *cam, struct parport* port)
+/*
+ *     Ugly and primitive i2c protocol functions
+ */
+
+/* Sets the data line on the i2c bus.
+   Expects a claimed pdev. */
+static void w9966_i2c_setsda(struct w9966 *cam, int state)
 {
-       if (cam->dev_state != 0)
-               return -1;
+       if (state)
+               cam->i2c_state |= W9966_I2C_W_DATA;
+       else
+               cam->i2c_state &= ~W9966_I2C_W_DATA;
 
-       cam->pport = port;
-       cam->brightness = 128;
-       cam->contrast = 64;
-       cam->color = 64;
-       cam->hue = 0;
+       w9966_write_reg(cam, 0x18, cam->i2c_state);
+       udelay(5);
+}
 
-/* Select requested transfer mode */
-       switch (parmode) {
-       default:        /* Auto-detect (priority: hw-ecp, hw-epp, sw-ecp) */
-       case 0:
-               if (port->modes & PARPORT_MODE_ECP)
-                       cam->ppmode = IEEE1284_MODE_ECP;
-               else if (port->modes & PARPORT_MODE_EPP)
-                       cam->ppmode = IEEE1284_MODE_EPP;
-               else
-                       cam->ppmode = IEEE1284_MODE_ECP;
-               break;
-       case 1:         /* hw- or sw-ecp */
-               cam->ppmode = IEEE1284_MODE_ECP;
-               break;
-       case 2:         /* hw- or sw-epp */
-               cam->ppmode = IEEE1284_MODE_EPP;
-       break;
+/* Get peripheral clock line
+   Expects a claimed pdev. */
+static int w9966_i2c_getscl(struct w9966 *cam)
+{
+       const unsigned char state = w9966_read_reg(cam, 0x18);
+       return ((state & W9966_I2C_R_CLOCK) > 0);
+}
+
+/* Sets the clock line on the i2c bus.
+   Expects a claimed pdev. -1 on error */
+static int w9966_i2c_setscl(struct w9966 *cam, int state)
+{
+       unsigned long timeout;
+
+       if (state)
+               cam->i2c_state |= W9966_I2C_W_CLOCK;
+       else
+               cam->i2c_state &= ~W9966_I2C_W_CLOCK;
+
+       w9966_write_reg(cam, 0x18, cam->i2c_state);
+       udelay(5);
+
+       /* we go to high, we also expect the peripheral to ack. */
+       if (state) {
+               timeout = jiffies + 100;
+               while (!w9966_i2c_getscl(cam)) {
+                       if (time_after(jiffies, timeout))
+                               return -1;
+               }
        }
+       return 0;
+}
 
-/* Tell the parport driver that we exists */
-       cam->pdev = parport_register_device(port, "w9966", NULL, NULL, NULL, 0, NULL);
-       if (cam->pdev == NULL) {
-               DPRINTF("parport_register_device() failed\n");
-               return -1;
+#if 0
+/* Get peripheral data line
+   Expects a claimed pdev. */
+static int w9966_i2c_getsda(struct w9966 *cam)
+{
+       const unsigned char state = w9966_read_reg(cam, 0x18);
+       return ((state & W9966_I2C_R_DATA) > 0);
+}
+#endif
+
+/* Write a byte with ack to the i2c bus.
+   Expects a claimed pdev. -1 on error */
+static int w9966_i2c_wbyte(struct w9966 *cam, int data)
+{
+       int i;
+
+       for (i = 7; i >= 0; i--) {
+               w9966_i2c_setsda(cam, (data >> i) & 0x01);
+
+               if (w9966_i2c_setscl(cam, 1) == -1)
+                       return -1;
+               w9966_i2c_setscl(cam, 0);
        }
-       w9966_setState(cam, W9966_STATE_PDEV, W9966_STATE_PDEV);
 
-       w9966_pdev_claim(cam);
+       w9966_i2c_setsda(cam, 1);
 
-/* Setup a default capture mode */
-       if (w9966_setup(cam, 0, 0, 1023, 1023, 200, 160) != 0) {
-               DPRINTF("w9966_setup() failed.\n");
+       if (w9966_i2c_setscl(cam, 1) == -1)
                return -1;
+       w9966_i2c_setscl(cam, 0);
+
+       return 0;
+}
+
+/* Read a data byte with ack from the i2c-bus
+   Expects a claimed pdev. -1 on error */
+#if 0
+static int w9966_i2c_rbyte(struct w9966 *cam)
+{
+       unsigned char data = 0x00;
+       int i;
+
+       w9966_i2c_setsda(cam, 1);
+
+       for (i = 0; i < 8; i++) {
+               if (w9966_i2c_setscl(cam, 1) == -1)
+                       return -1;
+               data = data << 1;
+               if (w9966_i2c_getsda(cam))
+                       data |= 0x01;
+
+               w9966_i2c_setscl(cam, 0);
        }
+       return data;
+}
+#endif
 
-       w9966_pdev_release(cam);
+/* Read a register from the i2c device.
+   Expects claimed pdev. -1 on error */
+#if 0
+static int w9966_read_reg_i2c(struct w9966 *cam, int reg)
+{
+       int data;
 
-/* Fill in the video_device struct and register us to v4l */
-       memcpy(&cam->vdev, &w9966_template, sizeof(struct video_device));
-       video_set_drvdata(&cam->vdev, cam);
+       w9966_i2c_setsda(cam, 0);
+       w9966_i2c_setscl(cam, 0);
 
-       if (video_register_device(&cam->vdev, VFL_TYPE_GRABBER, video_nr) < 0)
+       if (w9966_i2c_wbyte(cam, W9966_I2C_W_ID) == -1 ||
+           w9966_i2c_wbyte(cam, reg) == -1)
                return -1;
 
-       w9966_setState(cam, W9966_STATE_VDEV, W9966_STATE_VDEV);
+       w9966_i2c_setsda(cam, 1);
+       if (w9966_i2c_setscl(cam, 1) == -1)
+               return -1;
+       w9966_i2c_setsda(cam, 0);
+       w9966_i2c_setscl(cam, 0);
 
-       /* All ok */
-       printk(KERN_INFO "w9966cf: Found and initialized a webcam on %s.\n",
-               cam->pport->name);
-       return 0;
-}
+       if (w9966_i2c_wbyte(cam, W9966_I2C_R_ID) == -1)
+               return -1;
+       data = w9966_i2c_rbyte(cam);
+       if (data == -1)
+               return -1;
 
+       w9966_i2c_setsda(cam, 0);
 
-/* Terminate everything gracefully */
-static void w9966_term(struct w9966_dev *cam)
+       if (w9966_i2c_setscl(cam, 1) == -1)
+               return -1;
+       w9966_i2c_setsda(cam, 1);
+
+       return data;
+}
+#endif
+
+/* Write a register to the i2c device.
+   Expects claimed pdev. -1 on error */
+static int w9966_write_reg_i2c(struct w9966 *cam, int reg, int data)
 {
-/* Unregister from v4l */
-       if (w9966_getState(cam, W9966_STATE_VDEV, W9966_STATE_VDEV)) {
-               video_unregister_device(&cam->vdev);
-               w9966_setState(cam, W9966_STATE_VDEV, 0);
-       }
+       w9966_i2c_setsda(cam, 0);
+       w9966_i2c_setscl(cam, 0);
 
-/* Terminate from IEEE1284 mode and release pdev block */
-       if (w9966_getState(cam, W9966_STATE_PDEV, W9966_STATE_PDEV)) {
-               w9966_pdev_claim(cam);
-               parport_negotiate(cam->pport, IEEE1284_MODE_COMPAT);
-               w9966_pdev_release(cam);
-       }
+       if (w9966_i2c_wbyte(cam, W9966_I2C_W_ID) == -1 ||
+                       w9966_i2c_wbyte(cam, reg) == -1 ||
+                       w9966_i2c_wbyte(cam, data) == -1)
+               return -1;
 
-/* Unregister from parport */
-       if (w9966_getState(cam, W9966_STATE_PDEV, W9966_STATE_PDEV)) {
-               parport_unregister_device(cam->pdev);
-               w9966_setState(cam, W9966_STATE_PDEV, 0);
-       }
-}
+       w9966_i2c_setsda(cam, 0);
+       if (w9966_i2c_setscl(cam, 1) == -1)
+               return -1;
 
+       w9966_i2c_setsda(cam, 1);
+
+       return 0;
+}
 
 /* Find a good length for capture window (used both for W and H)
    A bit ugly but pretty functional. The capture length
@@ -451,7 +457,7 @@ static int w9966_calcscale(int size, int min, int max, int *beg, int *end, unsig
 /* Setup the cameras capture window etc.
    Expects a claimed pdev
    return -1 on error */
-static int w9966_setup(struct w9966_dev *cam, int x1, int y1, int x2, int y2, int w, int h)
+static int w9966_setup(struct w9966 *cam, int x1, int y1, int x2, int y2, int w, int h)
 {
        unsigned int i;
        unsigned int enh_s, enh_e;
@@ -486,15 +492,15 @@ static int w9966_setup(struct w9966_dev *cam, int x1, int y1, int x2, int y2, in
        enh_s = 0;
        enh_e = w * h * 2;
 
-/* Modify capture window if necessary and calculate downscaling */
+       /* Modify capture window if necessary and calculate downscaling */
        if (w9966_calcscale(w, W9966_WND_MIN_X, W9966_WND_MAX_X, &x1, &x2, &scale_x) != 0 ||
-           w9966_calcscale(h, W9966_WND_MIN_Y, W9966_WND_MAX_Y, &y1, &y2, &scale_y) != 0)
+                       w9966_calcscale(h, W9966_WND_MIN_Y, W9966_WND_MAX_Y, &y1, &y2, &scale_y) != 0)
                return -1;
 
        DPRINTF("%dx%d, x: %d<->%d, y: %d<->%d, sx: %d/64, sy: %d/64.\n",
-               w, h, x1, x2, y1, y2, scale_x & ~0x80, scale_y & ~0x80);
+                       w, h, x1, x2, y1, y2, scale_x & ~0x80, scale_y & ~0x80);
 
-/* Setup registers */
+       /* Setup registers */
        regs[0x00] = 0x00;                      /* Set normal operation */
        regs[0x01] = 0x18;                      /* Capture mode */
        regs[0x02] = scale_y;                   /* V-scaling */
@@ -533,348 +539,225 @@ static int w9966_setup(struct w9966_dev *cam, int x1, int y1, int x2, int y2, in
        saa7111_regs[0x0c] = cam->color;
        saa7111_regs[0x0d] = cam->hue;
 
-/* Reset (ECP-fifo & serial-bus) */
-       if (w9966_wReg(cam, 0x00, 0x03) == -1)
+       /* Reset (ECP-fifo & serial-bus) */
+       if (w9966_write_reg(cam, 0x00, 0x03) == -1)
                return -1;
 
-/* Write regs to w9966cf chip */
+       /* Write regs to w9966cf chip */
        for (i = 0; i < 0x1c; i++)
-               if (w9966_wReg(cam, i, regs[i]) == -1)
+               if (w9966_write_reg(cam, i, regs[i]) == -1)
                        return -1;
 
-/* Write regs to saa7111 chip */
+       /* Write regs to saa7111 chip */
        for (i = 0; i < 0x20; i++)
-               if (w9966_wReg_i2c(cam, i, saa7111_regs[i]) == -1)
+               if (w9966_write_reg_i2c(cam, i, saa7111_regs[i]) == -1)
                        return -1;
 
        return 0;
 }
 
 /*
- *     Ugly and primitive i2c protocol functions
+ *     Video4linux interfacing
  */
 
-/* Sets the data line on the i2c bus.
-   Expects a claimed pdev. */
-static inline void w9966_i2c_setsda(struct w9966_dev *cam, int state)
+static int cam_querycap(struct file *file, void  *priv,
+                                       struct v4l2_capability *vcap)
 {
-       if (state)
-               cam->i2c_state |= W9966_I2C_W_DATA;
-       else
-               cam->i2c_state &= ~W9966_I2C_W_DATA;
+       struct w9966 *cam = video_drvdata(file);
 
-       w9966_wReg(cam, 0x18, cam->i2c_state);
-       udelay(5);
+       strlcpy(vcap->driver, cam->v4l2_dev.name, sizeof(vcap->driver));
+       strlcpy(vcap->card, W9966_DRIVERNAME, sizeof(vcap->card));
+       strlcpy(vcap->bus_info, "parport", sizeof(vcap->bus_info));
+       vcap->version = KERNEL_VERSION(0, 33, 0);
+       vcap->capabilities = V4L2_CAP_VIDEO_CAPTURE | V4L2_CAP_READWRITE;
+       return 0;
 }
 
-/* Get peripheral clock line
-   Expects a claimed pdev. */
-static inline int w9966_i2c_getscl(struct w9966_dev *cam)
+static int cam_enum_input(struct file *file, void *fh, struct v4l2_input *vin)
 {
-       const unsigned char state = w9966_rReg(cam, 0x18);
-       return ((state & W9966_I2C_R_CLOCK) > 0);
+       if (vin->index > 0)
+               return -EINVAL;
+       strlcpy(vin->name, "Camera", sizeof(vin->name));
+       vin->type = V4L2_INPUT_TYPE_CAMERA;
+       vin->audioset = 0;
+       vin->tuner = 0;
+       vin->std = 0;
+       vin->status = 0;
+       return 0;
 }
 
-/* Sets the clock line on the i2c bus.
-   Expects a claimed pdev. -1 on error */
-static inline int w9966_i2c_setscl(struct w9966_dev *cam, int state)
+static int cam_g_input(struct file *file, void *fh, unsigned int *inp)
 {
-       unsigned long timeout;
-
-       if (state)
-               cam->i2c_state |= W9966_I2C_W_CLOCK;
-       else
-               cam->i2c_state &= ~W9966_I2C_W_CLOCK;
-
-       w9966_wReg(cam, 0x18, cam->i2c_state);
-       udelay(5);
-
-       /* we go to high, we also expect the peripheral to ack. */
-       if (state) {
-               timeout = jiffies + 100;
-               while (!w9966_i2c_getscl(cam)) {
-                       if (time_after(jiffies, timeout))
-                               return -1;
-               }
-       }
+       *inp = 0;
        return 0;
 }
 
-/* Get peripheral data line
-   Expects a claimed pdev. */
-static inline int w9966_i2c_getsda(struct w9966_dev *cam)
+static int cam_s_input(struct file *file, void *fh, unsigned int inp)
 {
-       const unsigned char state = w9966_rReg(cam, 0x18);
-       return ((state & W9966_I2C_R_DATA) > 0);
+       return (inp > 0) ? -EINVAL : 0;
 }
 
-/* Write a byte with ack to the i2c bus.
-   Expects a claimed pdev. -1 on error */
-static int w9966_i2c_wbyte(struct w9966_dev *cam, int data)
+static int cam_queryctrl(struct file *file, void *priv,
+                                       struct v4l2_queryctrl *qc)
 {
-       int i;
-
-       for (i = 7; i >= 0; i--) {
-               w9966_i2c_setsda(cam, (data >> i) & 0x01);
-
-               if (w9966_i2c_setscl(cam, 1) == -1)
-                       return -1;
-               w9966_i2c_setscl(cam, 0);
+       switch (qc->id) {
+       case V4L2_CID_BRIGHTNESS:
+               return v4l2_ctrl_query_fill(qc, 0, 255, 1, 128);
+       case V4L2_CID_CONTRAST:
+               return v4l2_ctrl_query_fill(qc, -64, 64, 1, 64);
+       case V4L2_CID_SATURATION:
+               return v4l2_ctrl_query_fill(qc, -64, 64, 1, 64);
+       case V4L2_CID_HUE:
+               return v4l2_ctrl_query_fill(qc, -128, 127, 1, 0);
        }
-
-       w9966_i2c_setsda(cam, 1);
-
-       if (w9966_i2c_setscl(cam, 1) == -1)
-               return -1;
-       w9966_i2c_setscl(cam, 0);
-
-       return 0;
+       return -EINVAL;
 }
 
-/* Read a data byte with ack from the i2c-bus
-   Expects a claimed pdev. -1 on error */
-#if 0
-static int w9966_i2c_rbyte(struct w9966_dev *cam)
+static int cam_g_ctrl(struct file *file, void *priv,
+                                       struct v4l2_control *ctrl)
 {
-       unsigned char data = 0x00;
-       int i;
-
-       w9966_i2c_setsda(cam, 1);
-
-       for (i = 0; i < 8; i++) {
-               if (w9966_i2c_setscl(cam, 1) == -1)
-                       return -1;
-               data = data << 1;
-               if (w9966_i2c_getsda(cam))
-                       data |= 0x01;
+       struct w9966 *cam = video_drvdata(file);
+       int ret = 0;
 
-               w9966_i2c_setscl(cam, 0);
+       switch (ctrl->id) {
+       case V4L2_CID_BRIGHTNESS:
+               ctrl->value = cam->brightness;
+               break;
+       case V4L2_CID_CONTRAST:
+               ctrl->value = cam->contrast;
+               break;
+       case V4L2_CID_SATURATION:
+               ctrl->value = cam->color;
+               break;
+       case V4L2_CID_HUE:
+               ctrl->value = cam->hue;
+               break;
+       default:
+               ret = -EINVAL;
+               break;
        }
-       return data;
+       return ret;
 }
-#endif
 
-/* Read a register from the i2c device.
-   Expects claimed pdev. -1 on error */
-#if 0
-static int w9966_rReg_i2c(struct w9966_dev *cam, int reg)
+static int cam_s_ctrl(struct file *file, void *priv,
+                                       struct v4l2_control *ctrl)
 {
-       int data;
-
-       w9966_i2c_setsda(cam, 0);
-       w9966_i2c_setscl(cam, 0);
-
-       if (w9966_i2c_wbyte(cam, W9966_I2C_W_ID) == -1 ||
-           w9966_i2c_wbyte(cam, reg) == -1)
-               return -1;
+       struct w9966 *cam = video_drvdata(file);
+       int ret = 0;
 
-       w9966_i2c_setsda(cam, 1);
-       if (w9966_i2c_setscl(cam, 1) == -1)
-               return -1;
-       w9966_i2c_setsda(cam, 0);
-       w9966_i2c_setscl(cam, 0);
-
-       if (w9966_i2c_wbyte(cam, W9966_I2C_R_ID) == -1)
-               return -1;
-       data = w9966_i2c_rbyte(cam);
-       if (data == -1)
-               return -1;
+       mutex_lock(&cam->lock);
+       switch (ctrl->id) {
+       case V4L2_CID_BRIGHTNESS:
+               cam->brightness = ctrl->value;
+               break;
+       case V4L2_CID_CONTRAST:
+               cam->contrast = ctrl->value;
+               break;
+       case V4L2_CID_SATURATION:
+               cam->color = ctrl->value;
+               break;
+       case V4L2_CID_HUE:
+               cam->hue = ctrl->value;
+               break;
+       default:
+               ret = -EINVAL;
+               break;
+       }
 
-       w9966_i2c_setsda(cam, 0);
+       if (ret == 0) {
+               w9966_pdev_claim(cam);
 
-       if (w9966_i2c_setscl(cam, 1) == -1)
-               return -1;
-       w9966_i2c_setsda(cam, 1);
+               if (w9966_write_reg_i2c(cam, 0x0a, cam->brightness) == -1 ||
+                   w9966_write_reg_i2c(cam, 0x0b, cam->contrast) == -1 ||
+                   w9966_write_reg_i2c(cam, 0x0c, cam->color) == -1 ||
+                   w9966_write_reg_i2c(cam, 0x0d, cam->hue) == -1) {
+                       ret = -EIO;
+               }
 
-       return data;
+               w9966_pdev_release(cam);
+       }
+       mutex_unlock(&cam->lock);
+       return ret;
 }
-#endif
 
-/* Write a register to the i2c device.
-   Expects claimed pdev. -1 on error */
-static int w9966_wReg_i2c(struct w9966_dev *cam, int reg, int data)
+static int cam_g_fmt_vid_cap(struct file *file, void *fh, struct v4l2_format *fmt)
 {
-       w9966_i2c_setsda(cam, 0);
-       w9966_i2c_setscl(cam, 0);
-
-       if (w9966_i2c_wbyte(cam, W9966_I2C_W_ID) == -1 ||
-           w9966_i2c_wbyte(cam, reg) == -1 ||
-           w9966_i2c_wbyte(cam, data) == -1)
-               return -1;
-
-       w9966_i2c_setsda(cam, 0);
-       if (w9966_i2c_setscl(cam, 1) == -1)
-               return -1;
-
-       w9966_i2c_setsda(cam, 1);
-
+       struct w9966 *cam = video_drvdata(file);
+       struct v4l2_pix_format *pix = &fmt->fmt.pix;
+
+       pix->width = cam->width;
+       pix->height = cam->height;
+       pix->pixelformat = V4L2_PIX_FMT_YUYV;
+       pix->field = V4L2_FIELD_NONE;
+       pix->bytesperline = 2 * cam->width;
+       pix->sizeimage = 2 * cam->width * cam->height;
+       /* Just a guess */
+       pix->colorspace = V4L2_COLORSPACE_SMPTE170M;
        return 0;
 }
 
-/*
- *     Video4linux interfacing
- */
-
-static long w9966_v4l_do_ioctl(struct file *file, unsigned int cmd, void *arg)
-{
-       struct w9966_dev *cam = video_drvdata(file);
-
-       switch (cmd) {
-       case VIDIOCGCAP:
-       {
-               static struct video_capability vcap = {
-                       .name      = W9966_DRIVERNAME,
-                       .type      = VID_TYPE_CAPTURE | VID_TYPE_SCALES,
-                       .channels  = 1,
-                       .maxwidth  = W9966_WND_MAX_W,
-                       .maxheight = W9966_WND_MAX_H,
-                       .minwidth  = 2,
-                       .minheight = 1,
-               };
-               struct video_capability *cap = arg;
-               *cap = vcap;
-               return 0;
-       }
-       case VIDIOCGCHAN:
-       {
-               struct video_channel *vch = arg;
-               if (vch->channel != 0)  /* We only support one channel (#0) */
-                       return -EINVAL;
-               memset(vch, 0, sizeof(*vch));
-               strcpy(vch->name, "CCD-input");
-               vch->type = VIDEO_TYPE_CAMERA;
-               return 0;
-       }
-       case VIDIOCSCHAN:
-       {
-               struct video_channel *vch = arg;
-               if (vch->channel != 0)
-                       return -EINVAL;
-               return 0;
-       }
-       case VIDIOCGTUNER:
-       {
-               struct video_tuner *vtune = arg;
-               if (vtune->tuner != 0)
-                       return -EINVAL;
-               strcpy(vtune->name, "no tuner");
-               vtune->rangelow = 0;
-               vtune->rangehigh = 0;
-               vtune->flags = VIDEO_TUNER_NORM;
-               vtune->mode = VIDEO_MODE_AUTO;
-               vtune->signal = 0xffff;
-               return 0;
-       }
-       case VIDIOCSTUNER:
-       {
-               struct video_tuner *vtune = arg;
-               if (vtune->tuner != 0)
-                       return -EINVAL;
-               if (vtune->mode != VIDEO_MODE_AUTO)
-                       return -EINVAL;
-               return 0;
-       }
-       case VIDIOCGPICT:
-       {
-               struct video_picture vpic = {
-                       cam->brightness << 8,   /* brightness */
-                       (cam->hue + 128) << 8,  /* hue */
-                       cam->color << 9,        /* color */
-                       cam->contrast << 9,     /* contrast */
-                       0x8000,                 /* whiteness */
-                       16, VIDEO_PALETTE_YUV422/* bpp, palette format */
-               };
-               struct video_picture *pic = arg;
-               *pic = vpic;
-               return 0;
-       }
-       case VIDIOCSPICT:
-       {
-               struct video_picture *vpic = arg;
-               if (vpic->depth != 16 || (vpic->palette != VIDEO_PALETTE_YUV422 && vpic->palette != VIDEO_PALETTE_YUYV))
-                       return -EINVAL;
-
-               cam->brightness = vpic->brightness >> 8;
-               cam->hue = (vpic->hue >> 8) - 128;
-               cam->color = vpic->colour >> 9;
-               cam->contrast = vpic->contrast >> 9;
+static int cam_try_fmt_vid_cap(struct file *file, void *fh, struct v4l2_format *fmt)
+{
+       struct v4l2_pix_format *pix = &fmt->fmt.pix;
+
+       if (pix->width < 2)
+               pix->width = 2;
+       if (pix->height < 1)
+               pix->height = 1;
+       if (pix->width > W9966_WND_MAX_W)
+               pix->width = W9966_WND_MAX_W;
+       if (pix->height > W9966_WND_MAX_H)
+               pix->height = W9966_WND_MAX_H;
+       pix->pixelformat = V4L2_PIX_FMT_YUYV;
+       pix->field = V4L2_FIELD_NONE;
+       pix->bytesperline = 2 * pix->width;
+       pix->sizeimage = 2 * pix->width * pix->height;
+       /* Just a guess */
+       pix->colorspace = V4L2_COLORSPACE_SMPTE170M;
+       return 0;
+}
 
-               w9966_pdev_claim(cam);
+static int cam_s_fmt_vid_cap(struct file *file, void *fh, struct v4l2_format *fmt)
+{
+       struct w9966 *cam = video_drvdata(file);
+       struct v4l2_pix_format *pix = &fmt->fmt.pix;
+       int ret = cam_try_fmt_vid_cap(file, fh, fmt);
 
-               if (
-                       w9966_wReg_i2c(cam, 0x0a, cam->brightness) == -1 ||
-                       w9966_wReg_i2c(cam, 0x0b, cam->contrast) == -1 ||
-                       w9966_wReg_i2c(cam, 0x0c, cam->color) == -1 ||
-                       w9966_wReg_i2c(cam, 0x0d, cam->hue) == -1
-               ) {
-                       w9966_pdev_release(cam);
-                       return -EIO;
-               }
+       if (ret)
+               return ret;
 
-               w9966_pdev_release(cam);
-               return 0;
-       }
-       case VIDIOCSWIN:
-       {
-               int ret;
-               struct video_window *vwin = arg;
-
-               if (vwin->flags != 0)
-                       return -EINVAL;
-               if (vwin->clipcount != 0)
-                       return -EINVAL;
-               if (vwin->width < 2 || vwin->width > W9966_WND_MAX_W)
-                       return -EINVAL;
-               if (vwin->height < 1 || vwin->height > W9966_WND_MAX_H)
-                       return -EINVAL;
-
-               /* Update camera regs */
-               w9966_pdev_claim(cam);
-               ret = w9966_setup(cam, 0, 0, 1023, 1023, vwin->width, vwin->height);
-               w9966_pdev_release(cam);
+       mutex_lock(&cam->lock);
+       /* Update camera regs */
+       w9966_pdev_claim(cam);
+       ret = w9966_setup(cam, 0, 0, 1023, 1023, pix->width, pix->height);
+       w9966_pdev_release(cam);
+       mutex_unlock(&cam->lock);
+       return ret;
+}
 
-               if (ret != 0) {
-                       DPRINTF("VIDIOCSWIN: w9966_setup() failed.\n");
-                       return -EIO;
-               }
+static int cam_enum_fmt_vid_cap(struct file *file, void *fh, struct v4l2_fmtdesc *fmt)
+{
+       static struct v4l2_fmtdesc formats[] = {
+               { 0, 0, 0,
+                 "YUV 4:2:2", V4L2_PIX_FMT_YUYV,
+                 { 0, 0, 0, 0 }
+               },
+       };
+       enum v4l2_buf_type type = fmt->type;
 
-               return 0;
-       }
-       case VIDIOCGWIN:
-       {
-               struct video_window *vwin = arg;
-               memset(vwin, 0, sizeof(*vwin));
-               vwin->width = cam->width;
-               vwin->height = cam->height;
-               return 0;
-       }
-       /* Unimplemented */
-       case VIDIOCCAPTURE:
-       case VIDIOCGFBUF:
-       case VIDIOCSFBUF:
-       case VIDIOCKEY:
-       case VIDIOCGFREQ:
-       case VIDIOCSFREQ:
-       case VIDIOCGAUDIO:
-       case VIDIOCSAUDIO:
+       if (fmt->index > 0)
                return -EINVAL;
-       default:
-               return -ENOIOCTLCMD;
-       }
-       return 0;
-}
 
-static long w9966_v4l_ioctl(struct file *file,
-                          unsigned int cmd, unsigned long arg)
-{
-       return video_usercopy(file, cmd, arg, w9966_v4l_do_ioctl);
+       *fmt = formats[fmt->index];
+       fmt->type = type;
+       return 0;
 }
 
 /* Capture data */
 static ssize_t w9966_v4l_read(struct file *file, char  __user *buf,
-                             size_t count, loff_t *ppos)
+               size_t count, loff_t *ppos)
 {
-       struct w9966_dev *cam = video_drvdata(file);
+       struct w9966 *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;
@@ -884,16 +767,18 @@ static ssize_t w9966_v4l_read(struct file *file, char  __user *buf,
        if (count > cam->width * cam->height * 2)
                return -EINVAL;
 
+       mutex_lock(&cam->lock);
        w9966_pdev_claim(cam);
-       w9966_wReg(cam, 0x00, 0x02);    /* Reset ECP-FIFO buffer */
-       w9966_wReg(cam, 0x00, 0x00);    /* Return to normal operation */
-       w9966_wReg(cam, 0x01, 0x98);    /* Enable capture */
+       w9966_write_reg(cam, 0x00, 0x02);       /* Reset ECP-FIFO buffer */
+       w9966_write_reg(cam, 0x00, 0x00);       /* Return to normal operation */
+       w9966_write_reg(cam, 0x01, 0x98);       /* Enable capture */
 
        /* write special capture-addr and negotiate into data transfer */
        if ((parport_negotiate(cam->pport, cam->ppmode|IEEE1284_ADDR) != 0) ||
-           (parport_write(cam->pport, &addr, 1) != 1) ||
-           (parport_negotiate(cam->pport, cam->ppmode|IEEE1284_DATA) != 0)) {
+                       (parport_write(cam->pport, &addr, 1) != 1) ||
+                       (parport_negotiate(cam->pport, cam->ppmode|IEEE1284_DATA) != 0)) {
                w9966_pdev_release(cam);
+               mutex_unlock(&cam->lock);
                return -EFAULT;
        }
 
@@ -918,15 +803,142 @@ static ssize_t w9966_v4l_read(struct file *file, char  __user *buf,
                dleft -= tsize;
        }
 
-       w9966_wReg(cam, 0x01, 0x18);    /* Disable capture */
+       w9966_write_reg(cam, 0x01, 0x18);       /* Disable capture */
 
 out:
        kfree(tbuf);
        w9966_pdev_release(cam);
+       mutex_unlock(&cam->lock);
 
        return count;
 }
 
+static const struct v4l2_file_operations w9966_fops = {
+       .owner          = THIS_MODULE,
+       .ioctl          = video_ioctl2,
+       .read           = w9966_v4l_read,
+};
+
+static const struct v4l2_ioctl_ops w9966_ioctl_ops = {
+       .vidioc_querycap                    = cam_querycap,
+       .vidioc_g_input                     = cam_g_input,
+       .vidioc_s_input                     = cam_s_input,
+       .vidioc_enum_input                  = cam_enum_input,
+       .vidioc_queryctrl                   = cam_queryctrl,
+       .vidioc_g_ctrl                      = cam_g_ctrl,
+       .vidioc_s_ctrl                      = cam_s_ctrl,
+       .vidioc_enum_fmt_vid_cap            = cam_enum_fmt_vid_cap,
+       .vidioc_g_fmt_vid_cap               = cam_g_fmt_vid_cap,
+       .vidioc_s_fmt_vid_cap               = cam_s_fmt_vid_cap,
+       .vidioc_try_fmt_vid_cap             = cam_try_fmt_vid_cap,
+};
+
+
+/* Initialize camera device. Setup all internal flags, set a
+   default video mode, setup ccd-chip, register v4l device etc..
+   Also used for 'probing' of hardware.
+   -1 on error */
+static int w9966_init(struct w9966 *cam, struct parport *port)
+{
+       struct v4l2_device *v4l2_dev = &cam->v4l2_dev;
+
+       if (cam->dev_state != 0)
+               return -1;
+
+       strlcpy(v4l2_dev->name, "w9966", sizeof(v4l2_dev->name));
+
+       if (v4l2_device_register(NULL, v4l2_dev) < 0) {
+               v4l2_err(v4l2_dev, "Could not register v4l2_device\n");
+               return -1;
+       }
+       cam->pport = port;
+       cam->brightness = 128;
+       cam->contrast = 64;
+       cam->color = 64;
+       cam->hue = 0;
+
+       /* Select requested transfer mode */
+       switch (parmode) {
+       default:        /* Auto-detect (priority: hw-ecp, hw-epp, sw-ecp) */
+       case 0:
+               if (port->modes & PARPORT_MODE_ECP)
+                       cam->ppmode = IEEE1284_MODE_ECP;
+               else if (port->modes & PARPORT_MODE_EPP)
+                       cam->ppmode = IEEE1284_MODE_EPP;
+               else
+                       cam->ppmode = IEEE1284_MODE_ECP;
+               break;
+       case 1:         /* hw- or sw-ecp */
+               cam->ppmode = IEEE1284_MODE_ECP;
+               break;
+       case 2:         /* hw- or sw-epp */
+               cam->ppmode = IEEE1284_MODE_EPP;
+               break;
+       }
+
+       /* Tell the parport driver that we exists */
+       cam->pdev = parport_register_device(port, "w9966", NULL, NULL, NULL, 0, NULL);
+       if (cam->pdev == NULL) {
+               DPRINTF("parport_register_device() failed\n");
+               return -1;
+       }
+       w9966_set_state(cam, W9966_STATE_PDEV, W9966_STATE_PDEV);
+
+       w9966_pdev_claim(cam);
+
+       /* Setup a default capture mode */
+       if (w9966_setup(cam, 0, 0, 1023, 1023, 200, 160) != 0) {
+               DPRINTF("w9966_setup() failed.\n");
+               return -1;
+       }
+
+       w9966_pdev_release(cam);
+
+       /* Fill in the video_device struct and register us to v4l */
+       strlcpy(cam->vdev.name, W9966_DRIVERNAME, sizeof(cam->vdev.name));
+       cam->vdev.v4l2_dev = v4l2_dev;
+       cam->vdev.fops = &w9966_fops;
+       cam->vdev.ioctl_ops = &w9966_ioctl_ops;
+       cam->vdev.release = video_device_release_empty;
+       video_set_drvdata(&cam->vdev, cam);
+
+       mutex_init(&cam->lock);
+
+       if (video_register_device(&cam->vdev, VFL_TYPE_GRABBER, video_nr) < 0)
+               return -1;
+
+       w9966_set_state(cam, W9966_STATE_VDEV, W9966_STATE_VDEV);
+
+       /* All ok */
+       v4l2_info(v4l2_dev, "Found and initialized a webcam on %s.\n",
+                       cam->pport->name);
+       return 0;
+}
+
+
+/* Terminate everything gracefully */
+static void w9966_term(struct w9966 *cam)
+{
+       /* Unregister from v4l */
+       if (w9966_get_state(cam, W9966_STATE_VDEV, W9966_STATE_VDEV)) {
+               video_unregister_device(&cam->vdev);
+               w9966_set_state(cam, W9966_STATE_VDEV, 0);
+       }
+
+       /* Terminate from IEEE1284 mode and release pdev block */
+       if (w9966_get_state(cam, W9966_STATE_PDEV, W9966_STATE_PDEV)) {
+               w9966_pdev_claim(cam);
+               parport_negotiate(cam->pport, IEEE1284_MODE_COMPAT);
+               w9966_pdev_release(cam);
+       }
+
+       /* Unregister from parport */
+       if (w9966_get_state(cam, W9966_STATE_PDEV, W9966_STATE_PDEV)) {
+               parport_unregister_device(cam->pdev);
+               w9966_set_state(cam, W9966_STATE_PDEV, 0);
+       }
+}
+
 
 /* Called once for every parport on init */
 static void w9966_attach(struct parport *port)