diff options
author | Tina Johnson | 2015-03-09 11:41:17 +0100 |
---|---|---|
committer | Greg Kroah-Hartman | 2015-03-09 13:39:10 +0100 |
commit | 90a9de83578fb7890a2f5a96dd393621f165ba5a (patch) | |
tree | df12c28624fc63cd56b9e6f37cc1c44168b95574 /drivers/staging | |
parent | drivers: staging: iio: meter: Removed unnecessary variable (diff) | |
download | kernel-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')
-rw-r--r-- | drivers/staging/iio/meter/ade7758_core.c | 5 |
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, |