[PATCH] cpqphp: add pci_enable_device()

Add pci_{enable,disable}_device() calls.  Without pci_enable_device(),
dev->irq is garbage, and cpqphp relies on it.

This fixes a problem reported by Bruno Redondi.  He reported a flood
of ACPI interrupts, that caused kacpid to run 100% of the time:
    http://bugzilla.kernel.org/show_bug.cgi?id=5312

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

 drivers/pci/hotplug/cpqphp_core.c |   24 +++++++++++++++++++-----
 1 file changed, 19 insertions(+), 5 deletions(-)

1 files changed, 19 insertions, 5 deletions

diff --git a/drivers/pci/hotplug/cpqphp_core.c b/drivers/pci/hotplug/cpqphp_core.c
index 8c6d3987d461..9aed8efe6a11 100644
--- a/drivers/pci/hotplug/cpqphp_core.c
+++ b/drivers/pci/hotplug/cpqphp_core.c
@@ -794,12 +794,21 @@ static int cpqhpc_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
         u32 rc;
         struct controller *ctrl;
         struct pci_func *func;
+        int err;
+
+        err = pci_enable_device(pdev);
+        if (err) {
+                printk(KERN_ERR MY_NAME ": cannot enable PCI device %s (%d)\n",
+                        pci_name(pdev), err);
+                return err;
+        }
 
         // Need to read VID early b/c it's used to differentiate CPQ and INTC discovery
         rc = pci_read_config_word(pdev, PCI_VENDOR_ID, &vendor_id);
         if (rc || ((vendor_id != PCI_VENDOR_ID_COMPAQ) && (vendor_id != PCI_VENDOR_ID_INTEL))) {
                 err(msg_HPC_non_compaq_or_intel);
-                return -ENODEV;
+                rc = -ENODEV;
+                goto err_disable_device;
         }
         dbg("Vendor ID: %x\n", vendor_id);
 
@@ -807,7 +816,8 @@ static int cpqhpc_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
         dbg("revision: %d\n", rev);
         if (rc || ((vendor_id == PCI_VENDOR_ID_COMPAQ) && (!rev))) {
                 err(msg_HPC_rev_error);
-                return -ENODEV;
+                rc = -ENODEV;
+                goto err_disable_device;
         }
 
         /* Check for the proper subsytem ID's
@@ -820,18 +830,20 @@ static int cpqhpc_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
                 rc = pci_read_config_word(pdev, PCI_SUBSYSTEM_VENDOR_ID, &subsystem_vid);
                 if (rc) {
                         err("%s : pci_read_config_word failed\n", __FUNCTION__);
-                        return rc;
+                        goto err_disable_device;
                 }
                 dbg("Subsystem Vendor ID: %x\n", subsystem_vid);
                 if ((subsystem_vid != PCI_VENDOR_ID_COMPAQ) && (subsystem_vid != PCI_VENDOR_ID_INTEL)) {
                         err(msg_HPC_non_compaq_or_intel);
-                        return -ENODEV;
+                        rc = -ENODEV;
+                        goto err_disable_device;
                 }
 
                 ctrl = (struct controller *) kmalloc(sizeof(struct controller), GFP_KERNEL);
                 if (!ctrl) {
                         err("%s : out of memory\n", __FUNCTION__);
-                        return -ENOMEM;
+                        rc = -ENOMEM;
+                        goto err_disable_device;
                 }
                 memset(ctrl, 0, sizeof(struct controller));
 
@@ -1264,6 +1276,8 @@ err_free_bus:
         kfree(ctrl->pci_bus);
 err_free_ctrl:
         kfree(ctrl);
+err_disable_device:
+        pci_disable_device(pdev);
         return rc;
 }