Commit a4e4f67f authored Mar 12, 2015 by Scott Wood Committed by Helge Deller Apr 24, 2015

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>

parent 27cf3a16

drivers/parisc/superio.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -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));

Admin message