| Message ID | 20240818172819.121697-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | dt-bindings: reset: socionext,uniphier-glue-reset: add top-level constraints | expand |
On Sun, 18 Aug 2024 19:28:19 +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 clock-names and reset-names. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/reset/socionext,uniphier-glue-reset.yaml | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Hi Krzysztof, On 2024/08/19 2:28, 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 clock-names and reset-names. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/reset/socionext,uniphier-glue-reset.yaml | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git > a/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.ya > ml > b/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.ya > ml > index fa253c518d79..babc563ae61e 100644 > --- > a/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.ya > ml > +++ > b/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.ya > ml > @@ -38,13 +38,17 @@ properties: > minItems: 1 > maxItems: 2 > > - clock-names: true > + clock-names: > + minItems: 1 > + maxItems: 2 > > resets: > minItems: 1 > maxItems: 2 > > - reset-names: true > + reset-names: > + minItems: 1 > + maxItems: 2 > > allOf: > - if: Reviewed-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> I'd appreciate if it could be applied in devicetree or reset. Thank you, --- Best Regards Kunihiko Hayashi
On Mi, 2024-08-21 at 19:31 +0900, Kunihiko Hayashi wrote: > On 2024/08/19 2:28, 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 clock-names and reset-names. > > > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [...] > Reviewed-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> > > I'd appreciate if it could be applied in devicetree or reset. Thank you, applied to reset/next. regards Philipp
diff --git a/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.yaml b/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.yaml index fa253c518d79..babc563ae61e 100644 --- a/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.yaml +++ b/Documentation/devicetree/bindings/reset/socionext,uniphier-glue-reset.yaml @@ -38,13 +38,17 @@ properties: minItems: 1 maxItems: 2 - clock-names: true + clock-names: + minItems: 1 + maxItems: 2 resets: minItems: 1 maxItems: 2 - reset-names: true + reset-names: + minItems: 1 + maxItems: 2 allOf: - if:
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 clock-names and reset-names. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../bindings/reset/socionext,uniphier-glue-reset.yaml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)