Re: [PATCH v6 2/4] dt-bindings: hwmon: Add Sensirion SHT30 series
From: Krzysztof Kozlowski
Date: Wed Apr 29 2026 - 02:29:11 EST
On Tue, Apr 28, 2026 at 07:09:27PM +0800, Zaixiang Xu wrote:
> Add YAML devicetree binding schema for Sensirion SHT30 series and the
> compatible GXCAS GXHT30 sensors.
>
> Use fallback compatibles for compatible chips and add optional
> interrupts and vdd-supply properties.
>
> Signed-off-by: Zaixiang Xu <zaixiang.xu.dev@xxxxxxxxx>
> ---
> .../bindings/hwmon/sensirion,sht30.yaml | 67 +++++++++++++++++++
> 1 file changed, 67 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/hwmon/sensirion,sht30.yaml
>
> diff --git a/Documentation/devicetree/bindings/hwmon/sensirion,sht30.yaml b/Documentation/devicetree/bindings/hwmon/sensirion,sht30.yaml
> new file mode 100644
> index 000000000000..9b120f8d303c
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/hwmon/sensirion,sht30.yaml
> @@ -0,0 +1,67 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/hwmon/sensirion,sht30.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Sensirion SHT3x and GXCAS GXHT30 humidity and temperature sensors
> +
> +maintainers:
> + - Zaixiang Xu <zaixiang.xu.dev@xxxxxxxxx>
> +
> +properties:
> + compatible:
> + oneOf:
> + - enum:
> + - sensirion,sht30
> + - sensirion,sht31
> + - sensirion,sht35
> + - sensirion,sht85
> + - sensirion,sts30
> + - sensirion,sts31
> + - sensirion,sts32
> + - sensirion,sts35
I don't understand this - it AGAIN does not match the driver. Your
driver clearly claims all of these are compatible. But binding does not.
Explain the reasons in commit msg.
Best regards,
Krzysztof