Merge branch 'topic/core-cleanup' into for-linus
[safe/jmp/linux-2.6] / drivers / media / video / gspca / pac7311.c
index 80af367..44fed96 100644 (file)
 
 #define MODULE_NAME "pac7311"
 
+#include <linux/input.h>
 #include "gspca.h"
 
 MODULE_AUTHOR("Thomas Kaiser thomas@kaiser-linux.li");
 MODULE_DESCRIPTION("Pixart PAC7311");
 MODULE_LICENSE("GPL");
 
-/* specific webcam descriptor */
+/* specific webcam descriptor for pac7311 */
 struct sd {
        struct gspca_dev gspca_dev;             /* !! must be the first item */
 
-       unsigned char brightness;
        unsigned char contrast;
-       unsigned char colors;
        unsigned char gain;
        unsigned char exposure;
        unsigned char autogain;
        __u8 hflip;
        __u8 vflip;
 
-       __u8 sensor;
-#define SENSOR_PAC7302 0
-#define SENSOR_PAC7311 1
-
        u8 sof_read;
        u8 autogain_ignore_frames;
 
@@ -81,12 +76,8 @@ struct sd {
 };
 
 /* V4L2 controls supported by the driver */
-static int sd_setbrightness(struct gspca_dev *gspca_dev, __s32 val);
-static int sd_getbrightness(struct gspca_dev *gspca_dev, __s32 *val);
 static int sd_setcontrast(struct gspca_dev *gspca_dev, __s32 val);
 static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val);
-static int sd_setcolors(struct gspca_dev *gspca_dev, __s32 val);
-static int sd_getcolors(struct gspca_dev *gspca_dev, __s32 *val);
 static int sd_setautogain(struct gspca_dev *gspca_dev, __s32 val);
 static int sd_getautogain(struct gspca_dev *gspca_dev, __s32 *val);
 static int sd_sethflip(struct gspca_dev *gspca_dev, __s32 val);
@@ -98,24 +89,7 @@ static int sd_getgain(struct gspca_dev *gspca_dev, __s32 *val);
 static int sd_setexposure(struct gspca_dev *gspca_dev, __s32 val);
 static int sd_getexposure(struct gspca_dev *gspca_dev, __s32 *val);
 
-static struct ctrl sd_ctrls[] = {
-/* This control is pac7302 only */
-#define BRIGHTNESS_IDX 0
-       {
-           {
-               .id      = V4L2_CID_BRIGHTNESS,
-               .type    = V4L2_CTRL_TYPE_INTEGER,
-               .name    = "Brightness",
-               .minimum = 0,
-#define BRIGHTNESS_MAX 0x20
-               .maximum = BRIGHTNESS_MAX,
-               .step    = 1,
-#define BRIGHTNESS_DEF 0x10
-               .default_value = BRIGHTNESS_DEF,
-           },
-           .set = sd_setbrightness,
-           .get = sd_getbrightness,
-       },
+static const struct ctrl sd_ctrls[] = {
 /* This control is for both the 7302 and the 7311 */
        {
            {
@@ -132,23 +106,6 @@ static struct ctrl sd_ctrls[] = {
            .set = sd_setcontrast,
            .get = sd_getcontrast,
        },
-/* This control is pac7302 only */
-#define SATURATION_IDX 2
-       {
-           {
-               .id      = V4L2_CID_SATURATION,
-               .type    = V4L2_CTRL_TYPE_INTEGER,
-               .name    = "Saturation",
-               .minimum = 0,
-#define COLOR_MAX 255
-               .maximum = COLOR_MAX,
-               .step    = 1,
-#define COLOR_DEF 127
-               .default_value = COLOR_DEF,
-           },
-           .set = sd_setcolors,
-           .get = sd_getcolors,
-       },
 /* All controls below are for both the 7302 and the 7311 */
        {
            {
@@ -226,7 +183,7 @@ static struct ctrl sd_ctrls[] = {
        },
 };
 
-static struct v4l2_pix_format vga_mode[] = {
+static const struct v4l2_pix_format vga_mode[] = {
        {160, 120, V4L2_PIX_FMT_PJPG, V4L2_FIELD_NONE,
                .bytesperline = 160,
                .sizeimage = 160 * 120 * 3 / 8 + 590,
@@ -244,101 +201,8 @@ static struct v4l2_pix_format vga_mode[] = {
                .priv = 0},
 };
 
-/* pac 7302 */
-static const __u8 init_7302[] = {
-/*     index,value */
-       0xff, 0x01,             /* page 1 */
-       0x78, 0x00,             /* deactivate */
-       0xff, 0x01,
-       0x78, 0x40,             /* led off */
-};
-static const __u8 start_7302[] = {
-/*     index, len, [value]* */
-       0xff, 1,        0x00,           /* page 0 */
-       0x00, 12,       0x01, 0x40, 0x40, 0x40, 0x01, 0xe0, 0x02, 0x80,
-                       0x00, 0x00, 0x00, 0x00,
-       0x0d, 24,       0x03, 0x01, 0x00, 0xb5, 0x07, 0xcb, 0x00, 0x00,
-                       0x07, 0xc8, 0x00, 0xea, 0x07, 0xcf, 0x07, 0xf7,
-                       0x07, 0x7e, 0x01, 0x0b, 0x00, 0x00, 0x00, 0x11,
-       0x26, 2,        0xaa, 0xaa,
-       0x2e, 1,        0x31,
-       0x38, 1,        0x01,
-       0x3a, 3,        0x14, 0xff, 0x5a,
-       0x43, 11,       0x00, 0x0a, 0x18, 0x11, 0x01, 0x2c, 0x88, 0x11,
-                       0x00, 0x54, 0x11,
-       0x55, 1,        0x00,
-       0x62, 4,        0x10, 0x1e, 0x1e, 0x18,
-       0x6b, 1,        0x00,
-       0x6e, 3,        0x08, 0x06, 0x00,
-       0x72, 3,        0x00, 0xff, 0x00,
-       0x7d, 23,       0x01, 0x01, 0x58, 0x46, 0x50, 0x3c, 0x50, 0x3c,
-                       0x54, 0x46, 0x54, 0x56, 0x52, 0x50, 0x52, 0x50,
-                       0x56, 0x64, 0xa4, 0x00, 0xda, 0x00, 0x00,
-       0xa2, 10,       0x22, 0x2c, 0x3c, 0x54, 0x69, 0x7c, 0x9c, 0xb9,
-                       0xd2, 0xeb,
-       0xaf, 1,        0x02,
-       0xb5, 2,        0x08, 0x08,
-       0xb8, 2,        0x08, 0x88,
-       0xc4, 4,        0xae, 0x01, 0x04, 0x01,
-       0xcc, 1,        0x00,
-       0xd1, 11,       0x01, 0x30, 0x49, 0x5e, 0x6f, 0x7f, 0x8e, 0xa9,
-                       0xc1, 0xd7, 0xec,
-       0xdc, 1,        0x01,
-       0xff, 1,        0x01,           /* page 1 */
-       0x12, 3,        0x02, 0x00, 0x01,
-       0x3e, 2,        0x00, 0x00,
-       0x76, 5,        0x01, 0x20, 0x40, 0x00, 0xf2,
-       0x7c, 1,        0x00,
-       0x7f, 10,       0x4b, 0x0f, 0x01, 0x2c, 0x02, 0x58, 0x03, 0x20,
-                       0x02, 0x00,
-       0x96, 5,        0x01, 0x10, 0x04, 0x01, 0x04,
-       0xc8, 14,       0x00, 0x00, 0x00, 0x00, 0x00, 0x07, 0x00, 0x00,
-                       0x07, 0x00, 0x01, 0x07, 0x04, 0x01,
-       0xd8, 1,        0x01,
-       0xdb, 2,        0x00, 0x01,
-       0xde, 7,        0x00, 0x01, 0x04, 0x04, 0x00, 0x00, 0x00,
-       0xe6, 4,        0x00, 0x00, 0x00, 0x01,
-       0xeb, 1,        0x00,
-       0xff, 1,        0x02,           /* page 2 */
-       0x22, 1,        0x00,
-       0xff, 1,        0x03,           /* page 3 */
-       0x00, 255,                      /* load the page 3 */
-       0x11, 1,        0x01,
-       0xff, 1,        0x02,           /* page 2 */
-       0x13, 1,        0x00,
-       0x22, 4,        0x1f, 0xa4, 0xf0, 0x96,
-       0x27, 2,        0x14, 0x0c,
-       0x2a, 5,        0xc8, 0x00, 0x18, 0x12, 0x22,
-       0x64, 8,        0x00, 0x00, 0xf0, 0x01, 0x14, 0x44, 0x44, 0x44,
-       0x6e, 1,        0x08,
-       0xff, 1,        0x01,           /* page 1 */
-       0x78, 1,        0x00,
-       0, 0                            /* end of sequence */
-};
-
-/* page 3 - the value 0xaa says skip the index - see reg_w_page() */
-static const __u8 page3_7302[] = {
-       0x90, 0x40, 0x03, 0x50, 0xc2, 0x01, 0x14, 0x16,
-       0x14, 0x12, 0x00, 0x00, 0x00, 0x02, 0x33, 0x00,
-       0x0f, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
-       0x00, 0x00, 0x00, 0x47, 0x01, 0xb3, 0x01, 0x00,
-       0x00, 0x08, 0x00, 0x00, 0x0d, 0x00, 0x00, 0x21,
-       0x00, 0x00, 0x00, 0x54, 0xf4, 0x02, 0x52, 0x54,
-       0xa4, 0xb8, 0xe0, 0x2a, 0xf6, 0x00, 0x00, 0x00,
-       0x00, 0x1e, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
-       0x00, 0xfc, 0x00, 0xf2, 0x1f, 0x04, 0x00, 0x00,
-       0x00, 0x00, 0x00, 0xc0, 0xc0, 0x10, 0x00, 0x00,
-       0x00, 0x40, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
-       0x00, 0x40, 0xff, 0x03, 0x19, 0x00, 0x00, 0x00,
-       0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
-       0x00, 0x00, 0x00, 0x00, 0x00, 0xc8, 0xc8, 0xc8,
-       0xc8, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x50,
-       0x08, 0x10, 0x24, 0x40, 0x00, 0x00, 0x00, 0x00,
-       0x01, 0x00, 0x02, 0x47, 0x00, 0x00, 0x00, 0x00,
-       0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
-       0x00, 0x02, 0xfa, 0x00, 0x64, 0x5a, 0x28, 0x00,
-       0x00
-};
+#define LOAD_PAGE4             254
+#define END_OF_SEQUENCE                0
 
 /* pac 7311 */
 static const __u8 init_7311[] = {
@@ -378,18 +242,19 @@ static const __u8 start_7311[] = {
        0xf0, 13,       0x01, 0x00, 0x00, 0x00, 0x22, 0x00, 0x20, 0x00,
                        0x3f, 0x00, 0x0a, 0x01, 0x00,
        0xff, 1,        0x04,           /* page 4 */
-       0x00, 254,                      /* load the page 4 */
+       0, LOAD_PAGE4,                  /* load the page 4 */
        0x11, 1,        0x01,
-       0, 0                            /* end of sequence */
+       0, END_OF_SEQUENCE              /* end of sequence */
 };
 
-/* page 4 - the value 0xaa says skip the index - see reg_w_page() */
+#define SKIP           0xaa
+/* page 4 - the value SKIP says skip the index - see reg_w_page() */
 static const __u8 page4_7311[] = {
-       0xaa, 0xaa, 0x04, 0x54, 0x07, 0x2b, 0x09, 0x0f,
-       0x09, 0x00, 0xaa, 0xaa, 0x07, 0x00, 0x00, 0x62,
-       0x08, 0xaa, 0x07, 0x00, 0x00, 0x00, 0x00, 0x00,
-       0x00, 0x00, 0x00, 0x03, 0xa0, 0x01, 0xf4, 0xaa,
-       0xaa, 0x00, 0x08, 0xaa, 0x03, 0xaa, 0x00, 0x68,
+       SKIP, SKIP, 0x04, 0x54, 0x07, 0x2b, 0x09, 0x0f,
+       0x09, 0x00, SKIP, SKIP, 0x07, 0x00, 0x00, 0x62,
+       0x08, SKIP, 0x07, 0x00, 0x00, 0x00, 0x00, 0x00,
+       0x00, 0x00, 0x00, 0x03, 0xa0, 0x01, 0xf4, SKIP,
+       SKIP, 0x00, 0x08, SKIP, 0x03, SKIP, 0x00, 0x68,
        0xca, 0x10, 0x06, 0x78, 0x00, 0x00, 0x00, 0x00,
        0x23, 0x28, 0x04, 0x11, 0x00, 0x00
 };
@@ -398,14 +263,24 @@ static void reg_w_buf(struct gspca_dev *gspca_dev,
                  __u8 index,
                  const char *buffer, int len)
 {
+       int ret;
+
+       if (gspca_dev->usb_err < 0)
+               return;
        memcpy(gspca_dev->usb_buf, buffer, len);
-       usb_control_msg(gspca_dev->dev,
+       ret = usb_control_msg(gspca_dev->dev,
                        usb_sndctrlpipe(gspca_dev->dev, 0),
                        1,              /* request */
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0,              /* value */
                        index, gspca_dev->usb_buf, len,
                        500);
+       if (ret < 0) {
+               PDEBUG(D_ERR, "reg_w_buf(): "
+               "Failed to write registers to index 0x%x, error %i",
+               index, ret);
+               gspca_dev->usb_err = ret;
+       }
 }
 
 
@@ -413,13 +288,23 @@ static void reg_w(struct gspca_dev *gspca_dev,
                  __u8 index,
                  __u8 value)
 {
+       int ret;
+
+       if (gspca_dev->usb_err < 0)
+               return;
        gspca_dev->usb_buf[0] = value;
-       usb_control_msg(gspca_dev->dev,
+       ret = usb_control_msg(gspca_dev->dev,
                        usb_sndctrlpipe(gspca_dev->dev, 0),
                        0,                      /* request */
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0, index, gspca_dev->usb_buf, 1,
                        500);
+       if (ret < 0) {
+               PDEBUG(D_ERR, "reg_w(): "
+               "Failed to write register to index 0x%x, value 0x%x, error %i",
+               index, value, ret);
+               gspca_dev->usb_err = ret;
+       }
 }
 
 static void reg_w_seq(struct gspca_dev *gspca_dev,
@@ -436,23 +321,35 @@ static void reg_w_page(struct gspca_dev *gspca_dev,
                        const __u8 *page, int len)
 {
        int index;
+       int ret = 0;
 
+       if (gspca_dev->usb_err < 0)
+               return;
        for (index = 0; index < len; index++) {
-               if (page[index] == 0xaa)                /* skip this index */
+               if (page[index] == SKIP)                /* skip this index */
                        continue;
                gspca_dev->usb_buf[0] = page[index];
-               usb_control_msg(gspca_dev->dev,
+               ret = usb_control_msg(gspca_dev->dev,
                                usb_sndctrlpipe(gspca_dev->dev, 0),
                                0,                      /* request */
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                                0, index, gspca_dev->usb_buf, 1,
                                500);
+               if (ret < 0) {
+                       PDEBUG(D_ERR, "reg_w_page(): "
+                       "Failed to write register to index 0x%x, "
+                       "value 0x%x, error %i",
+                       index, page[index], ret);
+                       gspca_dev->usb_err = ret;
+                       break;
+               }
        }
 }
 
 /* output a variable sequence */
 static void reg_w_var(struct gspca_dev *gspca_dev,
-                       const __u8 *seq)
+                       const __u8 *seq,
+                       const __u8 *page4, unsigned int page4_len)
 {
        int index, len;
 
@@ -460,23 +357,21 @@ static void reg_w_var(struct gspca_dev *gspca_dev,
                index = *seq++;
                len = *seq++;
                switch (len) {
-               case 0:
+               case END_OF_SEQUENCE:
                        return;
-               case 254:
-                       reg_w_page(gspca_dev, page4_7311, sizeof page4_7311);
-                       break;
-               case 255:
-                       reg_w_page(gspca_dev, page3_7302, sizeof page3_7302);
+               case LOAD_PAGE4:
+                       reg_w_page(gspca_dev, page4, page4_len);
                        break;
                default:
-                       if (len > 64) {
+                       if (len > USB_BUF_SZ) {
                                PDEBUG(D_ERR|D_STREAM,
                                        "Incorrect variable sequence");
                                return;
                        }
                        while (len > 0) {
                                if (len < 8) {
-                                       reg_w_buf(gspca_dev, index, seq, len);
+                                       reg_w_buf(gspca_dev,
+                                               index, seq, len);
                                        seq += len;
                                        break;
                                }
@@ -490,7 +385,7 @@ static void reg_w_var(struct gspca_dev *gspca_dev,
        /* not reached */
 }
 
-/* this function is called at probe time */
+/* this function is called at probe time for pac7311 */
 static int sd_config(struct gspca_dev *gspca_dev,
                        const struct usb_device_id *id)
 {
@@ -498,24 +393,12 @@ static int sd_config(struct gspca_dev *gspca_dev,
        struct cam *cam;
 
        cam = &gspca_dev->cam;
-       cam->epaddr = 0x05;
 
-       sd->sensor = id->driver_info;
-       if (sd->sensor == SENSOR_PAC7302) {
-               PDEBUG(D_CONF, "Find Sensor PAC7302");
-               cam->cam_mode = &vga_mode[2];   /* only 640x480 */
-               cam->nmodes = 1;
-       } else {
-               PDEBUG(D_CONF, "Find Sensor PAC7311");
-               cam->cam_mode = vga_mode;
-               cam->nmodes = ARRAY_SIZE(vga_mode);
-               gspca_dev->ctrl_dis = (1 << BRIGHTNESS_IDX)
-                               | (1 << SATURATION_IDX);
-       }
+       PDEBUG(D_CONF, "Find Sensor PAC7311");
+       cam->cam_mode = vga_mode;
+       cam->nmodes = ARRAY_SIZE(vga_mode);
 
-       sd->brightness = BRIGHTNESS_DEF;
        sd->contrast = CONTRAST_DEF;
-       sd->colors = COLOR_DEF;
        sd->gain = GAIN_DEF;
        sd->exposure = EXPOSURE_DEF;
        sd->autogain = AUTOGAIN_DEF;
@@ -524,33 +407,6 @@ static int sd_config(struct gspca_dev *gspca_dev,
        return 0;
 }
 
-/* This function is used by pac7302 only */
-static void setbrightcont(struct gspca_dev *gspca_dev)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       int i, v;
-       static const __u8 max[10] =
-               {0x29, 0x33, 0x42, 0x5a, 0x6e, 0x80, 0x9f, 0xbb,
-                0xd4, 0xec};
-       static const __u8 delta[10] =
-               {0x35, 0x33, 0x33, 0x2f, 0x2a, 0x25, 0x1e, 0x17,
-                0x11, 0x0b};
-
-       reg_w(gspca_dev, 0xff, 0x00);   /* page 0 */
-       for (i = 0; i < 10; i++) {
-               v = max[i];
-               v += (sd->brightness - BRIGHTNESS_MAX)
-                       * 150 / BRIGHTNESS_MAX;         /* 200 ? */
-               v -= delta[i] * sd->contrast / CONTRAST_MAX;
-               if (v < 0)
-                       v = 0;
-               else if (v > 0xff)
-                       v = 0xff;
-               reg_w(gspca_dev, 0xa2 + i, v);
-       }
-       reg_w(gspca_dev, 0xdc, 0x01);
-}
-
 /* This function is used by pac7311 only */
 static void setcontrast(struct gspca_dev *gspca_dev)
 {
@@ -562,46 +418,19 @@ static void setcontrast(struct gspca_dev *gspca_dev)
        reg_w(gspca_dev, 0x11, 0x01);
 }
 
-/* This function is used by pac7302 only */
-static void setcolors(struct gspca_dev *gspca_dev)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       int i, v;
-       static const int a[9] =
-               {217, -212, 0, -101, 170, -67, -38, -315, 355};
-       static const int b[9] =
-               {19, 106, 0, 19, 106, 1, 19, 106, 1};
-
-       reg_w(gspca_dev, 0xff, 0x03);   /* page 3 */
-       reg_w(gspca_dev, 0x11, 0x01);
-       reg_w(gspca_dev, 0xff, 0x00);   /* page 0 */
-       reg_w(gspca_dev, 0xff, 0x00);   /* page 0 */
-       for (i = 0; i < 9; i++) {
-               v = a[i] * sd->colors / COLOR_MAX + b[i];
-               reg_w(gspca_dev, 0x0f + 2 * i, (v >> 8) & 0x07);
-               reg_w(gspca_dev, 0x0f + 2 * i + 1, v);
-       }
-       reg_w(gspca_dev, 0xdc, 0x01);
-       PDEBUG(D_CONF|D_STREAM, "color: %i", sd->colors);
-}
-
 static void setgain(struct gspca_dev *gspca_dev)
 {
        struct sd *sd = (struct sd *) gspca_dev;
+       int gain = GAIN_MAX - sd->gain;
+
+       if (gain < 1)
+               gain = 1;
+       else if (gain > 245)
+               gain = 245;
+       reg_w(gspca_dev, 0xff, 0x04);                   /* page 4 */
+       reg_w(gspca_dev, 0x0e, 0x00);
+       reg_w(gspca_dev, 0x0f, gain);
 
-       if (sd->sensor == SENSOR_PAC7302) {
-               reg_w(gspca_dev, 0xff, 0x03);           /* page 3 */
-               reg_w(gspca_dev, 0x10, sd->gain >> 3);
-       } else {
-               int gain = GAIN_MAX - sd->gain;
-               if (gain < 1)
-                       gain = 1;
-               else if (gain > 245)
-                       gain = 245;
-               reg_w(gspca_dev, 0xff, 0x04);           /* page 4 */
-               reg_w(gspca_dev, 0x0e, 0x00);
-               reg_w(gspca_dev, 0x0f, gain);
-       }
        /* load registers to sensor (Bit 0, auto clear) */
        reg_w(gspca_dev, 0x11, 0x01);
 }
@@ -620,25 +449,19 @@ static void setexposure(struct gspca_dev *gspca_dev)
        else if (reg > 63)
                reg = 63;
 
-       if (sd->sensor == SENSOR_PAC7302) {
-               /* On the pac7302 reg2 MUST be a multiple of 3, so round it to
-                  the nearest multiple of 3, except when between 6 and 12? */
-               if (reg < 6 || reg > 12)
-                       reg = ((reg + 1) / 3) * 3;
-               reg_w(gspca_dev, 0xff, 0x03);           /* page 3 */
-               reg_w(gspca_dev, 0x02, reg);
+       reg_w(gspca_dev, 0xff, 0x04);                   /* page 4 */
+       reg_w(gspca_dev, 0x02, reg);
+
+       /* Page 1 register 8 must always be 0x08 except when not in
+          640x480 mode and Page3/4 reg 2 <= 3 then it must be 9 */
+       reg_w(gspca_dev, 0xff, 0x01);
+       if (gspca_dev->cam.cam_mode[(int)gspca_dev->curr_mode].priv &&
+                       reg <= 3) {
+               reg_w(gspca_dev, 0x08, 0x09);
        } else {
-               reg_w(gspca_dev, 0xff, 0x04);           /* page 4 */
-               reg_w(gspca_dev, 0x02, reg);
-               /* Page 1 register 8 must always be 0x08 except when not in
-                  640x480 mode and Page3/4 reg 2 <= 3 then it must be 9 */
-               reg_w(gspca_dev, 0xff, 0x01);
-               if (gspca_dev->cam.cam_mode[(int)gspca_dev->curr_mode].priv &&
-                               reg <= 3)
-                       reg_w(gspca_dev, 0x08, 0x09);
-               else
-                       reg_w(gspca_dev, 0x08, 0x08);
+               reg_w(gspca_dev, 0x08, 0x08);
        }
+
        /* load registers to sensor (Bit 0, auto clear) */
        reg_w(gspca_dev, 0x11, 0x01);
 }
@@ -648,31 +471,19 @@ static void sethvflip(struct gspca_dev *gspca_dev)
        struct sd *sd = (struct sd *) gspca_dev;
        __u8 data;
 
-       if (sd->sensor == SENSOR_PAC7302) {
-               reg_w(gspca_dev, 0xff, 0x03);           /* page 3 */
-               data = (sd->hflip ? 0x08 : 0x00)
-                       | (sd->vflip ? 0x04 : 0x00);
-       } else {
-               reg_w(gspca_dev, 0xff, 0x04);           /* page 4 */
-               data = (sd->hflip ? 0x04 : 0x00)
-                       | (sd->vflip ? 0x08 : 0x00);
-       }
+       reg_w(gspca_dev, 0xff, 0x04);                   /* page 4 */
+       data = (sd->hflip ? 0x04 : 0x00) | (sd->vflip ? 0x08 : 0x00);
        reg_w(gspca_dev, 0x21, data);
+
        /* load registers to sensor (Bit 0, auto clear) */
        reg_w(gspca_dev, 0x11, 0x01);
 }
 
-/* this function is called at probe and resume time */
+/* this function is called at probe and resume time for pac7311 */
 static int sd_init(struct gspca_dev *gspca_dev)
 {
-       struct sd *sd = (struct sd *) gspca_dev;
-
-       if (sd->sensor == SENSOR_PAC7302)
-               reg_w_seq(gspca_dev, init_7302, sizeof init_7302);
-       else
-               reg_w_seq(gspca_dev, init_7311, sizeof init_7311);
-
-       return 0;
+       reg_w_seq(gspca_dev, init_7311, sizeof(init_7311)/2);
+       return gspca_dev->usb_err;
 }
 
 static int sd_start(struct gspca_dev *gspca_dev)
@@ -681,14 +492,9 @@ static int sd_start(struct gspca_dev *gspca_dev)
 
        sd->sof_read = 0;
 
-       if (sd->sensor == SENSOR_PAC7302) {
-               reg_w_var(gspca_dev, start_7302);
-               setbrightcont(gspca_dev);
-               setcolors(gspca_dev);
-       } else {
-               reg_w_var(gspca_dev, start_7311);
-               setcontrast(gspca_dev);
-       }
+       reg_w_var(gspca_dev, start_7311,
+               page4_7311, sizeof(page4_7311));
+       setcontrast(gspca_dev);
        setgain(gspca_dev);
        setexposure(gspca_dev);
        sethvflip(gspca_dev);
@@ -706,8 +512,6 @@ static int sd_start(struct gspca_dev *gspca_dev)
                reg_w(gspca_dev, 0x87, 0x11);
                break;
        case 0:                                 /* 640x480 */
-               if (sd->sensor == SENSOR_PAC7302)
-                       break;
                reg_w(gspca_dev, 0xff, 0x01);
                reg_w(gspca_dev, 0x17, 0x00);
                reg_w(gspca_dev, 0x87, 0x12);
@@ -720,23 +524,13 @@ static int sd_start(struct gspca_dev *gspca_dev)
 
        /* start stream */
        reg_w(gspca_dev, 0xff, 0x01);
-       if (sd->sensor == SENSOR_PAC7302)
-               reg_w(gspca_dev, 0x78, 0x01);
-       else
-               reg_w(gspca_dev, 0x78, 0x05);
-       return 0;
+       reg_w(gspca_dev, 0x78, 0x05);
+
+       return gspca_dev->usb_err;
 }
 
 static void sd_stopN(struct gspca_dev *gspca_dev)
 {
-       struct sd *sd = (struct sd *) gspca_dev;
-
-       if (sd->sensor == SENSOR_PAC7302) {
-               reg_w(gspca_dev, 0xff, 0x01);
-               reg_w(gspca_dev, 0x78, 0x00);
-               reg_w(gspca_dev, 0x78, 0x00);
-               return;
-       }
        reg_w(gspca_dev, 0xff, 0x04);
        reg_w(gspca_dev, 0x27, 0x80);
        reg_w(gspca_dev, 0x28, 0xca);
@@ -749,17 +543,9 @@ static void sd_stopN(struct gspca_dev *gspca_dev)
        reg_w(gspca_dev, 0x78, 0x44); /* Bit_0=start stream, Bit_6=LED */
 }
 
-/* called on streamoff with alt 0 and on disconnect */
+/* called on streamoff with alt 0 and on disconnect for 7311 */
 static void sd_stop0(struct gspca_dev *gspca_dev)
 {
-       struct sd *sd = (struct sd *) gspca_dev;
-
-       if (!gspca_dev->present)
-               return;
-       if (sd->sensor == SENSOR_PAC7302) {
-               reg_w(gspca_dev, 0xff, 0x01);
-               reg_w(gspca_dev, 0x78, 0x40);
-       }
 }
 
 /* Include pac common sof detection functions */
@@ -774,22 +560,8 @@ static void do_autogain(struct gspca_dev *gspca_dev)
        if (avg_lum == -1)
                return;
 
-       if (sd->sensor == SENSOR_PAC7302) {
-               desired_lum = 270 + sd->brightness * 4;
-               /* Hack hack, with the 7202 the first exposure step is
-                  pretty large, so if we're about to make the first
-                  exposure increase make the deadzone large to avoid
-                  oscilating */
-               if (desired_lum > avg_lum && sd->gain == GAIN_DEF &&
-                               sd->exposure > EXPOSURE_DEF &&
-                               sd->exposure < 42)
-                       deadzone = 90;
-               else
-                       deadzone = 30;
-       } else {
-               desired_lum = 200;
-               deadzone = 20;
-       }
+       desired_lum = 200;
+       deadzone = 20;
 
        if (sd->autogain_ignore_frames > 0)
                sd->autogain_ignore_frames--;
@@ -798,53 +570,92 @@ static void do_autogain(struct gspca_dev *gspca_dev)
                sd->autogain_ignore_frames = PAC_AUTOGAIN_IGNORE_FRAMES;
 }
 
-static const unsigned char pac7311_jpeg_header1[] = {
-  0xff, 0xd8, 0xff, 0xc0, 0x00, 0x11, 0x08
+/* JPEG header, part 1 */
+static const unsigned char pac_jpeg_header1[] = {
+  0xff, 0xd8,          /* SOI: Start of Image */
+
+  0xff, 0xc0,          /* SOF0: Start of Frame (Baseline DCT) */
+  0x00, 0x11,          /* length = 17 bytes (including this length field) */
+  0x08                 /* Precision: 8 */
+  /* 2 bytes is placed here: number of image lines */
+  /* 2 bytes is placed here: samples per line */
 };
 
-static const unsigned char pac7311_jpeg_header2[] = {
-  0x03, 0x01, 0x21, 0x00, 0x02, 0x11, 0x01, 0x03, 0x11, 0x01, 0xff, 0xda,
-  0x00, 0x0c, 0x03, 0x01, 0x00, 0x02, 0x11, 0x03, 0x11, 0x00, 0x3f, 0x00
+/* JPEG header, continued */
+static const unsigned char pac_jpeg_header2[] = {
+  0x03,                        /* Number of image components: 3 */
+  0x01, 0x21, 0x00,    /* ID=1, Subsampling 1x1, Quantization table: 0 */
+  0x02, 0x11, 0x01,    /* ID=2, Subsampling 2x1, Quantization table: 1 */
+  0x03, 0x11, 0x01,    /* ID=3, Subsampling 2x1, Quantization table: 1 */
+
+  0xff, 0xda,          /* SOS: Start Of Scan */
+  0x00, 0x0c,          /* length = 12 bytes (including this length field) */
+  0x03,                        /* number of components: 3 */
+  0x01, 0x00,          /* selector 1, table 0x00 */
+  0x02, 0x11,          /* selector 2, table 0x11 */
+  0x03, 0x11,          /* selector 3, table 0x11 */
+  0x00, 0x3f,          /* Spectral selection: 0 .. 63 */
+  0x00                 /* Successive approximation: 0 */
 };
 
+static void pac_start_frame(struct gspca_dev *gspca_dev,
+               struct gspca_frame *frame,
+               __u16 lines, __u16 samples_per_line)
+{
+       unsigned char tmpbuf[4];
+
+       gspca_frame_add(gspca_dev, FIRST_PACKET,
+               pac_jpeg_header1, sizeof(pac_jpeg_header1));
+
+       tmpbuf[0] = lines >> 8;
+       tmpbuf[1] = lines & 0xff;
+       tmpbuf[2] = samples_per_line >> 8;
+       tmpbuf[3] = samples_per_line & 0xff;
+
+       gspca_frame_add(gspca_dev, INTER_PACKET,
+               tmpbuf, sizeof(tmpbuf));
+       gspca_frame_add(gspca_dev, INTER_PACKET,
+               pac_jpeg_header2, sizeof(pac_jpeg_header2));
+}
+
 /* this function is run at interrupt level */
 static void sd_pkt_scan(struct gspca_dev *gspca_dev,
-                       struct gspca_frame *frame,      /* target */
-                       __u8 *data,                     /* isoc packet */
+                       u8 *data,                       /* isoc packet */
                        int len)                        /* iso packet length */
 {
        struct sd *sd = (struct sd *) gspca_dev;
        unsigned char *sof;
+       struct gspca_frame *frame;
 
-       sof = pac_find_sof(gspca_dev, data, len);
+       sof = pac_find_sof(&sd->sof_read, data, len);
        if (sof) {
-               unsigned char tmpbuf[4];
                int n, lum_offset, footer_length;
 
-               if (sd->sensor == SENSOR_PAC7302) {
-                 /* 6 bytes after the FF D9 EOF marker a number of lumination
-                    bytes are send corresponding to different parts of the
-                    image, the 14th and 15th byte after the EOF seem to
-                    correspond to the center of the image */
-                 lum_offset = 61 + sizeof pac_sof_marker;
-                 footer_length = 74;
-               } else {
-                 lum_offset = 24 + sizeof pac_sof_marker;
-                 footer_length = 26;
+               frame = gspca_get_i_frame(gspca_dev);
+               if (frame == NULL) {
+                       gspca_dev->last_packet_type = DISCARD_PACKET;
+                       return;
                }
 
+               /* 6 bytes after the FF D9 EOF marker a number of lumination
+                  bytes are send corresponding to different parts of the
+                  image, the 14th and 15th byte after the EOF seem to
+                  correspond to the center of the image */
+               lum_offset = 24 + sizeof pac_sof_marker;
+               footer_length = 26;
+
                /* Finish decoding current frame */
                n = (sof - data) - (footer_length + sizeof pac_sof_marker);
                if (n < 0) {
                        frame->data_end += n;
                        n = 0;
                }
-               frame = gspca_frame_add(gspca_dev, INTER_PACKET, frame,
+               gspca_frame_add(gspca_dev, INTER_PACKET,
                                        data, n);
                if (gspca_dev->last_packet_type != DISCARD_PACKET &&
                                frame->data_end[-2] == 0xff &&
                                frame->data_end[-1] == 0xd9)
-                       frame = gspca_frame_add(gspca_dev, LAST_PACKET, frame,
+                       gspca_frame_add(gspca_dev, LAST_PACKET,
                                                NULL, 0);
 
                n = sof - data;
@@ -860,43 +671,10 @@ static void sd_pkt_scan(struct gspca_dev *gspca_dev,
                        atomic_set(&sd->avg_lum, -1);
 
                /* Start the new frame with the jpeg header */
-               gspca_frame_add(gspca_dev, FIRST_PACKET, frame,
-                       pac7311_jpeg_header1, sizeof(pac7311_jpeg_header1));
-               if (sd->sensor == SENSOR_PAC7302) {
-                       /* The PAC7302 has the image rotated 90 degrees */
-                       tmpbuf[0] = gspca_dev->width >> 8;
-                       tmpbuf[1] = gspca_dev->width & 0xff;
-                       tmpbuf[2] = gspca_dev->height >> 8;
-                       tmpbuf[3] = gspca_dev->height & 0xff;
-               } else {
-                       tmpbuf[0] = gspca_dev->height >> 8;
-                       tmpbuf[1] = gspca_dev->height & 0xff;
-                       tmpbuf[2] = gspca_dev->width >> 8;
-                       tmpbuf[3] = gspca_dev->width & 0xff;
-               }
-               gspca_frame_add(gspca_dev, INTER_PACKET, frame, tmpbuf, 4);
-               gspca_frame_add(gspca_dev, INTER_PACKET, frame,
-                       pac7311_jpeg_header2, sizeof(pac7311_jpeg_header2));
+               pac_start_frame(gspca_dev, frame,
+                       gspca_dev->height, gspca_dev->width);
        }
-       gspca_frame_add(gspca_dev, INTER_PACKET, frame, data, len);
-}
-
-static int sd_setbrightness(struct gspca_dev *gspca_dev, __s32 val)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-
-       sd->brightness = val;
-       if (gspca_dev->streaming)
-               setbrightcont(gspca_dev);
-       return 0;
-}
-
-static int sd_getbrightness(struct gspca_dev *gspca_dev, __s32 *val)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-
-       *val = sd->brightness;
-       return 0;
+       gspca_frame_add(gspca_dev, INTER_PACKET, data, len);
 }
 
 static int sd_setcontrast(struct gspca_dev *gspca_dev, __s32 val)
@@ -905,12 +683,9 @@ static int sd_setcontrast(struct gspca_dev *gspca_dev, __s32 val)
 
        sd->contrast = val;
        if (gspca_dev->streaming) {
-               if (sd->sensor == SENSOR_PAC7302)
-                       setbrightcont(gspca_dev);
-               else
-                       setcontrast(gspca_dev);
+               setcontrast(gspca_dev);
        }
-       return 0;
+       return gspca_dev->usb_err;
 }
 
 static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val)
@@ -921,24 +696,6 @@ static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val)
        return 0;
 }
 
-static int sd_setcolors(struct gspca_dev *gspca_dev, __s32 val)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-
-       sd->colors = val;
-       if (gspca_dev->streaming)
-               setcolors(gspca_dev);
-       return 0;
-}
-
-static int sd_getcolors(struct gspca_dev *gspca_dev, __s32 *val)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-
-       *val = sd->colors;
-       return 0;
-}
-
 static int sd_setgain(struct gspca_dev *gspca_dev, __s32 val)
 {
        struct sd *sd = (struct sd *) gspca_dev;
@@ -946,7 +703,7 @@ static int sd_setgain(struct gspca_dev *gspca_dev, __s32 val)
        sd->gain = val;
        if (gspca_dev->streaming)
                setgain(gspca_dev);
-       return 0;
+       return gspca_dev->usb_err;
 }
 
 static int sd_getgain(struct gspca_dev *gspca_dev, __s32 *val)
@@ -964,7 +721,7 @@ static int sd_setexposure(struct gspca_dev *gspca_dev, __s32 val)
        sd->exposure = val;
        if (gspca_dev->streaming)
                setexposure(gspca_dev);
-       return 0;
+       return gspca_dev->usb_err;
 }
 
 static int sd_getexposure(struct gspca_dev *gspca_dev, __s32 *val)
@@ -995,7 +752,7 @@ static int sd_setautogain(struct gspca_dev *gspca_dev, __s32 val)
                }
        }
 
-       return 0;
+       return gspca_dev->usb_err;
 }
 
 static int sd_getautogain(struct gspca_dev *gspca_dev, __s32 *val)
@@ -1013,7 +770,7 @@ static int sd_sethflip(struct gspca_dev *gspca_dev, __s32 val)
        sd->hflip = val;
        if (gspca_dev->streaming)
                sethvflip(gspca_dev);
-       return 0;
+       return gspca_dev->usb_err;
 }
 
 static int sd_gethflip(struct gspca_dev *gspca_dev, __s32 *val)
@@ -1031,7 +788,7 @@ static int sd_setvflip(struct gspca_dev *gspca_dev, __s32 val)
        sd->vflip = val;
        if (gspca_dev->streaming)
                sethvflip(gspca_dev);
-       return 0;
+       return gspca_dev->usb_err;
 }
 
 static int sd_getvflip(struct gspca_dev *gspca_dev, __s32 *val)
@@ -1042,8 +799,39 @@ static int sd_getvflip(struct gspca_dev *gspca_dev, __s32 *val)
        return 0;
 }
 
-/* sub-driver description */
-static struct sd_desc sd_desc = {
+#ifdef CONFIG_INPUT
+static int sd_int_pkt_scan(struct gspca_dev *gspca_dev,
+                       u8 *data,               /* interrupt packet data */
+                       int len)                /* interrupt packet length */
+{
+       int ret = -EINVAL;
+       u8 data0, data1;
+
+       if (len == 2) {
+               data0 = data[0];
+               data1 = data[1];
+               if ((data0 == 0x00 && data1 == 0x11) ||
+                   (data0 == 0x22 && data1 == 0x33) ||
+                   (data0 == 0x44 && data1 == 0x55) ||
+                   (data0 == 0x66 && data1 == 0x77) ||
+                   (data0 == 0x88 && data1 == 0x99) ||
+                   (data0 == 0xaa && data1 == 0xbb) ||
+                   (data0 == 0xcc && data1 == 0xdd) ||
+                   (data0 == 0xee && data1 == 0xff)) {
+                       input_report_key(gspca_dev->input_dev, KEY_CAMERA, 1);
+                       input_sync(gspca_dev->input_dev);
+                       input_report_key(gspca_dev->input_dev, KEY_CAMERA, 0);
+                       input_sync(gspca_dev->input_dev);
+                       ret = 0;
+               }
+       }
+
+       return ret;
+}
+#endif
+
+/* sub-driver description for pac7311 */
+static const struct sd_desc sd_desc = {
        .name = MODULE_NAME,
        .ctrls = sd_ctrls,
        .nctrls = ARRAY_SIZE(sd_ctrls),
@@ -1054,28 +842,25 @@ static struct sd_desc sd_desc = {
        .stop0 = sd_stop0,
        .pkt_scan = sd_pkt_scan,
        .dq_callback = do_autogain,
+#ifdef CONFIG_INPUT
+       .int_pkt_scan = sd_int_pkt_scan,
+#endif
 };
 
 /* -- module initialisation -- */
-static __devinitdata struct usb_device_id device_table[] = {
-       {USB_DEVICE(0x093a, 0x2600), .driver_info = SENSOR_PAC7311},
-       {USB_DEVICE(0x093a, 0x2601), .driver_info = SENSOR_PAC7311},
-       {USB_DEVICE(0x093a, 0x2603), .driver_info = SENSOR_PAC7311},
-       {USB_DEVICE(0x093a, 0x2608), .driver_info = SENSOR_PAC7311},
-       {USB_DEVICE(0x093a, 0x260e), .driver_info = SENSOR_PAC7311},
-       {USB_DEVICE(0x093a, 0x260f), .driver_info = SENSOR_PAC7311},
-       {USB_DEVICE(0x093a, 0x2621), .driver_info = SENSOR_PAC7302},
-       {USB_DEVICE(0x093a, 0x2622), .driver_info = SENSOR_PAC7302},
-       {USB_DEVICE(0x093a, 0x2624), .driver_info = SENSOR_PAC7302},
-       {USB_DEVICE(0x093a, 0x2626), .driver_info = SENSOR_PAC7302},
-       {USB_DEVICE(0x093a, 0x262a), .driver_info = SENSOR_PAC7302},
-       {USB_DEVICE(0x093a, 0x262c), .driver_info = SENSOR_PAC7302},
+static const struct usb_device_id device_table[] __devinitconst = {
+       {USB_DEVICE(0x093a, 0x2600)},
+       {USB_DEVICE(0x093a, 0x2601)},
+       {USB_DEVICE(0x093a, 0x2603)},
+       {USB_DEVICE(0x093a, 0x2608)},
+       {USB_DEVICE(0x093a, 0x260e)},
+       {USB_DEVICE(0x093a, 0x260f)},
        {}
 };
 MODULE_DEVICE_TABLE(usb, device_table);
 
 /* -- device connect -- */
-static int sd_probe(struct usb_interface *intf,
+static int __devinit sd_probe(struct usb_interface *intf,
                        const struct usb_device_id *id)
 {
        return gspca_dev_probe(intf, id, &sd_desc, sizeof(struct sd),
@@ -1096,8 +881,10 @@ static struct usb_driver sd_driver = {
 /* -- module insert / remove -- */
 static int __init sd_mod_init(void)
 {
-       if (usb_register(&sd_driver) < 0)
-               return -1;
+       int ret;
+       ret = usb_register(&sd_driver);
+       if (ret < 0)
+               return ret;
        PDEBUG(D_PROBE, "registered");
        return 0;
 }