diff options
author | Scott Wood <scottwood@freescale.com> | 2015-03-12 16:46:00 -0500 |
---|---|---|
committer | Helge Deller <deller@gmx.de> | 2015-04-24 13:45:54 +0200 |
commit | a4e4f67f41d9bfcc2a00849a75712f09a7d43798 (patch) | |
tree | d60de2e18f28af66c910e79a561e56c96d778248 /drivers/parisc | |
parent | 27cf3a16b2535a490f8cf1d29a6634f1c70f7831 (diff) | |
download | lwn-a4e4f67f41d9bfcc2a00849a75712f09a7d43798.tar.gz lwn-a4e4f67f41d9bfcc2a00849a75712f09a7d43798.zip |
parisc: %pf is only for function pointers
Use %ps for actual addresses, otherwise you'll get bad output
on arches like parisc64 where %pf expects a function descriptor.
This wasn't normally seen on parisc64 because the code is not built
unless DEBUG_SUPERIO_INIT is manually defined.
Patch modified by Helge Deller to utilize KERN_DEBUG.
Signed-off-by: Scott Wood <scottwood@freescale.com>
Cc: linux-parisc@vger.kernel.org
cc: James Bottomley <James.Bottomley@HansenPartnership.com>
Signed-off-by: Helge Deller <deller@gmx.de>
Diffstat (limited to 'drivers/parisc')
-rw-r--r-- | drivers/parisc/superio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/parisc/superio.c b/drivers/parisc/superio.c index 8be2096c8423..deeaed544222 100644 --- a/drivers/parisc/superio.c +++ b/drivers/parisc/superio.c @@ -348,7 +348,7 @@ int superio_fixup_irq(struct pci_dev *pcidev) BUG(); return -1; } - printk("superio_fixup_irq(%s) ven 0x%x dev 0x%x from %pf\n", + printk(KERN_DEBUG "superio_fixup_irq(%s) ven 0x%x dev 0x%x from %ps\n", pci_name(pcidev), pcidev->vendor, pcidev->device, __builtin_return_address(0)); |