2023-05-30 14:56:33

by Krzysztof Kozlowski

Subject: [PATCH 7/7] dt-bindings: watchdog: restrict node name suffixes

Make the pattern matching node names a bit stricter to improve DTS
consistency. The pattern is restricted to -N suffixes to decimal
numbers.

Suggested-by: Rob Herring <[email protected]>
Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

Cc: Tony Lindgren <[email protected]>
Cc: Oleksij Rempel <[email protected]>
---
Documentation/devicetree/bindings/watchdog/watchdog.yaml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
index 519b48889eb1..f0a584af1223 100644
--- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml
+++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
@@ -17,11 +17,11 @@ description: |
select:
properties:
$nodename:
- pattern: "^watchdog(@.*|-[0-9a-f])?$"
+ pattern: "^watchdog(@.*|-([0-9]|[1-9][0-9]+))?$"

properties:
$nodename:
- pattern: "^(timer|watchdog)(@.*|-[0-9a-f])?$"
+ pattern: "^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$"

timeout-sec:
description:
--
2.34.1

2023-05-30 17:56:58

by Guenter Roeck

Subject: Re: [PATCH 7/7] dt-bindings: watchdog: restrict node name suffixes

On Tue, May 30, 2023 at 04:48:51PM +0200, Krzysztof Kozlowski wrote:
> Make the pattern matching node names a bit stricter to improve DTS
> consistency. The pattern is restricted to -N suffixes to decimal
> numbers.
>
> Suggested-by: Rob Herring <[email protected]>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Guenter Roeck <[email protected]>

>
> ---
>
> Cc: Tony Lindgren <[email protected]>
> Cc: Oleksij Rempel <[email protected]>
> ---
> Documentation/devicetree/bindings/watchdog/watchdog.yaml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/watchdog.yaml b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> index 519b48889eb1..f0a584af1223 100644
> --- a/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/watchdog.yaml
> @@ -17,11 +17,11 @@ description: |
> select:
> properties:
> $nodename:
> - pattern: "^watchdog(@.*|-[0-9a-f])?$"
> + pattern: "^watchdog(@.*|-([0-9]|[1-9][0-9]+))?$"
>
> properties:
> $nodename:
> - pattern: "^(timer|watchdog)(@.*|-[0-9a-f])?$"
> + pattern: "^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$"
>
> timeout-sec:
> description:
> --
> 2.34.1
>

2023-05-31 04:29:50

by Tony Lindgren

Subject: Re: [PATCH 7/7] dt-bindings: watchdog: restrict node name suffixes

* Krzysztof Kozlowski <[email protected]> [230530 14:49]:
> Make the pattern matching node names a bit stricter to improve DTS
> consistency. The pattern is restricted to -N suffixes to decimal
> numbers.

Reviewed-by: Tony Lindgren <[email protected]>

2023-06-09 15:33:47

by Rob Herring (Arm)

Subject: Re: [PATCH 7/7] dt-bindings: watchdog: restrict node name suffixes

On Tue, 30 May 2023 16:48:51 +0200, Krzysztof Kozlowski wrote:
> Make the pattern matching node names a bit stricter to improve DTS
> consistency. The pattern is restricted to -N suffixes to decimal
> numbers.
>
> Suggested-by: Rob Herring <[email protected]>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Cc: Tony Lindgren <[email protected]>
> Cc: Oleksij Rempel <[email protected]>
> ---
> Documentation/devicetree/bindings/watchdog/watchdog.yaml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>

Acked-by: Rob Herring <[email protected]>

2023-06-30 17:12:09

by Rob Herring (Arm)

Subject: Re: [PATCH 7/7] dt-bindings: watchdog: restrict node name suffixes

On Tue, 30 May 2023 16:48:51 +0200, Krzysztof Kozlowski wrote:
> Make the pattern matching node names a bit stricter to improve DTS
> consistency. The pattern is restricted to -N suffixes to decimal
> numbers.
>
> Suggested-by: Rob Herring <[email protected]>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Cc: Tony Lindgren <[email protected]>
> Cc: Oleksij Rempel <[email protected]>
> ---
> Documentation/devicetree/bindings/watchdog/watchdog.yaml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>

Seems watchdog patches aren't getting applied... Applied, thanks!