summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLars-Peter Clausen <lars@metafoo.de>2017-09-29 15:24:05 +0200
committerJonathan Cameron <Jonathan.Cameron@huawei.com>2017-10-01 12:17:18 +0200
commitbe94a6f6d488b4767662e8949dc62361bd1d6311 (patch)
treed907b9a78cc2f96dc96f9afad24fa51b884ddb84
parentstaging: iio: ade7759: fix signed extension bug on shift of a u8 (diff)
downloadlinux-be94a6f6d488b4767662e8949dc62361bd1d6311.tar.xz
linux-be94a6f6d488b4767662e8949dc62361bd1d6311.zip
iio: dummy: events: Add missing break
Add missing break in iio_simple_dummy_write_event_config() for the voltage threshold event enable attribute. Without this writing to the in_voltage0_thresh_rising_en always returns -EINVAL even though the change was correctly applied. Fixes: 3e34e650db197 ("iio: dummy: Demonstrate the usage of new channel types") Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Cc: <Stable@vger.kernel.org> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
-rw-r--r--drivers/iio/dummy/iio_simple_dummy_events.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/iio/dummy/iio_simple_dummy_events.c b/drivers/iio/dummy/iio_simple_dummy_events.c
index ed63ffd849f8..7ec2a0bb0807 100644
--- a/drivers/iio/dummy/iio_simple_dummy_events.c
+++ b/drivers/iio/dummy/iio_simple_dummy_events.c
@@ -72,6 +72,7 @@ int iio_simple_dummy_write_event_config(struct iio_dev *indio_dev,
st->event_en = state;
else
return -EINVAL;
+ break;
default:
return -EINVAL;
}