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

[Wojciech Dubowik <Wojciech.Dubowik@mt.com>]

On Thu, Jul 09, 2026 at 12:17:05PM +0300, Laurent Pinchart wrote:
> 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@mt.com>
> > > 
> > > 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.

I will send v2 with data-lanes parsing instead of adding new dt property.
Regards,
Wojtek

> 
> > > Signed-off-by: Wojciech Dubowik <Wojciech.Dubowik@mt.com>
> > > ---
> > >  .../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