summaryrefslogtreecommitdiffstats
path: root/drivers/pci/pcie/portdrv_acpi.c
diff options
context:
space:
mode:
authorJon Derrick <jonathan.derrick@intel.com>2016-04-08 11:35:51 -0600
committerBjorn Helgaas <bhelgaas@google.com>2016-04-08 15:21:28 -0500
commit88a97da1eab7587036d8bc937d6bc874b8210df1 (patch)
tree8f473d01279a5be325f9561faabe323f9a39af9a /drivers/pci/pcie/portdrv_acpi.c
parent52966bd1c2a09fdd3149f00568cc18f45cc09785 (diff)
downloadop-kernel-dev-88a97da1eab7587036d8bc937d6bc874b8210df1.zip
op-kernel-dev-88a97da1eab7587036d8bc937d6bc874b8210df1.tar.gz
PCI: Remove return values from pcie_port_platform_notify() and relatives
Now that pcie_port_acpi_setup() always returns 0, make it and its callers void functions and stop checking the return values. [bhelgaas: changelog] Signed-off-by: Jon Derrick <jonathan.derrick@intel.com> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/pci/pcie/portdrv_acpi.c')
-rw-r--r--drivers/pci/pcie/portdrv_acpi.c10
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/pci/pcie/portdrv_acpi.c b/drivers/pci/pcie/portdrv_acpi.c
index f097a73..9f4ed71 100644
--- a/drivers/pci/pcie/portdrv_acpi.c
+++ b/drivers/pci/pcie/portdrv_acpi.c
@@ -32,22 +32,22 @@
* NOTE: It turns out that we cannot do that for individual port services
* separately, because that would make some systems work incorrectly.
*/
-int pcie_port_acpi_setup(struct pci_dev *port, int *srv_mask)
+void pcie_port_acpi_setup(struct pci_dev *port, int *srv_mask)
{
struct acpi_pci_root *root;
acpi_handle handle;
u32 flags;
if (acpi_pci_disabled)
- return 0;
+ return;
handle = acpi_find_root_bridge_handle(port);
if (!handle)
- return 0;
+ return;
root = acpi_pci_find_root(handle);
if (!root)
- return 0;
+ return;
flags = root->osc_control_set;
@@ -58,6 +58,4 @@ int pcie_port_acpi_setup(struct pci_dev *port, int *srv_mask)
*srv_mask |= PCIE_PORT_SERVICE_PME;
if (flags & OSC_PCI_EXPRESS_AER_CONTROL)
*srv_mask |= PCIE_PORT_SERVICE_AER;
-
- return 0;
}
OpenPOWER on IntegriCloud