On 09/05/2023 21:10, Fabio Estevam wrote: > From: Fabio Estevam <festevam@xxxxxxx> > > The Samsung DSIM IP block allows the inversion of the clock and > data lanes. > > Add an optional property called 'lane-polarities' that describes the > polarities of the MIPI DSI clock and data lanes. > > This property is useful for properly describing the hardware when the > board designer decided to switch the polarities of the MIPI DSI > clock and/or data lanes. > > Signed-off-by: Fabio Estevam <festevam@xxxxxxx> > --- > Changes since v2: > - Use video-interfaces.yaml (Rob). > > .../display/bridge/samsung,mipi-dsim.yaml | 29 +++++++++++++++++++ > 1 file changed, 29 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml b/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml > index e841659e20cd..dad6d06fbdd9 100644 > --- a/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml > @@ -105,6 +105,35 @@ properties: > DSI output port node to the panel or the next bridge > in the chain. > > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# > + unevaluatedProperties: false > + > + properties: > + data-lanes: > + oneOf: Drop oneOf, no need for it. > + - minItems: 1 > + maxItems: 4 > + uniqueItems: true > + items: > + enum: [ 1, 2, 3, 4 ] > + description: > + See ../../media/video-interfaces.yaml for details. Drop description, it's obvious. > + > + lane-polarities: > + minItems: 1 > + maxItems: 5 > + items: > + enum: [ 0, 1 ] Drop items, it's already in video-interfaces. > + description: > + See ../../media/video-interfaces.yaml for details. Drop this piece. > + The Samsung MIPI DSI IP requires that all the data lanes have > + the same polarity. > + > + dependencies: > + lane-polarities: [data-lanes] > + > required: > - clock-names > - clocks Best regards, Krzysztof