[PATCH 1/3] dt-bindings: watchdog: Restrict timeout-sec to one number

Krzysztof Kozlowski posted 3 patches 3 months, 3 weeks ago
[PATCH 1/3] dt-bindings: watchdog: Restrict timeout-sec to one number
Posted by Krzysztof Kozlowski 3 months, 3 weeks ago
Linux kernel expects only one number for the watchdog timeout and the
type is an array (defined in property-units.yaml in DT schema), so
restrict the property.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/watchdog/watchdog.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
index f0a584af1223..be0327f587eb 100644
--- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml
+++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
@@ -24,6 +24,7 @@ properties:
     pattern: "^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$"
 
   timeout-sec:
+    maxItems: 1
     description:
       Contains the watchdog timeout in seconds.
 

-- 
2.48.1
Re: [PATCH 1/3] dt-bindings: watchdog: Restrict timeout-sec to one number
Posted by Rob Herring (Arm) 3 months, 2 weeks ago
On Mon, 20 Oct 2025 18:52:20 +0200, Krzysztof Kozlowski wrote:
> Linux kernel expects only one number for the watchdog timeout and the
> type is an array (defined in property-units.yaml in DT schema), so
> restrict the property.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/watchdog/watchdog.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Re: [PATCH 1/3] dt-bindings: watchdog: Restrict timeout-sec to one number
Posted by Guenter Roeck 3 months, 3 weeks ago
On 10/20/25 09:52, Krzysztof Kozlowski wrote:
> Linux kernel expects only one number for the watchdog timeout and the
> type is an array (defined in property-units.yaml in DT schema), so
> restrict the property.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> ---
>   Documentation/devicetree/bindings/watchdog/watchdog.yaml | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> index f0a584af1223..be0327f587eb 100644
> --- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> @@ -24,6 +24,7 @@ properties:
>       pattern: "^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$"
>   
>     timeout-sec:
> +    maxItems: 1
>       description:
>         Contains the watchdog timeout in seconds.
>   
>