diff options
author | Jeff Garzik <jeff@garzik.org> | 2007-10-29 05:46:16 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:03:40 -0500 |
commit | 28fc1f5a0c375cb6375fa48e9a8b393f2a189be6 (patch) | |
tree | 95b3b517b02dd46ff0fc2728c1e70de8d6f78ed3 /drivers/net/cpmac.c | |
parent | 1b36efe07f83ecf50eddd431d067c7bf86318e87 (diff) |
[netdrvr] irq handler minor cleanups in several drivers
* use irq_handler_t where appropriate
* no need to use 'irq' function arg, its already stored in a data struct
* rename irq handler 'irq' argument to 'dummy', where the function
has been analyzed and proven not to use its first argument.
* remove always-false "dev_id == NULL" test from irq handlers
* remove pointless casts from void*
* declance: irq argument is not const
* add KERN_xxx printk prefix
* fix minor whitespace weirdness
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/net/cpmac.c')
-rw-r--r-- | drivers/net/cpmac.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/net/cpmac.c b/drivers/net/cpmac.c index 6e12d48351b8..6ccebb830ff9 100644 --- a/drivers/net/cpmac.c +++ b/drivers/net/cpmac.c | |||
@@ -661,9 +661,6 @@ static irqreturn_t cpmac_irq(int irq, void *dev_id) | |||
661 | int queue; | 661 | int queue; |
662 | u32 status; | 662 | u32 status; |
663 | 663 | ||
664 | if (!dev) | ||
665 | return IRQ_NONE; | ||
666 | |||
667 | priv = netdev_priv(dev); | 664 | priv = netdev_priv(dev); |
668 | 665 | ||
669 | status = cpmac_read(priv->regs, CPMAC_MAC_INT_VECTOR); | 666 | status = cpmac_read(priv->regs, CPMAC_MAC_INT_VECTOR); |