| Message ID | 20230530144851.92059-8-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 6cca6cf3f5ab2c6f7ce48e2d958151580079a6d4 |
| Headers | show |
| Series | dt-bindings: restrict node name suffixes | expand |
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 <robh@kernel.org> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Guenter Roeck <linux@roeck-us.net> > > --- > > Cc: Tony Lindgren <tony@atomide.com> > Cc: Oleksij Rempel <o.rempel@pengutronix.de> > --- > 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 >
* Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [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 <tony@atomide.com>
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 <robh@kernel.org> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Cc: Tony Lindgren <tony@atomide.com> > Cc: Oleksij Rempel <o.rempel@pengutronix.de> > --- > Documentation/devicetree/bindings/watchdog/watchdog.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
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 <robh@kernel.org> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Cc: Tony Lindgren <tony@atomide.com> > Cc: Oleksij Rempel <o.rempel@pengutronix.de> > --- > Documentation/devicetree/bindings/watchdog/watchdog.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Seems watchdog patches aren't getting applied... Applied, thanks!
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:
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 <robh@kernel.org> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Cc: Tony Lindgren <tony@atomide.com> Cc: Oleksij Rempel <o.rempel@pengutronix.de> --- Documentation/devicetree/bindings/watchdog/watchdog.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)