]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
iio: light: tsl2563: use correct event code
authorAkinobu Mita <akinobu.mita@gmail.com>
Tue, 20 Jun 2017 16:46:37 +0000 (01:46 +0900)
committerBen Hutchings <ben@decadent.org.uk>
Sat, 11 Nov 2017 13:34:23 +0000 (13:34 +0000)
commit a3507e48d3f99a93a3056a34a5365f310434570f upstream.

The TSL2563 driver provides three iio channels, two of which are raw ADC
channels (channel 0 and channel 1) in the device and the remaining one
is calculated by the two.  The ADC channel 0 only supports programmable
interrupt with threshold settings and this driver supports the event but
the generated event code does not contain the corresponding iio channel
type.

This is going to change userspace ABI.  Hopefully fixing this to be
what it should always have been won't break any userspace code.

Cc: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
[bwh: Backported to 3.2: adjust filename]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/staging/iio/light/tsl2563.c

index 7e984bcf8d7ed505dff5ceea4d411b85b8f57b6c..ee83ccb93af1f6ac58b42fec7512d7bafff88751 100644 (file)
@@ -606,7 +606,7 @@ static irqreturn_t tsl2563_event_handler(int irq, void *private)
        struct tsl2563_chip *chip = iio_priv(dev_info);
 
        iio_push_event(dev_info,
-                      IIO_UNMOD_EVENT_CODE(IIO_LIGHT,
+                      IIO_UNMOD_EVENT_CODE(IIO_INTENSITY,
                                            0,
                                            IIO_EV_TYPE_THRESH,
                                            IIO_EV_DIR_EITHER),