* [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
@ 2021-12-20 12:51 ` Thierry Reding
0 siblings, 0 replies; 8+ messages in thread
From: Thierry Reding @ 2021-12-20 12:51 UTC (permalink / raw)
To: Thomas Zimmermann, Marek Vasut, Rob Herring; +Cc: dri-devel, devicetree
From: Thierry Reding <treding@nvidia.com>
In order to validate multiple "if" conditionals, they must be part of an
"allOf:" list, otherwise they will cause a failure in parsing the schema
because of the duplicated "if" property.
Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
Signed-off-by: Thierry Reding <treding@nvidia.com>
---
.../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
1 file changed, 22 insertions(+), 21 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
index 708de84ac138..5079c1cc337b 100644
--- a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
@@ -94,31 +94,32 @@ properties:
power-supply: true
-if:
- not:
- properties:
- compatible:
- contains:
- const: lvds-decoder
-then:
- properties:
- ports:
+allOf:
+ - if:
+ not:
+ properties:
+ compatible:
+ contains:
+ const: lvds-decoder
+ then:
properties:
- port@0:
+ ports:
properties:
- endpoint:
+ port@0:
properties:
- data-mapping: false
+ endpoint:
+ properties:
+ data-mapping: false
-if:
- not:
- properties:
- compatible:
- contains:
- const: lvds-encoder
-then:
- properties:
- pclk-sample: false
+ - if:
+ not:
+ properties:
+ compatible:
+ contains:
+ const: lvds-encoder
+ then:
+ properties:
+ pclk-sample: false
required:
- compatible
--
2.34.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
@ 2021-12-20 12:51 ` Thierry Reding
0 siblings, 0 replies; 8+ messages in thread
From: Thierry Reding @ 2021-12-20 12:51 UTC (permalink / raw)
To: Thomas Zimmermann, Marek Vasut, Rob Herring; +Cc: devicetree, dri-devel
From: Thierry Reding <treding@nvidia.com>
In order to validate multiple "if" conditionals, they must be part of an
"allOf:" list, otherwise they will cause a failure in parsing the schema
because of the duplicated "if" property.
Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
Signed-off-by: Thierry Reding <treding@nvidia.com>
---
.../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
1 file changed, 22 insertions(+), 21 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
index 708de84ac138..5079c1cc337b 100644
--- a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
@@ -94,31 +94,32 @@ properties:
power-supply: true
-if:
- not:
- properties:
- compatible:
- contains:
- const: lvds-decoder
-then:
- properties:
- ports:
+allOf:
+ - if:
+ not:
+ properties:
+ compatible:
+ contains:
+ const: lvds-decoder
+ then:
properties:
- port@0:
+ ports:
properties:
- endpoint:
+ port@0:
properties:
- data-mapping: false
+ endpoint:
+ properties:
+ data-mapping: false
-if:
- not:
- properties:
- compatible:
- contains:
- const: lvds-encoder
-then:
- properties:
- pclk-sample: false
+ - if:
+ not:
+ properties:
+ compatible:
+ contains:
+ const: lvds-encoder
+ then:
+ properties:
+ pclk-sample: false
required:
- compatible
--
2.34.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
2021-12-20 12:51 ` Thierry Reding
@ 2021-12-21 2:12 ` Rob Herring
-1 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2021-12-21 2:12 UTC (permalink / raw)
To: Thierry Reding; +Cc: Thomas Zimmermann, Marek Vasut, dri-devel, devicetree
On Mon, Dec 20, 2021 at 8:51 AM Thierry Reding <thierry.reding@gmail.com> wrote:
>
> From: Thierry Reding <treding@nvidia.com>
>
> In order to validate multiple "if" conditionals, they must be part of an
> "allOf:" list, otherwise they will cause a failure in parsing the schema
> because of the duplicated "if" property.
>
> Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> .../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
> 1 file changed, 22 insertions(+), 21 deletions(-)
Thanks!
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
@ 2021-12-21 2:12 ` Rob Herring
0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2021-12-21 2:12 UTC (permalink / raw)
To: Thierry Reding; +Cc: Marek Vasut, devicetree, dri-devel, Thomas Zimmermann
On Mon, Dec 20, 2021 at 8:51 AM Thierry Reding <thierry.reding@gmail.com> wrote:
>
> From: Thierry Reding <treding@nvidia.com>
>
> In order to validate multiple "if" conditionals, they must be part of an
> "allOf:" list, otherwise they will cause a failure in parsing the schema
> because of the duplicated "if" property.
>
> Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> .../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
> 1 file changed, 22 insertions(+), 21 deletions(-)
Thanks!
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
2021-12-20 12:51 ` Thierry Reding
@ 2021-12-22 18:03 ` Rob Herring
-1 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2021-12-22 18:03 UTC (permalink / raw)
To: Thierry Reding
Cc: devicetree, Rob Herring, dri-devel, Thomas Zimmermann,
Marek Vasut
On Mon, 20 Dec 2021 13:51:47 +0100, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
>
> In order to validate multiple "if" conditionals, they must be part of an
> "allOf:" list, otherwise they will cause a failure in parsing the schema
> because of the duplicated "if" property.
>
> Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> .../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
> 1 file changed, 22 insertions(+), 21 deletions(-)
>
I went ahead and applied to drm-misc, so linux-next is fixed.
Rob
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
@ 2021-12-22 18:03 ` Rob Herring
0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2021-12-22 18:03 UTC (permalink / raw)
To: Thierry Reding
Cc: Marek Vasut, devicetree, Rob Herring, Thomas Zimmermann,
dri-devel
On Mon, 20 Dec 2021 13:51:47 +0100, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
>
> In order to validate multiple "if" conditionals, they must be part of an
> "allOf:" list, otherwise they will cause a failure in parsing the schema
> because of the duplicated "if" property.
>
> Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> .../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
> 1 file changed, 22 insertions(+), 21 deletions(-)
>
I went ahead and applied to drm-misc, so linux-next is fixed.
Rob
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
2021-12-22 18:03 ` Rob Herring
@ 2021-12-22 19:17 ` Marek Vasut
-1 siblings, 0 replies; 8+ messages in thread
From: Marek Vasut @ 2021-12-22 19:17 UTC (permalink / raw)
To: Rob Herring, Thierry Reding
Cc: devicetree, Rob Herring, dri-devel, Thomas Zimmermann
On 12/22/21 19:03, Rob Herring wrote:
> On Mon, 20 Dec 2021 13:51:47 +0100, Thierry Reding wrote:
>> From: Thierry Reding <treding@nvidia.com>
>>
>> In order to validate multiple "if" conditionals, they must be part of an
>> "allOf:" list, otherwise they will cause a failure in parsing the schema
>> because of the duplicated "if" property.
>>
>> Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
>> Signed-off-by: Thierry Reding <treding@nvidia.com>
>> ---
>> .../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
>> 1 file changed, 22 insertions(+), 21 deletions(-)
>>
>
> I went ahead and applied to drm-misc, so linux-next is fixed.
Thank you
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key
@ 2021-12-22 19:17 ` Marek Vasut
0 siblings, 0 replies; 8+ messages in thread
From: Marek Vasut @ 2021-12-22 19:17 UTC (permalink / raw)
To: Rob Herring, Thierry Reding
Cc: devicetree, Rob Herring, Thomas Zimmermann, dri-devel
On 12/22/21 19:03, Rob Herring wrote:
> On Mon, 20 Dec 2021 13:51:47 +0100, Thierry Reding wrote:
>> From: Thierry Reding <treding@nvidia.com>
>>
>> In order to validate multiple "if" conditionals, they must be part of an
>> "allOf:" list, otherwise they will cause a failure in parsing the schema
>> because of the duplicated "if" property.
>>
>> Fixes: d7df3948eb49 ("dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select")
>> Signed-off-by: Thierry Reding <treding@nvidia.com>
>> ---
>> .../bindings/display/bridge/lvds-codec.yaml | 43 ++++++++++---------
>> 1 file changed, 22 insertions(+), 21 deletions(-)
>>
>
> I went ahead and applied to drm-misc, so linux-next is fixed.
Thank you
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-12-22 19:17 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-20 12:51 [PATCH] dt-bindings: display: bridge: lvds-codec: Fix duplicate key Thierry Reding
2021-12-20 12:51 ` Thierry Reding
2021-12-21 2:12 ` Rob Herring
2021-12-21 2:12 ` Rob Herring
2021-12-22 18:03 ` Rob Herring
2021-12-22 18:03 ` Rob Herring
2021-12-22 19:17 ` Marek Vasut
2021-12-22 19:17 ` Marek Vasut
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.