| From: |
| Wadim Mueller <wafgo01-AT-gmail.com> |
| To: |
| |
| Subject: |
| [PATCH v2 0/3] iio: flow: Sensirion SLF3S liquid flow sensor |
| Date: |
| Wed, 27 May 2026 20:42:51 +0200 |
| Message-ID: |
| <20260527184257.141635-1-wafgo01@gmail.com> |
| Cc: |
| Jonathan Cameron <jic23-AT-kernel.org>, David Lechner <dlechner-AT-baylibre.com>, Nuno Sá <nuno.sa-AT-analog.com>, Andy Shevchenko <andy-AT-kernel.org>, Rob Herring <robh-AT-kernel.org>, Krzysztof Kozlowski <krzk+dt-AT-kernel.org>, Conor Dooley <conor+dt-AT-kernel.org>, Guenter Roeck <linux-AT-roeck-us.net>, Jean Delvare <jdelvare-AT-suse.com>, Andreas Klinger <ak-AT-it-klinger.de>, Lars-Peter Clausen <lars-AT-metafoo.de>, linux-iio-AT-vger.kernel.org, linux-hwmon-AT-vger.kernel.org, devicetree-AT-vger.kernel.org, linux-kernel-AT-vger.kernel.org, Maxwell Doose <m32285159-AT-gmail.com> |
| Archive-link: |
| Article |
Hi all,
v2 addresses the inline review feedback from Jonathan and Guenter on v1
(see https://lore.kernel.org/linux-iio/20260524205112.26638-1-...).
Per-patch replies have been sent in the v1 thread.
The series is dropped from RFC: subsystem placement (iio/flow/),
channel type (IIO_VOLUMEFLOW) and the new subdirectory have all been
ACKed in v1.
Changes since v1:
* rename driver/binding from slf3x to slf3s (Jonathan -- the
marketing family name is SLF3S)
* fold MAINTAINERS entry into the dt-binding patch instead of a
separate 4/4 patch (Jonathan)
* add SLF3S-1300F support (Guenter)
* dt-binding: split the single "sensirion,slf3s" compatible into
per-variant compatibles (-0600F, -1300F, -4000B) with a generic
fallback (Jonathan)
* dt-binding: add optional interrupts property (Guenter -- bindings
describe the hardware, not the current driver)
* driver: seed the variant from DT via i2c_get_match_data() instead
of probing the sensor first (Jonathan)
* driver: detect_variant() now trusts the DT-supplied variant and
only emits dev_dbg() on a sub_type/family mismatch (Jonathan)
* driver: per-instance crc8_populate_msb() rather than a global
crc table (Jonathan)
* driver: declare cmd as const u8 [static 2] in slf3s_send_cmd()
(Jonathan)
* driver: switch to fsleep() and sign_extend32() helpers (Jonathan)
* driver: use named initialisers for slf3s_variants[] entries
(Jonathan)
* driver: unsigned int loop counters (Jonathan)
* Kconfig: tab-indented bulleted list of supported parts (Jonathan)
* ABI sysfs doc: _scale entry folded into the shared block,
_raw block kept with its own KernelVersion (Guenter)
* re-checked all scale factors against the SLF3S datasheets
(Table 16 for -0600F, Table 15 for -1300F and -4000B)
Patches:
1/3 iio: types: add IIO_VOLUMEFLOW channel type
2/3 dt-bindings: iio: flow: add sensirion,slf3s binding
3/3 iio: flow: add Sensirion SLF3S liquid flow sensor driver
Tested with an SLF3S-0600F on a TI AM64x platform; W=1 build clean,
checkpatch --strict clean, dt_binding_check clean.
Thanks,
Wadim
Wadim Mueller (3):
iio: types: add IIO_VOLUMEFLOW channel type
dt-bindings: iio: flow: add Sensirion SLF3S liquid flow sensor
iio: flow: add Sensirion SLF3S liquid flow sensor driver
Documentation/ABI/testing/sysfs-bus-iio | 11 +
.../bindings/iio/flow/sensirion,slf3s.yaml | 68 ++++
MAINTAINERS | 8 +
drivers/iio/Kconfig | 1 +
drivers/iio/Makefile | 1 +
drivers/iio/flow/Kconfig | 27 ++
drivers/iio/flow/Makefile | 7 +
drivers/iio/flow/slf3s.c | 345 ++++++++++++++++++
drivers/iio/industrialio-core.c | 1 +
include/uapi/linux/iio/types.h | 1 +
tools/iio/iio_event_monitor.c | 2 +
11 files changed, 472 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/flow/sensirion,slf3s.yaml
create mode 100644 drivers/iio/flow/Kconfig
create mode 100644 drivers/iio/flow/Makefile
create mode 100644 drivers/iio/flow/slf3s.c
--
2.52.0
