[PATCH v5 12/15] tty: serial: 8250: Correct conversion specifiers

From: Anton Wuerfel
Date: Wed Jan 13 2016 - 11:41:10 EST


This patch fixes compiler warnings about wrong conversion specifiers used
in a debug output in 8250_pnp.c. The precise warning is:

drivers/tty/serial/8250/8250_pnp.c: In function âserial_pnp_probeâ:
include/linux/dynamic_debug.h:64:16: warning: format â%xâ expects argument
of [...]

include/linux/dynamic_debug.h:84:2: note: in expansion of macro
âDEFINE_DYNAMIC_DEBUG_METADATAâ

include/linux/device.h:1179:2: note: in expansion of macro
âdynamic_dev_dbgâ

drivers/tty/serial/8250/8250_pnp.c:467:2: note: in expansion of macro
âdev_dbgâ
dev_dbg(&dev->dev,
^
include/linux/dynamic_debug.h:64:16: warning: format â%lxâ expects argument
of [...]

include/linux/dynamic_debug.h:84:2: note: in expansion of macro
âDEFINE_DYNAMIC_DEBUG_METADATAâ

include/linux/device.h:1179:2: note: in expansion of macro
âdynamic_dev_dbgâ

drivers/tty/serial/8250/8250_pnp.c:467:2: note: in expansion of macro
âdev_dbgâ
dev_dbg(&dev->dev,
^

Those warnings never got triggered, because the command was nested
in an #ifdef, which is removed by a patch of this series.

Signed-off-by: Anton WÃrfel <anton.wuerfel@xxxxxx>
Signed-off-by: Phillip Raffeck <phillip.raffeck@xxxxxx>
Cc: linux-kernel@xxxxxxxxxxxx
---
drivers/tty/serial/8250/8250_pnp.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/8250/8250_pnp.c b/drivers/tty/serial/8250/8250_pnp.c
index ba3a1af..9305591 100644
--- a/drivers/tty/serial/8250/8250_pnp.c
+++ b/drivers/tty/serial/8250/8250_pnp.c
@@ -465,8 +465,8 @@ serial_pnp_probe(struct pnp_dev *dev, const struct pnp_device_id *dev_id)
return -ENODEV;

dev_dbg(&dev->dev,
- "Setup PNP port: port %x, mem 0x%lx, irq %d, type %d\n",
- uart.port.iobase, uart.port.mapbase,
+ "Setup PNP port: port %lx, mem 0x%lx, irq %d, type %d\n",
+ uart.port.iobase, (unsigned long)uart.port.mapbase,
uart.port.irq, uart.port.iotype);

if (flags & CIR_PORT) {
--
1.9.1