RE: [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux