diff mbox series

dt-bindings: display: Fix graph 'unevaluatedProperties' related warnings

Message ID 20210719195001.2412345-1-robh@kernel.org
State Accepted
Commit b4db237e1e23489fdde1fb195e687b50d0ec162f
Headers show
Series dt-bindings: display: Fix graph 'unevaluatedProperties' related warnings | expand

Commit Message

Rob Herring July 19, 2021, 7:50 p.m. UTC
The graph schema doesn't allow custom properties on endpoint nodes for
'#/properties/port' and '#/$defs/port-base' should be used instead. This
doesn't matter until 'unevaluatedProperties' support is implemented.

Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Sean Paul <sean@poorly.run>
Cc: Marek Vasut <marex@denx.de>
Cc: Krishna Manikandan <mkrishn@codeaurora.org>
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
 .../devicetree/bindings/display/bridge/ti,sn65dsi83.yaml    | 6 ++++--
 .../bindings/display/msm/dsi-controller-main.yaml           | 6 ++++--
 2 files changed, 8 insertions(+), 4 deletions(-)

Comments

Rob Herring July 23, 2021, 9:17 p.m. UTC | #1
On Mon, 19 Jul 2021 13:50:01 -0600, Rob Herring wrote:
> The graph schema doesn't allow custom properties on endpoint nodes for

> '#/properties/port' and '#/$defs/port-base' should be used instead. This

> doesn't matter until 'unevaluatedProperties' support is implemented.

> 

> Cc: David Airlie <airlied@linux.ie>

> Cc: Daniel Vetter <daniel@ffwll.ch>

> Cc: Rob Clark <robdclark@gmail.com>

> Cc: Sean Paul <sean@poorly.run>

> Cc: Marek Vasut <marex@denx.de>

> Cc: Krishna Manikandan <mkrishn@codeaurora.org>

> Cc: dri-devel@lists.freedesktop.org

> Signed-off-by: Rob Herring <robh@kernel.org>

> ---

>  .../devicetree/bindings/display/bridge/ti,sn65dsi83.yaml    | 6 ++++--

>  .../bindings/display/msm/dsi-controller-main.yaml           | 6 ++++--

>  2 files changed, 8 insertions(+), 4 deletions(-)

> 


Applied, thanks!
Rob Herring July 23, 2021, 9:25 p.m. UTC | #2
On Mon, Jul 19, 2021 at 1:57 PM Sam Ravnborg <sam@ravnborg.org> wrote:
>

> Hi Rob,

> On Mon, Jul 19, 2021 at 01:50:01PM -0600, Rob Herring wrote:

> > The graph schema doesn't allow custom properties on endpoint nodes for

> > '#/properties/port' and '#/$defs/port-base' should be used instead. This

> > doesn't matter until 'unevaluatedProperties' support is implemented.

> >

> > Cc: David Airlie <airlied@linux.ie>

> > Cc: Daniel Vetter <daniel@ffwll.ch>

> > Cc: Rob Clark <robdclark@gmail.com>

> > Cc: Sean Paul <sean@poorly.run>

> > Cc: Marek Vasut <marex@denx.de>

> > Cc: Krishna Manikandan <mkrishn@codeaurora.org>

> > Cc: dri-devel@lists.freedesktop.org

> > Signed-off-by: Rob Herring <robh@kernel.org>

> Acked-by: Sam Ravnborg <sam@ravnborg.org>


Thanks. FYI, your mail only went thru on dri-devel, but not DT or LKML lists.

Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
index d101233ae17f..07b20383cbca 100644
--- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
@@ -37,7 +37,8 @@  properties:
 
     properties:
       port@0:
-        $ref: /schemas/graph.yaml#/properties/port
+        $ref: /schemas/graph.yaml#/$defs/port-base
+        unevaluatedProperties: false
         description: Video port for MIPI DSI Channel-A input
 
         properties:
@@ -57,7 +58,8 @@  properties:
                   - const: 4
 
       port@1:
-        $ref: /schemas/graph.yaml#/properties/port
+        $ref: /schemas/graph.yaml#/$defs/port-base
+        unevaluatedProperties: false
         description: Video port for MIPI DSI Channel-B input
 
         properties:
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 76348b71f736..de23cab9b5f1 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -77,7 +77,8 @@  properties:
 
     properties:
       port@0:
-        $ref: "/schemas/graph.yaml#/properties/port"
+        $ref: "/schemas/graph.yaml#/$defs/port-base"
+        unevaluatedProperties: false
         description: |
           Input endpoints of the controller.
         properties:
@@ -92,7 +93,8 @@  properties:
                   enum: [ 0, 1, 2, 3 ]
 
       port@1:
-        $ref: "/schemas/graph.yaml#/properties/port"
+        $ref: "/schemas/graph.yaml#/$defs/port-base"
+        unevaluatedProperties: false
         description: |
           Output endpoints of the controller.
         properties: