Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1647776 > unrolled thread
| Started by | Nikita Yushchenko <nikita.yoush@cogentembedded.com> |
|---|---|
| First post | 2017-05-23 10:20 +0200 |
| Last post | 2017-05-24 21:30 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH/RFC] iio: hi8435: do not enable all events by default Nikita Yushchenko <nikita.yoush@cogentembedded.com> - 2017-05-23 10:20 +0200
Re: [PATCH/RFC] iio: hi8435: do not enable all events by default Jonathan Cameron <jic23@kernel.org> - 2017-05-24 21:30 +0200
| From | Nikita Yushchenko <nikita.yoush@cogentembedded.com> |
|---|---|
| Date | 2017-05-23 10:20 +0200 |
| Subject | [PATCH/RFC] iio: hi8435: do not enable all events by default |
| Message-ID | <tKdcS-x7-5@gated-at.bofh.it> |
Having all events enabled by default is misleading. Userspace should explicitly enable events they want to receive. Signed-off-by: Nikita Yushchenko <nikita.yoush@cogentembedded.com> --- 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 -- 2.11.0
[toc] | [next] | [standalone]
| From | Jonathan Cameron <jic23@kernel.org> |
|---|---|
| Date | 2017-05-24 21:30 +0200 |
| Message-ID | <tKK8O-6oz-21@gated-at.bofh.it> |
| In reply to | #1647776 |
On Tue, 23 May 2017 11:08:30 +0300 Nikita Yushchenko <nikita.yoush@cogentembedded.com> 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@cogentembedded.com> 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
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web