| Message ID | 20240818173014.122073-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/5] dt-bindings: clock: baikal,bt1-ccu-div: add top-level constraints | expand |
On Sun, Aug 18, 2024 at 07:30:10PM +0200, Krzysztof Kozlowski wrote: > Properties with variable number of items per each device are expected to > have widest constraints in top-level "properties:" block and further > customized (narrowed) in "if:then:". Add missing top-level constraints > for clocks and clock-names. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Serge Semin <fancer.lancer@gmail.com> -Serge(y) > --- > .../devicetree/bindings/clock/baikal,bt1-ccu-div.yaml | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml > index bd4cefbb1244..30252c95700c 100644 > --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml > +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml > @@ -134,9 +134,13 @@ properties: > "#reset-cells": > const: 1 > > - clocks: true > + clocks: > + minItems: 3 > + maxItems: 4 > > - clock-names: true > + clock-names: > + minItems: 3 > + maxItems: 4 > > additionalProperties: false > > -- > 2.43.0 >
On Sun, Aug 18, 2024 at 07:30:10PM +0200, Krzysztof Kozlowski wrote: > Properties with variable number of items per each device are expected to > have widest constraints in top-level "properties:" block and further > customized (narrowed) in "if:then:". Add missing top-level constraints > for clocks and clock-names. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> I really wish you'd sent cover letters so I don't "have" to send 5 different acks where one would do. Acked-by: Conor Dooley <conor.dooley@microchip.com> > --- > .../devicetree/bindings/clock/baikal,bt1-ccu-div.yaml | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml > index bd4cefbb1244..30252c95700c 100644 > --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml > +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml > @@ -134,9 +134,13 @@ properties: > "#reset-cells": > const: 1 > > - clocks: true > + clocks: > + minItems: 3 > + maxItems: 4 > > - clock-names: true > + clock-names: > + minItems: 3 > + maxItems: 4 > > additionalProperties: false > > -- > 2.43.0 >
Quoting Krzysztof Kozlowski (2024-08-18 10:30:10) > Properties with variable number of items per each device are expected to > have widest constraints in top-level "properties:" block and further > customized (narrowed) in "if:then:". Add missing top-level constraints > for clocks and clock-names. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- Applied to clk-next
On Sun, 18 Aug 2024 19:30:10 +0200, Krzysztof Kozlowski wrote: > Properties with variable number of items per each device are expected to > have widest constraints in top-level "properties:" block and further > customized (narrowed) in "if:then:". Add missing top-level constraints > for clocks and clock-names. > > Applied, thanks! [4/5] dt-bindings: clock: rockchip,rk3588-cru: drop unneeded assigned-clocks commit: 3529dc29fe65672ad9aeab9499fee901d0010901 Best regards,
diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml index bd4cefbb1244..30252c95700c 100644 --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml @@ -134,9 +134,13 @@ properties: "#reset-cells": const: 1 - clocks: true + clocks: + minItems: 3 + maxItems: 4 - clock-names: true + clock-names: + minItems: 3 + maxItems: 4 additionalProperties: false
Properties with variable number of items per each device are expected to have widest constraints in top-level "properties:" block and further customized (narrowed) in "if:then:". Add missing top-level constraints for clocks and clock-names. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../devicetree/bindings/clock/baikal,bt1-ccu-div.yaml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)