AcoSail/kernel_optimize_test
8
0
Fork 0
forked from luck/tmp_suning_uos_patched
Code Pull Requests Activity

staging: comedi: pcmmio: remove unnecessary for () loop and sanity check

Browse Source 
There is only one asic on the board so the for () loop in the interrupt
handler to check multiple asics is not needed.

The sanity check of the 'irq' in the interrupt handler is also not needed.
If it _is_ wrong we have bigger problems in the kernel.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
H Hartley Sweeten 2013-12-09 15:31:05 -07:00 committed by Greg Kroah-Hartman
parent 19e0bf1235
commit 1786bd7503
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/staging/comedi/drivers/pcmmio.c
View File

@ -402,10 +402,8 @@ static irqreturn_t interrupt_pcmmio(int irq, void *d)
struct pcmmio_private *devpriv = dev->private;
struct comedi_subdevice *s = dev->read_subdev;
struct pcmmio_subdev_private *subpriv = s->private;
int asic, got1 = 0;
int got1 = 0;
for (asic = 0; asic < MAX_ASICS; ++asic) {
if (irq == dev->irq) {
unsigned long flags;
unsigned triggered = 0;
/* it is an interrupt for ASIC #asic */
@ -456,7 +454,7 @@ static irqreturn_t interrupt_pcmmio(int irq, void *d)
* this is an interrupt subdev,
* and it matches this asic!
*/
if (subpriv->dio.intr.asic == asic) {
if (subpriv->dio.intr.asic == 0) {
unsigned long flags;
unsigned oldevents;
@ -539,8 +537,6 @@ static irqreturn_t interrupt_pcmmio(int irq, void *d)
}
}
}
if (!got1)
return IRQ_NONE; /* interrupt from other source */
return IRQ_HANDLED;