mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v2 0/2] dt-bindings: serial: uniphier: Fix clocks and resets properties
@ 2022-03-30  5:15 Kunihiko Hayashi
  2022-03-30  5:15 ` [PATCH v2 1/2] dt-bindings: serial: uniphier: Use maxItems for a single clock Kunihiko Hayashi
  2022-03-30  5:15 ` [PATCH v2 2/2] dt-bindings: serial: uniphier: Add "resets" property as optional Kunihiko Hayashi
  0 siblings, 2 replies; 5+ messages in thread
From: Kunihiko Hayashi @ 2022-03-30  5:15 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski
  Cc: Masami Hiramatsu, linux-serial, devicetree, linux-arm-kernel,
	linux-kernel, Kunihiko Hayashi

This series fixes dt-bindings for serial controller implemented in UniPhier SoCs.

- Fix "clocks" property for a single entry
- Add missing "resets" property

Changes since v1:
- Add patch 1
- Use maxItems for a single reset

Kunihiko Hayashi (2):
  dt-bindings: serial: uniphier: Use maxItems for a single clock
  dt-bindings: serial: uniphier: Add "resets" property as optional

 .../devicetree/bindings/serial/socionext,uniphier-uart.yaml  | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

-- 
2.25.1


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

* [PATCH v2 1/2] dt-bindings: serial: uniphier: Use maxItems for a single clock
  2022-03-30  5:15 [PATCH v2 0/2] dt-bindings: serial: uniphier: Fix clocks and resets properties Kunihiko Hayashi
@ 2022-03-30  5:15 ` Kunihiko Hayashi
  2022-03-30  8:16   ` Krzysztof Kozlowski
  2022-03-30  5:15 ` [PATCH v2 2/2] dt-bindings: serial: uniphier: Add "resets" property as optional Kunihiko Hayashi
  1 sibling, 1 reply; 5+ messages in thread
From: Kunihiko Hayashi @ 2022-03-30  5:15 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Rob Herring, Krzysztof Kozlowski
  Cc: Masami Hiramatsu, linux-serial, devicetree, linux-arm-kernel,
	linux-kernel, Kunihiko Hayashi

'maxItems' is preferred for a single entry, not 'minItems'.

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

diff --git a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
index d490c7c4b967..06353028b274 100644
--- a/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
+++ b/Documentation/devicetree/bindings/serial/socionext,uniphier-uart.yaml
@@ -20,7 +20,7 @@ properties:
     maxItems: 1
 
   clocks:
-    minItems: 1
+    maxItems: 1
 
   auto-flow-control:
     description: enable automatic flow control support.
-- 
2.25.1


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

* [PATCH v2 2/2] dt-bindings: serial: uniphier: Add "resets" property as optional
  2022-03-30  5:15 [PATCH v2 0/2] dt-bindings: serial: uniphier: Fix clocks and resets properties Kunihiko Hayashi
  2022-03-30  5:15 ` [PATCH v2 1/2] dt-bindings: serial: uniphier: Use maxItems for a single clock Kunihiko Hayashi
@ 2022-03-30  5:15 ` Kunihiko Hayashi
  2022-03-30  8:16   ` Krzysztof Kozlowski
  1 sibling, 1 reply; 5+ messages in thread
From: Kunihiko Hayashi @ 2022-03-30  5:15 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 06353028b274..3d01cc355778 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:
     maxItems: 1
 
+  resets:
+    maxItems: 1
+
   auto-flow-control:
     description: enable automatic flow control support.
     $ref: /schemas/types.yaml#/definitions/flag
-- 
2.25.1


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

* Re: [PATCH v2 1/2] dt-bindings: serial: uniphier: Use maxItems for a single clock
  2022-03-30  5:15 ` [PATCH v2 1/2] dt-bindings: serial: uniphier: Use maxItems for a single clock Kunihiko Hayashi
@ 2022-03-30  8:16   ` Krzysztof Kozlowski
  0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-03-30  8:16 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 30/03/2022 07:15, Kunihiko Hayashi wrote:
> 'maxItems' is preferred for a single entry, not 'minItems'.
> 
> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> ---
>  .../devicetree/bindings/serial/socionext,uniphier-uart.yaml     | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 


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


Best regards,
Krzysztof

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

* Re: [PATCH v2 2/2] dt-bindings: serial: uniphier: Add "resets" property as optional
  2022-03-30  5:15 ` [PATCH v2 2/2] dt-bindings: serial: uniphier: Add "resets" property as optional Kunihiko Hayashi
@ 2022-03-30  8:16   ` Krzysztof Kozlowski
  0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-03-30  8:16 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 30/03/2022 07:15, 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(+)
> 

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

Best regards,
Krzysztof

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

end of thread, other threads:[~2022-03-30  8:17 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-30  5:15 [PATCH v2 0/2] dt-bindings: serial: uniphier: Fix clocks and resets properties Kunihiko Hayashi
2022-03-30  5:15 ` [PATCH v2 1/2] dt-bindings: serial: uniphier: Use maxItems for a single clock Kunihiko Hayashi
2022-03-30  8:16   ` Krzysztof Kozlowski
2022-03-30  5:15 ` [PATCH v2 2/2] dt-bindings: serial: uniphier: Add "resets" property as optional Kunihiko Hayashi
2022-03-30  8:16   ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®