PCI: Simplify pcie_capability_clear_and_set_word() to ..._clear_word()

When using pcie_capability_clear_and_set_word() but not actually *setting*
anything, use pcie_capability_clear_word() instead.

Link: https://lore.kernel.org/r/20231026121924.2164-1-ilpo.jarvinen@linux.intel.com
Link: https://lore.kernel.org/r/20231026121924.2164-2-ilpo.jarvinen@linux.intel.com
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
[bhelgaas: squash]
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>

diff --git a/drivers/pci/pcie/aspm.c b/drivers/pci/pcie/aspm.c
index 1bf630059264478640c92ef9dc2ba61712b122fa..3b0508b4747276ecfd348be977e63ce853c428ff 100644 (file)
--- a/drivers/pci/pcie/aspm.c
+++ b/drivers/pci/pcie/aspm.c
@@ -689,10 +689,10 @@ static void pcie_config_aspm_l1ss(struct pcie_link_state *link, u32 state)
         * in pcie_config_aspm_link().
         */
        if (enable_req & (ASPM_STATE_L1_1 | ASPM_STATE_L1_2)) {
-               pcie_capability_clear_and_set_word(child, PCI_EXP_LNKCTL,
-                                                  PCI_EXP_LNKCTL_ASPM_L1, 0);
-               pcie_capability_clear_and_set_word(parent, PCI_EXP_LNKCTL,
-                                                  PCI_EXP_LNKCTL_ASPM_L1, 0);
+               pcie_capability_clear_word(child, PCI_EXP_LNKCTL,
+                                          PCI_EXP_LNKCTL_ASPM_L1);
+               pcie_capability_clear_word(parent, PCI_EXP_LNKCTL,
+                                          PCI_EXP_LNKCTL_ASPM_L1);
        }
 
        val = 0;

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index e3e915329510fbcc2aaff7ab6c75ac73cab20f14..3259798a8a644e3144cf515a5369165d49327961 100644 (file)
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -4555,9 +4555,9 @@ static void quirk_disable_root_port_attributes(struct pci_dev *pdev)
 
        pci_info(root_port, "Disabling No Snoop/Relaxed Ordering Attributes to avoid PCIe Completion erratum in %s\n",
                 dev_name(&pdev->dev));
-       pcie_capability_clear_and_set_word(root_port, PCI_EXP_DEVCTL,
-                                          PCI_EXP_DEVCTL_RELAX_EN |
-                                          PCI_EXP_DEVCTL_NOSNOOP_EN, 0);
+       pcie_capability_clear_word(root_port, PCI_EXP_DEVCTL,
+                                  PCI_EXP_DEVCTL_RELAX_EN |
+                                  PCI_EXP_DEVCTL_NOSNOOP_EN);
 }
 
 /*