[media] gspca - sq930x: Don't register a webcam when there are USB errors
authorJean-François Moine <moinejf@free.fr>
Sat, 25 Dec 2010 16:07:57 +0000 (13:07 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 29 Dec 2010 10:17:05 +0000 (08:17 -0200)
Signed-off-by: Jean-François Moine <moinejf@free.fr>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/gspca/sq930x.c

index 3e4b0b94c700b5a3b64c153555c9ea1949f7216c..74628eda7d96de82222e316b9a6d87e4f583f21d 100644 (file)
@@ -687,10 +687,12 @@ static void cmos_probe(struct gspca_dev *gspca_dev)
                if (gspca_dev->usb_buf[0] != 0)
                        break;
        }
-       if (i >= ARRAY_SIZE(probe_order))
+       if (i >= ARRAY_SIZE(probe_order)) {
                err("Unknown sensor");
-       else
-               sd->sensor = probe_order[i];
+               gspca_dev->usb_err = -EINVAL;
+               return;
+       }
+       sd->sensor = probe_order[i];
 }
 
 static void mt9v111_init(struct gspca_dev *gspca_dev)
@@ -867,6 +869,9 @@ static int sd_init(struct gspca_dev *gspca_dev)
  */
 
        reg_r(gspca_dev, SQ930_CTRL_GET_DEV_INFO, 8);
+       if (gspca_dev->usb_err < 0)
+               return gspca_dev->usb_err;
+
 /* it returns:
  * 03 00 12 93 0b f6 c9 00     live! ultra
  * 03 00 07 93 0b f6 ca 00     live! ultra for notebook
@@ -905,10 +910,10 @@ static int sd_init(struct gspca_dev *gspca_dev)
                else
                        cmos_probe(gspca_dev);
        }
-
-       PDEBUG(D_PROBE, "Sensor %s", sensor_tb[sd->sensor].name);
-
-       global_init(sd, 1);
+       if (gspca_dev->usb_err >= 0) {
+               PDEBUG(D_PROBE, "Sensor %s", sensor_tb[sd->sensor].name);
+               global_init(sd, 1);
+       }
        return gspca_dev->usb_err;
 }