On Mon, Oct 19, 2026 at 07:17:14AM -0300, Leonardo Costa wrote: > From: Leonardo Costa <[email protected]> > > Some simple VGA DACs may include a data-enable input that must be driven > at a specific polarity. Add the data-enable-active property from > video-interfaces to the input endpoint to describe that polarity. > > Signed-off-by: Leonardo Costa <[email protected]> > --- > .../bindings/display/bridge/simple-bridge.yaml | 28 > +++++++++++++++++++++- > 1 file changed, 27 insertions(+), 1 deletion(-) > > diff --git > a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml > b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml > index 7636c24906ba..b1b23d41245d 100644 > --- a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml > @@ -43,9 +43,18 @@ properties: > > properties: > port@0: > - $ref: /schemas/graph.yaml#/properties/port > + $ref: /schemas/graph.yaml#/$defs/port-base > + unevaluatedProperties: false > description: The bridge input > > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml > + unevaluatedProperties: false > + > + properties: > + data-enable-active: true > +
suppose needn't it because unevaluatedProperties: false, which allow all property in video-interfaces.yaml. If you only want to enable one property, should be additionalProperties: false but I think unevaluatedProperties: false make sense and needn't below if branch. Frank > port@1: > $ref: /schemas/graph.yaml#/properties/port > description: The bridge output > @@ -65,6 +74,23 @@ required: > - compatible > - ports > > +allOf: > + - if: > + not: > + properties: > + compatible: > + contains: > + const: dumb-vga-dac > + then: > + properties: > + ports: > + properties: > + port@0: > + properties: > + endpoint: > + properties: > + data-enable-active: false > + > additionalProperties: false > > examples: > > -- > 2.43.0 > >
