[v2] dt-bindings: reset: Add parent "resets" property as optional

[PATCH v2] dt-bindings: reset: Add parent "resets" property as optional

Posted by Kunihiko Hayashi 4 years, 2 months ago

LD11 mio reset controller has a reset lines from system controller.
Add parent "resets" property to fix the following warning.

  uniphier-ld11-global.dt.yaml: reset: 'resets' does not match any of the regexes: 'pinctrl-[0-9]+'
      From schema: Documentation/devicetree/bindings/reset/socionext,uniphier-reset.yaml

Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
---

Changes since v1:
- Use maxItems for a single reset

 .../devicetree/bindings/reset/socionext,uniphier-reset.yaml    | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/reset/socionext,uniphier-reset.yaml b/Documentation/devicetree/bindings/reset/socionext,uniphier-reset.yaml
index 377a7d242323..6566804ec567 100644
--- a/Documentation/devicetree/bindings/reset/socionext,uniphier-reset.yaml
+++ b/Documentation/devicetree/bindings/reset/socionext,uniphier-reset.yaml
@@ -55,6 +55,9 @@ properties:
   "#reset-cells":
     const: 1
 
+  resets:
+    maxItems: 1
+
 additionalProperties: false
 
 required:
-- 
2.25.1

Re: [PATCH v2] dt-bindings: reset: Add parent "resets" property as optional

Posted by Krzysztof Kozlowski 4 years, 2 months ago

On 30/03/2022 07:11, Kunihiko Hayashi wrote:
> LD11 mio reset controller has a reset lines from system controller.
> Add parent "resets" property to fix the following warning.
> 
>   uniphier-ld11-global.dt.yaml: reset: 'resets' does not match any of the regexes: 'pinctrl-[0-9]+'
>       From schema: Documentation/devicetree/bindings/reset/socionext,uniphier-reset.yaml
> 
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> ---
> 
> Changes since v1:
> - Use maxItems for a single reset
> 


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof

Re: [PATCH v2] dt-bindings: reset: Add parent "resets" property as optional

Posted by Philipp Zabel 4 years, 2 months ago

On Mi, 2022-03-30 at 14:11 +0900, Kunihiko Hayashi wrote:
> LD11 mio reset controller has a reset lines from system controller.
> Add parent "resets" property to fix the following warning.
> 
>   uniphier-ld11-global.dt.yaml: reset: 'resets' does not match any of
> the regexes: 'pinctrl-[0-9]+'
>       From schema:
> Documentation/devicetree/bindings/reset/socionext,uniphier-reset.yaml
> 
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> ---
> 
> Changes since v1:
> - Use maxItems for a single reset
> 
>  .../devicetree/bindings/reset/socionext,uniphier-reset.yaml    | 3
> +++
>  1 file changed, 3 insertions(+)
> 
> diff --git
> a/Documentation/devicetree/bindings/reset/socionext,uniphier-
> reset.yaml
> b/Documentation/devicetree/bindings/reset/socionext,uniphier-
> reset.yaml
> index 377a7d242323..6566804ec567 100644
> --- a/Documentation/devicetree/bindings/reset/socionext,uniphier-
> reset.yaml
> +++ b/Documentation/devicetree/bindings/reset/socionext,uniphier-
> reset.yaml
> @@ -55,6 +55,9 @@ properties:
>    "#reset-cells":
>      const: 1
>  
> +  resets:
> +    maxItems: 1
> +
>  additionalProperties: false
>  
>  required:

Thank you, applied to reset/fixes.

regards
Philipp