linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: serial: uniphier: Add "resets" property as optional
@ 2022-03-28  1:26 Kunihiko Hayashi
  2022-03-28 19:05 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 3+ messages in thread
From: Kunihiko Hayashi @ 2022-03-28  1:26 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski
  Cc: Masami Hiramatsu, linux-serial, devicetree, linux-arm-kernel,
	linux-kernel, Kunihiko Hayashi

UniPhier UART controller has a reset lines from system controller.
Add "resets" property to fix the following warning.

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

Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
---
 .../devicetree/bindings/serial/socionext,uniphier-uart.yaml    | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
index d490c7c4b967..d14d8c9cfe4d 100644
--- a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
+++ b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
@@ -22,6 +22,9 @@ properties:
   clocks:
     minItems: 1
 
+  resets:
+    minItems: 1
+
   auto-flow-control:
     description: enable automatic flow control support.
     $ref: /schemas/types.yaml#/definitions/flag
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: serial: uniphier: Add "resets" property as optional
  2022-03-28  1:26 [PATCH] dt-bindings: serial: uniphier: Add "resets" property as optional Kunihiko Hayashi
@ 2022-03-28 19:05 ` Krzysztof Kozlowski
  2022-03-29 10:56   ` Kunihiko Hayashi
  0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2022-03-28 19:05 UTC (permalink / raw)
  To: Kunihiko Hayashi, Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski
  Cc: Masami Hiramatsu, linux-serial, devicetree, linux-arm-kernel,
	linux-kernel

On 28/03/2022 03:26, Kunihiko Hayashi wrote:
> UniPhier UART controller has a reset lines from system controller.
> Add "resets" property to fix the following warning.
> 
>   uniphier-ld11-global.dtb: serial@54006800: 'resets' does not match any of the regexes: 'pinctrl-[0-9]+'
>       From schema: Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
> 
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> ---
>  .../devicetree/bindings/serial/socionext,uniphier-uart.yaml    | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
> index d490c7c4b967..d14d8c9cfe4d 100644
> --- a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
> +++ b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
> @@ -22,6 +22,9 @@ properties:
>    clocks:
>      minItems: 1

This should be fixed to maxItems as well (separate patch).

>  
> +  resets:
> +    minItems: 1

maxItems

> +
>    auto-flow-control:
>      description: enable automatic flow control support.
>      $ref: /schemas/types.yaml#/definitions/flag


Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: serial: uniphier: Add "resets" property as optional
  2022-03-28 19:05 ` Krzysztof Kozlowski
@ 2022-03-29 10:56   ` Kunihiko Hayashi
  0 siblings, 0 replies; 3+ messages in thread
From: Kunihiko Hayashi @ 2022-03-29 10:56 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Greg Kroah-Hartman, Rob Herring,
	Krzysztof Kozlowski
  Cc: Masami Hiramatsu, linux-serial, devicetree, linux-arm-kernel,
	linux-kernel

Hi Krzysztof,

On 2022/03/29 4:05, Krzysztof Kozlowski wrote:
> On 28/03/2022 03:26, Kunihiko Hayashi wrote:
>> UniPhier UART controller has a reset lines from system controller.
>> Add "resets" property to fix the following warning.
>>
>>    uniphier-ld11-global.dtb: serial@54006800: 'resets' does not match any
>> of the regexes: 'pinctrl-[0-9]+'
>>        From schema:
>> Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
>>
>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
>> ---
>>   .../devicetree/bindings/serial/socionext,uniphier-uart.yaml    | 3 +++
>>   1 file changed, 3 insertions(+)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
>> b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
>> index d490c7c4b967..d14d8c9cfe4d 100644
>> ---
>> a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
>> +++
>> b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
>> @@ -22,6 +22,9 @@ properties:
>>     clocks:
>>       minItems: 1
> 
> This should be fixed to maxItems as well (separate patch).

I see. In the case of one item, "maxItems" should be used.
I'll fix it in another patch.

>>
>> +  resets:
>> +    minItems: 1
> 
> maxItems

I'll fix it in this patch.

Thank you,

---
Best Regards
Kunihiko Hayashi

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-03-29 10:57 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-28  1:26 [PATCH] dt-bindings: serial: uniphier: Add "resets" property as optional Kunihiko Hayashi
2022-03-28 19:05 ` Krzysztof Kozlowski
2022-03-29 10:56   ` Kunihiko Hayashi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).