diff options
| author | Hartmut Knaack <knaack.h@gmx.de> | 2015-08-28 23:59:56 +0200 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2015-10-11 15:43:51 +0100 |
| commit | 11687d4aaeb074e0a51c8008c80a14c6b8937b3d (patch) | |
| tree | fceec029c16e5d75dec83b6a580796177fe3453f /drivers | |
| parent | 7d2da8eaa92585769dfbafe285b94c8f743510f6 (diff) | |
| download | op-kernel-dev-11687d4aaeb074e0a51c8008c80a14c6b8937b3d.zip op-kernel-dev-11687d4aaeb074e0a51c8008c80a14c6b8937b3d.tar.gz | |
iio:dac:m62332: drop unrequired variable
A return variable is not required in _write_raw(), and dropping it reduces
complexity, as well.
Signed-off-by: Hartmut Knaack <knaack.h@gmx.de>
Acked-by: Daniel Baluta <daniel.baluta@intel.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/iio/dac/m62332.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/iio/dac/m62332.c b/drivers/iio/dac/m62332.c index fe75098..1b65fc0 100644 --- a/drivers/iio/dac/m62332.c +++ b/drivers/iio/dac/m62332.c @@ -112,21 +112,17 @@ static int m62332_read_raw(struct iio_dev *indio_dev, static int m62332_write_raw(struct iio_dev *indio_dev, struct iio_chan_spec const *chan, int val, int val2, long mask) { - int ret; - switch (mask) { case IIO_CHAN_INFO_RAW: if (val < 0 || val > 255) return -EINVAL; - ret = m62332_set_value(indio_dev, val, chan->channel); - break; + return m62332_set_value(indio_dev, val, chan->channel); default: - ret = -EINVAL; break; } - return ret; + return -EINVAL; } #ifdef CONFIG_PM_SLEEP |
