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 <mche...@osg.samsung.com>
---
 drivers/media/rc/mceusb.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/media/rc/mceusb.c b/drivers/media/rc/mceusb.c
index 2cdb740cde48..35155ae500c7 100644
--- a/drivers/media/rc/mceusb.c
+++ b/drivers/media/rc/mceusb.c
@@ -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");
-- 
2.5.0

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to