PCI/ASPM: Simplify Clock Power Management setting

Update the Link Control Enable Clock Power Management bit the same
way we update the ASPM Control bits, with a single call of
pcie_capability_clear_and_set_word().

No functional change; this just makes both paths use the same style.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>

1 files changed, 5 insertions, 8 deletions

diff --git a/drivers/pci/pcie/aspm.c b/drivers/pci/pcie/aspm.c
index a84f87247d2f..317e3558a35e 100644
--- a/drivers/pci/pcie/aspm.c
+++ b/drivers/pci/pcie/aspm.c
@@ -127,15 +127,12 @@ static void pcie_set_clkpm_nocheck(struct pcie_link_state *link, int enable)
 {
         struct pci_dev *child;
         struct pci_bus *linkbus = link->pdev->subordinate;
+        u32 val = enable ? PCI_EXP_LNKCTL_CLKREQ_EN : 0;
 
-        list_for_each_entry(child, &linkbus->devices, bus_list) {
-                if (enable)
-                        pcie_capability_set_word(child, PCI_EXP_LNKCTL,
-                                                 PCI_EXP_LNKCTL_CLKREQ_EN);
-                else
-                        pcie_capability_clear_word(child, PCI_EXP_LNKCTL,
-                                                   PCI_EXP_LNKCTL_CLKREQ_EN);
-        }
+        list_for_each_entry(child, &linkbus->devices, bus_list)
+                pcie_capability_clear_and_set_word(child, PCI_EXP_LNKCTL,
+                                                   PCI_EXP_LNKCTL_CLKREQ_EN,
+                                                   val);
         link->clkpm_enabled = !!enable;
 }