* iio: imu: st_lsm6dsx: sample rate?
@ 2021-05-04 10:22 Sean Nyekjaer
2021-05-06 8:22 ` Lorenzo Bianconi
0 siblings, 1 reply; 4+ messages in thread
From: Sean Nyekjaer @ 2021-05-04 10:22 UTC (permalink / raw)
To: Lorenzo Bianconi, linux-iio
Hi,
Any specific reason to why the 12.5Hz sample rate is implemented as 13Hz ?
Br,
/Sean
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: iio: imu: st_lsm6dsx: sample rate?
2021-05-04 10:22 iio: imu: st_lsm6dsx: sample rate? Sean Nyekjaer
@ 2021-05-06 8:22 ` Lorenzo Bianconi
2021-05-06 8:27 ` Sean Nyekjaer
2021-05-07 10:01 ` Sean Nyekjaer
0 siblings, 2 replies; 4+ messages in thread
From: Lorenzo Bianconi @ 2021-05-06 8:22 UTC (permalink / raw)
To: Sean Nyekjaer; +Cc: linux-iio
[-- Attachment #1: Type: text/plain, Size: 230 bytes --]
> Hi,
>
> Any specific reason to why the 12.5Hz sample rate is implemented as 13Hz ?
>
Hi Sean,
what is the device you are referring to? I guess now all devices support
12.5Hz.
Regards,
Lorenzo
> Br,
> /Sean
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: iio: imu: st_lsm6dsx: sample rate?
2021-05-06 8:22 ` Lorenzo Bianconi
@ 2021-05-06 8:27 ` Sean Nyekjaer
2021-05-07 10:01 ` Sean Nyekjaer
1 sibling, 0 replies; 4+ messages in thread
From: Sean Nyekjaer @ 2021-05-06 8:27 UTC (permalink / raw)
To: Lorenzo Bianconi; +Cc: linux-iio
On 06/05/2021 10.22, Lorenzo Bianconi wrote:
>> Hi,
>>
>> Any specific reason to why the 12.5Hz sample rate is implemented as 13Hz ?
>>
>
> Hi Sean,
>
> what is the device you are referring to? I guess now all devices support
> 12.5Hz.
>
> Regards,
> Lorenzo
>
At least the two devices we are using.
ISM330DLC and LSM6DSLTR, they do not support 13Hz sampling rate, but 12.5Hz.
I think we should expose the correct 12.5Hz to user space.
Should do a patch? (It could potentially break some implementations)
/Sean
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: iio: imu: st_lsm6dsx: sample rate?
2021-05-06 8:22 ` Lorenzo Bianconi
2021-05-06 8:27 ` Sean Nyekjaer
@ 2021-05-07 10:01 ` Sean Nyekjaer
1 sibling, 0 replies; 4+ messages in thread
From: Sean Nyekjaer @ 2021-05-07 10:01 UTC (permalink / raw)
To: Lorenzo Bianconi; +Cc: linux-iio
On 06/05/2021 10.22, Lorenzo Bianconi wrote:
>> Hi,
>>
>> Any specific reason to why the 12.5Hz sample rate is implemented as 13Hz ?
>>
>
> Hi Sean,
>
> what is the device you are referring to? I guess now all devices support
> 12.5Hz.
>
> Regards,
> Lorenzo
>
>> Br,
>> /Sean
Hi Lorenzo,
I see the issue is fixed in:
commit f8710f0357bc36c740782c95f34ea847c75b7498
Author: Lorenzo Bianconi <lorenzo@kernel.org>
Date: Sun Nov 3 20:47:18 2019 +0100
iio: imu: st_lsm6dsx: express odr in mHZ
Express available frequencies in mHZ in order to support even
rational ODRs. This patch is need to fix an Android CTS failure
Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
/Sean
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-05-07 10:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-04 10:22 iio: imu: st_lsm6dsx: sample rate? Sean Nyekjaer
2021-05-06 8:22 ` Lorenzo Bianconi
2021-05-06 8:27 ` Sean Nyekjaer
2021-05-07 10:01 ` Sean Nyekjaer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).