summaryrefslogtreecommitdiffstats
path: root/drivers/net/irda
diff options
context:
space:
mode:
authorJeff Garzik <jeff@garzik.org>2007-10-29 05:18:12 -0400
committerJeff Garzik <jeff@garzik.org>2007-10-29 05:42:18 -0400
commite38c2c651a038b78fd01cf2e3f3a65cacf0e41cc (patch)
treed40990c037b97e2c113143e3466b4e8f7e3fc40e /drivers/net/irda
parent2a397e82c7db18019e408f953dd58dc1963a328c (diff)
downloadop-kernel-dev-e38c2c651a038b78fd01cf2e3f3a65cacf0e41cc.zip
op-kernel-dev-e38c2c651a038b78fd01cf2e3f3a65cacf0e41cc.tar.gz
drivers/net/irda/au1k_ir: fix obvious irq handler bugs
interrupt handlers return a return value these days. Also, kill always-true test and unneeded void* cast. Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/net/irda')
-rw-r--r--drivers/net/irda/au1k_ir.c11
1 files changed, 4 insertions, 7 deletions
diff --git a/drivers/net/irda/au1k_ir.c b/drivers/net/irda/au1k_ir.c
index 4dbdfaa..a1e4508 100644
--- a/drivers/net/irda/au1k_ir.c
+++ b/drivers/net/irda/au1k_ir.c
@@ -627,19 +627,16 @@ static int au1k_irda_rx(struct net_device *dev)
}
-void au1k_irda_interrupt(int irq, void *dev_id)
+static irqreturn_t au1k_irda_interrupt(int dummy, void *dev_id)
{
- struct net_device *dev = (struct net_device *) dev_id;
-
- if (dev == NULL) {
- printk(KERN_ERR "%s: isr: null dev ptr\n", dev->name);
- return;
- }
+ struct net_device *dev = dev_id;
writel(0, IR_INT_CLEAR); /* ack irda interrupts */
au1k_irda_rx(dev);
au1k_tx_ack(dev);
+
+ return IRQ_HANDLED;
}
OpenPOWER on IntegriCloud