[PATCH] PCI: Reduce nr of ptr derefs in drivers/pci/hotplug/pciehprm_acpi.c

Here's a small patch to reduce the nr. of pointer dereferences in
drivers/pci/hotplug/pciehprm_acpi.c

Benefits:
 - micro speed optimization due to fewer pointer derefs
 - generated code is slightly smaller
 - better readability

Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

1 files changed, 8 insertions, 5 deletions

diff --git a/drivers/pci/hotplug/pciehprm_acpi.c b/drivers/pci/hotplug/pciehprm_acpi.c
index ae244e218620..2bdb30f68bf8 100644
--- a/drivers/pci/hotplug/pciehprm_acpi.c
+++ b/drivers/pci/hotplug/pciehprm_acpi.c
@@ -174,7 +174,9 @@ int pciehp_get_hp_hw_control_from_firmware(struct pci_dev *dev)
         acpi_status status;
         acpi_handle chandle, handle = DEVICE_ACPI_HANDLE(&(dev->dev));
         struct pci_dev *pdev = dev;
+        struct pci_bus *parent;
         u8 *path_name;
+
         /*
          * Per PCI firmware specification, we should run the ACPI _OSC
          * method to get control of hotplug hardware before using it.
@@ -190,17 +192,18 @@ int pciehp_get_hp_hw_control_from_firmware(struct pci_dev *dev)
                  */
                 if (!pdev || !pdev->bus->parent)
                         break;
+                parent = pdev->bus->parent;
                 dbg("Could not find %s in acpi namespace, trying parent\n",
                                 pci_name(pdev));
-                if (!pdev->bus->parent->self)
+                if (!parent->self)
                         /* Parent must be a host bridge */
                         handle = acpi_get_pci_rootbridge_handle(
-                                        pci_domain_nr(pdev->bus->parent),
-                                        pdev->bus->parent->number);
+                                        pci_domain_nr(parent),
+                                        parent->number);
                 else
                         handle = DEVICE_ACPI_HANDLE(
-                                        &(pdev->bus->parent->self->dev));
-                pdev = pdev->bus->parent->self;
+                                        &(parent->self->dev));
+                pdev = parent->self;
         }
 
         while (handle) {