diff options
| author | Jeff Garzik <jeff@garzik.org> | 2006-10-06 14:56:04 -0400 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-10-06 14:56:04 -0400 |
| commit | c31f28e778ab299a5035ea2bda64f245b8915d7c (patch) | |
| tree | 92d1070b0ae0c3528ab2c8787c4402fd8adf5a5f /drivers/net/tokenring | |
| parent | 86d91bab4806191a8126502d80d729c2a4765ebe (diff) | |
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/tokenring')
| -rw-r--r-- | drivers/net/tokenring/smctr.c | 7 | ||||
| -rw-r--r-- | drivers/net/tokenring/tms380tr.c | 5 |
2 files changed, 0 insertions, 12 deletions
diff --git a/drivers/net/tokenring/smctr.c b/drivers/net/tokenring/smctr.c index 9bd4cba87872..46dabdb12071 100644 --- a/drivers/net/tokenring/smctr.c +++ b/drivers/net/tokenring/smctr.c | |||
| @@ -1990,15 +1990,8 @@ static irqreturn_t smctr_interrupt(int irq, void *dev_id) | |||
| 1990 | __u8 isb_type, isb_subtype; | 1990 | __u8 isb_type, isb_subtype; |
| 1991 | __u16 isb_index; | 1991 | __u16 isb_index; |
| 1992 | 1992 | ||
| 1993 | if(dev == NULL) | ||
| 1994 | { | ||
| 1995 | printk(KERN_CRIT "%s: irq %d for unknown device.\n", dev->name, irq); | ||
| 1996 | return IRQ_NONE; | ||
| 1997 | } | ||
| 1998 | |||
| 1999 | ioaddr = dev->base_addr; | 1993 | ioaddr = dev->base_addr; |
| 2000 | tp = netdev_priv(dev); | 1994 | tp = netdev_priv(dev); |
| 2001 | |||
| 2002 | 1995 | ||
| 2003 | if(tp->status == NOT_INITIALIZED) | 1996 | if(tp->status == NOT_INITIALIZED) |
| 2004 | return IRQ_NONE; | 1997 | return IRQ_NONE; |
diff --git a/drivers/net/tokenring/tms380tr.c b/drivers/net/tokenring/tms380tr.c index c0ab6e44eb1f..ea797ca2b988 100644 --- a/drivers/net/tokenring/tms380tr.c +++ b/drivers/net/tokenring/tms380tr.c | |||
| @@ -751,11 +751,6 @@ irqreturn_t tms380tr_interrupt(int irq, void *dev_id) | |||
| 751 | unsigned short irq_type; | 751 | unsigned short irq_type; |
| 752 | int handled = 0; | 752 | int handled = 0; |
| 753 | 753 | ||
| 754 | if(dev == NULL) { | ||
| 755 | printk(KERN_INFO "%s: irq %d for unknown device.\n", dev->name, irq); | ||
| 756 | return IRQ_NONE; | ||
| 757 | } | ||
| 758 | |||
| 759 | tp = netdev_priv(dev); | 754 | tp = netdev_priv(dev); |
| 760 | 755 | ||
| 761 | irq_type = SIFREADW(SIFSTS); | 756 | irq_type = SIFREADW(SIFSTS); |
