V4L/DVB (13422): gspca - ov534: ov772x changes from Richard Kaswy.
[safe/jmp/linux-2.6] / drivers / media / video / gspca / pac207.c
index 7e06745..57e13e2 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Pixart PAC207BCA library
  *
- * Copyright (C) 2008 Hans de Goede <j.w.r.degoede@hhs.nl>
+ * Copyright (C) 2008 Hans de Goede <hdgoede@redhat.com>
  * Copyright (C) 2005 Thomas Kaiser thomas@kaiser-linux.li
  * Copyleft (C) 2005 Michel Xhaard mxhaard@magic.fr
  *
 
 #include "gspca.h"
 
-#define DRIVER_VERSION_NUMBER  KERNEL_VERSION(2, 1, 4)
-static const char version[] = "2.1.4";
-
-MODULE_AUTHOR("Hans de Goede <j.w.r.degoede@hhs.nl>");
+MODULE_AUTHOR("Hans de Goede <hdgoede@redhat.com>");
 MODULE_DESCRIPTION("Pixart PAC207");
 MODULE_LICENSE("GPL");
 
@@ -38,54 +35,24 @@ MODULE_LICENSE("GPL");
 
 #define PAC207_BRIGHTNESS_MIN          0
 #define PAC207_BRIGHTNESS_MAX          255
-#define PAC207_BRIGHTNESS_DEFAULT      4 /* power on default: 4 */
+#define PAC207_BRIGHTNESS_DEFAULT      46
 
-#define PAC207_EXPOSURE_MIN            4
+#define PAC207_EXPOSURE_MIN            3
 #define PAC207_EXPOSURE_MAX            26
-#define PAC207_EXPOSURE_DEFAULT                4 /* power on default: 3 ?? */
-#define PAC207_EXPOSURE_KNEE           11 /* 4 = 30 fps, 11 = 8, 15 = 6 */
+#define PAC207_EXPOSURE_DEFAULT                5 /* power on default: 3 */
+#define PAC207_EXPOSURE_KNEE           8 /* 4 = 30 fps, 11 = 8, 15 = 6 */
 
 #define PAC207_GAIN_MIN                        0
 #define PAC207_GAIN_MAX                        31
 #define PAC207_GAIN_DEFAULT            9 /* power on default: 9 */
-#define PAC207_GAIN_KNEE               20
+#define PAC207_GAIN_KNEE               31
 
 #define PAC207_AUTOGAIN_DEADZONE       30
-/* We calculating the autogain at the end of the transfer of a frame, at this
-   moment a frame with the old settings is being transmitted, and a frame is
-   being captured with the old settings. So if we adjust the autogain we must
-   ignore atleast the 2 next frames for the new settings to come into effect
-   before doing any other adjustments */
-#define PAC207_AUTOGAIN_IGNORE_FRAMES  3
-
-enum pac207_line_state {
-       LINE_HEADER1,
-       LINE_HEADER2,
-       LINE_UNCOMPRESSED,
-       LINE_COMPRESSED,
-};
-
-struct pac207_decoder_state {
-       /* generic state */
-       u16 line_read;
-       u16 line_marker;
-       u8 line_state;
-       u8 header_read;
-       /* compression state */
-       u16 processed_bytes;
-       u8 remaining_bits;
-       s8 no_remaining_bits;
-       u8 get_abs;
-       u8 discard_byte;
-       u8 line_decode_buf[352];
-};
 
 /* specific webcam descriptor */
 struct sd {
        struct gspca_dev gspca_dev;             /* !! must be the first item */
 
-       struct pac207_decoder_state decoder_state;
-
        u8 mode;
 
        u8 brightness;
@@ -94,6 +61,7 @@ struct sd {
        u8 gain;
 
        u8 sof_read;
+       u8 header_read;
        u8 autogain_ignore_frames;
 
        atomic_t avg_lum;
@@ -149,7 +117,8 @@ static struct ctrl sd_ctrls[] = {
                .minimum = 0,
                .maximum = 1,
                .step   = 1,
-               .default_value = 1,
+#define AUTOGAIN_DEF 1
+               .default_value = AUTOGAIN_DEF,
                .flags = 0,
            },
            .set = sd_setautogain,
@@ -172,37 +141,41 @@ static struct ctrl sd_ctrls[] = {
        },
 };
 
-static struct cam_mode sif_mode[] = {
-       {V4L2_PIX_FMT_SBGGR8, 176, 144, 1},
-       {V4L2_PIX_FMT_SBGGR8, 352, 288, 0},
+static const struct v4l2_pix_format sif_mode[] = {
+       {176, 144, V4L2_PIX_FMT_PAC207, V4L2_FIELD_NONE,
+               .bytesperline = 176,
+               .sizeimage = (176 + 2) * 144,
+                       /* uncompressed, add 2 bytes / line for line header */
+               .colorspace = V4L2_COLORSPACE_SRGB,
+               .priv = 1},
+       {352, 288, V4L2_PIX_FMT_PAC207, V4L2_FIELD_NONE,
+               .bytesperline = 352,
+                       /* compressed, but only when needed (not compressed
+                          when the framerate is low) */
+               .sizeimage = (352 + 2) * 288,
+               .colorspace = V4L2_COLORSPACE_SRGB,
+               .priv = 0},
 };
 
 static const __u8 pac207_sensor_init[][8] = {
-       {0x10, 0x12, 0x0d, 0x12, 0x0c, 0x01, 0x29, 0xf0},
-       {0x00, 0x64, 0x64, 0x64, 0x04, 0x10, 0xf0, 0x30},
+       {0x10, 0x12, 0x0d, 0x12, 0x0c, 0x01, 0x29, 0x84},
+       {0x49, 0x64, 0x64, 0x64, 0x04, 0x10, 0xf0, 0x30},
        {0x00, 0x00, 0x00, 0x70, 0xa0, 0xf8, 0x00, 0x00},
-       {0x00, 0x00, 0x32, 0x00, 0x96, 0x00, 0xa2, 0x02},
        {0x32, 0x00, 0x96, 0x00, 0xA2, 0x02, 0xaf, 0x00},
 };
 
-                       /* 48 reg_72 Rate Control end BalSize_4a =0x36 */
-static const __u8 PacReg72[] = { 0x00, 0x00, 0x36, 0x00 };
-
-static const unsigned char pac207_sof_marker[5] =
-               { 0xff, 0xff, 0x00, 0xff, 0x96 };
-
-int pac207_write_regs(struct gspca_dev *gspca_dev, u16 index,
+static int pac207_write_regs(struct gspca_dev *gspca_dev, u16 index,
        const u8 *buffer, u16 length)
 {
        struct usb_device *udev = gspca_dev->dev;
        int err;
-       u8 kbuf[8];
 
-       memcpy(kbuf, buffer, length);
+       memcpy(gspca_dev->usb_buf, buffer, length);
 
        err = usb_control_msg(udev, usb_sndctrlpipe(udev, 0), 0x01,
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_INTERFACE,
-                       0x00, index, kbuf, length, PAC207_CTRL_TIMEOUT);
+                       0x00, index,
+                       gspca_dev->usb_buf, length, PAC207_CTRL_TIMEOUT);
        if (err < 0)
                PDEBUG(D_ERR,
                        "Failed to write registers to index 0x%04X, error %d)",
@@ -212,7 +185,7 @@ int pac207_write_regs(struct gspca_dev *gspca_dev, u16 index,
 }
 
 
-int pac207_write_reg(struct gspca_dev *gspca_dev, u16 index, u16 value)
+static int pac207_write_reg(struct gspca_dev *gspca_dev, u16 index, u16 value)
 {
        struct usb_device *udev = gspca_dev->dev;
        int err;
@@ -227,16 +200,15 @@ int pac207_write_reg(struct gspca_dev *gspca_dev, u16 index, u16 value)
        return err;
 }
 
-
-int pac207_read_reg(struct gspca_dev *gspca_dev, u16 index)
+static int pac207_read_reg(struct gspca_dev *gspca_dev, u16 index)
 {
        struct usb_device *udev = gspca_dev->dev;
-       u8 buff;
        int res;
 
        res = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), 0x00,
                        USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_INTERFACE,
-                       0x00, index, &buff, 1, PAC207_CTRL_TIMEOUT);
+                       0x00, index,
+                       gspca_dev->usb_buf, 1, PAC207_CTRL_TIMEOUT);
        if (res < 0) {
                PDEBUG(D_ERR,
                        "Failed to read a register (index 0x%04X, error %d)",
@@ -244,10 +216,9 @@ int pac207_read_reg(struct gspca_dev *gspca_dev, u16 index)
                return res;
        }
 
-       return buff;
+       return gspca_dev->usb_buf[0];
 }
 
-
 /* this function is called at probe time */
 static int sd_config(struct gspca_dev *gspca_dev,
                        const struct usb_device_id *id)
@@ -268,40 +239,35 @@ static int sd_config(struct gspca_dev *gspca_dev,
                return -ENODEV;
        }
 
-       pac207_write_reg(gspca_dev, 0x41, 0x00);
-                               /* Bit_0=Image Format,
-                                * Bit_1=LED,
-                                * Bit_2=Compression test mode enable */
-       pac207_write_reg(gspca_dev, 0x0f, 0x00); /* Power Control */
-       pac207_write_reg(gspca_dev, 0x11, 0x30); /* Analog Bias */
-
        PDEBUG(D_PROBE,
                "Pixart PAC207BCA Image Processor and Control Chip detected"
                " (vid/pid 0x%04X:0x%04X)", id->idVendor, id->idProduct);
 
        cam = &gspca_dev->cam;
-       cam->dev_name = (char *) id->driver_info;
-       cam->epaddr = 0x05;
        cam->cam_mode = sif_mode;
        cam->nmodes = ARRAY_SIZE(sif_mode);
        sd->brightness = PAC207_BRIGHTNESS_DEFAULT;
        sd->exposure = PAC207_EXPOSURE_DEFAULT;
        sd->gain = PAC207_GAIN_DEFAULT;
+       sd->autogain = AUTOGAIN_DEF;
 
        return 0;
 }
 
-/* this function is called at open time */
-static int sd_open(struct gspca_dev *gspca_dev)
+/* this function is called at probe and resume time */
+static int sd_init(struct gspca_dev *gspca_dev)
 {
-       struct sd *sd = (struct sd *) gspca_dev;
+       pac207_write_reg(gspca_dev, 0x41, 0x00);
+                               /* Bit_0=Image Format,
+                                * Bit_1=LED,
+                                * Bit_2=Compression test mode enable */
+       pac207_write_reg(gspca_dev, 0x0f, 0x00); /* Power Control */
 
-       sd->autogain = 1;
        return 0;
 }
 
 /* -- start the camera -- */
-static void sd_start(struct gspca_dev *gspca_dev)
+static int sd_start(struct gspca_dev *gspca_dev)
 {
        struct sd *sd = (struct sd *) gspca_dev;
        __u8 mode;
@@ -310,15 +276,13 @@ static void sd_start(struct gspca_dev *gspca_dev)
        pac207_write_regs(gspca_dev, 0x0002, pac207_sensor_init[0], 8);
        pac207_write_regs(gspca_dev, 0x000a, pac207_sensor_init[1], 8);
        pac207_write_regs(gspca_dev, 0x0012, pac207_sensor_init[2], 8);
-       pac207_write_regs(gspca_dev, 0x0040, pac207_sensor_init[3], 8);
-       pac207_write_regs(gspca_dev, 0x0042, pac207_sensor_init[4], 8);
-       pac207_write_regs(gspca_dev, 0x0048, PacReg72, 4);
+       pac207_write_regs(gspca_dev, 0x0042, pac207_sensor_init[3], 8);
 
        /* Compression Balance */
        if (gspca_dev->width == 176)
                pac207_write_reg(gspca_dev, 0x4a, 0xff);
        else
-               pac207_write_reg(gspca_dev, 0x4a, 0x88);
+               pac207_write_reg(gspca_dev, 0x4a, 0x30);
        pac207_write_reg(gspca_dev, 0x4b, 0x00); /* Sram test value */
        pac207_write_reg(gspca_dev, 0x08, sd->brightness);
 
@@ -343,6 +307,7 @@ static void sd_start(struct gspca_dev *gspca_dev)
        sd->sof_read = 0;
        sd->autogain_ignore_frames = 0;
        atomic_set(&sd->avg_lum, -1);
+       return 0;
 }
 
 static void sd_stopN(struct gspca_dev *gspca_dev)
@@ -352,410 +317,71 @@ static void sd_stopN(struct gspca_dev *gspca_dev)
        pac207_write_reg(gspca_dev, 0x0f, 0x00); /* Power Control */
 }
 
-static void sd_stop0(struct gspca_dev *gspca_dev)
-{
-}
-
-/* this function is called at close time */
-static void sd_close(struct gspca_dev *gspca_dev)
-{
-}
-
-/* -- convert pixart frames to Bayer -- */
-/* Sonix decompressor struct B.S.(2004) */
-static struct {
-       u8 is_abs;
-       u8 len;
-       s8 val;
-} table[256];
+/* Include pac common sof detection functions */
+#include "pac_common.h"
 
-void init_pixart_decoder(void)
-{
-       int i, is_abs, val, len;
-
-       for (i = 0; i < 256; i++) {
-               is_abs = 0;
-               val = 0;
-               len = 0;
-               if ((i & 0xC0) == 0) {
-                       /* code 00 */
-                       val = 0;
-                       len = 2;
-               } else if ((i & 0xC0) == 0x40) {
-                       /* code 01 */
-                       val = -5;
-                       len = 2;
-               } else if ((i & 0xC0) == 0x80) {
-                       /* code 10 */
-                       val = 5;
-                       len = 2;
-               } else if ((i & 0xF0) == 0xC0) {
-                       /* code 1100 */
-                       val = -10;
-                       len = 4;
-               } else if ((i & 0xF0) == 0xD0) {
-                       /* code 1101 */
-                       val = 10;
-                       len = 4;
-               } else if ((i & 0xF8) == 0xE0) {
-                       /* code 11100 */
-                       val = -15;
-                       len = 5;
-               } else if ((i & 0xF8) == 0xE8) {
-                       /* code 11101 */
-                       val = 15;
-                       len = 5;
-               } else if ((i & 0xFC) == 0xF0) {
-                       /* code 111100 */
-                       val = -20;
-                       len = 6;
-               } else if ((i & 0xFC) == 0xF4) {
-                       /* code 111101 */
-                       val = 20;
-                       len = 6;
-               } else if ((i & 0xF8) == 0xF8) {
-                       /* code 11111xxxxxx */
-                       is_abs = 1;
-                       val = 0;
-                       len = 5;
-               }
-               table[i].is_abs = is_abs;
-               table[i].val = val;
-               table[i].len = len;
-       }
-}
-
-/* auto gain and exposure algorithm based on the knee algorithm described here:
- * <http://ytse.tricolour.net/docs/LowLightOptimization.html> */
 static void pac207_do_auto_gain(struct gspca_dev *gspca_dev)
 {
        struct sd *sd = (struct sd *) gspca_dev;
-       int i, steps, desired_avg_lum;
-       int orig_gain = sd->gain;
-       int orig_exposure = sd->exposure;
        int avg_lum = atomic_read(&sd->avg_lum);
 
-       if (!sd->autogain || avg_lum == -1)
+       if (avg_lum == -1)
                return;
 
-       if (sd->autogain_ignore_frames > 0) {
+       if (sd->autogain_ignore_frames > 0)
                sd->autogain_ignore_frames--;
-               return;
-       }
-
-       /* correct desired lumination for the configured brightness */
-       desired_avg_lum = 100 + sd->brightness / 2;
-
-       /* If we are of a multiple of deadzone, do multiple step to reach the
-          desired lumination fast (with the risc of a slight overshoot) */
-       steps = abs(desired_avg_lum - avg_lum) / PAC207_AUTOGAIN_DEADZONE;
-
-       for (i = 0; i < steps; i++) {
-               if (avg_lum > desired_avg_lum) {
-                       if (sd->gain > PAC207_GAIN_KNEE)
-                               sd->gain--;
-                       else if (sd->exposure > PAC207_EXPOSURE_KNEE)
-                               sd->exposure--;
-                       else if (sd->gain > PAC207_GAIN_DEFAULT)
-                               sd->gain--;
-                       else if (sd->exposure > PAC207_EXPOSURE_MIN)
-                               sd->exposure--;
-                       else if (sd->gain > PAC207_GAIN_MIN)
-                               sd->gain--;
-                       else
-                               break;
-               } else {
-                       if (sd->gain < PAC207_GAIN_DEFAULT)
-                               sd->gain++;
-                       else if (sd->exposure < PAC207_EXPOSURE_KNEE)
-                               sd->exposure++;
-                       else if (sd->gain < PAC207_GAIN_KNEE)
-                               sd->gain++;
-                       else if (sd->exposure < PAC207_EXPOSURE_MAX)
-                               sd->exposure++;
-                       else if (sd->gain < PAC207_GAIN_MAX)
-                               sd->gain++;
-                       else
-                               break;
-               }
-       }
-
-       if (sd->exposure != orig_exposure || sd->gain != orig_gain) {
-               if (sd->exposure != orig_exposure)
-                       pac207_write_reg(gspca_dev, 0x0002, sd->exposure);
-               if (sd->gain != orig_gain)
-                       pac207_write_reg(gspca_dev, 0x000e, sd->gain);
-               pac207_write_reg(gspca_dev, 0x13, 0x01); /* load reg to sen */
-               pac207_write_reg(gspca_dev, 0x1c, 0x01); /* not documented */
-               sd->autogain_ignore_frames = PAC207_AUTOGAIN_IGNORE_FRAMES;
-       }
+       else if (gspca_auto_gain_n_exposure(gspca_dev, avg_lum,
+                       100, PAC207_AUTOGAIN_DEADZONE,
+                       PAC207_GAIN_KNEE, PAC207_EXPOSURE_KNEE))
+               sd->autogain_ignore_frames = PAC_AUTOGAIN_IGNORE_FRAMES;
 }
 
-static unsigned char *pac207_find_sof(struct gspca_dev *gspca_dev,
-                                       unsigned char *m, int len)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       int i;
-
-       /* Search for the SOF marker (fixed part) in the header */
-       for (i = 0; i < len; i++) {
-               if (m[i] == pac207_sof_marker[sd->sof_read]) {
-                       sd->sof_read++;
-                       if (sd->sof_read == sizeof(pac207_sof_marker)) {
-                               PDEBUG(D_STREAM,
-                                       "SOF found, bytes to analyze: %u."
-                                       " Frame starts at byte #%u",
-                                       len, i + 1);
-                               sd->sof_read = 0;
-                               return m + i + 1;
-                       }
-               } else {
-                       sd->sof_read = 0;
-               }
-       }
-
-       return NULL;
-}
-
-static int pac207_decompress_row(struct gspca_dev *gspca_dev,
-                               struct gspca_frame *f,
-                               __u8 *cdata,
-                               int len)
+static void sd_pkt_scan(struct gspca_dev *gspca_dev,
+                       struct gspca_frame *frame,
+                       __u8 *data,
+                       int len)
 {
        struct sd *sd = (struct sd *) gspca_dev;
-       struct pac207_decoder_state *decoder_state = &sd->decoder_state;
-       unsigned char *outp = decoder_state->line_decode_buf +
-                               decoder_state->line_read;
-       int val, bitlen, bitpos = -decoder_state->no_remaining_bits;
-       u8 code;
-
-       /* first two pixels are stored as raw 8-bit */
-       while (decoder_state->line_read < 2) {
-               *outp++ = *cdata++;
-               decoder_state->line_read++;
-               len--;
-               if (len == 0)
-                       return 0;
-       }
-
-       while (decoder_state->line_read < gspca_dev->width) {
-               if (bitpos < 0) {
-                       code = decoder_state->remaining_bits << (8 + bitpos) |
-                               cdata[0] >> -bitpos;
-               } else {
-                       u8 *addr = cdata + bitpos / 8;
-                       code = addr[0] << (bitpos & 7) |
-                               addr[1] >> (8 - (bitpos & 7));
-               }
-
-               bitlen = decoder_state->get_abs ?
-                               6 : table[code].len;
-
-               /* Stop decompressing if we're out of input data */
-               if ((bitpos + bitlen) > (len * 8))
-                       break;
-
-               if (decoder_state->get_abs) {
-                       *outp++ = code & 0xFC;
-                       decoder_state->line_read++;
-                       decoder_state->get_abs = 0;
-               } else {
-                       if (table[code].is_abs) {
-                               decoder_state->get_abs = 1;
-                       } else {
-                               /* relative to left pixel */
-                               val = outp[-2] +
-                                       table[code].val;
-                               if (val > 0xff)
-                                       val = 0xff;
-                               else if (val < 0)
-                                       val = 0;
-                               *outp++ = val;
-                               decoder_state->line_read++;
-                       }
-               }
-               bitpos += bitlen;
-       }
-
-       if (decoder_state->line_read == gspca_dev->width) {
-               int compressed_line_len;
-
-               gspca_frame_add(gspca_dev, INTER_PACKET, f,
-                               decoder_state->line_decode_buf,
-                               gspca_dev->width);
+       unsigned char *sof;
 
-               /* completely decompressed line, round pos to nearest word */
-               compressed_line_len = ((decoder_state->processed_bytes * 8 +
-                       bitpos + 15) / 16) * 2;
+       sof = pac_find_sof(&sd->sof_read, data, len);
+       if (sof) {
+               int n;
 
-               len -= compressed_line_len - decoder_state->processed_bytes;
-               if (len < 0) {
-                       decoder_state->discard_byte = 1;
-                       len = 0;
-               }
-       } else {
-               decoder_state->processed_bytes += len;
-               decoder_state->remaining_bits = cdata[bitpos/8];
-               decoder_state->no_remaining_bits = (8 - bitpos) & 7;
-               len = 0;
+               /* finish decoding current frame */
+               n = sof - data;
+               if (n > sizeof pac_sof_marker)
+                       n -= sizeof pac_sof_marker;
+               else
+                       n = 0;
+               frame = gspca_frame_add(gspca_dev, LAST_PACKET, frame,
+                                       data, n);
+               sd->header_read = 0;
+               gspca_frame_add(gspca_dev, FIRST_PACKET, frame, NULL, 0);
+               len -= sof - data;
+               data = sof;
        }
+       if (sd->header_read < 11) {
+               int needed;
 
-       return len;
-}
-
-static void pac207_decode_line_init(struct gspca_dev *gspca_dev)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       struct pac207_decoder_state *decoder_state = &sd->decoder_state;
-
-       decoder_state->line_read = 0;
-       decoder_state->line_state = LINE_HEADER1;
-       decoder_state->processed_bytes = 0;
-       decoder_state->no_remaining_bits = 0;
-       decoder_state->get_abs = 0;
-}
-
-static void pac207_decode_frame_init(struct gspca_dev *gspca_dev)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       struct pac207_decoder_state *decoder_state = &sd->decoder_state;
-
-       decoder_state->header_read = 0;
-       decoder_state->discard_byte = 0;
-
-       pac207_decode_line_init(gspca_dev);
-}
-
-static int pac207_decode_frame_data(struct gspca_dev *gspca_dev,
-       struct gspca_frame *f, unsigned char *data, int len)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       struct pac207_decoder_state *decoder_state = &sd->decoder_state;
-       int needed = 0;
-
-       /* first 11 bytes after sof marker: frame header */
-       if (decoder_state->header_read < 11) {
                /* get average lumination from frame header (byte 5) */
-               if (decoder_state->header_read < 5) {
-                       needed = 5 - decoder_state->header_read;
+               if (sd->header_read < 5) {
+                       needed = 5 - sd->header_read;
                        if (len >= needed)
-                               atomic_set(&sd->avg_lum, data[needed-1]);
+                               atomic_set(&sd->avg_lum, data[needed - 1]);
                }
                /* skip the rest of the header */
-               needed = 11 - decoder_state->header_read;
+               needed = 11 - sd->header_read;
                if (len <= needed) {
-                       decoder_state->header_read += len;
-                       return 0;
-               }
-               data += needed;
-               len -= needed;
-               decoder_state->header_read = 11;
-       }
-
-       while (len) {
-               if (decoder_state->discard_byte) {
-                       data++;
-                       len--;
-                       decoder_state->discard_byte = 0;
-                       continue;
-               }
-
-               switch (decoder_state->line_state) {
-               case LINE_HEADER1:
-                       decoder_state->line_marker = data[0] << 8;
-                       decoder_state->line_state = LINE_HEADER2;
-                       needed = 1;
-                       break;
-               case LINE_HEADER2:
-                       decoder_state->line_marker |= data[0];
-                       switch (decoder_state->line_marker) {
-                       case 0x0ff0:
-                               decoder_state->line_state = LINE_UNCOMPRESSED;
-                               break;
-                       case 0x1ee1:
-                               decoder_state->line_state = LINE_COMPRESSED;
-                               break;
-                       default:
-                               PDEBUG(D_STREAM,
-                                       "Error unknown line-header %04X",
-                                       (int) decoder_state->line_marker);
-                               gspca_dev->last_packet_type = DISCARD_PACKET;
-                               return 0;
-                       }
-                       needed = 1;
-                       break;
-               case LINE_UNCOMPRESSED:
-                       needed = gspca_dev->width - decoder_state->line_read;
-                       if (needed > len)
-                               needed = len;
-                       gspca_frame_add(gspca_dev, INTER_PACKET, f, data,
-                               needed);
-                       decoder_state->line_read += needed;
-                       break;
-               case LINE_COMPRESSED:
-                       needed = len -
-                               pac207_decompress_row(gspca_dev, f, data, len);
-                       break;
+                       sd->header_read += len;
+                       return;
                }
-
                data += needed;
                len -= needed;
-
-               if (decoder_state->line_read == gspca_dev->width) {
-                       if ((f->data_end - f->data) ==
-                               (gspca_dev->width * gspca_dev->height)) {
-                               /* eureka we've got a frame */
-                               return 1;
-                       }
-                       pac207_decode_line_init(gspca_dev);
-               }
+               sd->header_read = 11;
        }
 
-       return 0;
-}
-
-static void sd_pkt_scan(struct gspca_dev *gspca_dev,
-                       struct gspca_frame *frame,
-                       __u8 *data,
-                       int len)
-{
-       unsigned char *sof;
-       int n;
-
-       sof = pac207_find_sof(gspca_dev, data, len);
-
-       if (sof) {
-               /* finish decoding current frame */
-               if (gspca_dev->last_packet_type == INTER_PACKET) {
-                       n = sof - data;
-                       if (n > sizeof(pac207_sof_marker))
-                               n -= sizeof(pac207_sof_marker);
-                       else
-                               n = 0;
-                       n = pac207_decode_frame_data(gspca_dev, frame,
-                                                       data, n);
-                       if (n)
-                               frame = gspca_frame_add(gspca_dev,
-                                               LAST_PACKET,
-                                               frame,
-                                               NULL,
-                                               0);
-                       else
-                               PDEBUG(D_STREAM, "Incomplete frame");
-               }
-               pac207_decode_frame_init(gspca_dev);
-               gspca_frame_add(gspca_dev, FIRST_PACKET, frame, NULL, 0);
-               len -= sof - data;
-               data = sof;
-       }
-
-       if (gspca_dev->last_packet_type == DISCARD_PACKET)
-               return;
-
-       n = pac207_decode_frame_data(gspca_dev, frame, data, len);
-       if (n)
-               gspca_frame_add(gspca_dev, LAST_PACKET,
-                               frame, NULL, 0);
+       gspca_frame_add(gspca_dev, INTER_PACKET, frame, data, len);
 }
 
 static void setbrightness(struct gspca_dev *gspca_dev)
@@ -807,10 +433,6 @@ static int sd_setexposure(struct gspca_dev *gspca_dev, __s32 val)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       /* don't allow mucking with exposure when using autogain */
-       if (sd->autogain)
-               return -EINVAL;
-
        sd->exposure = val;
        if (gspca_dev->streaming)
                setexposure(gspca_dev);
@@ -829,10 +451,6 @@ static int sd_setgain(struct gspca_dev *gspca_dev, __s32 val)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       /* don't allow mucking with gain when using autogain */
-       if (sd->autogain)
-               return -EINVAL;
-
        sd->gain = val;
        if (gspca_dev->streaming)
                setgain(gspca_dev);
@@ -861,7 +479,7 @@ static int sd_setautogain(struct gspca_dev *gspca_dev, __s32 val)
                sd->gain = PAC207_GAIN_DEFAULT;
                if (gspca_dev->streaming) {
                        sd->autogain_ignore_frames =
-                               PAC207_AUTOGAIN_IGNORE_FRAMES;
+                               PAC_AUTOGAIN_IGNORE_FRAMES;
                        setexposure(gspca_dev);
                        setgain(gspca_dev);
                }
@@ -884,27 +502,28 @@ static const struct sd_desc sd_desc = {
        .ctrls = sd_ctrls,
        .nctrls = ARRAY_SIZE(sd_ctrls),
        .config = sd_config,
-       .open = sd_open,
+       .init = sd_init,
        .start = sd_start,
        .stopN = sd_stopN,
-       .stop0 = sd_stop0,
-       .close = sd_close,
        .dq_callback = pac207_do_auto_gain,
        .pkt_scan = sd_pkt_scan,
 };
 
 /* -- module initialisation -- */
-#define DVNM(name) .driver_info = (kernel_ulong_t) name
 static const __devinitdata struct usb_device_id device_table[] = {
-       {USB_DEVICE(0x041e, 0x4028), DVNM("Creative Webcam Vista Plus")},
-       {USB_DEVICE(0x093a, 0x2460), DVNM("Q-Tec Webcam 100")},
-       {USB_DEVICE(0x093a, 0x2463), DVNM("Philips spc200nc pac207")},
-       {USB_DEVICE(0x093a, 0x2464), DVNM("Labtec Webcam 1200")},
-       {USB_DEVICE(0x093a, 0x2468), DVNM("PAC207")},
-       {USB_DEVICE(0x093a, 0x2470), DVNM("Genius GF112")},
-       {USB_DEVICE(0x093a, 0x2471), DVNM("Genius VideoCam GE111")},
-       {USB_DEVICE(0x093a, 0x2472), DVNM("Genius VideoCam GE110")},
-       {USB_DEVICE(0x2001, 0xf115), DVNM("D-Link DSB-C120")},
+       {USB_DEVICE(0x041e, 0x4028)},
+       {USB_DEVICE(0x093a, 0x2460)},
+       {USB_DEVICE(0x093a, 0x2461)},
+       {USB_DEVICE(0x093a, 0x2463)},
+       {USB_DEVICE(0x093a, 0x2464)},
+       {USB_DEVICE(0x093a, 0x2468)},
+       {USB_DEVICE(0x093a, 0x2470)},
+       {USB_DEVICE(0x093a, 0x2471)},
+       {USB_DEVICE(0x093a, 0x2472)},
+       {USB_DEVICE(0x093a, 0x2474)},
+       {USB_DEVICE(0x093a, 0x2476)},
+       {USB_DEVICE(0x145f, 0x013a)},
+       {USB_DEVICE(0x2001, 0xf115)},
        {}
 };
 MODULE_DEVICE_TABLE(usb, device_table);
@@ -922,15 +541,20 @@ static struct usb_driver sd_driver = {
        .id_table = device_table,
        .probe = sd_probe,
        .disconnect = gspca_disconnect,
+#ifdef CONFIG_PM
+       .suspend = gspca_suspend,
+       .resume = gspca_resume,
+#endif
 };
 
 /* -- module insert / remove -- */
 static int __init sd_mod_init(void)
 {
-       init_pixart_decoder();
-       if (usb_register(&sd_driver) < 0)
-               return -1;
-       PDEBUG(D_PROBE, "v%s registered", version);
+       int ret;
+       ret = usb_register(&sd_driver);
+       if (ret < 0)
+               return ret;
+       PDEBUG(D_PROBE, "registered");
        return 0;
 }
 static void __exit sd_mod_exit(void)