Messages in this thread | | | From | Laurent Pinchart <> | Subject | Re: [PATCH v6 2/6] [media] ad5820: DT new optional field enable-gpios | Date | Tue, 02 Oct 2018 14:29:01 +0300 |
| |
Hi Ricardo,
Thank you for the patch.
On Tuesday, 2 October 2018 14:13:56 EEST Ricardo Ribalda Delgado wrote: > Document new enable-gpio field. It can be used to disable the part > without turning down its regulator. > > Cc: devicetree@vger.kernel.org > Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> > Acked-by: Pavel Machek <pavel@ucw.cz>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> --- > Documentation/devicetree/bindings/media/i2c/ad5820.txt | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/i2c/ad5820.txt > b/Documentation/devicetree/bindings/media/i2c/ad5820.txt index > 5940ca11c021..db596e8eb0ba 100644 > --- a/Documentation/devicetree/bindings/media/i2c/ad5820.txt > +++ b/Documentation/devicetree/bindings/media/i2c/ad5820.txt > @@ -8,6 +8,11 @@ Required Properties: > > - VANA-supply: supply of voltage for VANA pin > > +Optional properties: > + > + - enable-gpios : GPIO spec for the XSHUTDOWN pin. The XSHUTDOWN signal > is +active low, a high level on the pin enables the device. > + > Example: > > ad5820: coil@c { > @@ -15,5 +20,6 @@ Example: > reg = <0x0c>; > > VANA-supply = <&vaux4>; > + enable-gpios = <&msmgpio 26 GPIO_ACTIVE_HIGH>; > };
-- Regards,
Laurent Pinchart
| |