| Message ID | 1410722585-13393-4-git-send-email-heiko@sntech.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Sun, Sep 14, 2014 at 09:23:03PM +0200, Heiko Stuebner wrote: > +Optional properties: > + - fairchild,suspend-regulator: regulator number to use for suspend voltages > + possible values are either <0> or <1> I'm not sure what this means - What exactly is a "regulator" in this context? Are there really two separate regulators?
Am Montag, 15. September 2014, 14:54:10 schrieb Mark Brown: > On Sun, Sep 14, 2014 at 09:23:03PM +0200, Heiko Stuebner wrote: > > +Optional properties: > > + - fairchild,suspend-regulator: regulator number to use for suspend > > voltages + possible values are either <0> or <1> > > I'm not sure what this means - What exactly is a "regulator" in this > context? Are there really two separate regulators? The fan53555 has two voltage selection registers (VSEL0 and VSEL1) which can be switched through a gpio ... to use one as suspend voltage. So it's just one regulator output but can select one of two voltages.
On Tue, Sep 16, 2014 at 12:14:43AM +0200, Heiko Stübner wrote: > Am Montag, 15. September 2014, 14:54:10 schrieb Mark Brown: > > I'm not sure what this means - What exactly is a "regulator" in this > > context? Are there really two separate regulators? > The fan53555 has two voltage selection registers (VSEL0 and VSEL1) which can > be switched through a gpio ... to use one as suspend voltage. > So it's just one regulator output but can select one of two voltages. OK, that's what I thought was going on - please fix this to be described in terms of voltage selector registers rather than regulators as what's there now seems confusing.
diff --git a/Documentation/devicetree/bindings/regulator/fan53555.txt b/Documentation/devicetree/bindings/regulator/fan53555.txt new file mode 100644 index 0000000..2484497 --- /dev/null +++ b/Documentation/devicetree/bindings/regulator/fan53555.txt @@ -0,0 +1,25 @@ +Binding for Fairchild FAN53555 regulators + +Required properties: + - compatible: "fairchild,fan53555" + - reg: I2C address + +Optional properties: + - fairchild,suspend-regulator: regulator number to use for suspend voltages + possible values are either <0> or <1> + - fairchild,slew-rate-microvolt: slew rate to use for positive voltage + transistions in microvolts per us, possible values are: + 64000, 32000, 16000, 8000, 4000, 2000, 1000, 500 + - vin-supply: regulator supplying the vin pin + +Example: + + regulator@40 { + compatible = "fairchild,fan53555"; + regulator-name = "fan53555"; + regulator-min-microvolt = <1000000>; + regulator-max-microvolt = <1800000>; + vin-supply = <&parent_reg>; + fairchild,suspend-regulator = <1>; + fairchild,slew-rate-microvolt = <64000>; + }; diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt index ac7269f..99da41b 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.txt +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt @@ -49,6 +49,7 @@ epson Seiko Epson Corp. est ESTeem Wireless Modems eukrea Eukréa Electromatique excito Excito +fairchild Fairchild Semiconductor fsl Freescale Semiconductor GEFanuc GE Fanuc Intelligent Platforms Embedded Systems, Inc. gef GE Fanuc Intelligent Platforms Embedded Systems, Inc.
Document the regulator and add a fairchild vendor-prefix. Signed-off-by: Heiko Stuebner <heiko@sntech.de> --- .../devicetree/bindings/regulator/fan53555.txt | 25 ++++++++++++++++++++++ .../devicetree/bindings/vendor-prefixes.txt | 1 + 2 files changed, 26 insertions(+) create mode 100644 Documentation/devicetree/bindings/regulator/fan53555.txt