Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote on Sat [2019-Sep-14 11:11:02 +0100]: > 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. Ahhh, yes I'll fix that. > > Cheers, > --Prabhakar Lad
