Re: [PATCH v6 1/3] dt-bindings: thermal: samsung: Adjust '#thermal-sensor-cells' to 1

From: Krzysztof Kozlowski

Date: Fri Oct 10 2025 - 08:44:21 EST


On 30/09/2025 02:51, Shin Son wrote:
> reg:
> minItems: 1
> maxItems: 1
> + '#thermal-sensor-cells':
> + const: 0
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: samsung,exynosautov920-tmu
> + then:
> + properties:
> + clocks:
> + minItems: 1

You can drop minItems. Existing binding has it unnecessarily.

> + maxItems: 1

You also need clock-names restriction, just like clocks or just ": false".

> + reg:
> + minItems: 1

This also drop.

> + maxItems: 1
> + '#thermal-sensor-cells':
> + const: 1
>


Rest looks fine, so with changes above:


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>


Best regards,
Krzysztof