V4L/DVB (10027): m5602: convert the s5k83a sensor to use the common function
[pandora-kernel.git] / drivers / media / video / gspca / m5602 / m5602_s5k83a.c
index 8988a72..811766b 100644 (file)
@@ -18,6 +18,8 @@
 
 #include "m5602_s5k83a.h"
 
+static void s5k83a_dump_registers(struct sd *sd);
+
 int s5k83a_probe(struct sd *sd)
 {
        u8 prod_id = 0, ver_id = 0;
@@ -39,7 +41,7 @@ int s5k83a_probe(struct sd *sd)
        for (i = 0; i < ARRAY_SIZE(preinit_s5k83a) && !err; i++) {
                u8 data[2] = {preinit_s5k83a[i][2], preinit_s5k83a[i][3]};
                if (preinit_s5k83a[i][0] == SENSOR)
-                       err = s5k83a_write_sensor(sd, preinit_s5k83a[i][1],
+                       err = m5602_write_sensor(sd, preinit_s5k83a[i][1],
                                data, 2);
                else
                        err = m5602_write_bridge(sd, preinit_s5k83a[i][1],
@@ -49,10 +51,10 @@ int s5k83a_probe(struct sd *sd)
        /* We don't know what register (if any) that contain the product id
         * Just pick the first addresses that seem to produce the same results
         * on multiple machines */
-       if (s5k83a_read_sensor(sd, 0x00, &prod_id, 1))
+       if (m5602_read_sensor(sd, 0x00, &prod_id, 1))
                return -ENODEV;
 
-       if (s5k83a_read_sensor(sd, 0x01, &ver_id, 1))
+       if (m5602_read_sensor(sd, 0x01, &ver_id, 1))
                return -ENODEV;
 
        if ((prod_id == 0xff) || (ver_id == 0xff))
@@ -69,7 +71,7 @@ sensor_found:
 }
 
 int s5k83a_read_sensor(struct sd *sd, const u8 address,
-                             u8 *i2c_data, const u8 len)
+                      u8 *i2c_data, const u8 len)
 {
        int err, i;
 
@@ -89,15 +91,11 @@ int s5k83a_read_sensor(struct sd *sd, const u8 address,
                goto out;
 
        err = m5602_write_bridge(sd, M5602_XB_I2C_CTRL, 0x18 + len);
-       if (err < 0)
-               goto out;
 
        do {
                err = m5602_read_bridge(sd, M5602_XB_I2C_STATUS, i2c_data);
        } while ((*i2c_data & I2C_BUSY) && !err);
 
-       if (err < 0)
-               goto out;
        for (i = 0; i < len && !len; i++) {
                err = m5602_read_bridge(sd, M5602_XB_I2C_DATA, &(i2c_data[i]));
 
@@ -106,51 +104,7 @@ int s5k83a_read_sensor(struct sd *sd, const u8 address,
        }
 
 out:
-       return (err < 0) ? err : 0;
-}
-
-int s5k83a_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;
-
-       /* No sensor with a data width larger than 16 bits has yet been seen */
-       if (len > 2 || !len)
-               return -EINVAL;
-
-       memcpy(buf, sensor_urb_skeleton,
-              sizeof(sensor_urb_skeleton));
-
-       buf[11] = sd->sensor->i2c_slave_id;
-       buf[15] = address;
-
-       /* Special case larger sensor writes */
-       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 tailer */
-       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;
+       return err;
 }
 
 int s5k83a_init(struct sd *sd)
@@ -169,14 +123,14 @@ int s5k83a_init(struct sd *sd)
 
                case SENSOR:
                        data[0] = init_s5k83a[i][2];
-                       err = s5k83a_write_sensor(sd,
+                       err = m5602_write_sensor(sd,
                                init_s5k83a[i][1], data, 1);
                        break;
 
                case SENSOR_LONG:
                        data[0] = init_s5k83a[i][2];
                        data[1] = init_s5k83a[i][3];
-                       err = s5k83a_write_sensor(sd,
+                       err = m5602_write_sensor(sd,
                                init_s5k83a[i][1], data, 2);
                        break;
                default:
@@ -200,14 +154,14 @@ void s5k83a_dump_registers(struct sd *sd)
 {
        int address;
        u8 page, old_page;
-       s5k83a_read_sensor(sd, S5K83A_PAGE_MAP, &old_page, 1);
+       m5602_read_sensor(sd, S5K83A_PAGE_MAP, &old_page, 1);
 
        for (page = 0; page < 16; page++) {
-               s5k83a_write_sensor(sd, S5K83A_PAGE_MAP, &page, 1);
+               m5602_write_sensor(sd, S5K83A_PAGE_MAP, &page, 1);
                info("Dumping the s5k83a register state for page 0x%x", page);
                for (address = 0; address <= 0xff; address++) {
                        u8 val = 0;
-                       s5k83a_read_sensor(sd, address, &val, 1);
+                       m5602_read_sensor(sd, address, &val, 1);
                        info("register 0x%x contains 0x%x",
                             address, val);
                }
@@ -215,15 +169,15 @@ void s5k83a_dump_registers(struct sd *sd)
        info("s5k83a register state dump complete");
 
        for (page = 0; page < 16; page++) {
-               s5k83a_write_sensor(sd, S5K83A_PAGE_MAP, &page, 1);
+               m5602_write_sensor(sd, S5K83A_PAGE_MAP, &page, 1);
                info("Probing for which registers that are read/write "
                      "for page 0x%x", page);
                for (address = 0; address <= 0xff; address++) {
                        u8 old_val, ctrl_val, test_val = 0xff;
 
-                       s5k83a_read_sensor(sd, address, &old_val, 1);
-                       s5k83a_write_sensor(sd, address, &test_val, 1);
-                       s5k83a_read_sensor(sd, address, &ctrl_val, 1);
+                       m5602_read_sensor(sd, address, &old_val, 1);
+                       m5602_write_sensor(sd, address, &test_val, 1);
+                       m5602_read_sensor(sd, address, &ctrl_val, 1);
 
                        if (ctrl_val == test_val)
                                info("register 0x%x is writeable", address);
@@ -231,11 +185,11 @@ void s5k83a_dump_registers(struct sd *sd)
                                info("register 0x%x is read only", address);
 
                        /* Restore original val */
-                       s5k83a_write_sensor(sd, address, &old_val, 1);
+                       m5602_write_sensor(sd, address, &old_val, 1);
                }
        }
        info("Read/write register probing complete");
-       s5k83a_write_sensor(sd, S5K83A_PAGE_MAP, &old_page, 1);
+       m5602_write_sensor(sd, S5K83A_PAGE_MAP, &old_page, 1);
 }
 
 int s5k83a_get_brightness(struct gspca_dev *gspca_dev, __s32 *val)
@@ -244,11 +198,15 @@ int s5k83a_get_brightness(struct gspca_dev *gspca_dev, __s32 *val)
        u8 data[2];
        struct sd *sd = (struct sd *) gspca_dev;
 
-       err = s5k83a_read_sensor(sd, S5K83A_BRIGHTNESS, data, 2);
+       err = m5602_read_sensor(sd, S5K83A_BRIGHTNESS, data, 2);
+       if (err < 0)
+               goto out;
+
        data[1] = data[1] << 1;
        *val = data[1];
 
-       return (err < 0) ? err : 0;
+out:
+       return err;
 }
 
 int s5k83a_set_brightness(struct gspca_dev *gspca_dev, __s32 val)
@@ -259,23 +217,24 @@ int s5k83a_set_brightness(struct gspca_dev *gspca_dev, __s32 val)
 
        data[0] = 0x00;
        data[1] = 0x20;
-       err = s5k83a_write_sensor(sd, 0x14, data, 2);
+       err = m5602_write_sensor(sd, 0x14, data, 2);
        if (err < 0)
-               return err;
+               goto out;
 
        data[0] = 0x01;
        data[1] = 0x00;
-       err = s5k83a_write_sensor(sd, 0x0d, data, 2);
+       err = m5602_write_sensor(sd, 0x0d, data, 2);
        if (err < 0)
-               return err;
+               goto out;
 
        /* FIXME: This is not sane, we need to figure out the composition
                  of these registers */
        data[0] = val >> 3; /* brightness, high 5 bits */
        data[1] = val >> 1; /* brightness, high 7 bits */
-       err = s5k83a_write_sensor(sd, S5K83A_BRIGHTNESS, data, 2);
+       err = m5602_write_sensor(sd, S5K83A_BRIGHTNESS, data, 2);
 
-       return (err < 0) ? err : 0;
+out:
+       return err;
 }
 
 int s5k83a_get_whiteness(struct gspca_dev *gspca_dev, __s32 *val)
@@ -284,10 +243,14 @@ int s5k83a_get_whiteness(struct gspca_dev *gspca_dev, __s32 *val)
        u8 data;
        struct sd *sd = (struct sd *) gspca_dev;
 
-       err = s5k83a_read_sensor(sd, S5K83A_WHITENESS, &data, 1);
+       err = m5602_read_sensor(sd, S5K83A_WHITENESS, &data, 1);
+       if (err < 0)
+               goto out;
 
        *val = data;
-       return (err < 0) ? err : 0;
+
+out:
+       return err;
 }
 
 int s5k83a_set_whiteness(struct gspca_dev *gspca_dev, __s32 val)
@@ -297,9 +260,9 @@ int s5k83a_set_whiteness(struct gspca_dev *gspca_dev, __s32 val)
        struct sd *sd = (struct sd *) gspca_dev;
 
        data[0] = val;
-       err = s5k83a_write_sensor(sd, S5K83A_WHITENESS, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_WHITENESS, data, 1);
 
-       return (err < 0) ? err : 0;
+       return err;
 }
 
 int s5k83a_get_gain(struct gspca_dev *gspca_dev, __s32 *val)
@@ -308,7 +271,9 @@ int s5k83a_get_gain(struct gspca_dev *gspca_dev, __s32 *val)
        u8 data[2];
        struct sd *sd = (struct sd *) gspca_dev;
 
-       err = s5k83a_read_sensor(sd, S5K83A_GAIN, data, 2);
+       err = m5602_read_sensor(sd, S5K83A_GAIN, data, 2);
+       if (err < 0)
+               goto out;
 
        data[1] = data[1] & 0x3f;
        if (data[1] > S5K83A_MAXIMUM_GAIN)
@@ -316,7 +281,8 @@ int s5k83a_get_gain(struct gspca_dev *gspca_dev, __s32 *val)
 
        *val = data[1];
 
-       return (err < 0) ? err : 0;
+out:
+       return err;
 }
 
 int s5k83a_set_gain(struct gspca_dev *gspca_dev, __s32 val)
@@ -327,9 +293,8 @@ int s5k83a_set_gain(struct gspca_dev *gspca_dev, __s32 val)
 
        data[0] = 0;
        data[1] = val;
-       err = s5k83a_write_sensor(sd, S5K83A_GAIN, data, 2);
-
-       return (err < 0) ? err : 0;
+       err = m5602_write_sensor(sd, S5K83A_GAIN, data, 2);
+       return err;
 }
 
 int s5k83a_get_vflip(struct gspca_dev *gspca_dev, __s32 *val)
@@ -339,14 +304,15 @@ int s5k83a_get_vflip(struct gspca_dev *gspca_dev, __s32 *val)
        struct sd *sd = (struct sd *) gspca_dev;
 
        data[0] = 0x05;
-       err = s5k83a_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
-       err = s5k83a_read_sensor(sd, S5K83A_FLIP, data, 1);
+       err = m5602_read_sensor(sd, S5K83A_FLIP, data, 1);
        *val = (data[0] | 0x40) ? 1 : 0;
 
-       return (err < 0) ? err : 0;
+out:
+       return err;
 }
 
 int s5k83a_set_vflip(struct gspca_dev *gspca_dev, __s32 val)
@@ -356,25 +322,26 @@ int s5k83a_set_vflip(struct gspca_dev *gspca_dev, __s32 val)
        struct sd *sd = (struct sd *) gspca_dev;
 
        data[0] = 0x05;
-       err = s5k83a_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
-       err = s5k83a_read_sensor(sd, S5K83A_FLIP, data, 1);
+       err = m5602_read_sensor(sd, S5K83A_FLIP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
        /* set or zero six bit, seven is hflip */
        data[0] = (val) ? (data[0] & 0x80) | 0x40 | S5K83A_FLIP_MASK
                        : (data[0] & 0x80) | S5K83A_FLIP_MASK;
-       err = s5k83a_write_sensor(sd, S5K83A_FLIP, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_FLIP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
        data[0] = (val) ? 0x0b : 0x0a;
-       err = s5k83a_write_sensor(sd, S5K83A_VFLIP_TUNE, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_VFLIP_TUNE, data, 1);
 
-       return (err < 0) ? err : 0;
+out:
+       return err;
 }
 
 int s5k83a_get_hflip(struct gspca_dev *gspca_dev, __s32 *val)
@@ -384,14 +351,15 @@ int s5k83a_get_hflip(struct gspca_dev *gspca_dev, __s32 *val)
        struct sd *sd = (struct sd *) gspca_dev;
 
        data[0] = 0x05;
-       err = s5k83a_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
-       err = s5k83a_read_sensor(sd, S5K83A_FLIP, data, 1);
+       err = m5602_read_sensor(sd, S5K83A_FLIP, data, 1);
        *val = (data[0] | 0x80) ? 1 : 0;
 
-       return (err < 0) ? err : 0;
+out:
+       return err;
 }
 
 int s5k83a_set_hflip(struct gspca_dev *gspca_dev, __s32 val)
@@ -401,23 +369,23 @@ int s5k83a_set_hflip(struct gspca_dev *gspca_dev, __s32 val)
        struct sd *sd = (struct sd *) gspca_dev;
 
        data[0] = 0x05;
-       err = s5k83a_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_PAGE_MAP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
-       err = s5k83a_read_sensor(sd, S5K83A_FLIP, data, 1);
+       err = m5602_read_sensor(sd, S5K83A_FLIP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
        /* set or zero seven bit, six is vflip */
        data[0] = (val) ? (data[0] & 0x40) | 0x80 | S5K83A_FLIP_MASK
                        : (data[0] & 0x40) | S5K83A_FLIP_MASK;
-       err = s5k83a_write_sensor(sd, S5K83A_FLIP, data, 1);
+       err = m5602_write_sensor(sd, S5K83A_FLIP, data, 1);
        if (err < 0)
-               return err;
+               goto out;
 
        data[0] = (val) ? 0x0a : 0x0b;
-       err = s5k83a_write_sensor(sd, S5K83A_HFLIP_TUNE, data, 1);
-
-       return (err < 0) ? err : 0;
+       err = m5602_write_sensor(sd, S5K83A_HFLIP_TUNE, data, 1);
+out:
+       return err;
 }