On Mon, Jul 13, 2026 at 10:26:19AM +0200, Wojciech Dubowik wrote: > From: Wojciech Dubowik <[email protected]> > > Add an optional output lvds data lanes property with > two allowed values. The array <1 2 3 4> for standard > layout and <4 3 2 1> for reversed lvds output lanes. > The latter informs the driver that reverse lvds config > option has to be set in config register for the respective > output channel.
Please wrap commit message according to Linux coding style / submission process (neither too early nor over the limit): https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597 > > Signed-off-by: Wojciech Dubowik <[email protected]> > --- > .../bindings/display/bridge/ti,sn65dsi83.yaml | 52 +++++++++++++++++++ > 1 file changed, 52 insertions(+) > > diff --git > a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > index e69b6343a8eb..2693c0273f9a 100644 > --- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > @@ -86,10 +86,62 @@ properties: > description: Video port for LVDS Channel-A output (panel or bridge). > $ref: '#/$defs/lvds-port' > > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# > + unevaluatedProperties: false > + > + properties: > + data-lanes: > + $ref: /schemas/types.yaml#/definitions/uint32-array > + description: | Drop | > + Array of physical LVDS data lane indexes > + oneOf: > + - items: > + - const: 1 > + - const: 2 > + - const: 3 > + - const: 4 > + description: | As well > + This is the default layout > + - items: > + - const: 4 > + - const: 3 > + - const: 2 > + - const: 1 > + description: | Ditto > + This is the reversed layout Honestly, this previous description are redundant. I can read 4-3-2-1 and I do understand it is a reversed from 1-2-3-4. Do not explain the obvious code, but rather design behind. Just drop the descriptions except the first one. Reviewed-by: Krzysztof Kozlowski <[email protected]> Best regards, Krzysztof
