* [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: add top-level constraints
@ 2025-05-01 17:34 Krzysztof Kozlowski
2025-05-01 17:34 ` [PATCH RESEND v2 2/2] media: dt-bindings: renesas,vsp1: " Krzysztof Kozlowski
2025-05-14 13:32 ` [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: " Rob Herring (Arm)
0 siblings, 2 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-01 17:34 UTC (permalink / raw)
To: Laurent Pinchart, Mauro Carvalho Chehab, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Kieran Bingham,
Geert Uytterhoeven, Magnus Damm, linux-media, linux-renesas-soc,
devicetree, linux-kernel
Cc: Krzysztof Kozlowski, Conor Dooley
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.
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Resending because for three months these were not picked up by media
(still in "New" state).
Changes in v2:
1. Add tag
2. Move clocks description to top level
---
.../bindings/media/renesas,fcp.yaml | 23 ++++++++++---------
1 file changed, 12 insertions(+), 11 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.yaml b/Documentation/devicetree/bindings/media/renesas,fcp.yaml
index 5ed9427fb757..7bf1266223e8 100644
--- a/Documentation/devicetree/bindings/media/renesas,fcp.yaml
+++ b/Documentation/devicetree/bindings/media/renesas,fcp.yaml
@@ -36,9 +36,18 @@ properties:
reg:
maxItems: 1
- clocks: true
+ clocks:
+ minItems: 1
+ items:
+ - description: Main clock
+ - description: Register access clock
+ - description: Video clock
- clock-names: true
+ clock-names:
+ items:
+ - const: aclk
+ - const: pclk
+ - const: vclk
iommus:
maxItems: 1
@@ -71,15 +80,7 @@ allOf:
then:
properties:
clocks:
- items:
- - description: Main clock
- - description: Register access clock
- - description: Video clock
- clock-names:
- items:
- - const: aclk
- - const: pclk
- - const: vclk
+ minItems: 3
required:
- clock-names
else:
--
2.45.2
^ permalink raw reply [flat|nested] 4+ messages in thread* [PATCH RESEND v2 2/2] media: dt-bindings: renesas,vsp1: add top-level constraints
2025-05-01 17:34 [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: add top-level constraints Krzysztof Kozlowski
@ 2025-05-01 17:34 ` Krzysztof Kozlowski
2025-05-14 13:33 ` Rob Herring (Arm)
2025-05-14 13:32 ` [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: " Rob Herring (Arm)
1 sibling, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-01 17:34 UTC (permalink / raw)
To: Laurent Pinchart, Mauro Carvalho Chehab, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Kieran Bingham,
Geert Uytterhoeven, Magnus Damm, linux-media, linux-renesas-soc,
devicetree, linux-kernel
Cc: Krzysztof Kozlowski, Conor Dooley
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.
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Changes in v2:
1. Add tag
2. Move clocks description to top level
---
.../bindings/media/renesas,vsp1.yaml | 24 ++++++++++---------
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/renesas,vsp1.yaml b/Documentation/devicetree/bindings/media/renesas,vsp1.yaml
index 9d03b972f522..fcf7219b1f40 100644
--- a/Documentation/devicetree/bindings/media/renesas,vsp1.yaml
+++ b/Documentation/devicetree/bindings/media/renesas,vsp1.yaml
@@ -34,8 +34,18 @@ properties:
interrupts:
maxItems: 1
- clocks: true
- clock-names: true
+ clocks:
+ minItems: 1
+ items:
+ - description: Main clock
+ - description: Register access clock
+ - description: Video clock
+
+ clock-names:
+ items:
+ - const: aclk
+ - const: pclk
+ - const: vclk
power-domains:
maxItems: 1
@@ -79,15 +89,7 @@ allOf:
then:
properties:
clocks:
- items:
- - description: Main clock
- - description: Register access clock
- - description: Video clock
- clock-names:
- items:
- - const: aclk
- - const: pclk
- - const: vclk
+ minItems: 3
required:
- clock-names
else:
--
2.45.2
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH RESEND v2 2/2] media: dt-bindings: renesas,vsp1: add top-level constraints
2025-05-01 17:34 ` [PATCH RESEND v2 2/2] media: dt-bindings: renesas,vsp1: " Krzysztof Kozlowski
@ 2025-05-14 13:33 ` Rob Herring (Arm)
0 siblings, 0 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-05-14 13:33 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Magnus Damm, Kieran Bingham, Geert Uytterhoeven, Conor Dooley,
linux-kernel, Conor Dooley, Mauro Carvalho Chehab, linux-media,
linux-renesas-soc, devicetree, Laurent Pinchart,
Krzysztof Kozlowski
On Thu, 01 May 2025 19:34:13 +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.
>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> ---
>
> Changes in v2:
> 1. Add tag
> 2. Move clocks description to top level
> ---
> .../bindings/media/renesas,vsp1.yaml | 24 ++++++++++---------
> 1 file changed, 13 insertions(+), 11 deletions(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: add top-level constraints
2025-05-01 17:34 [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: add top-level constraints Krzysztof Kozlowski
2025-05-01 17:34 ` [PATCH RESEND v2 2/2] media: dt-bindings: renesas,vsp1: " Krzysztof Kozlowski
@ 2025-05-14 13:32 ` Rob Herring (Arm)
1 sibling, 0 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-05-14 13:32 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Conor Dooley, Conor Dooley, Laurent Pinchart, Geert Uytterhoeven,
linux-media, linux-kernel, Kieran Bingham, Mauro Carvalho Chehab,
Krzysztof Kozlowski, devicetree, Magnus Damm, linux-renesas-soc
On Thu, 01 May 2025 19:34:12 +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.
>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> ---
>
> Resending because for three months these were not picked up by media
> (still in "New" state).
>
> Changes in v2:
> 1. Add tag
> 2. Move clocks description to top level
> ---
> .../bindings/media/renesas,fcp.yaml | 23 ++++++++++---------
> 1 file changed, 12 insertions(+), 11 deletions(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-05-14 13:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-05-01 17:34 [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: add top-level constraints Krzysztof Kozlowski
2025-05-01 17:34 ` [PATCH RESEND v2 2/2] media: dt-bindings: renesas,vsp1: " Krzysztof Kozlowski
2025-05-14 13:33 ` Rob Herring (Arm)
2025-05-14 13:32 ` [PATCH RESEND v2 1/2] media: dt-bindings: renesas,fcp: " Rob Herring (Arm)
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®