]> err.no Git - linux-2.6/commitdiff
DM9000: Do not sleep with spinlock and IRQs held
authorBen Dooks <ben-linux@fluff.org>
Tue, 5 Feb 2008 00:02:07 +0000 (00:02 +0000)
committerJeff Garzik <jeff@garzik.org>
Mon, 11 Feb 2008 16:06:26 +0000 (11:06 -0500)
The phy read and write routines call udelay() with the board
lock held, and with the posibility of IRQs being disabled. Since
these delays can be up to 500usec, and are only required as we
have to save the chip's address register.

To improve the behaviour, hold the lock whilst we are writing
and then restore the state before the delay and then repeat
the process once the delay has happened.

Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/net/dm9000.c

index 709fd674ec384cb6880f57b338bfa34b5c96c251..071aad1af577e1c6c4d6cfa7e8afac2f4588cd4b 100644 (file)
@@ -1127,7 +1127,15 @@ dm9000_phy_read(struct net_device *dev, int phy_reg_unused, int reg)
        iow(db, DM9000_EPAR, DM9000_PHY | reg);
 
        iow(db, DM9000_EPCR, 0xc);      /* Issue phyxcer read command */
+
+       writeb(reg_save, db->io_addr);
+       spin_unlock_irqrestore(&db->lock,flags);
+
        udelay(100);            /* Wait read complete */
+
+       spin_lock_irqsave(&db->lock,flags);
+       reg_save = readb(db->io_addr);
+
        iow(db, DM9000_EPCR, 0x0);      /* Clear phyxcer read command */
 
        /* The read data keeps on REG_0D & REG_0E */
@@ -1135,7 +1143,6 @@ dm9000_phy_read(struct net_device *dev, int phy_reg_unused, int reg)
 
        /* restore the previous address */
        writeb(reg_save, db->io_addr);
-
        spin_unlock_irqrestore(&db->lock,flags);
 
        return ret;
@@ -1164,7 +1171,15 @@ dm9000_phy_write(struct net_device *dev, int phyaddr_unused, int reg, int value)
        iow(db, DM9000_EPDRH, ((value >> 8) & 0xff));
 
        iow(db, DM9000_EPCR, 0xa);      /* Issue phyxcer write command */
+
+       writeb(reg_save, db->io_addr);
+       spin_unlock_irqrestore(&db->lock,flags);
+
        udelay(500);            /* Wait write complete */
+
+       spin_lock_irqsave(&db->lock,flags);
+       reg_save = readb(db->io_addr);
+
        iow(db, DM9000_EPCR, 0x0);      /* Clear phyxcer write command */
 
        /* restore the previous address */