summaryrefslogtreecommitdiffstats
path: root/drivers/staging/iio/meter
diff options
context:
space:
mode:
authorTina Johnson2015-03-09 11:41:17 +0100
committerGreg Kroah-Hartman2015-03-09 13:39:10 +0100
commit90a9de83578fb7890a2f5a96dd393621f165ba5a (patch)
treedf12c28624fc63cd56b9e6f37cc1c44168b95574 /drivers/staging/iio/meter
parentdrivers: staging: iio: meter: Removed unnecessary variable (diff)
downloadkernel-qcow2-linux-90a9de83578fb7890a2f5a96dd393621f165ba5a.tar.gz
kernel-qcow2-linux-90a9de83578fb7890a2f5a96dd393621f165ba5a.tar.xz
kernel-qcow2-linux-90a9de83578fb7890a2f5a96dd393621f165ba5a.zip
drivers: staging: iio: meter: Removed unnecessary variable
Variable len is used only to store the return value. Hence len is removed and the return statement modified. Coccinelle was used to detect such removable variables: @rule1@ identifier ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: Tina Johnson <tinajohnson.1234@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/iio/meter')
-rw-r--r--drivers/staging/iio/meter/ade7758_core.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/staging/iio/meter/ade7758_core.c b/drivers/staging/iio/meter/ade7758_core.c
index 7e287dae7b44..652aa1069b06 100644
--- a/drivers/staging/iio/meter/ade7758_core.c
+++ b/drivers/staging/iio/meter/ade7758_core.c
@@ -485,7 +485,7 @@ static ssize_t ade7758_read_frequency(struct device *dev,
struct device_attribute *attr,
char *buf)
{
- int ret, len = 0;
+ int ret;
u8 t;
int sps;
@@ -498,8 +498,7 @@ static ssize_t ade7758_read_frequency(struct device *dev,
t = (t >> 5) & 0x3;
sps = 26040 / (1 << t);
- len = sprintf(buf, "%d SPS\n", sps);
- return len;
+ return sprintf(buf, "%d SPS\n", sps);
}
static ssize_t ade7758_write_frequency(struct device *dev,