[media] em28xx: respect the message size constraints for i2c transfers
authorFrank Schaefer <fschaefer.oss@googlemail.com>
Thu, 3 Jan 2013 17:27:02 +0000 (14:27 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 5 Jan 2013 03:13:29 +0000 (01:13 -0200)
The em2800 can transfer up to 4 bytes per i2c message.
All other em25xx/em27xx/28xx chips can transfer at least 64 bytes per message.
I2C adapters should never split messages transferred via the I2C subsystem
into multiple message transfers, because the result will almost always NOT be
the same as when the whole data is transferred to the I2C client in a single
message.
If the message size exceeds the capabilities of the I2C adapter, -EOPNOTSUPP
should be returned.

Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/usb/em28xx/em28xx-i2c.c

index 44533e4574ffe6fd2cc746aab0d249b48a0510b1..c508c1297a26a0745cae07c2eb1b27aa27f01e2a 100644 (file)
@@ -50,14 +50,18 @@ do {                                                        \
 } while (0)
 
 /*
- * em2800_i2c_send_max4()
- * send up to 4 bytes to the i2c device
+ * em2800_i2c_send_bytes()
+ * send up to 4 bytes to the em2800 i2c device
  */
-static int em2800_i2c_send_max4(struct em28xx *dev, u8 addr, u8 *buf, u16 len)
+static int em2800_i2c_send_bytes(struct em28xx *dev, u8 addr, u8 *buf, u16 len)
 {
        int ret;
        int write_timeout;
        u8 b2[6];
+
+       if (len < 1 || len > 4)
+               return -EOPNOTSUPP;
+
        BUG_ON(len < 1 || len > 4);
        b2[5] = 0x80 + len - 1;
        b2[4] = addr;
@@ -85,29 +89,6 @@ static int em2800_i2c_send_max4(struct em28xx *dev, u8 addr, u8 *buf, u16 len)
        return -EIO;
 }
 
-/*
- * em2800_i2c_send_bytes()
- */
-static int em2800_i2c_send_bytes(struct em28xx *dev, u8 addr, u8 *buf, u16 len)
-{
-       u8 *bufPtr = buf;
-       int ret;
-       int wrcount = 0;
-       int count;
-       int maxLen = 4;
-       while (len > 0) {
-               count = (len > maxLen) ? maxLen : len;
-               ret = em2800_i2c_send_max4(dev, addr, bufPtr, count);
-               if (ret > 0) {
-                       len -= count;
-                       bufPtr += count;
-                       wrcount += count;
-               } else
-                       return (ret < 0) ? ret : -EFAULT;
-       }
-       return wrcount;
-}
-
 /*
  * em2800_i2c_check_for_device()
  * check if there is a i2c_device at the supplied address
@@ -150,6 +131,10 @@ static int em2800_i2c_check_for_device(struct em28xx *dev, u8 addr)
 static int em2800_i2c_recv_bytes(struct em28xx *dev, u8 addr, u8 *buf, u16 len)
 {
        int ret;
+
+       if (len < 1 || len > 4)
+               return -EOPNOTSUPP;
+
        /* check for the device and set i2c read address */
        ret = em2800_i2c_check_for_device(dev, addr);
        if (ret) {
@@ -176,6 +161,9 @@ static int em28xx_i2c_send_bytes(struct em28xx *dev, u16 addr, u8 *buf,
        int wrcount = 0;
        int write_timeout, ret;
 
+       if (len < 1 || len > 64)
+               return -EOPNOTSUPP;
+
        wrcount = dev->em28xx_write_regs_req(dev, stop ? 2 : 3, addr, buf, len);
 
        /* Seems to be required after a write */
@@ -197,6 +185,10 @@ static int em28xx_i2c_send_bytes(struct em28xx *dev, u16 addr, u8 *buf,
 static int em28xx_i2c_recv_bytes(struct em28xx *dev, u16 addr, u8 *buf, u16 len)
 {
        int ret;
+
+       if (len < 1 || len > 64)
+               return -EOPNOTSUPP;
+
        ret = dev->em28xx_read_reg_req_len(dev, 2, addr, buf, len);
        if (ret < 0) {
                em28xx_warn("reading i2c device failed (error=%i)\n", ret);