Wait for the link to come up on kirkwood network init
authorSimon Kagstrom <simon.kagstrom@netinsight.net>
Thu, 20 Aug 2009 08:13:06 +0000 (10:13 +0200)
committerBen Warren <biggerbadderben@gmail.com>
Fri, 21 Aug 2009 16:57:33 +0000 (09:57 -0700)
This patch makes the device wait for up to 5 seconds for the link to
come up, similar to what many of the other network drivers do. This
avoids confusing situations where, e.g., a tftp fails when initiated
early after U-boot has started (before the link has come up).

Signed-off-by: Simon Kagstrom <simon.kagstrom@netinsight.net>
Signed-off-by: Ben Warren <biggerbadderben@gmail.com>
drivers/net/kirkwood_egiga.c

index 065e335..9f36633 100644 (file)
@@ -400,6 +400,7 @@ static int kwgbe_init(struct eth_device *dev)
 {
        struct kwgbe_device *dkwgbe = to_dkwgbe(dev);
        struct kwgbe_registers *regs = dkwgbe->regs;
+       int i;
 
        /* setup RX rings */
        kwgbe_init_rx_desc_ring(dkwgbe);
@@ -447,13 +448,20 @@ static int kwgbe_init(struct eth_device *dev)
 
 #if (defined (CONFIG_MII) || defined (CONFIG_CMD_MII)) \
         && defined (CONFIG_SYS_FAULT_ECHO_LINK_DOWN)
-       u16 phyadr;
-       miiphy_read(dev->name, KIRKWOOD_PHY_ADR_REQUEST,
-                       KIRKWOOD_PHY_ADR_REQUEST, &phyadr);
-       if (!miiphy_link(dev->name, phyadr)) {
-               printf("%s: No link on %s\n", __FUNCTION__, dev->name);
-               return -1;
+       /* Wait up to 5s for the link status */
+       for (i = 0; i < 5; i++) {
+               u16 phyadr;
+
+               miiphy_read(dev->name, KIRKWOOD_PHY_ADR_REQUEST,
+                               KIRKWOOD_PHY_ADR_REQUEST, &phyadr);
+               /* Return if we get link up */
+               if (miiphy_link(dev->name, phyadr))
+                       return 0;
+               udelay(1000000);
        }
+
+       printf("No link on %s\n", dev->name);
+       return -1;
 #endif
        return 0;
 }