Message ID | 20220424150333.75172-3-krzysztof.kozlowski@linaro.org (mailing list archive) |
---|---|
State | Accepted |
Commit | ee77ef0d09e3ca0509833f6fbd49d925630a27f4 |
Headers | show |
Series | [1/3] dt-bindings: timer: cdns,ttc: drop unneeded minItems | expand |
Hello Krzysztof, >-----Original Message----- >From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org] >Sent: Sunday, April 24, 2022 8:34 PM >To: Daniel Lezcano <daniel.lezcano@linaro.org>; Thomas Gleixner ><tglx@linutronix.de>; Rob Herring <robh+dt@kernel.org>; Krzysztof >Kozlowski <krzysztof.kozlowski+dt@linaro.org>; Alim Akhtar ><alim.akhtar@samsung.com>; Michal Simek <michal.simek@xilinx.com>; >linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; linux-arm- >kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org >Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >Subject: [PATCH 3/3] dt-bindings: timer: samsung,exynos4210-mct: define >strict clock order > >The DTS should always have fixed clock order, even if it comes with clock- >names property. Drop the pattern to make the order strict. >Existing DTS already match this. > >Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >--- Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com> > .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/Documentation/devicetree/bindings/timer/samsung,exynos4210- >mct.yaml >b/Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml >index 0e28d9bdb8cb..9c81d00b12e0 100644 >--- a/Documentation/devicetree/bindings/timer/samsung,exynos4210- >mct.yaml >+++ b/Documentation/devicetree/bindings/timer/samsung,exynos4210- >mct.yam >+++ l >@@ -39,8 +39,8 @@ properties: > > clock-names: > items: >- - pattern: "^(fin_pll|mct)$" >- - pattern: "^(fin_pll|mct)$" >+ - const: fin_pll >+ - const: mct > > reg: > maxItems: 1 >-- >2.32.0
On Sun, 24 Apr 2022 17:03:33 +0200, Krzysztof Kozlowski wrote: > The DTS should always have fixed clock order, even if it comes with > clock-names property. Drop the pattern to make the order strict. > Existing DTS already match this. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml b/Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml index 0e28d9bdb8cb..9c81d00b12e0 100644 --- a/Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml +++ b/Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml @@ -39,8 +39,8 @@ properties: clock-names: items: - - pattern: "^(fin_pll|mct)$" - - pattern: "^(fin_pll|mct)$" + - const: fin_pll + - const: mct reg: maxItems: 1
The DTS should always have fixed clock order, even if it comes with clock-names property. Drop the pattern to make the order strict. Existing DTS already match this. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)