2023-05-30 14:54:47

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 3/7] dt-bindings: rtc: restrict node name suffixes

Make the pattern matching node names a bit stricter to improve DTS
consistency. The pattern is restricted to:
1. Only one unit address or one -N suffix,
2. -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/rtc/rtc.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/rtc/rtc.yaml b/Documentation/devicetree/bindings/rtc/rtc.yaml
index c6fff5486fe6..efb66df82782 100644
--- a/Documentation/devicetree/bindings/rtc/rtc.yaml
+++ b/Documentation/devicetree/bindings/rtc/rtc.yaml
@@ -15,7 +15,7 @@ description: |

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

aux-voltage-chargeable:
$ref: /schemas/types.yaml#/definitions/uint32
--
2.34.1



2023-05-31 04:46:13

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH 3/7] dt-bindings: rtc: 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:
> 1. Only one unit address or one -N suffix,
> 2. -N suffixes to decimal numbers.

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

2023-06-06 21:50:19

by Alexandre Belloni

[permalink] [raw]
Subject: Re: (subset) [PATCH 3/7] dt-bindings: rtc: restrict node name suffixes


On Tue, 30 May 2023 16:48:47 +0200, Krzysztof Kozlowski wrote:
> Make the pattern matching node names a bit stricter to improve DTS
> consistency. The pattern is restricted to:
> 1. Only one unit address or one -N suffix,
> 2. -N suffixes to decimal numbers.
>
>

Applied, thanks!

[3/7] dt-bindings: rtc: restrict node name suffixes
commit: 93eff1e0e67a0b54fc16f07f0ca88f8b5afd9f89

Best regards,

--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com