V4L/DVB (11648): gspca - m5602: Remove some needless error checking and add comments

Signed-off-by: Erik Andrén <erik.andren@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

diff --git a/drivers/media/video/gspca/m5602/m5602_core.c b/drivers/media/video/gspca/m5602/m5602_core.c
index ca77dab907504548ec28330f83c5301961465871..8ca5643f2c7a4add7ad00b26a6445e877c35c72d 100644 (file)
--- a/drivers/media/video/gspca/m5602/m5602_core.c
+++ b/drivers/media/video/gspca/m5602/m5602_core.c
@@ -112,18 +112,16 @@ int m5602_read_sensor(struct sd *sd, const u8 address,
        if (err < 0)
                return err;
 
+       /* Sensors with registers that only are one byte width are differently read */
+       /* FIXME: This works with the ov9650, but has issues with the po1030 */
        if (sd->sensor->i2c_regW == 1) {
-               err = m5602_write_bridge(sd, M5602_XB_I2C_CTRL, len);
+               err = m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 1);
                if (err < 0)
                        return err;
 
                err = m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 0x08);
-               if (err < 0)
-                       return err;
        } else {
                err = m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 0x18 + len);
-               if (err < 0)
-                       return err;
        }
 
        for (i = 0; (i < len) && !err; i++) {