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 ? 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
