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.