PCI hotplug: cpqphp: use pci_dev->vendor
authorSergei Shtylyov <sshtylyov@ru.mvista.com>
Wed, 13 Jul 2011 15:20:14 +0000 (19:20 +0400)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Fri, 22 Jul 2011 15:25:43 +0000 (08:25 -0700)
The driver reads PCI vendor ID from the PCI configuration register while it is
already stored by the PCI subsystem in the 'vendor' field of 'struct pci_dev'...

Signed-off-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/hotplug/cpqphp_core.c

index 627274cc571fca6ccae82a33e3ca73a875315a2d..f1ce99cceac60a2135cf8292da0013eebeadb2a1 100644 (file)
@@ -840,8 +840,9 @@ static int cpqhpc_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
        /* 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))) {
+       vendor_id = pdev->vendor;
+       if ((vendor_id != PCI_VENDOR_ID_COMPAQ) &&
+           (vendor_id != PCI_VENDOR_ID_INTEL)) {
                err(msg_HPC_non_compaq_or_intel);
                rc = -ENODEV;
                goto err_disable_device;