[media] gspca: Fix some warnings tied to 'no debug'
authorJean-François Moine <moinejf@free.fr>
Tue, 17 May 2011 08:08:59 +0000 (05:08 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 21 May 2011 12:35:43 +0000 (09:35 -0300)
This patch fixes the warnings raised by unsetting the option GSPCA_DEBUG.

Signed-off-by: Jean-François Moine <moinejf@free.fr>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/gspca/gl860/gl860.c
drivers/media/video/gspca/spca508.c
drivers/media/video/gspca/zc3xx.c

index 9908303..e8e071a 100644 (file)
@@ -499,21 +499,8 @@ MODULE_DEVICE_TABLE(usb, device_table);
 static int sd_probe(struct usb_interface *intf,
                                const struct usb_device_id *id)
 {
-       struct gspca_dev *gspca_dev;
-       s32 ret;
-
-       ret = gspca_dev_probe(intf, id,
+       return gspca_dev_probe(intf, id,
                        &sd_desc_mi1320, sizeof(struct sd), THIS_MODULE);
-
-       if (ret >= 0) {
-               gspca_dev = usb_get_intfdata(intf);
-
-               PDEBUG(D_PROBE,
-                       "Camera is now controlling video device %s",
-                       video_device_node_name(&gspca_dev->vdev));
-       }
-
-       return ret;
 }
 
 static void sd_disconnect(struct usb_interface *intf)
index 41dce49..9d0b460 100644 (file)
@@ -1375,7 +1375,6 @@ static int sd_config(struct gspca_dev *gspca_dev,
 {
        struct sd *sd = (struct sd *) gspca_dev;
        struct cam *cam;
-       int data1, data2;
        const u16 (*init_data)[2];
        static const u16 (*(init_data_tb[]))[2] = {
                spca508_vista_init_data,        /* CreativeVista 0 */
@@ -1386,6 +1385,9 @@ static int sd_config(struct gspca_dev *gspca_dev,
                spca508_init_data,              /* ViewQuestVQ110 5 */
        };
 
+#ifdef GSPCA_DEBUG
+       int data1, data2;
+
        /* Read from global register the USB product and vendor IDs, just to
         * prove that we can communicate with the device.  This works, which
         * confirms at we are communicating properly and that the device
@@ -1400,6 +1402,7 @@ static int sd_config(struct gspca_dev *gspca_dev,
 
        data1 = reg_read(gspca_dev, 0x8621);
        PDEBUG(D_PROBE, "Window 1 average luminance: %d", data1);
+#endif
 
        cam = &gspca_dev->cam;
        cam->cam_mode = sif_mode;
index dfb1f81..61cdd56 100644 (file)
@@ -6266,7 +6266,6 @@ static int vga_3wr_probe(struct gspca_dev *gspca_dev)
 {
        struct sd *sd = (struct sd *) gspca_dev;
        int i;
-       u8 retbyte;
        u16 retword;
 
 /*fixme: lack of 8b=b3 (11,12)-> 10, 8b=e0 (14,15,16)-> 12 found in gspcav1*/
@@ -6359,8 +6358,12 @@ static int vga_3wr_probe(struct gspca_dev *gspca_dev)
        retword |= i2c_read(gspca_dev, 0x01);           /* ID 1 */
        PDEBUG(D_PROBE, "probe 3wr vga 2 0x%04x", retword);
        if (retword == 0x2030) {
+#ifdef GSPCA_DEBUG
+               u8 retbyte;
+
                retbyte = i2c_read(gspca_dev, 0x02);    /* revision number */
                PDEBUG(D_PROBE, "sensor PO2030 rev 0x%02x", retbyte);
+#endif
                send_unknown(gspca_dev, SENSOR_PO2030);
                return retword;
        }