diff options
author | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2016-03-06 08:00:09 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2016-03-10 11:37:44 -0500 |
commit | eef8fc374c131647ea9eea3301f06f4eee7f51ae (patch) | |
tree | 6272b0f8e99fec7e1a5353db63b06a269b194583 /drivers | |
parent | de08b5a8be0df1eb7c796b0fe6b30cf1d03d14a6 (diff) |
[media] mceusb: use %*ph for small buffer dumps
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>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/media/rc/mceusb.c | 5 |
1 files 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, | |||
587 | if (len == 2) | 587 | if (len == 2) |
588 | dev_dbg(dev, "Get hw/sw rev?"); | 588 | dev_dbg(dev, "Get hw/sw rev?"); |
589 | else | 589 | else |
590 | dev_dbg(dev, "hw/sw rev 0x%02x 0x%02x 0x%02x 0x%02x", | 590 | dev_dbg(dev, "hw/sw rev %*ph", |
591 | data1, data2, | 591 | 4, &buf[start + 2]); |
592 | buf[start + 4], buf[start + 5]); | ||
593 | break; | 592 | break; |
594 | case MCE_CMD_RESUME: | 593 | case MCE_CMD_RESUME: |
595 | dev_dbg(dev, "Device resume requested"); | 594 | dev_dbg(dev, "Device resume requested"); |