Re: [PATCH/RFC] iio: hi8435: do not enable all events by default

From: Jonathan Cameron
Date: Wed May 24 2017 - 15:27:46 EST


On Tue, 23 May 2017 11:08:30 +0300
Nikita Yushchenko <nikita.yoush@xxxxxxxxxxxxxxxxxx> wrote:

> Having all events enabled by default is misleading.
> Userspace should explicitly enable events they want to receive.
>
> Signed-off-by: Nikita Yushchenko <nikita.yoush@xxxxxxxxxxxxxxxxxx>
I agree in principle, but this is a userspace ABI change. Sadly we
can't do it with out risking breaking userspace code...

One of those we should have caught in review, but now it's there
we can't actually do anything about it unless we are absolutely
sure no one will notice!

Jonathan
> ---
> drivers/iio/adc/hi8435.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/iio/adc/hi8435.c b/drivers/iio/adc/hi8435.c
> index ef5c286c8e67..0331739f235c 100644
> --- a/drivers/iio/adc/hi8435.c
> +++ b/drivers/iio/adc/hi8435.c
> @@ -506,8 +506,6 @@ static int hi8435_probe(struct spi_device *spi)
> idev->channels = hi8435_channels;
> idev->num_channels = ARRAY_SIZE(hi8435_channels);
>
> - /* unmask all events */
> - priv->event_scan_mask = ~(0);
> /*
> * There is a restriction in the chip - the hysteresis can not be odd.
> * If the hysteresis is set to odd value then chip gets into lock state