Hello Prabhakar, On Sat, Aug 15, 2020 at 11:34:33AM +0100, Lad, Prabhakar wrote: > Hi Jacopo, > > On Mon, Aug 3, 2020 at 12:39 PM Lad Prabhakar > <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > > > > Document endpoint properties required for parallel interface > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > --- > > .../devicetree/bindings/media/i2c/ov772x.txt | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > I see you already have a patch for YAML conversion for OV772x binding > [1], if you plan to post a v2 would you be OK to pick these changes as > part of your conversion changes ? Sure thing, I'll add the following properties to the series! Thanks j > > [1] https://www.spinics.net/lists/linux-media/msg173201.html > > Cheers, > Prabhakar > > > diff --git a/Documentation/devicetree/bindings/media/i2c/ov772x.txt b/Documentation/devicetree/bindings/media/i2c/ov772x.txt > > index 0b3ede5b8e6a..1f4153484717 100644 > > --- a/Documentation/devicetree/bindings/media/i2c/ov772x.txt > > +++ b/Documentation/devicetree/bindings/media/i2c/ov772x.txt > > @@ -21,6 +21,22 @@ subnode for its digital output video port, in accordance with the video > > interface bindings defined in Documentation/devicetree/bindings/media/ > > video-interfaces.txt. > > > > +Endpoint node required properties for parallel connection are: > > +- remote-endpoint: a phandle to the bus receiver's endpoint node. > > +- bus-width: shall be set to <8> for 8 bits parallel bus > > + or <10> for 10 bits parallel bus > > +- data-shift: shall be set to <2> for 8 bits parallel bus > > + (lines 9:2 are used) or <0> for 10 bits parallel bus > > +- hsync-active: active state of the HSYNC signal, 0/1 for LOW/HIGH respectively. > > + (Not required for bus-type equal 6) > > +- vsync-active: active state of the VSYNC signal, 0/1 for LOW/HIGH respectively. > > + (Not required for bus-type equal 6) > > +- pclk-sample: sample data on rising (1) or falling (0) edge of the pixel clock > > + signal. (Not required for bus-type equal 6) > > +- bus-type: data bus type. Possible values are: > > + 5 - Parallel > > + 6 - Bt.656 > > + > > Example: > > > > &i2c0 { > > -- > > 2.17.1 > >
