staging: iio: compress return logic
authorGargi Sharma <gs051095@gmail.com>
Fri, 17 Feb 2017 17:17:21 +0000 (22:47 +0530)
committerJonathan Cameron <jic23@kernel.org>
Sat, 18 Feb 2017 18:58:08 +0000 (18:58 +0000)
Simplify function returns by merging assignment and return.
Found with Coccinelle.

Semantic patch:
@@
local idexpression ret;
expression e;
@@

-ret =
+return
     e;
-return ret;

Signed-off-by: Gargi Sharma <gs051095@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/meter/ade7854.c

index e8007f0c5186cd1835423574160bfb1c306e832a..c6cffc11b0ba96e24797ce328bbfc64a8277f276 100644 (file)
@@ -426,9 +426,7 @@ static int ade7854_set_irq(struct device *dev, bool enable)
        else
                irqen &= ~BIT(17);
 
-       ret = st->write_reg_32(dev, ADE7854_MASK0, irqen);
-
-       return ret;
+       return st->write_reg_32(dev, ADE7854_MASK0, irqen);
 }
 
 static int ade7854_initial_setup(struct iio_dev *indio_dev)