Re: [PATCH v2 1/3] dt-bindings: display: bridge: analogix-dp: Add data-lanes support for endpoint

[Damon Ding]

On 5/22/2026 3:50 AM, Conor Dooley wrote:
> On Thu, May 21, 2026 at 07:44:57PM +0800, Damon Ding wrote:
> > Add data-lanes property support to the port@1 endpoint for physical
> > lane mapping configuration.
> >
> > Lane mapping is mainly used for below scenarios:
> > 1. Correct PCB lane swap and differential line routing crossover
> >     without hardware changes;
> > 2. Adapt mismatched lane pin definitions between SoC and eDP panel;
> > 3. Support multiple panel hardware variants on the same board
> >     by configuring data-lanes in device tree only.
> >
> > Signed-off-by: Damon Ding <damon.ding@xxxxxxxxxxxxxx>
>
> Sashiko complaint here looks valid.
> pw-bot: changes-requested

Yes, unevaluatedProperties: false should be added to both the port@1
and endpoint nodes.

Will fix in v3.

Best regards,
Damon

> > ---
> >
> > Changes in v2:
> > - Add lane mapping application scenarios in commit message.
> > - Remove redundant deprecated property 'data-lanes' for eDP node.
> > - Update port@1 $ref to /schemas/graph.yaml#/$defs/port-base.
> > ---
> >   .../bindings/display/bridge/analogix,dp.yaml    | 17 ++++++++++++-----
> >   1 file changed, 12 insertions(+), 5 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/display/bridge/analogix,dp.yaml b/Documentation/devicetree/bindings/display/bridge/analogix,dp.yaml
> > index 62f0521b0924..e34fdb21adb4 100644
> > --- a/Documentation/devicetree/bindings/display/bridge/analogix,dp.yaml
> > +++ b/Documentation/devicetree/bindings/display/bridge/analogix,dp.yaml
> > @@ -42,13 +42,20 @@ properties:
> >       properties:
> >         port@0:
> >           $ref: /schemas/graph.yaml#/properties/port
> > -        description:
> > -          Input node to receive pixel data.
> > +        description: Input node to receive pixel data.
> >   
> >         port@1:
> > -        $ref: /schemas/graph.yaml#/properties/port
> > -        description:
> > -          Port node with one endpoint connected to a dp-connector node.
> > +        $ref: /schemas/graph.yaml#/$defs/port-base
> > +        description: Port node with one endpoint connected to sink device node.
> > +        properties:
> > +          endpoint:
> > +            $ref: /schemas/media/video-interfaces.yaml#
> > +            properties:
> > +              data-lanes:
> > +                minItems: 1
> > +                maxItems: 4
> > +                items:
> > +                  enum: [ 0, 1, 2, 3 ]
> >   
> >       required:
> >         - port@0
> > --
> > 2.34.1