diff options
author | Jeff Garzik <jeff@garzik.org> | 2007-10-19 03:14:03 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-10-23 19:53:16 -0400 |
commit | 1daec86ad11383845274e032d1b90620258dc87d (patch) | |
tree | 01fb555646b490dd494e1d51869c6f1d51cba7e7 /drivers/net/lib82596.c | |
parent | 06efcad0d43a5491602f7d7bfc1ce997cdb0d062 (diff) |
[NETDRVR] lib82596, netxen: delete pointless tests from irq handler
Remove always-false tests in irq handler.
Also a few other minor cleanups.
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/net/lib82596.c')
-rw-r--r-- | drivers/net/lib82596.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/drivers/net/lib82596.c b/drivers/net/lib82596.c index ffaa14f2cd0..9a855e51214 100644 --- a/drivers/net/lib82596.c +++ b/drivers/net/lib82596.c | |||
@@ -1124,12 +1124,6 @@ static irqreturn_t i596_interrupt(int irq, void *dev_id) | |||
1124 | struct i596_dma *dma; | 1124 | struct i596_dma *dma; |
1125 | unsigned short status, ack_cmd = 0; | 1125 | unsigned short status, ack_cmd = 0; |
1126 | 1126 | ||
1127 | if (dev == NULL) { | ||
1128 | printk(KERN_WARNING "%s: irq %d for unknown device.\n", | ||
1129 | __FUNCTION__, irq); | ||
1130 | return IRQ_NONE; | ||
1131 | } | ||
1132 | |||
1133 | lp = netdev_priv(dev); | 1127 | lp = netdev_priv(dev); |
1134 | dma = lp->dma; | 1128 | dma = lp->dma; |
1135 | 1129 | ||
@@ -1140,7 +1134,7 @@ static irqreturn_t i596_interrupt(int irq, void *dev_id) | |||
1140 | 1134 | ||
1141 | DEB(DEB_INTS, printk(KERN_DEBUG | 1135 | DEB(DEB_INTS, printk(KERN_DEBUG |
1142 | "%s: i596 interrupt, IRQ %d, status %4.4x.\n", | 1136 | "%s: i596 interrupt, IRQ %d, status %4.4x.\n", |
1143 | dev->name, irq, status)); | 1137 | dev->name, dev->irq, status)); |
1144 | 1138 | ||
1145 | ack_cmd = status & 0xf000; | 1139 | ack_cmd = status & 0xf000; |
1146 | 1140 | ||