OSDN Git Service

Staging: iio: Remove unnecessory immediate else after return statement
authorVaishali Thakkar <vthakkar1994@gmail.com>
Thu, 25 Sep 2014 10:39:57 +0000 (16:09 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 29 Sep 2014 02:19:04 +0000 (22:19 -0400)
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>
drivers/staging/iio/adc/ad7816.c

index d83b3fc..734a7e4 100644 (file)
@@ -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,