Re: [Patch v4 5/8] media: dt-bindings: ov2659: add powerdown/reset-gpios optional property

[Lad, Prabhakar]

On Fri, Sep 27, 2019 at 7:47 PM Benoit Parrot <bparrot@xxxxxx> wrote:
>
> Add powerdown-gpios and reset-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 | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
Acked-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx>

Cheers,
--Prabhakar Lad

> diff --git a/Documentation/devicetree/bindings/media/i2c/ov2659.txt b/Documentation/devicetree/bindings/media/i2c/ov2659.txt
> index cabc7d827dfb..92989a619f29 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ov2659.txt
> +++ b/Documentation/devicetree/bindings/media/i2c/ov2659.txt
> @@ -12,6 +12,12 @@ 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 high with internal pull down resistor.
> +- reset-gpios: reference to the GPIO connected to the resetb pin, if any.
> +  Active low with internal pull up resistor.
> +
>  For further reading on port node refer to
>  Documentation/devicetree/bindings/media/video-interfaces.txt.
>
> @@ -27,6 +33,9 @@ Example:
>                         clocks = <&clk_ov2659 0>;
>                         clock-names = "xvclk";
>
> +                       powerdown-gpios = <&gpio6 14 GPIO_ACTIVE_HIGH>;
> +                       reset-gpios = <&gpio6 15 GPIO_ACTIVE_LOW>;
> +
>                         port {
>                                 ov2659_0: endpoint {
>                                         remote-endpoint = <&vpfe_ep>;
> --
> 2.17.1
>