Re: [PATCH v3 1/2] dt-bindings: iio: pressure: add Sensirion SDP31

[Jonathan Cameron]

On Sat, 26 Sep 2026 20:41:17 +0500
Muhammad Abu Bakar <m.abubakar365@xxxxxxxxx> wrote:

> Add device tree bindings for the Sensirion SDP31 differential pressure
> sensor, an I2C device that also reports temperature.
> 
> Signed-off-by: Muhammad Abu Bakar <m.abubakar365@xxxxxxxxx>
I think you have been missing some replies for some reason.

See https://lore.kernel.org/all/20260920014032.2cc3c851@jic23-hlaptop/
Also the reply Krzysztof sent to point out you hadn't addressed all
the feedback on v1.

If you disagree with any suggestions then reply to that email.

Reviewers tend to stop reviewing very quickly if they see that stuff
they raised in previous versions hasn't been addressed yet.

Jonathan


> ---
>  .../iio/pressure/sensirion,sdp31.yaml         | 41 +++++++++++++++++++
>  MAINTAINERS                                   |  5 +++
>  2 files changed, 46 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/iio/pressure/sensirion,sdp31.yaml
> 
> diff --git a/Documentation/devicetree/bindings/iio/pressure/sensirion,sdp31.yaml b/Documentation/devicetree/bindings/iio/pressure/sensirion,sdp31.yaml
> new file mode 100644
> index 000000000..a98e7c97e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/pressure/sensirion,sdp31.yaml
> @@ -0,0 +1,41 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/pressure/sensirion,sdp31.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Sensirion SDP31 differential pressure sensor
> +
> +maintainers:
> +  - Muhammad Abu Bakar <m.abubakar365@xxxxxxxxx>
> +
> +description: |
> +  Differential pressure sensor from Sensirion with an I2C interface.
> +
> +properties:
> +  compatible:
> +    const: sensirion,sdp31
> +
> +  reg:
> +    maxItems: 1
> +
> +  vdd-supply: true
> +
> +required:
> +  - compatible
> +  - reg
> +  - vdd-supply
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    i2c {
> +      #address-cells = <1>;
> +      #size-cells = <0>;
> +      pressure@21 {
> +        compatible = "sensirion,sdp31";
> +        reg = <0x21>;
> +        vdd-supply = <&vdd_3v3>;
> +      };
> +    };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 214aeee76..e285147be 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -24855,6 +24855,11 @@ S:	Maintained
>  F:	Documentation/devicetree/bindings/iio/chemical/sensirion,scd4x.yaml
>  F:	drivers/iio/chemical/scd4x.c
>  
> +SENSIRION SDP31 DIFFERENTIAL PRESSURE SENSOR DRIVER
> +M:	Muhammad Abu Bakar <m.abubakar365@xxxxxxxxx>
> +S:	Maintained
> +F:	Documentation/devicetree/bindings/iio/pressure/sensirion,sdp31.yaml
> +
>  SENSIRION SDP500 DIFFERENTIAL PRESSURE SENSOR DRIVER
>  M:	Petar Stoykov <petar.stoykov@xxxxxxxxxxxxxxxxxxxxxxxxx>
>  S:	Maintained