On 10/24/13 12:53, Sachin Kamat wrote: > 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@xxxxxxxxxx> Applied to the togreg branch of iio.git Thanks for the series. > --- > drivers/iio/light/vcnl4000.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/drivers/iio/light/vcnl4000.c b/drivers/iio/light/vcnl4000.c > index 2bb3042..ecb3341 100644 > --- a/drivers/iio/light/vcnl4000.c > +++ b/drivers/iio/light/vcnl4000.c > @@ -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) > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
