The newly added iio-gts-helper (gain time scale helper) errorneously reports available integration times as micro seconds. The same mistake is in newly added BU27034 light sensor driver. Fix these by adding new IIO_VAL type, IIO_VAL_INT_MICRO - which can be used for micro <unit> values where the integer part is zero. NOTE: I did not have the time to test the gts-helpers with integration times which are greater than 1 second. Currently there is no other in-tree users besides the bu27034, which does always use integration times smaller than 1 second. When greater than 1 second times are needed, this needs to be revised. (Or, when the devm_* interface kunit test support gets added). Right now this should be a quick fix to integration time handling before the bug manifests itself in the user-space). I am planning to test (and re-work if needed) the gts-helpers available integration time list for > 1Sec times - but I most probably don't have the time for that during this or next week. (Don't know about the weekend though - but probably not.) -- Matti Vaittinen (3): iio: core: add IIO_VAL_INT_MICRO iio: gts: fix units of available integration times iio: bu27034: Fix integration time units drivers/iio/industrialio-core.c | 4 ++++ drivers/iio/industrialio-gts-helper.c | 2 +- drivers/iio/light/rohm-bu27034.c | 7 +++++-- include/linux/iio/types.h | 1 + 4 files changed, 11 insertions(+), 3 deletions(-) base-commit: c86b0e73f0bebbb0245ef2bac4cf269d61ff828c -- 2.39.2 -- Matti Vaittinen, Linux device drivers ROHM Semiconductors, Finland SWDC Kiviharjunlenkki 1E 90220 OULU FINLAND ~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~ Simon says - in Latin please. ~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~ Thanks to Simon Glass for the translation =]
Attachment:
signature.asc
Description: PGP signature