On Tue, May 02, 2023 at 02:00:36PM +0200, Krzysztof Kozlowski wrote: > The panel-common schema does not define what "ports" property is, so > bring the definition to enforce the type. All panels described by > binding are dual-link, thus require both ports. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > > --- > > Changes since v1: > 1. Rework to add ports to device schema, not to panel-common. > https://lore.kernel.org/all/20230420160905.GA2952736-robh@xxxxxxxxxx/ > --- > .../bindings/display/panel/novatek,nt36523.yaml | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml > index 0039561ef04c..a2a8c4e39573 100644 > --- a/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml > +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt36523.yaml > @@ -32,8 +32,14 @@ properties: > vddio-supply: > description: regulator that supplies the I/O voltage > > + ports: > + $ref: /schemas/graph.yaml#/properties/ports > + > + required: > + - port@0 > + - port@1 What is each port? You need: additionalProperties: false properties: port@0: $ref: ... description: ... port@1: $ref: ... description: ...
