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 -- 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
