| Message ID | 20230807143515.7882-3-clamor95@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v3,1/5] ARM: dts: tegra: Drop unit-address from parallel RGB output port | expand |
On Mon, Aug 07, 2023 at 05:35:12PM +0300, Svyatoslav Ryhel wrote: > From: Maxim Schwalm <maxim.schwalm@gmail.com> > > Either this node, which is optional, or the nvidia,panel property can be > present. > > Signed-off-by: Maxim Schwalm <maxim.schwalm@gmail.com> > Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com> > --- > .../display/tegra/nvidia,tegra20-dc.yaml | 31 +++++++++++++++++++ > 1 file changed, 31 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml > index 69be95afd562..102304703062 100644 > --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml > +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml > @@ -127,6 +127,37 @@ allOf: > $ref: /schemas/types.yaml#/definitions/phandle > description: phandle of a display panel > > + port: > + $ref: /schemas/graph.yaml#/$defs/port-base > + description: Parallel RGB output port > + > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# Just to make sure, what properties are you using from this? Usually we'll list them though not a hard requirement. If none, then you just need to ref graph.yaml#/properties/port instead and can drop the rest. > + unevaluatedProperties: false > + > + unevaluatedProperties: false In the indented cases, it's easier to read if this is before properties/patternProperties. > + > + anyOf: > + - if: > + not: > + properties: > + nvidia,panel: false > + then: > + not: > + properties: > + port: true > + - if: > + not: > + properties: > + port: false > + then: > + not: > + properties: > + nvidia,panel: true I would prefer to drop this and mark "nvidia,panel" as deprecated. Eventually I plan to add a mode to the tools to warn on using deprecated properties. Having both could be perfectly fine too. You have the "nvidia,panel" for compatibility with an old OS version and 'port' to work with newer users. > + > + additionalProperties: false Move this up too. Rob
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml index 69be95afd562..102304703062 100644 --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-dc.yaml @@ -127,6 +127,37 @@ allOf: $ref: /schemas/types.yaml#/definitions/phandle description: phandle of a display panel + port: + $ref: /schemas/graph.yaml#/$defs/port-base + description: Parallel RGB output port + + properties: + endpoint: + $ref: /schemas/media/video-interfaces.yaml# + unevaluatedProperties: false + + unevaluatedProperties: false + + anyOf: + - if: + not: + properties: + nvidia,panel: false + then: + not: + properties: + port: true + - if: + not: + properties: + port: false + then: + not: + properties: + nvidia,panel: true + + additionalProperties: false + - if: properties: compatible: