summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorH Hartley Sweeten <hsweeten@visionengravers.com>2013-11-26 16:42:09 -0700
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-12-03 09:20:22 -0800
commitc1ce7f10543d616920a1eb772b993c7d7815d653 (patch)
tree6a71825a057869dd3f07844e641924af5f4e63f8
parentb1ca6b1d479931a682ef443b23e00b4abae8ce25 (diff)
downloadlinux-c1ce7f10543d616920a1eb772b993c7d7815d653.tar.bz2
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.c4
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,
* We always write 0 to GNSWA bit, so the channel range is +-/10.1Vdc
*/
outb(insn->chanspec, dev->iobase + MPC624_GNMUXCH);
- if (!insn->n) {
- printk(KERN_INFO "MPC624: Warning, no data to acquire\n");
- return 0;
- }
for (n = 0; n < insn->n; n++) {
/* Trigger the conversion */