lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 25 Aug 2022 11:26:01 +0300 From: Laurent Pinchart <laurent.pinchart@...asonboard.com> To: Rob Herring <robh@...nel.org> Cc: Andrzej Hajda <andrzej.hajda@...el.com>, Neil Armstrong <narmstrong@...libre.com>, Robert Foss <robert.foss@...aro.org>, Jonas Karlman <jonas@...boo.se>, Jernej Skrabec <jernej.skrabec@...il.com>, David Airlie <airlied@...ux.ie>, Daniel Vetter <daniel@...ll.ch>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, dri-devel@...ts.freedesktop.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] dt-bindings: display: adi,adv75xx: Add missing graph schema references Hi Rob, Thank you for the patch. On Tue, Aug 23, 2022 at 09:56:43AM -0500, Rob Herring wrote: > DT bindings using the graph binding must have references to the graph > binding schema. These are missing from the adi,adv7511 and adi,adv7533 > bindings, so add them. > > Signed-off-by: Rob Herring <robh@...nel.org> Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com> > --- > .../bindings/display/bridge/adi,adv7511.yaml | 14 ++++++-------- > .../bindings/display/bridge/adi,adv7533.yaml | 14 ++++++-------- > 2 files changed, 12 insertions(+), 16 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml b/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml > index f08a01dfedf3..5bbe81862c8f 100644 > --- a/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml > @@ -117,23 +117,21 @@ properties: > > ports: > description: > - The ADV7511(W)/13 has two video ports and one audio port. This node > - models their connections as documented in > - Documentation/devicetree/bindings/media/video-interfaces.txt > - Documentation/devicetree/bindings/graph.txt > - type: object > + The ADV7511(W)/13 has two video ports and one audio port. > + $ref: /schemas/graph.yaml#/properties/ports > + > properties: > port@0: > description: Video port for the RGB or YUV input. > - type: object > + $ref: /schemas/graph.yaml#/properties/port > > port@1: > description: Video port for the HDMI output. > - type: object > + $ref: /schemas/graph.yaml#/properties/port > > port@2: > description: Audio port for the HDMI output. > - type: object > + $ref: /schemas/graph.yaml#/properties/port > > # adi,input-colorspace and adi,input-clock are required except in > # "rgb 1x" and "yuv444 1x" modes, in which case they must not be > diff --git a/Documentation/devicetree/bindings/display/bridge/adi,adv7533.yaml b/Documentation/devicetree/bindings/display/bridge/adi,adv7533.yaml > index f36209137c8a..987aa83c2649 100644 > --- a/Documentation/devicetree/bindings/display/bridge/adi,adv7533.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/adi,adv7533.yaml > @@ -91,25 +91,23 @@ properties: > > ports: > description: > - The ADV7533/35 has two video ports and one audio port. This node > - models their connections as documented in > - Documentation/devicetree/bindings/media/video-interfaces.txt > - Documentation/devicetree/bindings/graph.txt > - type: object > + The ADV7533/35 has two video ports and one audio port. > + $ref: /schemas/graph.yaml#/properties/ports > + > properties: > port@0: > description: > Video port for the DSI input. The remote endpoint phandle > should be a reference to a valid mipi_dsi_host_device. > - type: object > + $ref: /schemas/graph.yaml#/properties/port > > port@1: > description: Video port for the HDMI output. > - type: object > + $ref: /schemas/graph.yaml#/properties/port > > port@2: > description: Audio port for the HDMI output. > - type: object > + $ref: /schemas/graph.yaml#/properties/port > > required: > - compatible -- Regards, Laurent Pinchart
Powered by blists - more mailing lists