]> Pileus Git - ~andy/linux/commitdiff
iio: light: vcnl4000: Remove redundant code
authorSachin Kamat <sachin.kamat@linaro.org>
Thu, 24 Oct 2013 11:53:00 +0000 (12:53 +0100)
committerJonathan Cameron <jic23@kernel.org>
Thu, 24 Oct 2013 13:48:14 +0000 (14:48 +0100)
The if check is redundant as the value obtained from
iio_device_register() is already in the required format.
Hence return the function directly.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/light/vcnl4000.c

index 2bb304215b1d88e2c19189215806ec7311f41739..ecb3341ef9c07a8e56ddaf1de8d76d31456e6d2a 100644 (file)
@@ -179,11 +179,7 @@ static int vcnl4000_probe(struct i2c_client *client,
        indio_dev->name = VCNL4000_DRV_NAME;
        indio_dev->modes = INDIO_DIRECT_MODE;
 
-       ret = iio_device_register(indio_dev);
-       if (ret < 0)
-               return ret;
-
-       return 0;
+       return iio_device_register(indio_dev);
 }
 
 static int vcnl4000_remove(struct i2c_client *client)