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

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dt-bindings: iio: sdp31 omits vdd-supply from required list
--

commit 61ed9cca92b973d2ff70fa365d3b2d923c296955
Author: Muhammad Abu Bakar <m.abubakar365@yahoo.com>

dt-bindings: iio: pressure: add Sensirion SDP31

This commit adds device tree bindings for the Sensirion SDP31 differential
pressure sensor. It defines the properties compatible, reg, and vdd-supply
for the I2C device.

> 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 0000000000000..fc646eca2d6f1
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/pressure/sensirion,sdp31.yaml

[ ... ]

> +properties:
> +  compatible:
> +    const: sensirion,sdp31
> +
> +  reg:
> +    maxItems: 1
> +
> +  vdd-supply: true
> +
> +required:
> +  - compatible
> +  - reg

[Severity: Low]
Should vdd-supply be added to this required list to enforce complete device
tree configurations? 

Since the SDP31 physical sensor requires a power supply to operate, defining
vdd-supply: true but omitting it from the required list means incomplete
device tree nodes missing the power supply definition will silently pass
schema validation when developers run make dtbs_check.

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260919223854.13437-1-m.abubakar365@yahoo.com?part=1