staging: iio: tsl2583: clarified comment about clearing interrupts
authorBrian Masney <masneyb@onstation.org>
Sat, 12 Nov 2016 18:19:36 +0000 (13:19 -0500)
committerJonathan Cameron <jic23@kernel.org>
Sun, 13 Nov 2016 13:07:30 +0000 (13:07 +0000)
The comment that describes the code that clears the interrupt bit was
vague and didn't provide much value. This patch adds more detail about
why that bit needs to be cleared.

Signed-off-by: Brian Masney <masneyb@onstation.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/light/tsl2583.c

index b031fc32ec5f0eeb66747196ba3cc768e9e7a829..65b29ed60cc6e32f566cfb963499f964ed6420cf 100644 (file)
@@ -205,8 +205,9 @@ static int tsl2583_get_lux(struct iio_dev *indio_dev)
        }
 
        /*
-        * clear status, really interrupt status (interrupts are off), but
-        * we use the bit anyway - don't forget 0x80 - this is a command
+        * Clear the pending interrupt status bit on the chip to allow the next
+        * integration cycle to start. This has to be done even though this
+        * driver currently does not support interrupts.
         */
        ret = i2c_smbus_write_byte(chip->client,
                                   (TSL2583_CMD_REG | TSL2583_CMD_SPL_FN |