Re: [PATCH 0/2] iio: pressure: add Sensirion SDP31 driver

[Jonathan Cameron <jic23@kernel.org>]

On Sun, 20 Sep 2026 03:38:52 +0500
Muhammad Abu Bakar <m.abubakar365@yahoo.com> wrote:

> This series adds support for the Sensirion SDP31 differential pressure
> sensor, an I2C device that also reports temperature.
> 
> Patch 1 adds the device tree binding.
> Patch 2 adds the IIO driver, which exposes differential pressure and
> temperature channels and validates each measurement using the sensor's
> CRC-8 checksum.

Hi,

Given we have another sensirion pressure sensor supported already (the SDP500)
I'd expect a brief note here on why it doesn't make sense to add support to
that driver for this new part.  Also check other similar drivers as sometimes
we get compatible parts from multiple vendors.

FWIW the two parts don't look to be that similar so probably a very short note :)

Jonathan


> 
> The driver was developed and tested on an SDP31 wired to the I2C bus of
> a Raspberry Pi 4.
> 
> Muhammad Abu Bakar (2):
>   dt-bindings: iio: pressure: add Sensirion SDP31
>   iio: pressure: add Sensirion SDP31 driver
> 
>  .../iio/pressure/sensirion,sdp31.yaml         |  40 ++++
>  MAINTAINERS                                   |   6 +
>  drivers/iio/pressure/Kconfig                  |  11 +
>  drivers/iio/pressure/Makefile                 |   1 +
>  drivers/iio/pressure/sdp31.c                  | 194 ++++++++++++++++++
>  5 files changed, 252 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/pressure/sensirion,sdp31.yaml
>  create mode 100644 drivers/iio/pressure/sdp31.c
>