Hi Prabhakar, 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 ? It's not an issue for me. From a process perspective it might be better if we know that comment and this series gets merged in the same release to avoid 'holes', but if you're confident the BT.656 series will land at the same time please go ahead! > > Cheers, > Prabhakar > > > Renesas Electronics Europe GmbH, Geschaeftsfuehrer/President: Carsten Jauch, Sitz der Gesellschaft/Registered office: Duesseldorf, Arcadiastrasse 10, 40472 Duesseldorf, Germany, Handelsregister/Commercial Register: Duesseldorf, HRB 3708 USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE reg. no.: DE 14978647
