On Fri, Sep 11, 2020 at 08:59:13AM +0000, Prabhakar Mahadev Lad wrote: > Hi Jacopo, > > > -----Original Message----- > > From: Jacopo Mondi <jacopo@xxxxxxxxxx> > > Sent: 11 September 2020 09:59 > > To: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > Cc: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>; robh+dt@xxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; Lad, > > Prabhakar <prabhakar.csengg@xxxxxxxxx>; mchehab@xxxxxxxxxx; hverkuil-cisco@xxxxxxxxx; laurent.pinchart@xxxxxxxxxxxxxxxx; linux- > > renesas-soc@xxxxxxxxxxxxxxx; Rob Herring <robh@xxxxxxxxxx>; Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > Subject: Re: [PATCH v5 2/3] dt-bindings: media: ov772x: Make bus-type mandatory > > > > Hi Sakari, > > > > On Fri, Sep 11, 2020 at 08:37:57AM +0300, Sakari Ailus wrote: > > > Hi Jacopo, > > > > > > On Thu, Sep 10, 2020 at 06:20:54PM +0200, Jacopo Mondi wrote: > > > > In order to establish required properties based on the selected > > > > bus type, make the 'bus-type' property mandatory. As this change > > > > documents an endpoint property, also document the 'remote-endpoint' > > > > one now that the 'endpoint' schema has been expanded. > > > > > > > > Binary compatibility with existing DTB is kept as the driver does not > > > > enforce the property to be present, and shall fall-back to default > > > > parallel bus configuration, which was the only supported bus type, if > > > > the property is not specified. > > > > > > Could you add a comment on this to the driver, so this feature isn't > > > accidentally removed? > > > > Sure, can I send a patch in reply to this series to avoid a v6 ? > If you don’t mind Ill handle this as part of bt656 additions ? Both are fine for me. -- Sakari Ailus
