iio: adc: vf610: fix simple_return.cocci warnings
authorkbuild test robot <fengguang.wu@intel.com>
Tue, 22 Sep 2015 02:16:48 +0000 (10:16 +0800)
committerJonathan Cameron <jic23@kernel.org>
Wed, 23 Sep 2015 19:23:28 +0000 (20:23 +0100)
drivers/iio/adc/vf610_adc.c:766:1-4: WARNING: end returns can be simpified and declaration on line 755 can be dropped

 Simplify a trivial if-return sequence.  Possibly combine with a
 preceding function call.

Generated by: scripts/coccinelle/misc/simple_return.cocci

CC: Sanchayan Maity <maitysanchayan@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/adc/vf610_adc.c

index 635ccd86620eb3e363b0917637789e969b278a40..599cde3d03a117358f86defec5dcd10ac25b1ed7 100644 (file)
@@ -752,7 +752,7 @@ static int vf610_adc_buffer_predisable(struct iio_dev *indio_dev)
 {
        struct vf610_adc *info = iio_priv(indio_dev);
        unsigned int hc_cfg = 0;
-       int val, ret;
+       int val;
 
        val = readl(info->regs + VF610_REG_ADC_GC);
        val &= ~VF610_ADC_ADCON;
@@ -763,11 +763,7 @@ static int vf610_adc_buffer_predisable(struct iio_dev *indio_dev)
 
        writel(hc_cfg, info->regs + VF610_REG_ADC_HC0);
 
-       ret = iio_triggered_buffer_predisable(indio_dev);
-       if (ret)
-               return ret;
-
-       return 0;
+       return iio_triggered_buffer_predisable(indio_dev);
 }
 
 static const struct iio_buffer_setup_ops iio_triggered_buffer_setup_ops = {