diff options
author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-06-18 16:22:07 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-06-24 18:51:01 -0400 |
commit | d9990ceacc39b4320661adf7778538858a5c4bad (patch) | |
tree | aac6a1e7cac041d419e82bce43b55d761651ecb5 | |
parent | 3de38f258f95b6dc9f614e1f64b8d596a75533e0 (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.c | 11 |
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; |