diff options
author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-11-26 18:42:09 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-12-03 12:20:22 -0500 |
commit | c1ce7f10543d616920a1eb772b993c7d7815d653 (patch) | |
tree | 6a71825a057869dd3f07844e641924af5f4e63f8 | |
parent | b1ca6b1d479931a682ef443b23e00b4abae8ce25 (diff) |
staging: comedi: mpc624: remove unnecessary test
It's not necessaary to vaidate insn->n passed to the (*insn_read)
function. The for() loop following it will just be a NOP.
Remove the test as well as the printk noise.
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/mpc624.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/staging/comedi/drivers/mpc624.c b/drivers/staging/comedi/drivers/mpc624.c index 2d6add4f97f7..b105dd98c358 100644 --- a/drivers/staging/comedi/drivers/mpc624.c +++ b/drivers/staging/comedi/drivers/mpc624.c | |||
@@ -159,10 +159,6 @@ static int mpc624_ai_rinsn(struct comedi_device *dev, | |||
159 | * We always write 0 to GNSWA bit, so the channel range is +-/10.1Vdc | 159 | * We always write 0 to GNSWA bit, so the channel range is +-/10.1Vdc |
160 | */ | 160 | */ |
161 | outb(insn->chanspec, dev->iobase + MPC624_GNMUXCH); | 161 | outb(insn->chanspec, dev->iobase + MPC624_GNMUXCH); |
162 | if (!insn->n) { | ||
163 | printk(KERN_INFO "MPC624: Warning, no data to acquire\n"); | ||
164 | return 0; | ||
165 | } | ||
166 | 162 | ||
167 | for (n = 0; n < insn->n; n++) { | 163 | for (n = 0; n < insn->n; n++) { |
168 | /* Trigger the conversion */ | 164 | /* Trigger the conversion */ |