PCI/portdrv: Rename and reverse sense of pcie_ports_auto
authorBjorn Helgaas <bhelgaas@google.com>
Fri, 9 Mar 2018 17:21:30 +0000 (11:21 -0600)
committerBjorn Helgaas <helgaas@kernel.org>
Fri, 30 Mar 2018 22:26:58 +0000 (17:26 -0500)
The platform may restrict the OS's use of PCIe services, e.g., via the ACPI
_OSC method.  The user may use "pcie_ports=native" to force the port driver
to use PCIe services even if the platform asked us not to.

The "pcie_ports=native" parameter determines the setting of
pcie_ports_auto.  Rename this to pcie_ports_native and reverse the
sense to simplify the code.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/pcie/portdrv.h
drivers/pci/pcie/portdrv_core.c
drivers/pci/pcie/portdrv_pci.c

index 62e28b5afa517b27808ae20db197c80e25ce20d2..3e0058a5500f10d540b1b3b5fee0016da6296618 100644 (file)
@@ -12,7 +12,7 @@
 
 #include <linux/compiler.h>
 
 
 #include <linux/compiler.h>
 
-extern bool pcie_ports_auto;
+extern bool pcie_ports_native;
 
 /* Service Type */
 #define PCIE_PORT_SERVICE_PME_SHIFT    0       /* Power Management Event */
 
 /* Service Type */
 #define PCIE_PORT_SERVICE_PME_SHIFT    0       /* Power Management Event */
index 6ed67cbf61483d7ca97286119db87f0f15cfd157..6890aea4a550e679dca580a7a876afb9c0e640a3 100644 (file)
@@ -193,17 +193,10 @@ legacy_irq:
 static int get_port_device_capability(struct pci_dev *dev)
 {
        struct pci_host_bridge *host = pci_find_host_bridge(dev->bus);
 static int get_port_device_capability(struct pci_dev *dev)
 {
        struct pci_host_bridge *host = pci_find_host_bridge(dev->bus);
-       bool native;
        int services = 0;
 
        int services = 0;
 
-       /*
-        * If the user specified "pcie_ports=native", use the PCIe services
-        * regardless of whether the platform has given us permission.  On
-        * ACPI systems, this means we ignore _OSC.
-        */
-       native = !pcie_ports_auto;
-
-       if (dev->is_hotplug_bridge && (native || host->native_hotplug)) {
+       if (dev->is_hotplug_bridge &&
+           (pcie_ports_native || host->native_hotplug)) {
                services |= PCIE_PORT_SERVICE_HP;
 
                /*
                services |= PCIE_PORT_SERVICE_HP;
 
                /*
@@ -215,7 +208,7 @@ static int get_port_device_capability(struct pci_dev *dev)
        }
 
        if (pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR) &&
        }
 
        if (pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR) &&
-           pci_aer_available() && (native || host->native_aer)) {
+           pci_aer_available() && (pcie_ports_native || host->native_aer)) {
                services |= PCIE_PORT_SERVICE_AER;
 
                /*
                services |= PCIE_PORT_SERVICE_AER;
 
                /*
@@ -231,7 +224,7 @@ static int get_port_device_capability(struct pci_dev *dev)
         * those yet.
         */
        if (pci_pcie_type(dev) == PCI_EXP_TYPE_ROOT_PORT &&
         * those yet.
         */
        if (pci_pcie_type(dev) == PCI_EXP_TYPE_ROOT_PORT &&
-           (native || host->native_pme)) {
+           (pcie_ports_native || host->native_pme)) {
                services |= PCIE_PORT_SERVICE_PME;
 
                /*
                services |= PCIE_PORT_SERVICE_PME;
 
                /*
index 8b62192342ac080d08702570848f80af8a8e6be5..8e4260d2594148d258f0603921e54c4bd6be1f19 100644 (file)
 bool pcie_ports_disabled;
 
 /*
 bool pcie_ports_disabled;
 
 /*
- * If this switch is set, ACPI _OSC will be used to determine whether or not to
- * enable PCIe port native services.
+ * If the user specified "pcie_ports=native", use the PCIe services regardless
+ * of whether the platform has given us permission.  On ACPI systems, this
+ * means we ignore _OSC.
  */
  */
-bool pcie_ports_auto = true;
+bool pcie_ports_native;
 
 static int __init pcie_port_setup(char *str)
 {
 
 static int __init pcie_port_setup(char *str)
 {
-       if (!strncmp(str, "compat", 6)) {
+       if (!strncmp(str, "compat", 6))
                pcie_ports_disabled = true;
                pcie_ports_disabled = true;
-       } else if (!strncmp(str, "native", 6)) {
-               pcie_ports_disabled = false;
-               pcie_ports_auto = false;
-       }
+       else if (!strncmp(str, "native", 6))
+               pcie_ports_native = true;
 
        return 1;
 }
 
        return 1;
 }