dmfe: enforce consistent timing delay.
authorfrançois romieu <romieu@fr.zoreil.com>
Tue, 17 Apr 2012 11:11:40 +0000 (11:11 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Apr 2012 02:54:13 +0000 (22:54 -0400)
The driver does not always use the same timing for what looks like
the same operations.

- DCR0
  Use the same udelay everywhere for reset. Upper bound is 100 us.
- DCR9
  Use 5us delay for srom clock. 1us delay for phy_write_1bit (writes
  PHY_DATA_[01]) are not changed as they stay withing a 2,5MHz MDIO
  clock range.

Signed-off-by: Francois Romieu <romieu@fr.zoreil.com>
Reviewed-by: Grant Grundler <grundler@parisc-linux.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/dec/tulip/dmfe.c

index 0ef5b68acd0509d05c6fb14077426b185baadc0c..4d6fe604fa64b920d58b05f3989a365355f951c3 100644 (file)
@@ -767,7 +767,7 @@ static int dmfe_stop(struct DEVICE *dev)
 
        /* Reset & stop DM910X board */
        dw32(DCR0, DM910X_RESET);
-       udelay(5);
+       udelay(100);
        phy_write(ioaddr, db->phy_addr, 0, 0x8000, db->chip_id);
 
        /* free interrupt */
@@ -1601,7 +1601,9 @@ static u16 read_srom_word(void __iomem *ioaddr, int offset)
        int i;
 
        dw32(DCR9, CR9_SROM_READ);
+       udelay(5);
        dw32(DCR9, CR9_SROM_READ | CR9_SRCS);
+       udelay(5);
 
        /* Send the Read Command 110b */
        srom_clk_write(ioaddr, SROM_DATA_1);
@@ -1615,6 +1617,7 @@ static u16 read_srom_word(void __iomem *ioaddr, int offset)
        }
 
        dw32(DCR9, CR9_SROM_READ | CR9_SRCS);
+       udelay(5);
 
        for (i = 16; i > 0; i--) {
                dw32(DCR9, CR9_SROM_READ | CR9_SRCS | CR9_SRCLK);
@@ -1626,6 +1629,7 @@ static u16 read_srom_word(void __iomem *ioaddr, int offset)
        }
 
        dw32(DCR9, CR9_SROM_READ);
+       udelay(5);
        return srom_data;
 }