r8169: enable ASPM on RTL8106E

khfeng · davem330 · commit 0866cd15029b · 2018-09-17T18:45:55.000-07:00
The Intel SoC was prevented from entering lower idle state because
of RTL8106E's ASPM was not enabled.

So enable ASPM on RTL8106E (chip version 39).
Now the Intel SoC can enter lower idle state, power consumption and
temperature are much lower.

Signed-off-by: Kai-Heng Feng &lt;kai.heng.feng@canonical.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git a/drivers/net/ethernet/realtek/r8169.c b/drivers/net/ethernet/realtek/r8169.c
@@ -5627,6 +5627,8 @@ static void rtl_hw_start_8402(struct rtl8169_private *tp)
 
 static void rtl_hw_start_8106(struct rtl8169_private *tp)
 {
+	rtl_hw_aspm_clkreq_enable(tp, false);
+
 	/* Force LAN exit from ASPM if Rx/Tx are not idle */
 	RTL_W32(tp, FuncEvent, RTL_R32(tp, FuncEvent) | 0x002800);
 
@@ -5635,6 +5637,7 @@ static void rtl_hw_start_8106(struct rtl8169_private *tp)
 	RTL_W8(tp, DLLPR, RTL_R8(tp, DLLPR) & ~PFM_EN);
 
 	rtl_pcie_state_l2l3_enable(tp, false);
+	rtl_hw_aspm_clkreq_enable(tp, true);
 }
 
 static void rtl_hw_start_8101(struct rtl8169_private *tp)

Original file line number	Diff line number	Diff line change
`@@ -5627,6 +5627,8 @@ static void rtl_hw_start_8402(struct rtl8169_private *tp)`
`5627`	`5627`
`5628`	`5628`	`static void rtl_hw_start_8106(struct rtl8169_private *tp)`
`5629`	`5629`	`{`
	`5630`	`+ rtl_hw_aspm_clkreq_enable(tp, false);`
	`5631`	`+`
`5630`	`5632`	`/* Force LAN exit from ASPM if Rx/Tx are not idle */`
`5631`	`5633`	`RTL_W32(tp, FuncEvent, RTL_R32(tp, FuncEvent) \| 0x002800);`
`5632`	`5634`
`@@ -5635,6 +5637,7 @@ static void rtl_hw_start_8106(struct rtl8169_private *tp)`
`5635`	`5637`	`RTL_W8(tp, DLLPR, RTL_R8(tp, DLLPR) & ~PFM_EN);`
`5636`	`5638`
`5637`	`5639`	`rtl_pcie_state_l2l3_enable(tp, false);`
	`5640`	`+ rtl_hw_aspm_clkreq_enable(tp, true);`
`5638`	`5641`	`}`
`5639`	`5642`
`5640`	`5643`	`static void rtl_hw_start_8101(struct rtl8169_private *tp)`