diff options
author | Bjorn Helgaas <bhelgaas@google.com> | 2013-08-22 11:24:44 +0800 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2013-08-22 10:47:02 -0600 |
commit | a58674ff8383f5b8f6a77f03c48f6a47840b9325 (patch) | |
tree | b07c7f08dc3bd929131d6dc164d27cf8c0903cff /drivers/pci/hotplug | |
parent | 2c25e34c7531ca1849b85cbcdb5a2f507ffe240c (diff) | |
download | linux-a58674ff8383f5b8f6a77f03c48f6a47840b9325.tar.bz2 |
PCI: Simplify pcie_bus_configure_settings() interface
Based on a patch by Jon Mason (see URL below).
All users of pcie_bus_configure_settings() pass arguments of the form
"bus, bus->self->pcie_mpss". The "mpss" argument is redundant since we
can easily look it up internally. In addition, all callers check
"bus->self" for NULL, which we can also do internally.
This patch simplifies the interface and the callers. No functional change.
Reference: http://lkml.kernel.org/r/1317048850-30728-2-git-send-email-mason@myri.com
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/pci/hotplug')
-rw-r--r-- | drivers/pci/hotplug/pcihp_slot.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/pci/hotplug/pcihp_slot.c b/drivers/pci/hotplug/pcihp_slot.c index fec2d5b75440..16f920352317 100644 --- a/drivers/pci/hotplug/pcihp_slot.c +++ b/drivers/pci/hotplug/pcihp_slot.c @@ -160,9 +160,8 @@ void pci_configure_slot(struct pci_dev *dev) (dev->class >> 8) == PCI_CLASS_BRIDGE_PCI))) return; - if (dev->bus && dev->bus->self) - pcie_bus_configure_settings(dev->bus, - dev->bus->self->pcie_mpss); + if (dev->bus) + pcie_bus_configure_settings(dev->bus); memset(&hpp, 0, sizeof(hpp)); ret = pci_get_hp_params(dev, &hpp); |