]> git.karo-electronics.de Git - linux-beck.git/commitdiff
[media] cx231xx: handle errors at read_eeprom()
authorMauro Carvalho Chehab <m.chehab@samsung.com>
Sun, 27 Jul 2014 19:27:30 +0000 (16:27 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Sun, 27 Jul 2014 20:03:51 +0000 (17:03 -0300)
Fix the following warnings:
drivers/media/usb/cx231xx/cx231xx-cards.c: In function 'read_eeprom':
drivers/media/usb/cx231xx/cx231xx-cards.c:979:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable]

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/usb/cx231xx/cx231xx-cards.c

index 5812c63e1e4b1eca949511ea1778f73038022db0..8039b769f2580fd7409d56b0a4e7cd1078a75ee4 100644 (file)
@@ -991,13 +991,20 @@ static int read_eeprom(struct cx231xx *dev, u8 *eedata, int len)
 
        /* start reading at offset 0 */
        ret = i2c_transfer(&dev->i2c_bus[1].i2c_adap, &msg_write, 1);
+       if (ret < 0) {
+               cx231xx_err("Can't read eeprom\n");
+               return ret;
+       }
 
        while (len_todo > 0) {
                msg_read.len = (len_todo > 64) ? 64 : len_todo;
                msg_read.buf = eedata_cur;
 
                ret = i2c_transfer(&dev->i2c_bus[1].i2c_adap, &msg_read, 1);
-
+               if (ret < 0) {
+                       cx231xx_err("Can't read eeprom\n");
+                       return ret;
+               }
                eedata_cur += msg_read.len;
                len_todo -= msg_read.len;
        }