[PATCH net v4] r8169: Added delay after power on and reset.

Subsystems: 8169 10/100/1000 gigabit ethernet driver, networking drivers, the rest

STALE3841d

2 messages, 2 authors, 2016-02-01 · open the first message on its own page

[PATCH net v4] r8169: Added delay after power on and reset.

From: Corcodel Marian <hidden>
Date: 2016-02-01 16:05:58

On reset and power on need 2msecs delay to autoload complete data from
 eeprom to VPD.

Signed-off-by: Corcodel Marian <redacted>
---
 drivers/net/ethernet/realtek/r8169.c | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/drivers/net/ethernet/realtek/r8169.c b/drivers/net/ethernet/realtek/r8169.c
index 8476659..5186ef9 100644
--- a/drivers/net/ethernet/realtek/r8169.c
+++ b/drivers/net/ethernet/realtek/r8169.c
@@ -97,6 +97,7 @@ static const int multicast_filter_limit = 32;
 
 #define RTL8169_TX_TIMEOUT	(6*HZ)
 #define RTL8169_PHY_TIMEOUT	(10*HZ)
+#define RTL8169_EE_TIMEOUT	2  /* 2ms Autoload from eeprom */
 
 /* write/read MMIO register */
 #define RTL_W8(reg, val8)	writeb ((val8), ioaddr + (reg))
@@ -5137,6 +5138,7 @@ static void rtl_hw_reset(struct rtl8169_private *tp)
 	RTL_W8(ChipCmd, CmdReset);
 
 	rtl_udelay_loop_wait_low(tp, &rtl_chipcmd_cond, 100, 100);
+	mdelay(RTL8169_EE_TIMEOUT);
 }
 
 static void rtl_request_uncached_firmware(struct rtl8169_private *tp)
@@ -8183,6 +8185,7 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
 	pci_disable_link_state(pdev, PCIE_LINK_STATE_L0S | PCIE_LINK_STATE_L1 |
 				     PCIE_LINK_STATE_CLKPM);
 
+	mdelay(RTL8169_EE_TIMEOUT);
 	/* enable device (incl. PCI PM wakeup and hotplug setup) */
 	rc = pci_enable_device(pdev);
 	if (rc < 0) {
-- 
2.5.0

Re: [PATCH net v4] r8169: Added delay after power on and reset.

From: David Miller <davem@davemloft.net>
Date: 2016-02-01 19:40:19

From: Corcodel Marian <redacted>
Date: Mon,  1 Feb 2016 18:05:20 +0200
On reset and power on need 2msecs delay to autoload complete data
 from eeprom to VPD.

Signed-off-by: Corcodel Marian <redacted>
Where is this needed delay documented?

Unless you are a Realtek engineer, or learned about the requirement
from them, I want to see the HW document that explains this.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help