[media] touptek: cast char types on %x printk
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Sun, 6 Mar 2016 13:15:24 +0000 (10:15 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Thu, 10 Mar 2016 16:37:45 +0000 (13:37 -0300)
This fixes those two smatch warnings:
drivers/media/usb/gspca/touptek.c:206 val_reply() warn: argument 3 to %02x specifier has type 'char'
drivers/media/usb/gspca/touptek.c:222 reg_w() warn: argument 4 to %02x specifier has type 'char'

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/usb/gspca/touptek.c

index 8063b8e..b8af437 100644 (file)
@@ -203,7 +203,7 @@ static int val_reply(struct gspca_dev *gspca_dev, const char *reply, int rc)
                return -EIO;
        }
        if (reply[0] != 0x08) {
-               PERR("Bad reply 0x%02X", reply[0]);
+               PERR("Bad reply 0x%02x", (int)reply[0]);
                return -EIO;
        }
        return 0;
@@ -219,7 +219,7 @@ static void reg_w(struct gspca_dev *gspca_dev, u16 value, u16 index)
                value, index);
        rc = usb_control_msg(gspca_dev->dev, usb_rcvctrlpipe(gspca_dev->dev, 0),
                0x0B, 0xC0, value, index, buff, 1, 500);
-       PDEBUG(D_USBO, "rc=%d, ret={0x%02X}", rc, buff[0]);
+       PDEBUG(D_USBO, "rc=%d, ret={0x%02x}", rc, (int)buff[0]);
        if (rc < 0) {
                PERR("Failed reg_w(0x0B, 0xC0, 0x%04X, 0x%04X) w/ rc %d\n",
                        value, index, rc);