Commit e2f6df94 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: pcl812: tidy up the irq support in pcl812_attach()

Tidy up the code that does the request_irq().
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: default avatarIan Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5d24b7de
...@@ -1088,7 +1088,6 @@ static int pcl812_attach(struct comedi_device *dev, struct comedi_devconfig *it) ...@@ -1088,7 +1088,6 @@ static int pcl812_attach(struct comedi_device *dev, struct comedi_devconfig *it)
const struct pcl812_board *board = comedi_board(dev); const struct pcl812_board *board = comedi_board(dev);
struct pcl812_private *devpriv; struct pcl812_private *devpriv;
int ret, subdev; int ret, subdev;
unsigned int irq;
unsigned int dma; unsigned int dma;
unsigned long pages; unsigned long pages;
struct comedi_subdevice *s; struct comedi_subdevice *s;
...@@ -1102,31 +1101,13 @@ static int pcl812_attach(struct comedi_device *dev, struct comedi_devconfig *it) ...@@ -1102,31 +1101,13 @@ static int pcl812_attach(struct comedi_device *dev, struct comedi_devconfig *it)
if (!devpriv) if (!devpriv)
return -ENOMEM; return -ENOMEM;
irq = 0; if ((1 << it->options[1]) & board->IRQbits) {
if (board->IRQbits != 0) { /* board support IRQ */ ret = request_irq(it->options[1], interrupt_pcl812, 0,
irq = it->options[1]; dev->board_name, dev);
if (irq) { /* we want to use IRQ */ if (ret == 0)
if (((1 << irq) & board->IRQbits) == 0) { dev->irq = it->options[1];
printk
(", IRQ %u is out of allowed range, "
"DISABLING IT", irq);
irq = 0; /* Bad IRQ */
} else {
if (request_irq(irq, interrupt_pcl812, 0,
dev->board_name, dev)) {
printk
(", unable to allocate IRQ %u, "
"DISABLING IT", irq);
irq = 0; /* Can't use IRQ */
} else {
printk(KERN_INFO ", irq=%u", irq);
}
}
}
} }
dev->irq = irq;
dma = 0; dma = 0;
devpriv->dma = dma; devpriv->dma = dma;
if (!dev->irq) if (!dev->irq)
...@@ -1395,7 +1376,7 @@ static int pcl812_attach(struct comedi_device *dev, struct comedi_devconfig *it) ...@@ -1395,7 +1376,7 @@ static int pcl812_attach(struct comedi_device *dev, struct comedi_devconfig *it)
break; break;
case boardA821: case boardA821:
devpriv->max_812_ai_mode0_rangewait = 1; devpriv->max_812_ai_mode0_rangewait = 1;
devpriv->mode_reg_int = (irq << 4) & 0xf0; devpriv->mode_reg_int = (dev->irq << 4) & 0xf0;
break; break;
case boardPCL813B: case boardPCL813B:
case boardPCL813: case boardPCL813:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment