iio: dac: ad7303: remove an unneeded check
Subsystems (3)
PATCH 1/1 iio: dac: ad7303: remove an unneeded check
"ret" is zero here. There is no need to check again.
Signed-off-by Dan Carpenter <[email protected]>
drivers/iio/dac/ad7303.c +1 -1
@@ -92,7 +92,7 @@ static ssize_t ad7303_write_dac_powerdown(struct iio_dev *indio_dev,ad7303_write (st, chan->channel, st->dac_cache[chan->channel]);mutex_unlock (&indio_dev->mlock); -return ret ? ret : len; +return len; } static int ad7303_get_vref(struct ad7303_state *st,
Quoted 3 lines · Dan Carpenter
"ret" is zero here. There is no need to check again.
Signed-off-by Dan Carpenter <[email protected]>
Thanks.
Quoted 15 lines
drivers/iio/dac/ad7303.c +1
-1@@ -92,7 +92,7 @@ static ssize_t ad7303_write_dac_powerdown(struct iio_dev *indio_dev,ad7303_write (st, chan->channel, st->dac_cache[chan->channel]);mutex_unlock (&indio_dev->mlock); -return ret ? ret : len; +return len; } static int ad7303_get_vref(struct ad7303_state *st,
Quoted 6 lines · Lars-Peter Clausen
Quoted 3 lines · Dan Carpenter
"ret" is zero here. There is no need to check again.
Signed-off-by Dan Carpenter <[email protected]>
Applied to the togreg branch of iio.git
Thanks,
Quoted 19 lines
Thanks.
Quoted 15 lines
drivers/iio/dac/ad7303.c +1
-1@@ -92,7 +92,7 @@ static ssize_t ad7303_write_dac_powerdown(struct iio_dev *indio_dev,ad7303_write (st, chan->channel, st->dac_cache[chan->channel]);mutex_unlock (&indio_dev->mlock); -return ret ? ret : len; +return len; } static int ad7303_get_vref(struct ad7303_state *st,