iio: core: Get rid of redundant 'else'
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 21 Jul 2023 17:00:19 +0000 (20:00 +0300)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 23 Jul 2023 09:15:42 +0000 (10:15 +0100)
In the snippets like the following

if (...)
return / goto / break / continue ...;
else
...

the 'else' is redundant. Get rid of it.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Nuno Sa <nuno.sa@analog.com>
Link: https://lore.kernel.org/r/20230721170022.3461-6-andriy.shevchenko@linux.intel.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/industrialio-core.c

index 8fcf54f..bba4fe4 100644 (file)
@@ -524,7 +524,7 @@ ssize_t iio_enum_read(struct iio_dev *indio_dev,
        i = e->get(indio_dev, chan);
        if (i < 0)
                return i;
-       else if (i >= e->num_items || !e->items[i])
+       if (i >= e->num_items || !e->items[i])
                return -EINVAL;
 
        return sysfs_emit(buf, "%s\n", e->items[i]);
@@ -1217,7 +1217,7 @@ static int iio_device_add_info_mask_type(struct iio_dev *indio_dev,
                                             &iio_dev_opaque->channel_attr_list);
                if ((ret == -EBUSY) && (shared_by != IIO_SEPARATE))
                        continue;
-               else if (ret < 0)
+               if (ret < 0)
                        return ret;
                attrcount++;
        }
@@ -1255,7 +1255,7 @@ static int iio_device_add_info_mask_type_avail(struct iio_dev *indio_dev,
                kfree(avail_postfix);
                if ((ret == -EBUSY) && (shared_by != IIO_SEPARATE))
                        continue;
-               else if (ret < 0)
+               if (ret < 0)
                        return ret;
                attrcount++;
        }