[media] mceusb: use %*ph for small buffer dumps
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Sun, 6 Mar 2016 13:00:09 +0000 (10:00 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Thu, 10 Mar 2016 16:37:44 +0000 (13:37 -0300)
It makes the printk cleaner. As a side efect, it also fixes those smatch
warnings:
drivers/media/rc/mceusb.c:590 mceusb_dev_printdata() warn: argument 6 to %02x specifier has type 'char'
drivers/media/rc/mceusb.c:590 mceusb_dev_printdata() warn: argument 7 to %02x specifier has type 'char'

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

index 2cdb740cde481e2e8884b7ddb05c90f6f399a954..35155ae500c747082ce6cf0f4e8216687f109982 100644 (file)
@@ -587,9 +587,8 @@ static void mceusb_dev_printdata(struct mceusb_dev *ir, char *buf,
                        if (len == 2)
                                dev_dbg(dev, "Get hw/sw rev?");
                        else
-                               dev_dbg(dev, "hw/sw rev 0x%02x 0x%02x 0x%02x 0x%02x",
-                                        data1, data2,
-                                        buf[start + 4], buf[start + 5]);
+                               dev_dbg(dev, "hw/sw rev %*ph",
+                                       4, &buf[start + 2]);
                        break;
                case MCE_CMD_RESUME:
                        dev_dbg(dev, "Device resume requested");