Re: [PATCH 1/3] iio: accel: bma220_spi: set up buffer timestamps for non-zero values

From: Jonathan Cameron
Date: Sun Jul 24 2016 - 08:05:56 EST


On 11/07/16 16:53, Breana, Tiberiu A wrote:
>> -----Original Message-----
>> From: Alison Schofield [mailto:amsfield22@xxxxxxxxx]
>> Sent: Monday, July 11, 2016 6:25 PM
>> To: jic23@xxxxxxxxxx
>> Cc: Breana, Tiberiu A <tiberiu.a.breana@xxxxxxxxx>; mranostay@xxxxxxxxx;
>> knaack.h@xxxxxx; lars@xxxxxxxxxx; pmeerw@xxxxxxxxxx; linux-
>> iio@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
>> Subject: [PATCH 1/3] iio: accel: bma220_spi: set up buffer timestamps for non-
>> zero values
>>
>> Use the iio_pollfunc_store_time parameter during triggered buffer set-up to get
>> valid timestamps.
>>
>> Signed-off-by: Alison Schofield <amsfield22@xxxxxxxxx>
>> Cc: Daniel Baluta <daniel.baluta@xxxxxxxxx>
>
> +1
> Reviewed-By: Tiberiu Breana <tiberiu.a.breana@xxxxxxxxx>
>
Applied to the fixes-togreg-post-rc1 branch of iio.git
(for my convenience) and marked for stable.

Jonathan
>> ---
>> drivers/iio/accel/bma220_spi.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/iio/accel/bma220_spi.c b/drivers/iio/accel/bma220_spi.c
>> index 1098d10..5099f29 100644
>> --- a/drivers/iio/accel/bma220_spi.c
>> +++ b/drivers/iio/accel/bma220_spi.c
>> @@ -253,7 +253,7 @@ static int bma220_probe(struct spi_device *spi)
>> if (ret < 0)
>> return ret;
>>
>> - ret = iio_triggered_buffer_setup(indio_dev, NULL,
>> + ret = iio_triggered_buffer_setup(indio_dev, iio_pollfunc_store_time,
>> bma220_trigger_handler, NULL);
>> if (ret < 0) {
>> dev_err(&spi->dev, "iio triggered buffer setup failed\n");
>> --
>> 2.1.4
>