Re: [PATCH v3 0/8] Expose cros_ec_sensors frequency range via iio sysfs

From: Fabien Lahoudere
Date: Wed Jun 19 2019 - 08:29:36 EST


Le mardi 18 juin 2019 Ã 11:06 +0200, Fabien Lahoudere a Ãcrit :
> Chromebooks EC sensors must expose a range of frequencies for each
> sensors using
> the standard ABI sampling_frquency_available.
>
> Changes since v2:
>
> - use read_avail callback
> - rework core functions to avoid code duplication

- now sample-frequency-available sysfs return a list of known
frequencies instead of a range, because the step depend on sensors
(fast vs slow) and is hidden by EC.
See https://patchwork.kernel.org/patch/10957141/#22662201

>
> Changes since v1:
> - Add a cover letter
> - Add Nick Vaccaro SoB to patch 1
> - Drop fifo size related code
>
> Fabien Lahoudere (8):
> iio: common: cros_ec_sensors: move iio_info management to core
> iio: common: cros_ec_sensors: move channels to core structure
> iio: common: cros_ec_sensors: move registration to core
> iio: common: cros_ec_sensors: clean code
> iio: common: cros_ec_sensors: use core structure
> iio: common: cros_ec_sensors: support protocol v3 message
> iio: common: cros_ec_sensors: add sysfs attribute for frequencies
> docs: iio: add precision about sampling_frequency_available
>
> Documentation/ABI/testing/sysfs-bus-iio | 7 +-
> .../common/cros_ec_sensors/cros_ec_sensors.c | 148 +++++------
> .../cros_ec_sensors/cros_ec_sensors_core.c | 230 +++++++++++++++-
> --
> drivers/iio/light/cros_ec_light_prox.c | 124 ++++------
> drivers/iio/pressure/cros_ec_baro.c | 101 +++-----
> .../linux/iio/common/cros_ec_sensors_core.h | 43 +++-
> 6 files changed, 357 insertions(+), 296 deletions(-)
>