Re: [PATCH 1/2] dt-bindings: iio: pressure: infineon,dps310: Add interrupts

[Jonathan Cameron <jic23@kernel.org>]

On Tue, 10 Nov 2026 17:42:53 +0200
Rupesh Majhi <zoone.rupert@gmail.com> wrote:

> DPS310 signals new results and a full FIFO on its SDO pin. In I2C mode
> the same pin sets the address LSB, and interrupt polarity must be the
> opposite of how SDO is strapped.

Some hardware engineer is presumably proud they made that work!

I'm not sure why we never had the SPI part of this binding but
given it's not there and it gets more complicated wrt to this
that can be addressed if anyone ever adds SPI.

Anyhow, to me this looks fine.

Jonathan


> 
> Signed-off-by: Rupesh Majhi <zoone.rupert@gmail.com>
> ---
> Cc: Paulius Zaleckas <paulius.zaleckas@gmail.com>
> 
>  .../bindings/iio/pressure/infineon,dps310.yaml         | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml b/Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
> index 416117dfd927..cded6e0af147 100644
> --- a/Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
> +++ b/Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
> @@ -23,6 +23,12 @@ properties:
>    reg:
>      maxItems: 1
>  
> +  interrupts:
> +    maxItems: 1
> +    description:
> +      New result or full FIFO, on SDO pin. SDO also selects I2C address, so
> +      interrupt is active high at 0x76 (SDO low) and active low at 0x77.
> +
>    "#io-channel-cells":
>      const: 0
>  
> @@ -42,6 +48,8 @@ additionalProperties: false
>  
>  examples:
>    - |
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +
>      i2c {
>          #address-cells = <1>;
>          #size-cells = <0>;
> @@ -49,6 +57,8 @@ examples:
>          dps: pressure-sensor@76 {
>              compatible = "infineon,dps310";
>              reg = <0x76>;
> +            interrupt-parent = <&gpio0>;
> +            interrupts = <17 IRQ_TYPE_LEVEL_HIGH>;
>              #io-channel-cells = <0>;
>              vdd-supply = <&vref1>;
>              vddio-supply = <&vref2>;