iio: adc: ina2xx: Align timestamp with conversion ready flag
authorStefan Brüns <stefan.bruens@rwth-aachen.de>
Thu, 21 Dec 2017 18:31:37 +0000 (19:31 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Jan 2018 15:03:43 +0000 (16:03 +0100)
As the timestamp is no longer (ab-)used to measure the function run time,
it can be taken at the correct time, i.e. when the conversion has finished.

Signed-off-by: Stefan Brüns <stefan.bruens@rwth-aachen.de>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/iio/adc/ina2xx-adc.c

index 144d5c9..0845aca 100644 (file)
@@ -708,8 +708,6 @@ static int ina2xx_work_buffer(struct iio_dev *indio_dev)
        s64 time;
        unsigned int alert;
 
-       time = iio_get_time_ns(indio_dev);
-
        /*
         * Because the timer thread and the chip conversion clock
         * are asynchronous, the period difference will eventually
@@ -738,6 +736,8 @@ static int ina2xx_work_buffer(struct iio_dev *indio_dev)
 
                } while (!alert);
 
+       time = iio_get_time_ns(indio_dev);
+
        /*
         * Single register reads: bulk_read will not work with ina226/219
         * as there is no auto-increment of the register pointer.