]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
PCI: print quirk name in debug messages
authorbjorn.helgaas@hp.com <bjorn.helgaas@hp.com>
Mon, 17 Dec 2007 21:09:38 +0000 (14:09 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 1 Feb 2008 23:04:26 +0000 (15:04 -0800)
Instead of printing this:

    PCI: Calling quirk c023b250 for 0000:00:00.0

we can print this:

    pci 0000:00:00.0: calling quirk 0xc023b270: quirk_cardbus_legacy+0x0/0x30()

The address is superfluous because sprint_symbol() includes the
address if the symbol lookup fails, but this is the same style used
in do_initcalls() and pnp_fixup_device().

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/pci/quirks.c

index 637903a2df3a6f709c341883637d27c456b341e9..873125b725d4bc3df38a64ab96a958e45e700f40 100644 (file)
@@ -21,6 +21,7 @@
 #include <linux/init.h>
 #include <linux/delay.h>
 #include <linux/acpi.h>
+#include <linux/kallsyms.h>
 #include "pci.h"
 
 /* The Mellanox Tavor device gives false positive parity errors
@@ -1500,7 +1501,11 @@ static void pci_do_fixups(struct pci_dev *dev, struct pci_fixup *f, struct pci_f
        while (f < end) {
                if ((f->vendor == dev->vendor || f->vendor == (u16) PCI_ANY_ID) &&
                    (f->device == dev->device || f->device == (u16) PCI_ANY_ID)) {
-                       pr_debug("PCI: Calling quirk %p for %s\n", f->hook, pci_name(dev));
+#ifdef DEBUG
+                       dev_dbg(&dev->dev, "calling quirk 0x%p", f->hook);
+                       print_fn_descriptor_symbol(": %s()\n",
+                               (unsigned long) f->hook);
+#endif
                        f->hook(dev);
                }
                f++;