Message ID | 20200310014836.23656-8-acelan.kao@canonical.com |
---|---|
State | New |
Headers | show |
Series | enable realtek ethernet device ASPM function | expand |
diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c index a2cef6a004e7..7613022c344c 100644 --- a/drivers/net/ethernet/realtek/r8169_main.c +++ b/drivers/net/ethernet/realtek/r8169_main.c @@ -7094,8 +7094,7 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) /* Disable ASPM completely as that cause random device stop working * problems as well as full system hangs for some PCIe devices users. */ - rc = pci_disable_link_state(pdev, PCIE_LINK_STATE_L0S | - PCIE_LINK_STATE_L1); + rc = pci_disable_link_state(pdev, PCIE_LINK_STATE_L1_1); tp->aspm_manageable = !rc; /* enable device (incl. PCI PM wakeup and hotplug setup) */
BguLink: https://bugs.launchpad.net/bugs/1836030 r8169 doesn't suport ASPM L1.1, so we don't have to disable ASPM completely. Disable ASPM L1.1 doesn't affect the power consumption and the network function keeps working after S3 test 30 times. Signed-off-by: AceLan Kao <acelan.kao@canonical.com> --- drivers/net/ethernet/realtek/r8169_main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)