| Message ID | 20230509191059.3327960-1-festevam@gmail.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | [v3,1/2] dt-bindings: samsung,mipi-dsim: Add 'lane-polarities' | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dt-meta-schema | fail | build log |
On 09/05/2023 21:10, Fabio Estevam wrote: > From: Fabio Estevam <festevam@denx.de> > > 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@denx.de> > --- > 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
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: + - minItems: 1 + maxItems: 4 + uniqueItems: true + items: + enum: [ 1, 2, 3, 4 ] + description: + See ../../media/video-interfaces.yaml for details. + + lane-polarities: + minItems: 1 + maxItems: 5 + items: + enum: [ 0, 1 ] + description: + See ../../media/video-interfaces.yaml for details. + The Samsung MIPI DSI IP requires that all the data lanes have + the same polarity. + + dependencies: + lane-polarities: [data-lanes] + required: - clock-names - clocks