Re: [PATCH 1/2] dt-bindings: display: sn65dsi83: Add reverse lanes property

[Laurent Pinchart]

On Thu, Jul 09, 2026 at 10:33:36AM +0200, Alexander Stein wrote:
> Am Dienstag, 7. Juli 2026, 09:02:42 CEST schrieb Wojciech Dubowik:
> > From: Wojciech Dubowik <Wojciech.Dubowik@xxxxxx>
> > 
> > Add reverse lvds lanes property for endpoints. It reverses
> > the data-lanes order i.e. <4 3 2 1> instead of default
> > <1 2 3 4>.
> 
> Wouldn't it be more reasonable to parse the data-lanes property and check
> for the order instead of introducing a new property?

I think so. That's the intended usage of data-lanes, the property is
meant to convey the order of lanes in addition to the count. Otherwise
we would have used a data-lanes-count property instead.

> > Signed-off-by: Wojciech Dubowik <Wojciech.Dubowik@xxxxxx>
> > ---
> >  .../devicetree/bindings/display/bridge/ti,sn65dsi83.yaml      | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> > index e69b6343a8eb..44483e808af9 100644
> > --- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> > +++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> > @@ -115,6 +115,10 @@ $defs:
> >              enum: [100, 200]
> >              default: 200
> >  
> > +          ti,lvds-reverse-lanes:
> > +            description: Reverse the order of LVDS lanes.
> > +            type: boolean
> > +
> >            ti,lvds-vod-swing-clock-microvolt:
> >              description: LVDS diferential output voltage <min max> for clock
> >                lanes in microvolts.

-- 
Regards,

Laurent Pinchart