V4L/DVB (10014): gspca - m5602: Remove all sensor specific write functions.
[pandora-kernel.git] / drivers / media / video / gspca / m5602 / m5602_po1030.c
index d17ac52..f52a3cd 100644 (file)
@@ -18,6 +18,8 @@
 
 #include "m5602_po1030.h"
 
+static void po1030_dump_registers(struct sd *sd);
+
 int po1030_probe(struct sd *sd)
 {
        u8 prod_id = 0, ver_id = 0, i;
@@ -38,7 +40,7 @@ int po1030_probe(struct sd *sd)
        for (i = 0; i < ARRAY_SIZE(preinit_po1030); i++) {
                u8 data = preinit_po1030[i][2];
                if (preinit_po1030[i][0] == SENSOR)
-                       po1030_write_sensor(sd,
+                       m5602_write_sensor(sd,
                                preinit_po1030[i][1], &data, 1);
                else
                        m5602_write_bridge(sd, preinit_po1030[i][1], data);
@@ -73,61 +75,28 @@ int po1030_read_sensor(struct sd *sd, const u8 address,
                err = m5602_read_bridge(sd, M5602_XB_I2C_STATUS, i2c_data);
        } while ((*i2c_data & I2C_BUSY) && !err);
 
-       m5602_write_bridge(sd, M5602_XB_I2C_DEV_ADDR,
-                          sd->sensor->i2c_slave_id);
-       m5602_write_bridge(sd, M5602_XB_I2C_REG_ADDR, address);
-       m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 0x10 + len);
-       m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 0x08);
+       err = m5602_write_bridge(sd, M5602_XB_I2C_DEV_ADDR,
+                                sd->sensor->i2c_slave_id);
+       if (err < 0)
+               goto out;
+       err = m5602_write_bridge(sd, M5602_XB_I2C_REG_ADDR, address);
+       if (err < 0)
+               goto out;
+       err = m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 0x10 + len);
+       if (err < 0)
+               goto out;
+       err = m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 0x08);
+       if (err < 0)
+               goto out;
 
-       for (i = 0; i < len; i++) {
+       for (i = 0; (i < len) && !err; i++) {
                err = m5602_read_bridge(sd, M5602_XB_I2C_DATA, &(i2c_data[i]));
 
                PDEBUG(D_CONF, "Reading sensor register "
                                "0x%x containing 0x%x ", address, *i2c_data);
        }
-       return (err < 0) ? err : 0;
-}
-
-int po1030_write_sensor(struct sd *sd, const u8 address,
-                       u8 *i2c_data, const u8 len)
-{
-       int err, i;
-       u8 *p;
-       struct usb_device *udev = sd->gspca_dev.dev;
-       __u8 *buf = sd->gspca_dev.usb_buf;
-
-       /* The po1030 only supports one byte writes */
-       if (len > 1 || !len)
-               return -EINVAL;
-
-       memcpy(buf, sensor_urb_skeleton, sizeof(sensor_urb_skeleton));
-
-       buf[11] = sd->sensor->i2c_slave_id;
-       buf[15] = address;
-
-       p = buf + 16;
-
-       /* Copy a four byte write sequence for each byte to be written to */
-       for (i = 0; i < len; i++) {
-               memcpy(p, sensor_urb_skeleton + 16, 4);
-               p[3] = i2c_data[i];
-               p += 4;
-               PDEBUG(D_CONF, "Writing sensor register 0x%x with 0x%x",
-                      address, i2c_data[i]);
-       }
-
-       /* Copy the footer */
-       memcpy(p, sensor_urb_skeleton + 20, 4);
-
-       /* Set the total length */
-       p[3] = 0x10 + len;
-
-       err = usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
-                             0x04, 0x40, 0x19,
-                             0x0000, buf,
-                             20 + len * 4, M5602_URB_MSG_TIMEOUT);
-
-       return (err < 0) ? err : 0;
+out:
+       return err;
 }
 
 int po1030_init(struct sd *sd)
@@ -135,7 +104,7 @@ int po1030_init(struct sd *sd)
        int i, err = 0;
 
        /* Init the sensor */
-       for (i = 0; i < ARRAY_SIZE(init_po1030); i++) {
+       for (i = 0; i < ARRAY_SIZE(init_po1030) && !err; i++) {
                u8 data[2] = {0x00, 0x00};
 
                switch (init_po1030[i][0]) {
@@ -147,14 +116,14 @@ int po1030_init(struct sd *sd)
 
                case SENSOR:
                        data[0] = init_po1030[i][2];
-                       err = po1030_write_sensor(sd,
+                       err = m5602_write_sensor(sd,
                                init_po1030[i][1], data, 1);
                        break;
 
                case SENSOR_LONG:
                        data[0] = init_po1030[i][2];
                        data[1] = init_po1030[i][3];
-                       err = po1030_write_sensor(sd,
+                       err = m5602_write_sensor(sd,
                                init_po1030[i][1], data, 2);
                        break;
                default:
@@ -166,7 +135,7 @@ int po1030_init(struct sd *sd)
        if (dump_sensor)
                po1030_dump_registers(sd);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_get_exposure(struct gspca_dev *gspca_dev, __s32 *val)
@@ -187,7 +156,7 @@ int po1030_get_exposure(struct gspca_dev *gspca_dev, __s32 *val)
 
        PDEBUG(D_V4L2, "Exposure read as %d", *val);
 out:
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_set_exposure(struct gspca_dev *gspca_dev, __s32 val)
@@ -202,7 +171,7 @@ int po1030_set_exposure(struct gspca_dev *gspca_dev, __s32 val)
        PDEBUG(D_V4L2, "Set exposure to high byte to 0x%x",
               i2c_data);
 
-       err = po1030_write_sensor(sd, PO1030_REG_INTEGLINES_H,
+       err = m5602_write_sensor(sd, PO1030_REG_INTEGLINES_H,
                                  &i2c_data, 1);
        if (err < 0)
                goto out;
@@ -210,11 +179,11 @@ int po1030_set_exposure(struct gspca_dev *gspca_dev, __s32 val)
        i2c_data = (val & 0xff);
        PDEBUG(D_V4L2, "Set exposure to low byte to 0x%x",
               i2c_data);
-       err = po1030_write_sensor(sd, PO1030_REG_INTEGLINES_M,
+       err = m5602_write_sensor(sd, PO1030_REG_INTEGLINES_M,
                                  &i2c_data, 1);
 
 out:
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_get_gain(struct gspca_dev *gspca_dev, __s32 *val)
@@ -228,7 +197,7 @@ int po1030_get_gain(struct gspca_dev *gspca_dev, __s32 *val)
        *val = i2c_data;
        PDEBUG(D_V4L2, "Read global gain %d", *val);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_get_hflip(struct gspca_dev *gspca_dev, __s32 *val)
@@ -244,7 +213,7 @@ int po1030_get_hflip(struct gspca_dev *gspca_dev, __s32 *val)
 
        PDEBUG(D_V4L2, "Read hflip %d", *val);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_set_hflip(struct gspca_dev *gspca_dev, __s32 val)
@@ -257,10 +226,10 @@ int po1030_set_hflip(struct gspca_dev *gspca_dev, __s32 val)
 
        i2c_data = (val & 0x01) << 7;
 
-       err = po1030_write_sensor(sd, PO1030_REG_CONTROL2,
+       err = m5602_write_sensor(sd, PO1030_REG_CONTROL2,
                                  &i2c_data, 1);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_get_vflip(struct gspca_dev *gspca_dev, __s32 *val)
@@ -276,7 +245,7 @@ int po1030_get_vflip(struct gspca_dev *gspca_dev, __s32 *val)
 
        PDEBUG(D_V4L2, "Read vflip %d", *val);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_set_vflip(struct gspca_dev *gspca_dev, __s32 val)
@@ -289,10 +258,10 @@ int po1030_set_vflip(struct gspca_dev *gspca_dev, __s32 val)
 
        i2c_data = (val & 0x01) << 6;
 
-       err = po1030_write_sensor(sd, PO1030_REG_CONTROL2,
+       err = m5602_write_sensor(sd, PO1030_REG_CONTROL2,
                                  &i2c_data, 1);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_set_gain(struct gspca_dev *gspca_dev, __s32 val)
@@ -303,9 +272,9 @@ int po1030_set_gain(struct gspca_dev *gspca_dev, __s32 val)
 
        i2c_data = val & 0xff;
        PDEBUG(D_V4L2, "Set global gain to %d", i2c_data);
-       err = po1030_write_sensor(sd, PO1030_REG_GLOBALGAIN,
+       err = m5602_write_sensor(sd, PO1030_REG_GLOBALGAIN,
                                  &i2c_data, 1);
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_get_red_balance(struct gspca_dev *gspca_dev, __s32 *val)
@@ -318,7 +287,7 @@ int po1030_get_red_balance(struct gspca_dev *gspca_dev, __s32 *val)
                                 &i2c_data, 1);
        *val = i2c_data;
        PDEBUG(D_V4L2, "Read red gain %d", *val);
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_set_red_balance(struct gspca_dev *gspca_dev, __s32 val)
@@ -329,9 +298,9 @@ int po1030_set_red_balance(struct gspca_dev *gspca_dev, __s32 val)
 
        i2c_data = val & 0xff;
        PDEBUG(D_V4L2, "Set red gain to %d", i2c_data);
-       err = po1030_write_sensor(sd, PO1030_REG_RED_GAIN,
+       err = m5602_write_sensor(sd, PO1030_REG_RED_GAIN,
                                  &i2c_data, 1);
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_get_blue_balance(struct gspca_dev *gspca_dev, __s32 *val)
@@ -345,7 +314,7 @@ int po1030_get_blue_balance(struct gspca_dev *gspca_dev, __s32 *val)
        *val = i2c_data;
        PDEBUG(D_V4L2, "Read blue gain %d", *val);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_set_blue_balance(struct gspca_dev *gspca_dev, __s32 val)
@@ -355,10 +324,10 @@ int po1030_set_blue_balance(struct gspca_dev *gspca_dev, __s32 val)
        int err;
        i2c_data = val & 0xff;
        PDEBUG(D_V4L2, "Set blue gain to %d", i2c_data);
-       err = po1030_write_sensor(sd, PO1030_REG_BLUE_GAIN,
+       err = m5602_write_sensor(sd, PO1030_REG_BLUE_GAIN,
                                  &i2c_data, 1);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int po1030_power_down(struct sd *sd)
@@ -366,7 +335,7 @@ int po1030_power_down(struct sd *sd)
        return 0;
 }
 
-void po1030_dump_registers(struct sd *sd)
+static void po1030_dump_registers(struct sd *sd)
 {
        int address;
        u8 value = 0;
@@ -386,7 +355,7 @@ void po1030_dump_registers(struct sd *sd)
                u8 test_value[2] = {0xff, 0xff};
 
                po1030_read_sensor(sd, address, &old_value, 1);
-               po1030_write_sensor(sd, address, test_value, 1);
+               m5602_write_sensor(sd, address, test_value, 1);
                po1030_read_sensor(sd, address, &ctrl_value, 1);
 
                if (ctrl_value == test_value[0])
@@ -395,6 +364,6 @@ void po1030_dump_registers(struct sd *sd)
                        info("register 0x%x is read only", address);
 
                /* Restore original value */
-               po1030_write_sensor(sd, address, &old_value, 1);
+               m5602_write_sensor(sd, address, &old_value, 1);
        }
 }