]> xenbits.xensource.com Git - legacy/linux-2.6.18-xen.git/commitdiff
Backport: PCI: remove unnecessary condition check in pci_restore_bars()
authorKeir Fraser <keir.fraser@citrix.com>
Fri, 13 Mar 2009 08:50:44 +0000 (08:50 +0000)
committerKeir Fraser <keir.fraser@citrix.com>
Fri, 13 Mar 2009 08:50:44 +0000 (08:50 +0000)
    commit bc5f5a8277cb353161454b6704b3186ebcf3a2a3
    Author: Yu Zhao <yu.zhao@intel.com>
    Date:   Sat Nov 22 02:40:00 2008 +0800

    PCI: remove unnecessary condition check in pci_restore_bars()

    Remove the unnecessary number of resources condition checks
    because
    the pci_update_resource() will check availability of the
    resources.

Signed-off-by: Yu Zhao <yu.zhao@intel.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Yu Zhao <yu.zhao@intel.com>
drivers/pci/pci.c

index 172e5a75fe2178921ca03f8be180bbad3bad3fc5..f810e9eeb2cffce65d152b5aaf1738297be35299 100644 (file)
@@ -256,25 +256,10 @@ pci_find_parent_resource(const struct pci_dev *dev, struct resource *res)
 void
 pci_restore_bars(struct pci_dev *dev)
 {
-       int i, numres;
-
-       switch (dev->hdr_type) {
-       case PCI_HEADER_TYPE_NORMAL:
-               numres = 6;
-               break;
-       case PCI_HEADER_TYPE_BRIDGE:
-               numres = 2;
-               break;
-       case PCI_HEADER_TYPE_CARDBUS:
-               numres = 1;
-               break;
-       default:
-               /* Should never get here, but just in case... */
-               return;
-       }
+       int i;
 
-       for (i = 0; i < numres; i++)
-               pci_update_resource(dev, &dev->resource[i]);
+       for (i = 0; i < PCI_BRIDGE_RESOURCES; i++)
+               pci_update_resource(dev, i);
 }
 
 int (*platform_pci_set_power_state)(struct pci_dev *dev, pci_power_t t);