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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0d6a950b-7794-1413-a74b-b73026a006f2@kernel.org>
Date:	Sun, 24 Jul 2016 13:05:47 +0100
From:	Jonathan Cameron <jic23@...nel.org>
To:	"Breana, Tiberiu A" <tiberiu.a.breana@...el.com>,
	Alison Schofield <amsfield22@...il.com>
Cc:	"mranostay@...il.com" <mranostay@...il.com>,
	"knaack.h@....de" <knaack.h@....de>,
	"lars@...afoo.de" <lars@...afoo.de>,
	"pmeerw@...erw.net" <pmeerw@...erw.net>,
	"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/3] iio: accel: bma220_spi: set up buffer timestamps for
 non-zero values

On 11/07/16 16:53, Breana, Tiberiu A wrote:
>> -----Original Message-----
>> From: Alison Schofield [mailto:amsfield22@...il.com]
>> Sent: Monday, July 11, 2016 6:25 PM
>> To: jic23@...nel.org
>> Cc: Breana, Tiberiu A <tiberiu.a.breana@...el.com>; mranostay@...il.com;
>> knaack.h@....de; lars@...afoo.de; pmeerw@...erw.net; linux-
>> iio@...r.kernel.org; linux-kernel@...r.kernel.org
>> 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@...il.com>
>> Cc: Daniel Baluta <daniel.baluta@...il.com>
> 
> +1
> Reviewed-By: Tiberiu Breana <tiberiu.a.breana@...el.com>
> 
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
>