Hi Fabio Estevam, yes, 7 is for DPI, please remove it.
Reviewed-by: Xin Ji <xji@xxxxxxxxxxxxxxxx>
> -----Original Message-----
> From: Fabio Estevam <festevam@xxxxxxxxx>
> Sent: Tuesday, November 28, 2023 8:31 AM
> To: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>; Xin Ji
> <xji@xxxxxxxxxxxxxxxx>
> Cc: sakari.ailus@xxxxxxxxxxxxxxx; rfoss@xxxxxxxxxx; robh+dt@xxxxxxxxxx;
> krzysztof.kozlowski+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; mchehab@xxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; Fabio Estevam
> <festevam@xxxxxxx>
> Subject: Re: [PATCH 2/2] media: dt-bindings: Use bus-types definitions from
> video-interfaces.h
>
> CAUTION: This email originated from outside of the organization. Please do not
> click links or open attachments unless you recognize the sender, and know the
> content is safe.
>
>
> Hi Laurent,
>
> On Mon, Nov 27, 2023 at 9:00 PM Laurent Pinchart
> <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote:
>
> > > @@ -157,7 +158,7 @@ examples:
> > > reg = <0>;
> > > anx7625_in: endpoint {
> > > remote-endpoint = <&mipi_dsi>;
> > > - bus-type = <7>;
> >
> > 7 seems to have been a mistake, as this is a DSI endpoint, not a DPI
> > endpoint.
>
> bus-type = <7> was added in commit 0a61ef9cc30d
> ("dt-bindings:drm/bridge:anx7625: add port@0 property").
>
> Adding Xin Ji on CC for clarification.
