Skip to content

Commit 25edb25

Browse files
Ajay Agarwalbjorn-helgaas
authored andcommitted
PCI/ASPM: Set only ASPM_STATE_L1 when driver enables L1
Previously pci_enable_link_state(PCIE_LINK_STATE_L1) enabled L1SS as well as L1. Enable only ASPM_STATE_L1 when the caller enables L1. The only current caller is vmd_pm_enable_quirk(), which enables *all* ASPM states, so this should have no functional effect. [bhelgaas: commit log] Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Ajay Agarwal <[email protected]> Signed-off-by: Bjorn Helgaas <[email protected]> Reviewed-by: Kuppuswamy Sathyanarayanan <[email protected]>
1 parent fb097dc commit 25edb25

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

drivers/pci/pcie/aspm.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1173,8 +1173,7 @@ int pci_enable_link_state(struct pci_dev *pdev, int state)
11731173
if (state & PCIE_LINK_STATE_L0S)
11741174
link->aspm_default |= ASPM_STATE_L0S;
11751175
if (state & PCIE_LINK_STATE_L1)
1176-
/* L1 PM substates require L1 */
1177-
link->aspm_default |= ASPM_STATE_L1 | ASPM_STATE_L1SS;
1176+
link->aspm_default |= ASPM_STATE_L1;
11781177
if (state & PCIE_LINK_STATE_L1_1)
11791178
link->aspm_default |= ASPM_STATE_L1_1;
11801179
if (state & PCIE_LINK_STATE_L1_2)

0 commit comments

Comments
 (0)