OK, thanks for all Sakari ! On 01/03/2018 10:10 AM, Sakari Ailus wrote: > On Wed, Jan 03, 2018 at 08:47:09AM +0000, Hugues FRUCHET wrote: >> Hi Sakari, >> this is fine for me to drop those two lines so sync signals become >> mandatory. >> Must I repost a v5 serie ? > > Here's the diff: > > diff --git a/Documentation/devicetree/bindings/media/i2c/ov5640.txt b/Documentation/devicetree/bindings/media/i2c/ov5640.txt > index e26a84646603..8e36da0d8406 100644 > --- a/Documentation/devicetree/bindings/media/i2c/ov5640.txt > +++ b/Documentation/devicetree/bindings/media/i2c/ov5640.txt > @@ -31,8 +31,6 @@ Endpoint node required properties for parallel connection are: > 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 > - > -Endpoint node optional properties for parallel connection are: > - hsync-active: active state of the HSYNC signal, 0/1 for LOW/HIGH respectively. > - vsync-active: active state of the VSYNC signal, 0/1 for LOW/HIGH respectively. > - pclk-sample: sample data on rising (1) or falling (0) edge of the pixel clock >