Hi Benoit, Thank you for the patch. On Thu, Sep 12, 2019 at 1:58 PM Benoit Parrot <bparrot@xxxxxx> wrote: > > Add powerdown-gpios to the list of optional properties for the OV2659 > camera sensor. > > Signed-off-by: Benoit Parrot <bparrot@xxxxxx> > --- > Documentation/devicetree/bindings/media/i2c/ov2659.txt | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/i2c/ov2659.txt b/Documentation/devicetree/bindings/media/i2c/ov2659.txt > index cabc7d827dfb..f55204cce0cd 100644 > --- a/Documentation/devicetree/bindings/media/i2c/ov2659.txt > +++ b/Documentation/devicetree/bindings/media/i2c/ov2659.txt > @@ -12,6 +12,10 @@ Required Properties: > - clock-names: should be "xvclk". > - link-frequencies: target pixel clock frequency. > > +Optional Properties: > +- powerdown-gpios: reference to the GPIO connected to the pwdn pin, if any. > + Active is low. > + as per the datasheet this should be active high with pull-down resistor. Cheers, --Prabhakar Lad