Skip to contentSkip to conversation

tulip: Poll link status more frequently for Comet chips

PATCH Applied Ondrej Zary 2 participants 2 messages Jun 22, 2014 – Jun 23, 2014

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 tulip: Poll link status more frequently for Comet chips

It now takes up to 60 seconds to detect cable (un)plug on ADMtek Comet chips. That's too slow and might cause people to think that it doesn't work at all.

Poll link status every 2 seconds instead of 60 for ADMtek Comet chips. That should be fast enough while not stressing the system too much.

Tested with ADMtek AN983B.

---
 drivers/net/ethernet/dec/tulip/timer.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/dec/tulip/timer.c +1 -1
index 768379b..523d9dd 100644
@@ -158,7 +158,7 @@ void comet_timer(unsigned long data)
 {
 	struct net_device *dev = (struct net_device *)data;
 	struct tulip_private *tp = netdev_priv(dev);
-	int next_tick = 60*HZ;
+	int next_tick = 2*HZ;
 
 	if (tulip_debug > 1)
 		netdev_dbg(dev, "Comet link status %04x partner capability %04x\n",
Signature · 1 line
Ondrej Zary
maintainer Applied
#1 lore raw

From: Ondrej Zary <[email protected]>
Date: Sun, 22 Jun 2014 12:01:12 +0200

Quoted 9 lines

It now takes up to 60 seconds to detect cable (un)plug on ADMtek Comet chips. That's too slow and might cause people to think that it doesn't work at all.

Poll link status every 2 seconds instead of 60 for ADMtek Comet chips. That should be fast enough while not stressing the system too much.

Tested with ADMtek AN983B.

Looks good, and this is consistent with tulip_media_timer().

Applied, thanks.