tree-wide: fix assorted typos all over the place
[safe/jmp/linux-2.6] / drivers / media / video / gspca / spca500.c
index 3a42506..7dbd5ee 100644 (file)
@@ -24,9 +24,6 @@
 #include "gspca.h"
 #include "jpeg.h"
 
-#define DRIVER_VERSION_NUMBER  KERNEL_VERSION(2, 1, 4)
-static const char version[] = "2.1.4";
-
 MODULE_AUTHOR("Michel Xhaard <mxhaard@users.sourceforge.net>");
 MODULE_DESCRIPTION("GSPCA/SPCA500 USB Camera Driver");
 MODULE_LICENSE("GPL");
@@ -35,14 +32,14 @@ MODULE_LICENSE("GPL");
 struct sd {
        struct gspca_dev gspca_dev;             /* !! must be the first item */
 
-       __u8 packet[ISO_MAX_SIZE + 128];
-                                /* !! no more than 128 ff in an ISO packet */
-
        unsigned char brightness;
        unsigned char contrast;
        unsigned char colors;
+       u8 quality;
+#define QUALITY_MIN 70
+#define QUALITY_MAX 95
+#define QUALITY_DEF 85
 
-       char qindex;
        char subtype;
 #define AgfaCl20 0
 #define AiptekPocketDV 1
@@ -59,6 +56,8 @@ struct sd {
 #define Optimedia 12
 #define PalmPixDC85 13
 #define ToptroIndus 14
+
+       u8 *jpeg_hdr;
 };
 
 /* V4L2 controls supported by the driver */
@@ -114,14 +113,30 @@ static struct ctrl sd_ctrls[] = {
        },
 };
 
-static struct cam_mode vga_mode[] = {
-       {V4L2_PIX_FMT_JPEG, 320, 240, 1},
-       {V4L2_PIX_FMT_JPEG, 640, 480, 0},
+static const struct v4l2_pix_format vga_mode[] = {
+       {320, 240, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
+               .bytesperline = 320,
+               .sizeimage = 320 * 240 * 3 / 8 + 590,
+               .colorspace = V4L2_COLORSPACE_JPEG,
+               .priv = 1},
+       {640, 480, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
+               .bytesperline = 640,
+               .sizeimage = 640 * 480 * 3 / 8 + 590,
+               .colorspace = V4L2_COLORSPACE_JPEG,
+               .priv = 0},
 };
 
-static struct cam_mode sif_mode[] = {
-       {V4L2_PIX_FMT_JPEG, 176, 144, 1},
-       {V4L2_PIX_FMT_JPEG, 352, 288, 0},
+static const struct v4l2_pix_format sif_mode[] = {
+       {176, 144, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
+               .bytesperline = 176,
+               .sizeimage = 176 * 144 * 3 / 8 + 590,
+               .colorspace = V4L2_COLORSPACE_JPEG,
+               .priv = 1},
+       {352, 288, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
+               .bytesperline = 352,
+               .sizeimage = 352 * 288 * 3 / 8 + 590,
+               .colorspace = V4L2_COLORSPACE_JPEG,
+               .priv = 0},
 };
 
 /* Frame packet header offsets for the spca500 */
@@ -356,26 +371,27 @@ static const __u8 qtable_pocketdv[2][64] = {
         0x28, 0x28, 0x28, 0x28, 0x28, 0x28, 0x28, 0x28}
 };
 
-static void reg_r(struct usb_device *dev,
-                          __u16 index,
-                          __u8 *buffer, __u16 length)
+/* read 'len' bytes to gspca_dev->usb_buf */
+static void reg_r(struct gspca_dev *gspca_dev,
+                 __u16 index,
+                 __u16 length)
 {
-       usb_control_msg(dev,
-                       usb_rcvctrlpipe(dev, 0),
+       usb_control_msg(gspca_dev->dev,
+                       usb_rcvctrlpipe(gspca_dev->dev, 0),
                        0,
                        USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0,              /* value */
-                       index, buffer, length, 500);
+                       index, gspca_dev->usb_buf, length, 500);
 }
 
-static int reg_w(struct usb_device *dev,
+static int reg_w(struct gspca_dev *gspca_dev,
                     __u16 req, __u16 index, __u16 value)
 {
        int ret;
 
        PDEBUG(D_USBO, "reg write: [0x%02x] = 0x%02x", index, value);
-       ret = usb_control_msg(dev,
-                       usb_sndctrlpipe(dev, 0),
+       ret = usb_control_msg(gspca_dev->dev,
+                       usb_sndctrlpipe(gspca_dev->dev, 0),
                        req,
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        value, index, NULL, 0, 500);
@@ -385,28 +401,27 @@ static int reg_w(struct usb_device *dev,
 }
 
 /* returns: negative is error, pos or zero is data */
-static int reg_r_12(struct usb_device *dev,
+static int reg_r_12(struct gspca_dev *gspca_dev,
                        __u16 req,      /* bRequest */
                        __u16 index,    /* wIndex */
                        __u16 length)   /* wLength (1 or 2 only) */
 {
        int ret;
-       __u8 buf[2];
 
-       buf[1] = 0;
-       ret = usb_control_msg(dev,
-                       usb_rcvctrlpipe(dev, 0),
+       gspca_dev->usb_buf[1] = 0;
+       ret = usb_control_msg(gspca_dev->dev,
+                       usb_rcvctrlpipe(gspca_dev->dev, 0),
                        req,
                        USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0,              /* value */
                        index,
-                       buf, length,
+                       gspca_dev->usb_buf, length,
                        500);           /* timeout */
        if (ret < 0) {
                PDEBUG(D_ERR, "reg_r_12 err %d", ret);
                return -1;
        }
-       return (buf[1] << 8) + buf[0];
+       return (gspca_dev->usb_buf[1] << 8) + gspca_dev->usb_buf[0];
 }
 
 /*
@@ -414,13 +429,13 @@ static int reg_r_12(struct usb_device *dev,
  * a reg_read call.
  * Returns: negative is error or timeout, zero is success.
  */
-static int reg_r_wait(struct usb_device *dev,
+static int reg_r_wait(struct gspca_dev *gspca_dev,
                        __u16 reg, __u16 index, __u16 value)
 {
        int ret, cnt = 20;
 
        while (--cnt > 0) {
-               ret = reg_r_12(dev, reg, index, 1);
+               ret = reg_r_12(gspca_dev, reg, index, 1);
                if (ret == value)
                        return 0;
                msleep(50);
@@ -431,11 +446,10 @@ static int reg_r_wait(struct usb_device *dev,
 static int write_vector(struct gspca_dev *gspca_dev,
                        const __u16 data[][3])
 {
-       struct usb_device *dev = gspca_dev->dev;
        int ret, i = 0;
 
        while (data[i][0] != 0 || data[i][1] != 0 || data[i][2] != 0) {
-               ret = reg_w(dev, data[i][0], data[i][2], data[i][1]);
+               ret = reg_w(gspca_dev, data[i][0], data[i][2], data[i][1]);
                if (ret < 0)
                        return ret;
                i++;
@@ -449,19 +463,18 @@ static int spca50x_setup_qtable(struct gspca_dev *gspca_dev,
                                unsigned int cbase,
                                const __u8 qtable[2][64])
 {
-       struct usb_device *dev = gspca_dev->dev;
        int i, err;
 
        /* loop over y components */
        for (i = 0; i < 64; i++) {
-               err = reg_w(dev, request, ybase + i, qtable[0][i]);
+               err = reg_w(gspca_dev, request, ybase + i, qtable[0][i]);
                if (err < 0)
                        return err;
        }
 
        /* loop over c components */
        for (i = 0; i < 64; i++) {
-               err = reg_w(dev, request, cbase + i, qtable[1][i]);
+               err = reg_w(gspca_dev, request, cbase + i, qtable[1][i]);
                if (err < 0)
                        return err;
        }
@@ -470,37 +483,33 @@ static int spca50x_setup_qtable(struct gspca_dev *gspca_dev,
 
 static void spca500_ping310(struct gspca_dev *gspca_dev)
 {
-       __u8 Data[2];
-
-       reg_r(gspca_dev->dev, 0x0d04, Data, 2);
+       reg_r(gspca_dev, 0x0d04, 2);
        PDEBUG(D_STREAM, "ClickSmart310 ping 0x0d04 0x%02x 0x%02x",
-               Data[0], Data[1]);
+               gspca_dev->usb_buf[0], gspca_dev->usb_buf[1]);
 }
 
 static void spca500_clksmart310_init(struct gspca_dev *gspca_dev)
 {
-       __u8 Data[2];
-
-       reg_r(gspca_dev->dev, 0x0d05, Data, 2);
+       reg_r(gspca_dev, 0x0d05, 2);
        PDEBUG(D_STREAM, "ClickSmart310 init 0x0d05 0x%02x 0x%02x",
-               Data[0], Data[1]);
-       reg_w(gspca_dev->dev, 0x00, 0x8167, 0x5a);
+               gspca_dev->usb_buf[0], gspca_dev->usb_buf[1]);
+       reg_w(gspca_dev, 0x00, 0x8167, 0x5a);
        spca500_ping310(gspca_dev);
 
-       reg_w(gspca_dev->dev, 0x00, 0x8168, 0x22);
-       reg_w(gspca_dev->dev, 0x00, 0x816a, 0xc0);
-       reg_w(gspca_dev->dev, 0x00, 0x816b, 0x0b);
-       reg_w(gspca_dev->dev, 0x00, 0x8169, 0x25);
-       reg_w(gspca_dev->dev, 0x00, 0x8157, 0x5b);
-       reg_w(gspca_dev->dev, 0x00, 0x8158, 0x5b);
-       reg_w(gspca_dev->dev, 0x00, 0x813f, 0x03);
-       reg_w(gspca_dev->dev, 0x00, 0x8151, 0x4a);
-       reg_w(gspca_dev->dev, 0x00, 0x8153, 0x78);
-       reg_w(gspca_dev->dev, 0x00, 0x0d01, 0x04);
+       reg_w(gspca_dev, 0x00, 0x8168, 0x22);
+       reg_w(gspca_dev, 0x00, 0x816a, 0xc0);
+       reg_w(gspca_dev, 0x00, 0x816b, 0x0b);
+       reg_w(gspca_dev, 0x00, 0x8169, 0x25);
+       reg_w(gspca_dev, 0x00, 0x8157, 0x5b);
+       reg_w(gspca_dev, 0x00, 0x8158, 0x5b);
+       reg_w(gspca_dev, 0x00, 0x813f, 0x03);
+       reg_w(gspca_dev, 0x00, 0x8151, 0x4a);
+       reg_w(gspca_dev, 0x00, 0x8153, 0x78);
+       reg_w(gspca_dev, 0x00, 0x0d01, 0x04);
                                                /* 00 for adjust shutter */
-       reg_w(gspca_dev->dev, 0x00, 0x0d02, 0x01);
-       reg_w(gspca_dev->dev, 0x00, 0x8169, 0x25);
-       reg_w(gspca_dev->dev, 0x00, 0x0d01, 0x02);
+       reg_w(gspca_dev, 0x00, 0x0d02, 0x01);
+       reg_w(gspca_dev, 0x00, 0x8169, 0x25);
+       reg_w(gspca_dev, 0x00, 0x0d01, 0x02);
 }
 
 static void spca500_setmode(struct gspca_dev *gspca_dev,
@@ -509,14 +518,14 @@ static void spca500_setmode(struct gspca_dev *gspca_dev,
        int mode;
 
        /* set x multiplier */
-       reg_w(gspca_dev->dev, 0, 0x8001, xmult);
+       reg_w(gspca_dev, 0, 0x8001, xmult);
 
        /* set y multiplier */
-       reg_w(gspca_dev->dev, 0, 0x8002, ymult);
+       reg_w(gspca_dev, 0, 0x8002, ymult);
 
        /* use compressed mode, VGA, with mode specific subsample */
-       mode = gspca_dev->cam.cam_mode[(int) gspca_dev->curr_mode].mode;
-       reg_w(gspca_dev->dev, 0, 0x8003, mode << 4);
+       mode = gspca_dev->cam.cam_mode[(int) gspca_dev->curr_mode].priv;
+       reg_w(gspca_dev, 0, 0x8003, mode << 4);
 }
 
 static int spca500_full_reset(struct gspca_dev *gspca_dev)
@@ -524,18 +533,18 @@ static int spca500_full_reset(struct gspca_dev *gspca_dev)
        int err;
 
        /* send the reset command */
-       err = reg_w(gspca_dev->dev, 0xe0, 0x0001, 0x0000);
+       err = reg_w(gspca_dev, 0xe0, 0x0001, 0x0000);
        if (err < 0)
                return err;
 
        /* wait for the reset to complete */
-       err = reg_r_wait(gspca_dev->dev, 0x06, 0x0000, 0x0000);
+       err = reg_r_wait(gspca_dev, 0x06, 0x0000, 0x0000);
        if (err < 0)
                return err;
-       err = reg_w(gspca_dev->dev, 0xe0, 0x0000, 0x0000);
+       err = reg_w(gspca_dev, 0xe0, 0x0000, 0x0000);
        if (err < 0)
                return err;
-       err = reg_r_wait(gspca_dev->dev, 0x06, 0, 0);
+       err = reg_r_wait(gspca_dev, 0x06, 0, 0);
        if (err < 0) {
                PDEBUG(D_ERR, "reg_r_wait() failed");
                return err;
@@ -552,15 +561,13 @@ static int spca500_full_reset(struct gspca_dev *gspca_dev)
 /* up-port the same feature as in 2.4.x kernel */
 static int spca500_synch310(struct gspca_dev *gspca_dev)
 {
-       __u8 Data;
-
        if (usb_set_interface(gspca_dev->dev, gspca_dev->iface, 0) < 0) {
                PDEBUG(D_ERR, "Set packet size: set interface error");
                goto error;
        }
        spca500_ping310(gspca_dev);
 
-       reg_r(gspca_dev->dev, 0x0d00, &Data, 1);
+       reg_r(gspca_dev, 0x0d00, 1);
 
        /* need alt setting here */
        PDEBUG(D_PACK, "ClickSmart310 sync alt: %d", gspca_dev->alt);
@@ -582,14 +589,14 @@ static void spca500_reinit(struct gspca_dev *gspca_dev)
        int err;
        __u8 Data;
 
-       /* some unknow command from Aiptek pocket dv and family300 */
+       /* some unknown command from Aiptek pocket dv and family300 */
 
-       reg_w(gspca_dev->dev, 0x00, 0x0d01, 0x01);
-       reg_w(gspca_dev->dev, 0x00, 0x0d03, 0x00);
-       reg_w(gspca_dev->dev, 0x00, 0x0d02, 0x01);
+       reg_w(gspca_dev, 0x00, 0x0d01, 0x01);
+       reg_w(gspca_dev, 0x00, 0x0d03, 0x00);
+       reg_w(gspca_dev, 0x00, 0x0d02, 0x01);
 
        /* enable drop packet */
-       reg_w(gspca_dev->dev, 0x00, 0x850a, 0x0001);
+       reg_w(gspca_dev, 0x00, 0x850a, 0x0001);
 
        err = spca50x_setup_qtable(gspca_dev, 0x00, 0x8800, 0x8840,
                                 qtable_pocketdv);
@@ -597,22 +604,23 @@ static void spca500_reinit(struct gspca_dev *gspca_dev)
                PDEBUG(D_ERR|D_STREAM, "spca50x_setup_qtable failed on init");
 
        /* set qtable index */
-       reg_w(gspca_dev->dev, 0x00, 0x8880, 2);
+       reg_w(gspca_dev, 0x00, 0x8880, 2);
        /* family cam Quicksmart stuff */
-       reg_w(gspca_dev->dev, 0x00, 0x800a, 0x00);
+       reg_w(gspca_dev, 0x00, 0x800a, 0x00);
        /* Set agc transfer: synced inbetween frames */
-       reg_w(gspca_dev->dev, 0x00, 0x820f, 0x01);
+       reg_w(gspca_dev, 0x00, 0x820f, 0x01);
        /* Init SDRAM - needed for SDRAM access */
-       reg_w(gspca_dev->dev, 0x00, 0x870a, 0x04);
+       reg_w(gspca_dev, 0x00, 0x870a, 0x04);
        /*Start init sequence or stream */
-
-       reg_w(gspca_dev->dev, 0, 0x8003, 0x00);
+       reg_w(gspca_dev, 0, 0x8003, 0x00);
        /* switch to video camera mode */
-       reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
+       reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
        msleep(2000);
-       if (reg_r_wait(gspca_dev->dev, 0, 0x8000, 0x44) != 0)
-               reg_r(gspca_dev->dev, 0x816b, &Data, 1);
-       reg_w(gspca_dev->dev, 0x00, 0x816b, Data);
+       if (reg_r_wait(gspca_dev, 0, 0x8000, 0x44) != 0) {
+               reg_r(gspca_dev, 0x816b, 1);
+               Data = gspca_dev->usb_buf[0];
+               reg_w(gspca_dev, 0x00, 0x816b, Data);
+       }
 }
 
 /* this function is called at probe time */
@@ -621,125 +629,25 @@ static int sd_config(struct gspca_dev *gspca_dev,
 {
        struct sd *sd = (struct sd *) gspca_dev;
        struct cam *cam;
-       __u16 vendor;
-       __u16 product;
-
-       vendor = id->idVendor;
-       product = id->idProduct;
-       switch (vendor) {
-       case 0x040a:            /* Kodak cameras */
-/*             switch (product) { */
-/*             case 0x0300: */
-                       sd->subtype = KodakEZ200;
-/*                     break; */
-/*             } */
-               break;
-       case 0x041e:            /* Creative cameras */
-/*             switch (product) { */
-/*             case 0x400a: */
-                       sd->subtype = CreativePCCam300;
-/*                     break; */
-/*             } */
-               break;
-       case 0x046d:            /* Logitech Labtec */
-               switch (product) {
-               case 0x0890:
-                       sd->subtype = LogitechTraveler;
-                       break;
-               case 0x0900:
-                       sd->subtype = LogitechClickSmart310;
-                       break;
-               case 0x0901:
-                       sd->subtype = LogitechClickSmart510;
-                       break;
-               }
-               break;
-       case 0x04a5:            /* Benq */
-/*             switch (product) { */
-/*             case 0x300c: */
-                       sd->subtype = BenqDC1016;
-/*                     break; */
-/*             } */
-               break;
-       case 0x04fc:            /* SunPlus */
-/*             switch (product) { */
-/*             case 0x7333: */
-                       sd->subtype = PalmPixDC85;
-/*                     break; */
-/*             } */
-               break;
-       case 0x055f:            /* Mustek cameras */
-               switch (product) {
-               case 0xc200:
-                       sd->subtype = MustekGsmart300;
-                       break;
-               case 0xc220:
-                       sd->subtype = Gsmartmini;
-                       break;
-               }
-               break;
-       case 0x06bd:            /* Agfa Cl20 */
-/*             switch (product) { */
-/*             case 0x0404: */
-                       sd->subtype = AgfaCl20;
-/*                     break; */
-/*             } */
-               break;
-       case 0x06be:            /* Optimedia */
-/*             switch (product) { */
-/*             case 0x0800: */
-                       sd->subtype = Optimedia;
-/*                     break; */
-/*             } */
-               break;
-       case 0x084d:            /* D-Link / Minton */
-/*             switch (product) { */
-/*             case 0x0003:     * DSC-350 / S-Cam F5 */
-                       sd->subtype = DLinkDSC350;
-/*                     break; */
-/*             } */
-               break;
-       case 0x08ca:            /* Aiptek */
-/*             switch (product) { */
-/*             case 0x0103: */
-                       sd->subtype = AiptekPocketDV;
-/*                     break; */
-/*             } */
-               break;
-       case 0x2899:            /* ToptroIndustrial */
-/*             switch (product) { */
-/*             case 0x012c: */
-                       sd->subtype = ToptroIndus;
-/*                     break; */
-/*             } */
-               break;
-       case 0x8086:            /* Intel */
-/*             switch (product) { */
-/*             case 0x0630:     * Pocket PC Camera */
-                       sd->subtype = IntelPocketPCCamera;
-/*                     break; */
-/*             } */
-               break;
-       }
+
        cam = &gspca_dev->cam;
-       cam->dev_name = (char *) id->driver_info;
-       cam->epaddr = 0x01;
+       sd->subtype = id->driver_info;
        if (sd->subtype != LogitechClickSmart310) {
                cam->cam_mode = vga_mode;
-               cam->nmodes = sizeof vga_mode / sizeof vga_mode[0];
+               cam->nmodes = ARRAY_SIZE(vga_mode);
        } else {
                cam->cam_mode = sif_mode;
-               cam->nmodes = sizeof sif_mode / sizeof sif_mode[0];
+               cam->nmodes = ARRAY_SIZE(sif_mode);
        }
-       sd->qindex = 5;
        sd->brightness = BRIGHTNESS_DEF;
        sd->contrast = CONTRAST_DEF;
        sd->colors = COLOR_DEF;
+       sd->quality = QUALITY_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;
 
@@ -753,13 +661,21 @@ static int sd_open(struct gspca_dev *gspca_dev)
        return 0;
 }
 
-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;
        int err;
        __u8 Data;
        __u8 xmult, ymult;
 
+       /* create the JPEG header */
+       sd->jpeg_hdr = kmalloc(JPEG_HDR_SZ, GFP_KERNEL);
+       if (!sd->jpeg_hdr)
+               return -ENOMEM;
+       jpeg_define(sd->jpeg_hdr, gspca_dev->height, gspca_dev->width,
+                       0x22);          /* JPEG 411 */
+       jpeg_set_qual(sd->jpeg_hdr, sd->quality);
+
        if (sd->subtype == LogitechClickSmart310) {
                xmult = 0x16;
                ymult = 0x12;
@@ -769,9 +685,10 @@ static void sd_start(struct gspca_dev *gspca_dev)
        }
 
        /* is there a sensor here ? */
-       reg_r(gspca_dev->dev, 0x8a04, &Data, 1);
-       PDEBUG(D_STREAM, "Spca500 Sensor Address 0x%02X", Data);
-       PDEBUG(D_STREAM, "Spca500 curr_mode: %d Xmult: 0x%02X, Ymult: 0x%02X",
+       reg_r(gspca_dev, 0x8a04, 1);
+       PDEBUG(D_STREAM, "Spca500 Sensor Address 0x%02x",
+               gspca_dev->usb_buf[0]);
+       PDEBUG(D_STREAM, "Spca500 curr_mode: %d Xmult: 0x%02x, Ymult: 0x%02x",
                gspca_dev->curr_mode, xmult, ymult);
 
        /* setup qtable */
@@ -780,33 +697,35 @@ static void sd_start(struct gspca_dev *gspca_dev)
                 spca500_setmode(gspca_dev, xmult, ymult);
 
                /* enable drop packet */
-               reg_w(gspca_dev->dev, 0x00, 0x850a, 0x0001);
-               reg_w(gspca_dev->dev, 0x00, 0x8880, 3);
+               reg_w(gspca_dev, 0x00, 0x850a, 0x0001);
+               reg_w(gspca_dev, 0x00, 0x8880, 3);
                err = spca50x_setup_qtable(gspca_dev,
                                           0x00, 0x8800, 0x8840,
                                           qtable_creative_pccam);
                if (err < 0)
                        PDEBUG(D_ERR, "spca50x_setup_qtable failed");
                /* Init SDRAM - needed for SDRAM access */
-               reg_w(gspca_dev->dev, 0x00, 0x870a, 0x04);
+               reg_w(gspca_dev, 0x00, 0x870a, 0x04);
 
                /* switch to video camera mode */
-               reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
+               reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
                msleep(500);
-               if (reg_r_wait(gspca_dev->dev, 0, 0x8000, 0x44) != 0)
+               if (reg_r_wait(gspca_dev, 0, 0x8000, 0x44) != 0)
                        PDEBUG(D_ERR, "reg_r_wait() failed");
 
-               reg_r(gspca_dev->dev, 0x816b, &Data, 1);
-               reg_w(gspca_dev->dev, 0x00, 0x816b, Data);
+               reg_r(gspca_dev, 0x816b, 1);
+               Data = gspca_dev->usb_buf[0];
+               reg_w(gspca_dev, 0x00, 0x816b, Data);
 
                spca500_synch310(gspca_dev);
 
                write_vector(gspca_dev, spca500_visual_defaults);
                spca500_setmode(gspca_dev, xmult, ymult);
                /* enable drop packet */
-               reg_w(gspca_dev->dev, 0x00, 0x850a, 0x0001);
+               err = reg_w(gspca_dev, 0x00, 0x850a, 0x0001);
+               if (err < 0)
                        PDEBUG(D_ERR, "failed to enable drop packet");
-               reg_w(gspca_dev->dev, 0x00, 0x8880, 3);
+               reg_w(gspca_dev, 0x00, 0x8880, 3);
                err = spca50x_setup_qtable(gspca_dev,
                                           0x00, 0x8800, 0x8840,
                                           qtable_creative_pccam);
@@ -814,16 +733,17 @@ static void sd_start(struct gspca_dev *gspca_dev)
                        PDEBUG(D_ERR, "spca50x_setup_qtable failed");
 
                /* Init SDRAM - needed for SDRAM access */
-               reg_w(gspca_dev->dev, 0x00, 0x870a, 0x04);
+               reg_w(gspca_dev, 0x00, 0x870a, 0x04);
 
                /* switch to video camera mode */
-               reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
+               reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
 
-               if (reg_r_wait(gspca_dev->dev, 0, 0x8000, 0x44) != 0)
+               if (reg_r_wait(gspca_dev, 0, 0x8000, 0x44) != 0)
                        PDEBUG(D_ERR, "reg_r_wait() failed");
 
-               reg_r(gspca_dev->dev, 0x816b, &Data, 1);
-               reg_w(gspca_dev->dev, 0x00, 0x816b, Data);
+               reg_r(gspca_dev, 0x816b, 1);
+               Data = gspca_dev->usb_buf[0];
+               reg_w(gspca_dev, 0x00, 0x816b, Data);
                break;
        case CreativePCCam300:          /* Creative PC-CAM 300 640x480 CCD */
        case IntelPocketPCCamera:       /* FIXME: Temporary fix for
@@ -836,10 +756,10 @@ static void sd_start(struct gspca_dev *gspca_dev)
                        PDEBUG(D_ERR, "spca500_full_reset failed");
 
                /* enable drop packet */
-               err = reg_w(gspca_dev->dev, 0x00, 0x850a, 0x0001);
+               err = reg_w(gspca_dev, 0x00, 0x850a, 0x0001);
                if (err < 0)
                        PDEBUG(D_ERR, "failed to enable drop packet");
-               reg_w(gspca_dev->dev, 0x00, 0x8880, 3);
+               reg_w(gspca_dev, 0x00, 0x8880, 3);
                err = spca50x_setup_qtable(gspca_dev,
                                           0x00, 0x8800, 0x8840,
                                           qtable_creative_pccam);
@@ -847,16 +767,17 @@ static void sd_start(struct gspca_dev *gspca_dev)
                        PDEBUG(D_ERR, "spca50x_setup_qtable failed");
 
                spca500_setmode(gspca_dev, xmult, ymult);
-               reg_w(gspca_dev->dev, 0x20, 0x0001, 0x0004);
+               reg_w(gspca_dev, 0x20, 0x0001, 0x0004);
 
                /* switch to video camera mode */
-               reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
+               reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
 
-               if (reg_r_wait(gspca_dev->dev, 0, 0x8000, 0x44) != 0)
+               if (reg_r_wait(gspca_dev, 0, 0x8000, 0x44) != 0)
                        PDEBUG(D_ERR, "reg_r_wait() failed");
 
-               reg_r(gspca_dev->dev, 0x816b, &Data, 1);
-               reg_w(gspca_dev->dev, 0x00, 0x816b, Data);
+               reg_r(gspca_dev, 0x816b, 1);
+               Data = gspca_dev->usb_buf[0];
+               reg_w(gspca_dev, 0x00, 0x816b, Data);
 
 /*             write_vector(gspca_dev, spca500_visual_defaults); */
                break;
@@ -867,8 +788,8 @@ static void sd_start(struct gspca_dev *gspca_dev)
                if (err < 0)
                        PDEBUG(D_ERR, "spca500_full_reset failed");
                /* enable drop packet */
-               reg_w(gspca_dev->dev, 0x00, 0x850a, 0x0001);
-               reg_w(gspca_dev->dev, 0x00, 0x8880, 0);
+               reg_w(gspca_dev, 0x00, 0x850a, 0x0001);
+               reg_w(gspca_dev, 0x00, 0x8880, 0);
                err = spca50x_setup_qtable(gspca_dev,
                                           0x00, 0x8800, 0x8840,
                                           qtable_kodak_ez200);
@@ -876,16 +797,17 @@ static void sd_start(struct gspca_dev *gspca_dev)
                        PDEBUG(D_ERR, "spca50x_setup_qtable failed");
                spca500_setmode(gspca_dev, xmult, ymult);
 
-               reg_w(gspca_dev->dev, 0x20, 0x0001, 0x0004);
+               reg_w(gspca_dev, 0x20, 0x0001, 0x0004);
 
                /* switch to video camera mode */
-               reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
+               reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
 
-               if (reg_r_wait(gspca_dev->dev, 0, 0x8000, 0x44) != 0)
+               if (reg_r_wait(gspca_dev, 0, 0x8000, 0x44) != 0)
                        PDEBUG(D_ERR, "reg_r_wait() failed");
 
-               reg_r(gspca_dev->dev, 0x816b, &Data, 1);
-               reg_w(gspca_dev->dev, 0x00, 0x816b, Data);
+               reg_r(gspca_dev, 0x816b, 1);
+               Data = gspca_dev->usb_buf[0];
+               reg_w(gspca_dev, 0x00, 0x816b, Data);
 
 /*             write_vector(gspca_dev, spca500_visual_defaults); */
                break;
@@ -900,79 +822,80 @@ static void sd_start(struct gspca_dev *gspca_dev)
        case ToptroIndus:
        case AgfaCl20:
                spca500_reinit(gspca_dev);
-               reg_w(gspca_dev->dev, 0x00, 0x0d01, 0x01);
+               reg_w(gspca_dev, 0x00, 0x0d01, 0x01);
                /* enable drop packet */
-               reg_w(gspca_dev->dev, 0x00, 0x850a, 0x0001);
+               reg_w(gspca_dev, 0x00, 0x850a, 0x0001);
 
                err = spca50x_setup_qtable(gspca_dev,
                                   0x00, 0x8800, 0x8840, qtable_pocketdv);
                if (err < 0)
                        PDEBUG(D_ERR, "spca50x_setup_qtable failed");
-               reg_w(gspca_dev->dev, 0x00, 0x8880, 2);
+               reg_w(gspca_dev, 0x00, 0x8880, 2);
 
                /* familycam Quicksmart pocketDV stuff */
-               reg_w(gspca_dev->dev, 0x00, 0x800a, 0x00);
+               reg_w(gspca_dev, 0x00, 0x800a, 0x00);
                /* Set agc transfer: synced inbetween frames */
-               reg_w(gspca_dev->dev, 0x00, 0x820f, 0x01);
+               reg_w(gspca_dev, 0x00, 0x820f, 0x01);
                /* Init SDRAM - needed for SDRAM access */
-               reg_w(gspca_dev->dev, 0x00, 0x870a, 0x04);
+               reg_w(gspca_dev, 0x00, 0x870a, 0x04);
 
                spca500_setmode(gspca_dev, xmult, ymult);
                /* switch to video camera mode */
-               reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
+               reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
 
-               reg_r_wait(gspca_dev->dev, 0, 0x8000, 0x44);
+               reg_r_wait(gspca_dev, 0, 0x8000, 0x44);
 
-               reg_r(gspca_dev->dev, 0x816b, &Data, 1);
-               reg_w(gspca_dev->dev, 0x00, 0x816b, Data);
+               reg_r(gspca_dev, 0x816b, 1);
+               Data = gspca_dev->usb_buf[0];
+               reg_w(gspca_dev, 0x00, 0x816b, Data);
                break;
        case LogitechTraveler:
        case LogitechClickSmart510:
-               reg_w(gspca_dev->dev, 0x02, 0x00, 0x00);
+               reg_w(gspca_dev, 0x02, 0x00, 0x00);
                /* enable drop packet */
-               reg_w(gspca_dev->dev, 0x00, 0x850a, 0x0001);
+               reg_w(gspca_dev, 0x00, 0x850a, 0x0001);
 
                err = spca50x_setup_qtable(gspca_dev,
                                        0x00, 0x8800,
                                        0x8840, qtable_creative_pccam);
                if (err < 0)
                        PDEBUG(D_ERR, "spca50x_setup_qtable failed");
-               reg_w(gspca_dev->dev, 0x00, 0x8880, 3);
-               reg_w(gspca_dev->dev, 0x00, 0x800a, 0x00);
+               reg_w(gspca_dev, 0x00, 0x8880, 3);
+               reg_w(gspca_dev, 0x00, 0x800a, 0x00);
                /* Init SDRAM - needed for SDRAM access */
-               reg_w(gspca_dev->dev, 0x00, 0x870a, 0x04);
+               reg_w(gspca_dev, 0x00, 0x870a, 0x04);
 
                spca500_setmode(gspca_dev, xmult, ymult);
 
                /* switch to video camera mode */
-               reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
-               reg_r_wait(gspca_dev->dev, 0, 0x8000, 0x44);
+               reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
+               reg_r_wait(gspca_dev, 0, 0x8000, 0x44);
 
-               reg_r(gspca_dev->dev, 0x816b, &Data, 1);
-               reg_w(gspca_dev->dev, 0x00, 0x816b, Data);
+               reg_r(gspca_dev, 0x816b, 1);
+               Data = gspca_dev->usb_buf[0];
+               reg_w(gspca_dev, 0x00, 0x816b, Data);
                write_vector(gspca_dev, Clicksmart510_defaults);
                break;
        }
+       return 0;
 }
 
 static void sd_stopN(struct gspca_dev *gspca_dev)
 {
-       __u8 data;
-
-       reg_w(gspca_dev->dev, 0, 0x8003, 0x00);
+       reg_w(gspca_dev, 0, 0x8003, 0x00);
 
        /* switch to video camera mode */
-       reg_w(gspca_dev->dev, 0x00, 0x8000, 0x0004);
-       reg_r(gspca_dev->dev, 0x8000, &data, 1);
-       PDEBUG(D_STREAM, "stop SPCA500 done reg8000: 0x%2x", data);
+       reg_w(gspca_dev, 0x00, 0x8000, 0x0004);
+       reg_r(gspca_dev, 0x8000, 1);
+       PDEBUG(D_STREAM, "stop SPCA500 done reg8000: 0x%2x",
+               gspca_dev->usb_buf[0]);
 }
 
 static void sd_stop0(struct gspca_dev *gspca_dev)
 {
-}
+       struct sd *sd = (struct sd *) gspca_dev;
 
-static void sd_close(struct gspca_dev *gspca_dev)
-{
+       kfree(sd->jpeg_hdr);
 }
 
 static void sd_pkt_scan(struct gspca_dev *gspca_dev,
@@ -982,7 +905,6 @@ static void sd_pkt_scan(struct gspca_dev *gspca_dev,
 {
        struct sd *sd = (struct sd *) gspca_dev;
        int i;
-       __u8 *s, *d;
        static __u8 ffd9[] = {0xff, 0xd9};
 
 /* frames are jpeg 4.1.1 without 0xff escape */
@@ -995,7 +917,8 @@ static void sd_pkt_scan(struct gspca_dev *gspca_dev,
                                        ffd9, 2);
 
                /* put the JPEG header in the new frame */
-               jpeg_put_header(gspca_dev, frame, sd->qindex, 0x22);
+               gspca_frame_add(gspca_dev, FIRST_PACKET, frame,
+                       sd->jpeg_hdr, JPEG_HDR_SZ);
 
                data += SPCA500_OFFSET_DATA;
                len -= SPCA500_OFFSET_DATA;
@@ -1005,74 +928,41 @@ static void sd_pkt_scan(struct gspca_dev *gspca_dev,
        }
 
        /* add 0x00 after 0xff */
-       for (i = len; --i >= 0; )
-               if (data[i] == 0xff)
-                       break;
-       if (i < 0) {                    /* no 0xff */
-               gspca_frame_add(gspca_dev, INTER_PACKET, frame, data, len);
-               return;
-       }
-       s = data;
-       d = sd->packet;
-       for (i = 0; i < len; i++) {
-               *d++ = *s++;
-               if (s[-1] == 0xff)
-                       *d++ = 0x00;
-       }
-       gspca_frame_add(gspca_dev, INTER_PACKET, frame,
-                       sd->packet, d - sd->packet);
+       i = 0;
+       do {
+               if (data[i] == 0xff) {
+                       gspca_frame_add(gspca_dev, INTER_PACKET, frame,
+                                       data, i + 1);
+                       len -= i;
+                       data += i;
+                       *data = 0x00;
+                       i = 0;
+               }
+               i++;
+       } while (i < len);
+       gspca_frame_add(gspca_dev, INTER_PACKET, frame, data, len);
 }
 
 static void setbrightness(struct gspca_dev *gspca_dev)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       reg_w(gspca_dev->dev, 0x00, 0x8167,
+       reg_w(gspca_dev, 0x00, 0x8167,
                        (__u8) (sd->brightness - 128));
 }
 
-static void getbrightness(struct gspca_dev *gspca_dev)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       int ret;
-
-       ret = reg_r_12(gspca_dev->dev, 0x00, 0x8167, 1);
-       if (ret >= 0)
-               sd->brightness = ret + 128;
-}
-
 static void setcontrast(struct gspca_dev *gspca_dev)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       reg_w(gspca_dev->dev, 0x00, 0x8168, sd->contrast);
-}
-
-static void getcontrast(struct gspca_dev *gspca_dev)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       int ret;
-
-       ret = reg_r_12(gspca_dev->dev, 0x0, 0x8168, 1);
-       if (ret >= 0)
-               sd->contrast = ret;
+       reg_w(gspca_dev, 0x00, 0x8168, sd->contrast);
 }
 
 static void setcolors(struct gspca_dev *gspca_dev)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       reg_w(gspca_dev->dev, 0x00, 0x8169, sd->colors);
-}
-
-static void getcolors(struct gspca_dev *gspca_dev)
-{
-       struct sd *sd = (struct sd *) gspca_dev;
-       int ret;
-
-       ret = reg_r_12(gspca_dev->dev, 0x0, 0x8169, 1);
-       if (ret >= 0)
-               sd->colors = ret;
+       reg_w(gspca_dev, 0x00, 0x8169, sd->colors);
 }
 
 static int sd_setbrightness(struct gspca_dev *gspca_dev, __s32 val)
@@ -1089,7 +979,6 @@ static int sd_getbrightness(struct gspca_dev *gspca_dev, __s32 *val)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       getbrightness(gspca_dev);
        *val = sd->brightness;
        return 0;
 }
@@ -1108,7 +997,6 @@ static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       getcontrast(gspca_dev);
        *val = sd->contrast;
        return 0;
 }
@@ -1127,43 +1015,70 @@ static int sd_getcolors(struct gspca_dev *gspca_dev, __s32 *val)
 {
        struct sd *sd = (struct sd *) gspca_dev;
 
-       getcolors(gspca_dev);
        *val = sd->colors;
        return 0;
 }
 
+static int sd_set_jcomp(struct gspca_dev *gspca_dev,
+                       struct v4l2_jpegcompression *jcomp)
+{
+       struct sd *sd = (struct sd *) gspca_dev;
+
+       if (jcomp->quality < QUALITY_MIN)
+               sd->quality = QUALITY_MIN;
+       else if (jcomp->quality > QUALITY_MAX)
+               sd->quality = QUALITY_MAX;
+       else
+               sd->quality = jcomp->quality;
+       if (gspca_dev->streaming)
+               jpeg_set_qual(sd->jpeg_hdr, sd->quality);
+       return 0;
+}
+
+static int sd_get_jcomp(struct gspca_dev *gspca_dev,
+                       struct v4l2_jpegcompression *jcomp)
+{
+       struct sd *sd = (struct sd *) gspca_dev;
+
+       memset(jcomp, 0, sizeof *jcomp);
+       jcomp->quality = sd->quality;
+       jcomp->jpeg_markers = V4L2_JPEG_MARKER_DHT
+                       | V4L2_JPEG_MARKER_DQT;
+       return 0;
+}
+
 /* sub-driver description */
 static struct sd_desc sd_desc = {
        .name = MODULE_NAME,
        .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,
        .pkt_scan = sd_pkt_scan,
+       .get_jcomp = sd_get_jcomp,
+       .set_jcomp = sd_set_jcomp,
 };
 
 /* -- module initialisation -- */
-#define DVNM(name) .driver_info = (kernel_ulong_t) name
 static const __devinitdata struct usb_device_id device_table[] = {
-       {USB_DEVICE(0x040a, 0x0300), DVNM("Kodak EZ200")},
-       {USB_DEVICE(0x041e, 0x400a), DVNM("Creative PC-CAM 300")},
-       {USB_DEVICE(0x046d, 0x0890), DVNM("Logitech QuickCam traveler")},
-       {USB_DEVICE(0x046d, 0x0900), DVNM("Logitech Inc. ClickSmart 310")},
-       {USB_DEVICE(0x046d, 0x0901), DVNM("Logitech Inc. ClickSmart 510")},
-       {USB_DEVICE(0x04a5, 0x300c), DVNM("Benq DC1016")},
-       {USB_DEVICE(0x04fc, 0x7333), DVNM("PalmPixDC85")},
-       {USB_DEVICE(0x055f, 0xc200), DVNM("Mustek Gsmart 300")},
-       {USB_DEVICE(0x055f, 0xc220), DVNM("Gsmart Mini")},
-       {USB_DEVICE(0x06bd, 0x0404), DVNM("Agfa CL20")},
-       {USB_DEVICE(0x06be, 0x0800), DVNM("Optimedia")},
-       {USB_DEVICE(0x084d, 0x0003), DVNM("D-Link DSC-350")},
-       {USB_DEVICE(0x08ca, 0x0103), DVNM("Aiptek PocketDV")},
-       {USB_DEVICE(0x2899, 0x012c), DVNM("Toptro Industrial")},
-       {USB_DEVICE(0x8086, 0x0630), DVNM("Intel Pocket PC Camera")},
+       {USB_DEVICE(0x040a, 0x0300), .driver_info = KodakEZ200},
+       {USB_DEVICE(0x041e, 0x400a), .driver_info = CreativePCCam300},
+       {USB_DEVICE(0x046d, 0x0890), .driver_info = LogitechTraveler},
+       {USB_DEVICE(0x046d, 0x0900), .driver_info = LogitechClickSmart310},
+       {USB_DEVICE(0x046d, 0x0901), .driver_info = LogitechClickSmart510},
+       {USB_DEVICE(0x04a5, 0x300c), .driver_info = BenqDC1016},
+       {USB_DEVICE(0x04fc, 0x7333), .driver_info = PalmPixDC85},
+       {USB_DEVICE(0x055f, 0xc200), .driver_info = MustekGsmart300},
+       {USB_DEVICE(0x055f, 0xc220), .driver_info = Gsmartmini},
+       {USB_DEVICE(0x06bd, 0x0404), .driver_info = AgfaCl20},
+       {USB_DEVICE(0x06be, 0x0800), .driver_info = Optimedia},
+       {USB_DEVICE(0x084d, 0x0003), .driver_info = DLinkDSC350},
+       {USB_DEVICE(0x08ca, 0x0103), .driver_info = AiptekPocketDV},
+       {USB_DEVICE(0x2899, 0x012c), .driver_info = ToptroIndus},
+       {USB_DEVICE(0x8086, 0x0630), .driver_info = IntelPocketPCCamera},
        {}
 };
 MODULE_DEVICE_TABLE(usb, device_table);
@@ -1181,14 +1096,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)
 {
-       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)