aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorH Hartley Sweeten <hsweeten@visionengravers.com>2013-06-18 16:22:07 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-06-24 18:51:01 -0400
commitd9990ceacc39b4320661adf7778538858a5c4bad (patch)
treeaac6a1e7cac041d419e82bce43b55d761651ecb5
parent3de38f258f95b6dc9f614e1f64b8d596a75533e0 (diff)
staging: comedi: pcmuio: remove 'first_chan'
This member of the subdevice private data is always initialize to '0' due to the 'byte_no' always being zero when it is set during the attach. Simplify the driver a bit by removing it. 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>
-rw-r--r--drivers/staging/comedi/drivers/pcmuio.c11
1 files changed, 2 insertions, 9 deletions
diff --git a/drivers/staging/comedi/drivers/pcmuio.c b/drivers/staging/comedi/drivers/pcmuio.c
index 0e71eacae14b..8771e4b5d4cb 100644
--- a/drivers/staging/comedi/drivers/pcmuio.c
+++ b/drivers/staging/comedi/drivers/pcmuio.c
@@ -152,8 +152,6 @@ struct pcmuio_subdev_private {
152 struct { 152 struct {
153 /* if non-negative, this subdev has an interrupt asic */ 153 /* if non-negative, this subdev has an interrupt asic */
154 int asic; 154 int asic;
155 /* if nonnegative, the first channel id for interrupts */
156 int first_chan;
157 /* 155 /*
158 * the number of asic channels in this 156 * the number of asic channels in this
159 * subdev that have interrutps 157 * subdev that have interrutps
@@ -377,7 +375,6 @@ static void pcmuio_handle_intr_subdev(struct comedi_device *dev,
377 375
378 mytrig = triggered >> subpriv->intr.asic_chan; 376 mytrig = triggered >> subpriv->intr.asic_chan;
379 mytrig &= ((0x1 << subpriv->intr.num_asic_chans) - 1); 377 mytrig &= ((0x1 << subpriv->intr.num_asic_chans) - 1);
380 mytrig <<= subpriv->intr.first_chan;
381 378
382 if (!(mytrig & subpriv->intr.enabled_mask)) 379 if (!(mytrig & subpriv->intr.enabled_mask))
383 goto done; 380 goto done;
@@ -520,8 +517,7 @@ static int pcmuio_start_intr(struct comedi_device *dev,
520 << CR_CHAN(cmd->chanlist[n]); 517 << CR_CHAN(cmd->chanlist[n]);
521 } 518 }
522 } 519 }
523 bits &= ((0x1 << subpriv->intr.num_asic_chans) - 520 bits &= ((0x1 << subpriv->intr.num_asic_chans) - 1);
524 1) << subpriv->intr.first_chan;
525 subpriv->intr.enabled_mask = bits; 521 subpriv->intr.enabled_mask = bits;
526 522
527 /* set pol and enab intrs for this subdev.. */ 523 /* set pol and enab intrs for this subdev.. */
@@ -727,7 +723,6 @@ static int pcmuio_attach(struct comedi_device *dev, struct comedi_devconfig *it)
727 s->insn_config = pcmuio_dio_insn_config; 723 s->insn_config = pcmuio_dio_insn_config;
728 s->n_chan = min(chans_left, MAX_CHANS_PER_SUBDEV); 724 s->n_chan = min(chans_left, MAX_CHANS_PER_SUBDEV);
729 subpriv->intr.asic = -1; 725 subpriv->intr.asic = -1;
730 subpriv->intr.first_chan = -1;
731 subpriv->intr.asic_chan = -1; 726 subpriv->intr.asic_chan = -1;
732 subpriv->intr.num_asic_chans = -1; 727 subpriv->intr.num_asic_chans = -1;
733 subpriv->intr.active = 0; 728 subpriv->intr.active = 0;
@@ -752,10 +747,8 @@ static int pcmuio_attach(struct comedi_device *dev, struct comedi_devconfig *it)
752 subpriv->intr.asic = asic; 747 subpriv->intr.asic = asic;
753 subpriv->intr.active = 0; 748 subpriv->intr.active = 0;
754 subpriv->intr.stop_count = 0; 749 subpriv->intr.stop_count = 0;
755 subpriv->intr.first_chan = byte_no * 8;
756 subpriv->intr.asic_chan = thisasic_chanct; 750 subpriv->intr.asic_chan = thisasic_chanct;
757 subpriv->intr.num_asic_chans = 751 subpriv->intr.num_asic_chans = s->n_chan;
758 s->n_chan - subpriv->intr.first_chan;
759 dev->read_subdev = s; 752 dev->read_subdev = s;
760 s->subdev_flags |= SDF_CMD_READ; 753 s->subdev_flags |= SDF_CMD_READ;
761 s->cancel = pcmuio_cancel; 754 s->cancel = pcmuio_cancel;