Re: [PATCH v2 2/3] iio: add documentation for iio_chan_info_enum

From: Matti Vaittinen
Date: Wed Apr 12 2023 - 04:53:15 EST


On 4/8/23 13:30, Jonathan Cameron wrote:
On Sat, 25 Feb 2023 15:55:25 +0200
Matti Vaittinen <mazziesaccount@xxxxxxxxx> wrote:

+ * @IIO_CHAN_INFO_INT_TIME: Integration time. Time during which the data is
+ * accumulated by the device.

Unit? (seconds I think).

Holy moly. Thanks for bringing this up now. I just checked this and the API doc indeed says clearly and loud the unit is in seconds. This means the BU27034 driver as well as the gain-time-scale helper does this wrong. I hope you can postpone sending them upstream until this gets fixed. I'll try to cook incremental patch on top of the iio/togreg - but I am not sure if I can do it today as I need to run after an hour... Sorry and thanks!

Yours,
-- Matti

--
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland

~~ When things go utterly wrong vim users can always type :help! ~~