| Message ID | fef9412087f3961e03b77c067e227a31860a1417.1500368529.git.sean.wang@mediatek.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
On Tue, Jul 18, 2017 at 05:49:22PM +0800, sean.wang@mediatek.com wrote: > From: Sean Wang <sean.wang@mediatek.com> > > add dt-binding document for MediaTek MT6380 PMIC > > Signed-off-by: Chenglin Xu <chenglin.xu@mediatek.com> > Signed-off-by: Sean Wang <sean.wang@mediatek.com> > --- > .../bindings/regulator/mt6380-regulator.txt | 90 ++++++++++++++++++++++ > 1 file changed, 90 insertions(+) > create mode 100644 Documentation/devicetree/bindings/regulator/mt6380-regulator.txt > > diff --git a/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt b/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt > new file mode 100644 > index 0000000..2a559f0 > --- /dev/null > +++ b/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt > @@ -0,0 +1,90 @@ > +MediaTek MT6380 Regulator Driver Bindings don't describe drivers. > + > +All voltage regulators are defined as subnodes of the regulators node. A list What's the regulators node a child of? > +of regulators provided by this controller are defined as subnodes of the > +PMIC's node. Each regulator is named according to its regulator type, > +buck-<name> and ldo-<name>. The definition for each of these nodes is defined > +using the standard binding for regulators at > +Documentation/devicetree/bindings/regulator/regulator.txt. > + > +The valid names for regulators are: > +BUCK: > + buck-core1, buck-vcore, buck-vrf > +LDO: > + ldo-vm ,ldo-va , ldo-vphy, ldo-vddr, ldo-vt -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 2017-07-24 at 13:22 -0500, Rob Herring wrote: > On Tue, Jul 18, 2017 at 05:49:22PM +0800, sean.wang@mediatek.com wrote: > > From: Sean Wang <sean.wang@mediatek.com> > > > > add dt-binding document for MediaTek MT6380 PMIC > > > > Signed-off-by: Chenglin Xu <chenglin.xu@mediatek.com> > > Signed-off-by: Sean Wang <sean.wang@mediatek.com> > > --- > > .../bindings/regulator/mt6380-regulator.txt | 90 ++++++++++++++++++++++ > > 1 file changed, 90 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/regulator/mt6380-regulator.txt > > > > diff --git a/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt b/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt > > new file mode 100644 > > index 0000000..2a559f0 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt > > @@ -0,0 +1,90 @@ > > +MediaTek MT6380 Regulator Driver > > Bindings don't describe drivers. > will fix it. > > + > > +All voltage regulators are defined as subnodes of the regulators node. A list > > What's the regulators node a child of? > this two line i felt are a little confusion, i will change them into " all voltage regulators provided by the MT6380 PMIC are described as the subnodes of the mt6380regulators node " > > +of regulators provided by this controller are defined as subnodes of the > > +PMIC's node. Each regulator is named according to its regulator type, > > +buck-<name> and ldo-<name>. The definition for each of these nodes is defined > > +using the standard binding for regulators at > > +Documentation/devicetree/bindings/regulator/regulator.txt. > > + > > +The valid names for regulators are: > > +BUCK: > > + buck-core1, buck-vcore, buck-vrf > > +LDO: > > + ldo-vm ,ldo-va , ldo-vphy, ldo-vddr, ldo-vt -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt b/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt new file mode 100644 index 0000000..2a559f0 --- /dev/null +++ b/Documentation/devicetree/bindings/regulator/mt6380-regulator.txt @@ -0,0 +1,90 @@ +MediaTek MT6380 Regulator Driver + +All voltage regulators are defined as subnodes of the regulators node. A list +of regulators provided by this controller are defined as subnodes of the +PMIC's node. Each regulator is named according to its regulator type, +buck-<name> and ldo-<name>. The definition for each of these nodes is defined +using the standard binding for regulators at +Documentation/devicetree/bindings/regulator/regulator.txt. + +The valid names for regulators are: +BUCK: + buck-core1, buck-vcore, buck-vrf +LDO: + ldo-vm ,ldo-va , ldo-vphy, ldo-vddr, ldo-vt + +Example: + + regulators { + compatible = "mediatek,mt6380-regulator"; + + mt6380_vcpu_reg: buck-vcore1 { + regulator-name = "vcore1"; + regulator-min-microvolt = < 600000>; + regulator-max-microvolt = <1393750>; + regulator-ramp-delay = <6250>; + regulator-always-on; + regulator-boot-on; + }; + + mt6380_vcore_reg: buck-vcore { + regulator-name = "vcore"; + regulator-min-microvolt = <600000>; + regulator-max-microvolt = <1393750>; + regulator-ramp-delay = <6250>; + }; + + mt6380_vrf_reg: buck-vrf { + regulator-name = "vrf"; + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1575000>; + regulator-ramp-delay = <0>; + regulator-always-on; + regulator-boot-on; + }; + + mt6380_vm_reg: ldo-vm { + regulator-name = "vm"; + regulator-min-microvolt = <1050000>; + regulator-max-microvolt = <1400000>; + regulator-ramp-delay = <0>; + regulator-always-on; + regulator-boot-on; + }; + + mt6380_va_reg: ldo-va { + regulator-name = "va"; + regulator-min-microvolt = <2200000>; + regulator-max-microvolt = <3300000>; + regulator-ramp-delay = <0>; + regulator-always-on; + regulator-boot-on; + }; + + mt6380_vphy_reg: ldo-vphy { + regulator-name = "vphy"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-ramp-delay = <0>; + regulator-always-on; + regulator-boot-on; + }; + + mt6380_vddr_reg: ldo-vddr { + regulator-name = "vddr"; + regulator-min-microvolt = <1240000>; + regulator-max-microvolt = <1840000>; + regulator-ramp-delay = <0>; + regulator-always-on; + regulator-boot-on; + }; + + mt6380_vt_reg: ldo-vt { + regulator-name = "vt"; + regulator-min-microvolt = <2200000>; + regulator-max-microvolt = <3300000>; + regulator-ramp-delay = <0>; + regulator-always-on; + regulator-boot-on; + }; + };