staging:iio:light:tsl2563 channel spec buglet / always reading same adc.
authorJonathan Cameron <jic23@kernel.org>
Fri, 13 Apr 2012 09:24:18 +0000 (10:24 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 13 Apr 2012 17:59:38 +0000 (10:59 -0700)
The IIO_LIGHT channel was not marked as being a processed_val despite
clearly being in lux.

The IIO_INTENSITY channel reads were dependent on channel and that isn't
specified for either adc (as they now use modifiers).  Hence use the
modifier instead.

Reported-by: Jon Brenner <jbrenner@taosinc.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/iio/light/tsl2563.c

index 546c95a..beb51d7 100644 (file)
@@ -465,7 +465,7 @@ static int tsl2563_write_raw(struct iio_dev *indio_dev,
 {
        struct tsl2563_chip *chip = iio_priv(indio_dev);
 
-       if (chan->channel == 0)
+       if (chan->channel == IIO_MOD_LIGHT_BOTH)
                chip->calib0 = calib_from_sysfs(val);
        else
                chip->calib1 = calib_from_sysfs(val);
@@ -534,6 +534,7 @@ static const struct iio_chan_spec tsl2563_channels[] = {
        {
                .type = IIO_LIGHT,
                .indexed = 1,
+               .processed_val = 1,
                .channel = 0,
        }, {
                .type = IIO_INTENSITY,