projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
2ccfad1
)
mfd: ipaq-micro: Use %*ph for printing hexdump of a small buffer
author
Andy Shevchenko
<andriy.shevchenko@linux.intel.com>
Mon, 3 Jul 2023 14:09:22 +0000
(17:09 +0300)
committer
Lee Jones
<lee@kernel.org>
Fri, 18 Aug 2023 20:47:59 +0000
(21:47 +0100)
The kernel already has a helper to print a hexdump of a small
buffer via pointer extension. Use that instead of open coded
variant.
In long term it helps to kill pr_cont() or at least narrow down
its use.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link:
https://lore.kernel.org/r/20230703140923.2840-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/mfd/ipaq-micro.c
patch
|
blob
|
history
diff --git
a/drivers/mfd/ipaq-micro.c
b/drivers/mfd/ipaq-micro.c
index 6d3968458e8164c8f3945398d0610715a6ce2a13..cddfd2e808f293dc348639ff9fe7f330e0b828f1 100644
(file)
--- a/
drivers/mfd/ipaq-micro.c
+++ b/
drivers/mfd/ipaq-micro.c
@@
-131,10
+131,7
@@
static void micro_rx_msg(struct ipaq_micro *micro, u8 id, int len, u8 *data)
break;
default:
dev_err(micro->dev,
- "unknown msg %d [%d] ", id, len);
- for (i = 0; i < len; ++i)
- pr_cont("0x%02x ", data[i]);
- pr_cont("\n");
+ "unknown msg %d [%d] %*ph\n", id, len, len, data);
}
spin_unlock(µ->lock);
}