diff options
| author | Jeff Garzik <jeff@garzik.org> | 2006-10-06 20:56:04 +0200 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-10-06 20:56:04 +0200 |
| commit | c31f28e778ab299a5035ea2bda64f245b8915d7c (patch) | |
| tree | 92d1070b0ae0c3528ab2c8787c4402fd8adf5a5f /drivers/net/sonic.c | |
| parent | arch/i386/kernel/time: don't shadow 'irq' function arg (diff) | |
| download | linux-c31f28e778ab299a5035ea2bda64f245b8915d7c.tar.xz linux-c31f28e778ab299a5035ea2bda64f245b8915d7c.zip | |
drivers/net: eliminate irq handler impossible checks, needless casts
- Eliminate check for irq handler 'dev_id==NULL' where the
condition never occurs.
- Eliminate needless casts to/from void*
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net/sonic.c')
| -rw-r--r-- | drivers/net/sonic.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/net/sonic.c b/drivers/net/sonic.c index cfece9676aff..ed7aa0a5acca 100644 --- a/drivers/net/sonic.c +++ b/drivers/net/sonic.c @@ -295,15 +295,10 @@ static int sonic_send_packet(struct sk_buff *skb, struct net_device *dev) */ static irqreturn_t sonic_interrupt(int irq, void *dev_id) { - struct net_device *dev = (struct net_device *) dev_id; + struct net_device *dev = dev_id; struct sonic_local *lp = netdev_priv(dev); int status; - if (dev == NULL) { - printk(KERN_ERR "sonic_interrupt: irq %d for unknown device.\n", irq); - return IRQ_NONE; - } - if (!(status = SONIC_READ(SONIC_ISR) & SONIC_IMR_DEFAULT)) return IRQ_NONE; |