diff mbox

[6/6] dt-bindings: pwm-regulator: Document enable-gpio property

Message ID 20160622082558.20935-7-acourbot@nvidia.com
State Changes Requested, archived
Headers show

Commit Message

Alexandre Courbot June 22, 2016, 8:25 a.m. UTC
Add and document the enable-gpio property. Its behavior is similar to
the property of the same name found in GPIO regulator and fixed
regulator.

Cc: devicetree@vger.kernel.org
Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
---
 Documentation/devicetree/bindings/regulator/pwm-regulator.txt | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Comments

Mark Brown June 22, 2016, 10:36 a.m. UTC | #1
On Wed, Jun 22, 2016 at 05:25:58PM +0900, Alexandre Courbot wrote:
> Add and document the enable-gpio property. Its behavior is similar to
> the property of the same name found in GPIO regulator and fixed
> regulator.

Please submit patches using subject lines reflecting the style for the
subsystem.  This makes it easier for people to identify relevant
patches.

> +Optional properties:
> +--------------------
> +- enable-gpio:		GPIO to use to enable/disable the regulator
> +

The DT people are going to want this to be -gpios.
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/regulator/pwm-regulator.txt b/Documentation/devicetree/bindings/regulator/pwm-regulator.txt
index ed936f0f34f2..4f9dd27f52f5 100644
--- a/Documentation/devicetree/bindings/regulator/pwm-regulator.txt
+++ b/Documentation/devicetree/bindings/regulator/pwm-regulator.txt
@@ -38,13 +38,18 @@  NB: To be clear, if voltage-table is provided, then the device will be used
 in Voltage Table Mode.  If no voltage-table is provided, then the device will
 be used in Continuous Voltage Mode.
 
+Optional properties:
+--------------------
+- enable-gpio:		GPIO to use to enable/disable the regulator
+
 Any property defined as part of the core regulator binding can also be used.
 (See: ../regulator/regulator.txt)
 
-Continuous Voltage Example:
+Continuous Voltage With Enable GPIO Example:
 	pwm_regulator {
 		compatible = "pwm-regulator;
 		pwms = <&pwm1 0 8448 0>;
+		enable-gpio = <&gpio0 23 GPIO_ACTIVE_HIGH>;
 		regulator-min-microvolt = <1016000>;
 		regulator-max-microvolt = <1114000>;
 		regulator-name = "vdd_logic";