diff options
| author | Vaishali Thakkar <vthakkar1994@gmail.com> | 2014-09-25 16:09:57 +0530 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-28 22:19:04 -0400 |
| commit | da96aecdc59d086926d49becd981acab8d98bb9a (patch) | |
| tree | 1297c14c01a86a4d3f7db0815fae385796cb0c2c | |
| parent | 7bf6513eeb51d1ae0a2a9b820368c13e55f44835 (diff) | |
Staging: iio: Remove unnecessory immediate else after return statement
This patch fixes following checkpatch.pl warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Vaishali Thakkar <vthakkar1994@gmail.com>
Acked-by: Daniel Baluta <daniel.baluta@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/iio/adc/ad7816.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/staging/iio/adc/ad7816.c b/drivers/staging/iio/adc/ad7816.c index d83b3fc9b95..734a7e4886a 100644 --- a/drivers/staging/iio/adc/ad7816.c +++ b/drivers/staging/iio/adc/ad7816.c @@ -118,8 +118,7 @@ static ssize_t ad7816_show_mode(struct device *dev, if (chip->mode) return sprintf(buf, "power-save\n"); - else - return sprintf(buf, "full\n"); + return sprintf(buf, "full\n"); } static ssize_t ad7816_store_mode(struct device *dev, @@ -226,8 +225,8 @@ static ssize_t ad7816_show_value(struct device *dev, if (value < 0) data = (1 << AD7816_TEMP_FLOAT_OFFSET) - data; return sprintf(buf, "%d.%.2d\n", value, data * 25); - } else - return sprintf(buf, "%u\n", data); + } + return sprintf(buf, "%u\n", data); } static IIO_DEVICE_ATTR(value, S_IRUGO, ad7816_show_value, NULL, 0); @@ -275,8 +274,8 @@ static ssize_t ad7816_show_oti(struct device *dev, (chip->oti_data[chip->channel_id] - AD7816_BOUND_VALUE_BASE); return sprintf(buf, "%d\n", value); - } else - return sprintf(buf, "%u\n", chip->oti_data[chip->channel_id]); + } + return sprintf(buf, "%u\n", chip->oti_data[chip->channel_id]); } static inline ssize_t ad7816_set_oti(struct device *dev, |