summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Abbott <abbotti@mev.co.uk>2015-10-12 19:21:26 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-10-13 20:29:48 +0300
commitf84e27adfc4cb0c4aa97d830c43f8c82aca0547c (patch)
treea1e9d2d8bf5899a211e4d5f74b6d2ca10378f38d
parent42ea907d785d522e5ae661c3774b8a801ac7f00b (diff)
downloadlinux-f84e27adfc4cb0c4aa97d830c43f8c82aca0547c.tar.xz
staging: comedi: remove superfluous retval = 0 in comedi_read()
`comedi_read()` initializes `retval` to 0. The other `retval = 0` assignments are superfluous, so remove them. Signed-off-by: Ian Abbott <abbotti@mev.co.uk> Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/staging/comedi/comedi_fops.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/staging/comedi/comedi_fops.c b/drivers/staging/comedi/comedi_fops.c
index a979f621f932..88ba0b648805 100644
--- a/drivers/staging/comedi/comedi_fops.c
+++ b/drivers/staging/comedi/comedi_fops.c
@@ -2504,8 +2504,6 @@ static ssize_t comedi_read(struct file *file, char __user *buf, size_t nbytes,
if (!comedi_is_runflags_running(runflags)) {
if (comedi_is_runflags_in_error(runflags))
retval = -EPIPE;
- else
- retval = 0;
become_nonbusy = true;
break;
}
@@ -2519,7 +2517,6 @@ static ssize_t comedi_read(struct file *file, char __user *buf, size_t nbytes,
break;
}
if (!s->busy) {
- retval = 0;
break;
}
if (s->busy != file) {