diff mbox

[V7,1/2] regulator: Add document for MT6323 regulator

Message ID 1456396631-12788-1-git-send-email-blogic@openwrt.org (mailing list archive)
State New, archived
Headers show

Commit Message

John Crispin Feb. 25, 2016, 10:37 a.m. UTC
Signed-off-by: John Crispin <blogic@openwrt.org>
Cc: devicetree@vger.kernel.org
---
This patch needs to be merged post-rc1 as it depends on the MFD series of
the PMIC which is already inside Lee's tree pending the PR.

Changes in V7
* remove another compatible string
* small wording changes

Changes in V6
* remove the compatible string

Changes in V5
* regulator vs regulators

Changes in V4
* regulator is a subnode and not a property

 .../bindings/regulator/mt6323-regulator.txt        |  239 ++++++++++++++++++++
 1 file changed, 239 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/regulator/mt6323-regulator.txt

Comments

Rob Herring (Arm) March 2, 2016, 6:07 p.m. UTC | #1
On Thu, Feb 25, 2016 at 11:37:10AM +0100, John Crispin wrote:
> Signed-off-by: John Crispin <blogic@openwrt.org>
> Cc: devicetree@vger.kernel.org
> ---
> This patch needs to be merged post-rc1 as it depends on the MFD series of
> the PMIC which is already inside Lee's tree pending the PR.

Why? It is an indpendent change with no actual dependency.

> 
> Changes in V7
> * remove another compatible string
> * small wording changes
> 
> Changes in V6
> * remove the compatible string
> 
> Changes in V5
> * regulator vs regulators
> 
> Changes in V4
> * regulator is a subnode and not a property
> 
>  .../bindings/regulator/mt6323-regulator.txt        |  239 ++++++++++++++++++++
>  1 file changed, 239 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/regulator/mt6323-regulator.txt

Acked-by: Rob Herring <robh@kernel.org>

> 
> diff --git a/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
> new file mode 100644
> index 0000000..9fd95e7
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
> @@ -0,0 +1,239 @@
> +Mediatek MT6323 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_vproc, buck_vsys, buck_vpa
> +LDO:
> +  ldo_vtcxo, ldo_vcn28, ldo_vcn33_bt, ldo_vcn33_wifi, ldo_va, ldo_vcama,
> +  ldo_vio28, ldo_vusb, ldo_vmc, ldo_vmch, ldo_vemc3v3, ldo_vgp1, ldo_vgp2,
> +  ldo_vgp3, ldo_vcn18, ldo_vsim1, ldo_vsim2, ldo_vrtc, ldo_vcamaf, ldo_vibr,
> +  ldo_vrf18, ldo_vm, ldo_vio18, ldo_vcamd, ldo_vcamio
> +
> +Example:
> +
> +	pmic: mt6323 {
> +		compatible = "mediatek,mt6323";
> +
> +		mt6323regulator: regulators {
> +			mt6323_vproc_reg: buck_vproc{
> +				regulator-name = "vproc";
> +				regulator-min-microvolt = < 700000>;
> +				regulator-max-microvolt = <1350000>;
> +				regulator-ramp-delay = <12500>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vsys_reg: buck_vsys{
> +				regulator-name = "vsys";
> +				regulator-min-microvolt = <1400000>;
> +				regulator-max-microvolt = <2987500>;
> +				regulator-ramp-delay = <25000>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vpa_reg: buck_vpa{
> +				regulator-name = "vpa";
> +				regulator-min-microvolt = < 500000>;
> +				regulator-max-microvolt = <3650000>;
> +			};
> +
> +			mt6323_vtcxo_reg: ldo_vtcxo{
> +				regulator-name = "vtcxo";
> +				regulator-min-microvolt = <2800000>;
> +				regulator-max-microvolt = <2800000>;
> +				regulator-enable-ramp-delay = <90>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vcn28_reg: ldo_vcn28{
> +				regulator-name = "vcn28";
> +				regulator-min-microvolt = <2800000>;
> +				regulator-max-microvolt = <2800000>;
> +				regulator-enable-ramp-delay = <185>;
> +			};
> +
> +			mt6323_vcn33_bt_reg: ldo_vcn33_bt{
> +				regulator-name = "vcn33_bt";
> +				regulator-min-microvolt = <3300000>;
> +				regulator-max-microvolt = <3600000>;
> +				regulator-enable-ramp-delay = <185>;
> +			};
> +
> +			mt6323_vcn33_wifi_reg: ldo_vcn33_wifi{
> +				regulator-name = "vcn33_wifi";
> +				regulator-min-microvolt = <3300000>;
> +				regulator-max-microvolt = <3600000>;
> +				regulator-enable-ramp-delay = <185>;
> +			};
> +
> +			mt6323_va_reg: ldo_va{
> +				regulator-name = "va";
> +				regulator-min-microvolt = <2800000>;
> +				regulator-max-microvolt = <2800000>;
> +				regulator-enable-ramp-delay = <216>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vcama_reg: ldo_vcama{
> +				regulator-name = "vcama";
> +				regulator-min-microvolt = <1500000>;
> +				regulator-max-microvolt = <2800000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vio28_reg: ldo_vio28{
> +				regulator-name = "vio28";
> +				regulator-min-microvolt = <2800000>;
> +				regulator-max-microvolt = <2800000>;
> +				regulator-enable-ramp-delay = <216>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vusb_reg: ldo_vusb{
> +				regulator-name = "vusb";
> +				regulator-min-microvolt = <3300000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-enable-ramp-delay = <216>;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vmc_reg: ldo_vmc{
> +				regulator-name = "vmc";
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-enable-ramp-delay = <36>;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vmch_reg: ldo_vmch{
> +				regulator-name = "vmch";
> +				regulator-min-microvolt = <3000000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-enable-ramp-delay = <36>;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vemc3v3_reg: ldo_vemc3v3{
> +				regulator-name = "vemc3v3";
> +				regulator-min-microvolt = <3000000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-enable-ramp-delay = <36>;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vgp1_reg: ldo_vgp1{
> +				regulator-name = "vgp1";
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vgp2_reg: ldo_vgp2{
> +				regulator-name = "vgp2";
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <3000000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vgp3_reg: ldo_vgp3{
> +				regulator-name = "vgp3";
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <1800000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vcn18_reg: ldo_vcn18{
> +				regulator-name = "vcn18";
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <1800000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vsim1_reg: ldo_vsim1{
> +				regulator-name = "vsim1";
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <3000000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vsim2_reg: ldo_vsim2{
> +				regulator-name = "vsim2";
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <3000000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vrtc_reg: ldo_vrtc{
> +				regulator-name = "vrtc";
> +				regulator-min-microvolt = <2800000>;
> +				regulator-max-microvolt = <2800000>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vcamaf_reg: ldo_vcamaf{
> +				regulator-name = "vcamaf";
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vibr_reg: ldo_vibr{
> +				regulator-name = "vibr";
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-enable-ramp-delay = <36>;
> +			};
> +
> +			mt6323_vrf18_reg: ldo_vrf18{
> +				regulator-name = "vrf18";
> +				regulator-min-microvolt = <1825000>;
> +				regulator-max-microvolt = <1825000>;
> +				regulator-enable-ramp-delay = <187>;
> +			};
> +
> +			mt6323_vm_reg: ldo_vm{
> +				regulator-name = "vm";
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <1800000>;
> +				regulator-enable-ramp-delay = <216>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vio18_reg: ldo_vio18{
> +				regulator-name = "vio18";
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <1800000>;
> +				regulator-enable-ramp-delay = <216>;
> +				regulator-always-on;
> +				regulator-boot-on;
> +			};
> +
> +			mt6323_vcamd_reg: ldo_vcamd{
> +				regulator-name = "vcamd";
> +				regulator-min-microvolt = <1200000>;
> +				regulator-max-microvolt = <1800000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +
> +			mt6323_vcamio_reg: ldo_vcamio{
> +				regulator-name = "vcamio";
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <1800000>;
> +				regulator-enable-ramp-delay = <216>;
> +			};
> +		};
> +	};
> -- 
> 1.7.10.4
> --
> 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
John Crispin March 2, 2016, 6:11 p.m. UTC | #2
On 02/03/2016 19:07, Rob Herring wrote:
> On Thu, Feb 25, 2016 at 11:37:10AM +0100, John Crispin wrote:
>> Signed-off-by: John Crispin <blogic@openwrt.org>
>> Cc: devicetree@vger.kernel.org
>> ---
>> This patch needs to be merged post-rc1 as it depends on the MFD series of
>> the PMIC which is already inside Lee's tree pending the PR.
> 
> Why? It is an indpendent change with no actual dependency.
> 

that comment applies to the whole series and not just the one patch as
2/2 uses registers contained within a header file which is part of the
series adding the actual MFD driver. adding the doc without the driver
seemed wrong to me.


>>
>> Changes in V7
>> * remove another compatible string
>> * small wording changes
>>
>> Changes in V6
>> * remove the compatible string
>>
>> Changes in V5
>> * regulator vs regulators
>>
>> Changes in V4
>> * regulator is a subnode and not a property
>>
>>  .../bindings/regulator/mt6323-regulator.txt        |  239 ++++++++++++++++++++
>>  1 file changed, 239 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
> 
> Acked-by: Rob Herring <robh@kernel.org>
> 
>>
>> diff --git a/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
>> new file mode 100644
>> index 0000000..9fd95e7
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
>> @@ -0,0 +1,239 @@
>> +Mediatek MT6323 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_vproc, buck_vsys, buck_vpa
>> +LDO:
>> +  ldo_vtcxo, ldo_vcn28, ldo_vcn33_bt, ldo_vcn33_wifi, ldo_va, ldo_vcama,
>> +  ldo_vio28, ldo_vusb, ldo_vmc, ldo_vmch, ldo_vemc3v3, ldo_vgp1, ldo_vgp2,
>> +  ldo_vgp3, ldo_vcn18, ldo_vsim1, ldo_vsim2, ldo_vrtc, ldo_vcamaf, ldo_vibr,
>> +  ldo_vrf18, ldo_vm, ldo_vio18, ldo_vcamd, ldo_vcamio
>> +
>> +Example:
>> +
>> +	pmic: mt6323 {
>> +		compatible = "mediatek,mt6323";
>> +
>> +		mt6323regulator: regulators {
>> +			mt6323_vproc_reg: buck_vproc{
>> +				regulator-name = "vproc";
>> +				regulator-min-microvolt = < 700000>;
>> +				regulator-max-microvolt = <1350000>;
>> +				regulator-ramp-delay = <12500>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vsys_reg: buck_vsys{
>> +				regulator-name = "vsys";
>> +				regulator-min-microvolt = <1400000>;
>> +				regulator-max-microvolt = <2987500>;
>> +				regulator-ramp-delay = <25000>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vpa_reg: buck_vpa{
>> +				regulator-name = "vpa";
>> +				regulator-min-microvolt = < 500000>;
>> +				regulator-max-microvolt = <3650000>;
>> +			};
>> +
>> +			mt6323_vtcxo_reg: ldo_vtcxo{
>> +				regulator-name = "vtcxo";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <90>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcn28_reg: ldo_vcn28{
>> +				regulator-name = "vcn28";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <185>;
>> +			};
>> +
>> +			mt6323_vcn33_bt_reg: ldo_vcn33_bt{
>> +				regulator-name = "vcn33_bt";
>> +				regulator-min-microvolt = <3300000>;
>> +				regulator-max-microvolt = <3600000>;
>> +				regulator-enable-ramp-delay = <185>;
>> +			};
>> +
>> +			mt6323_vcn33_wifi_reg: ldo_vcn33_wifi{
>> +				regulator-name = "vcn33_wifi";
>> +				regulator-min-microvolt = <3300000>;
>> +				regulator-max-microvolt = <3600000>;
>> +				regulator-enable-ramp-delay = <185>;
>> +			};
>> +
>> +			mt6323_va_reg: ldo_va{
>> +				regulator-name = "va";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcama_reg: ldo_vcama{
>> +				regulator-name = "vcama";
>> +				regulator-min-microvolt = <1500000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vio28_reg: ldo_vio28{
>> +				regulator-name = "vio28";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vusb_reg: ldo_vusb{
>> +				regulator-name = "vusb";
>> +				regulator-min-microvolt = <3300000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vmc_reg: ldo_vmc{
>> +				regulator-name = "vmc";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vmch_reg: ldo_vmch{
>> +				regulator-name = "vmch";
>> +				regulator-min-microvolt = <3000000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vemc3v3_reg: ldo_vemc3v3{
>> +				regulator-name = "vemc3v3";
>> +				regulator-min-microvolt = <3000000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vgp1_reg: ldo_vgp1{
>> +				regulator-name = "vgp1";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vgp2_reg: ldo_vgp2{
>> +				regulator-name = "vgp2";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3000000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vgp3_reg: ldo_vgp3{
>> +				regulator-name = "vgp3";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vcn18_reg: ldo_vcn18{
>> +				regulator-name = "vcn18";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vsim1_reg: ldo_vsim1{
>> +				regulator-name = "vsim1";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <3000000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vsim2_reg: ldo_vsim2{
>> +				regulator-name = "vsim2";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <3000000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vrtc_reg: ldo_vrtc{
>> +				regulator-name = "vrtc";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcamaf_reg: ldo_vcamaf{
>> +				regulator-name = "vcamaf";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vibr_reg: ldo_vibr{
>> +				regulator-name = "vibr";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +			};
>> +
>> +			mt6323_vrf18_reg: ldo_vrf18{
>> +				regulator-name = "vrf18";
>> +				regulator-min-microvolt = <1825000>;
>> +				regulator-max-microvolt = <1825000>;
>> +				regulator-enable-ramp-delay = <187>;
>> +			};
>> +
>> +			mt6323_vm_reg: ldo_vm{
>> +				regulator-name = "vm";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vio18_reg: ldo_vio18{
>> +				regulator-name = "vio18";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcamd_reg: ldo_vcamd{
>> +				regulator-name = "vcamd";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vcamio_reg: ldo_vcamio{
>> +				regulator-name = "vcamio";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +		};
>> +	};
>> -- 
>> 1.7.10.4
>> --
>> 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
> 
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek
>
John Crispin March 2, 2016, 6:13 p.m. UTC | #3
On 02/03/2016 19:07, Rob Herring wrote:
> On Thu, Feb 25, 2016 at 11:37:10AM +0100, John Crispin wrote:
>> Signed-off-by: John Crispin <blogic@openwrt.org>
>> Cc: devicetree@vger.kernel.org
>> ---
>> This patch needs to be merged post-rc1 as it depends on the MFD series of
>> the PMIC which is already inside Lee's tree pending the PR.
> 
> Why? It is an indpendent change with no actual dependency.
> 

that comment applies to the whole series and not just the one patch as
2/2 uses registers contained within a header file which is part of the
series adding the actual MFD driver. adding the doc without the driver
seemed wrong to me.



>>
>> Changes in V7
>> * remove another compatible string
>> * small wording changes
>>
>> Changes in V6
>> * remove the compatible string
>>
>> Changes in V5
>> * regulator vs regulators
>>
>> Changes in V4
>> * regulator is a subnode and not a property
>>
>>  .../bindings/regulator/mt6323-regulator.txt        |  239 ++++++++++++++++++++
>>  1 file changed, 239 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
> 
> Acked-by: Rob Herring <robh@kernel.org>
> 
>>
>> diff --git a/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
>> new file mode 100644
>> index 0000000..9fd95e7
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
>> @@ -0,0 +1,239 @@
>> +Mediatek MT6323 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_vproc, buck_vsys, buck_vpa
>> +LDO:
>> +  ldo_vtcxo, ldo_vcn28, ldo_vcn33_bt, ldo_vcn33_wifi, ldo_va, ldo_vcama,
>> +  ldo_vio28, ldo_vusb, ldo_vmc, ldo_vmch, ldo_vemc3v3, ldo_vgp1, ldo_vgp2,
>> +  ldo_vgp3, ldo_vcn18, ldo_vsim1, ldo_vsim2, ldo_vrtc, ldo_vcamaf, ldo_vibr,
>> +  ldo_vrf18, ldo_vm, ldo_vio18, ldo_vcamd, ldo_vcamio
>> +
>> +Example:
>> +
>> +	pmic: mt6323 {
>> +		compatible = "mediatek,mt6323";
>> +
>> +		mt6323regulator: regulators {
>> +			mt6323_vproc_reg: buck_vproc{
>> +				regulator-name = "vproc";
>> +				regulator-min-microvolt = < 700000>;
>> +				regulator-max-microvolt = <1350000>;
>> +				regulator-ramp-delay = <12500>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vsys_reg: buck_vsys{
>> +				regulator-name = "vsys";
>> +				regulator-min-microvolt = <1400000>;
>> +				regulator-max-microvolt = <2987500>;
>> +				regulator-ramp-delay = <25000>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vpa_reg: buck_vpa{
>> +				regulator-name = "vpa";
>> +				regulator-min-microvolt = < 500000>;
>> +				regulator-max-microvolt = <3650000>;
>> +			};
>> +
>> +			mt6323_vtcxo_reg: ldo_vtcxo{
>> +				regulator-name = "vtcxo";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <90>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcn28_reg: ldo_vcn28{
>> +				regulator-name = "vcn28";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <185>;
>> +			};
>> +
>> +			mt6323_vcn33_bt_reg: ldo_vcn33_bt{
>> +				regulator-name = "vcn33_bt";
>> +				regulator-min-microvolt = <3300000>;
>> +				regulator-max-microvolt = <3600000>;
>> +				regulator-enable-ramp-delay = <185>;
>> +			};
>> +
>> +			mt6323_vcn33_wifi_reg: ldo_vcn33_wifi{
>> +				regulator-name = "vcn33_wifi";
>> +				regulator-min-microvolt = <3300000>;
>> +				regulator-max-microvolt = <3600000>;
>> +				regulator-enable-ramp-delay = <185>;
>> +			};
>> +
>> +			mt6323_va_reg: ldo_va{
>> +				regulator-name = "va";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcama_reg: ldo_vcama{
>> +				regulator-name = "vcama";
>> +				regulator-min-microvolt = <1500000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vio28_reg: ldo_vio28{
>> +				regulator-name = "vio28";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vusb_reg: ldo_vusb{
>> +				regulator-name = "vusb";
>> +				regulator-min-microvolt = <3300000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vmc_reg: ldo_vmc{
>> +				regulator-name = "vmc";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vmch_reg: ldo_vmch{
>> +				regulator-name = "vmch";
>> +				regulator-min-microvolt = <3000000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vemc3v3_reg: ldo_vemc3v3{
>> +				regulator-name = "vemc3v3";
>> +				regulator-min-microvolt = <3000000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vgp1_reg: ldo_vgp1{
>> +				regulator-name = "vgp1";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vgp2_reg: ldo_vgp2{
>> +				regulator-name = "vgp2";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3000000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vgp3_reg: ldo_vgp3{
>> +				regulator-name = "vgp3";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vcn18_reg: ldo_vcn18{
>> +				regulator-name = "vcn18";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vsim1_reg: ldo_vsim1{
>> +				regulator-name = "vsim1";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <3000000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vsim2_reg: ldo_vsim2{
>> +				regulator-name = "vsim2";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <3000000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vrtc_reg: ldo_vrtc{
>> +				regulator-name = "vrtc";
>> +				regulator-min-microvolt = <2800000>;
>> +				regulator-max-microvolt = <2800000>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcamaf_reg: ldo_vcamaf{
>> +				regulator-name = "vcamaf";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vibr_reg: ldo_vibr{
>> +				regulator-name = "vibr";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <3300000>;
>> +				regulator-enable-ramp-delay = <36>;
>> +			};
>> +
>> +			mt6323_vrf18_reg: ldo_vrf18{
>> +				regulator-name = "vrf18";
>> +				regulator-min-microvolt = <1825000>;
>> +				regulator-max-microvolt = <1825000>;
>> +				regulator-enable-ramp-delay = <187>;
>> +			};
>> +
>> +			mt6323_vm_reg: ldo_vm{
>> +				regulator-name = "vm";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vio18_reg: ldo_vio18{
>> +				regulator-name = "vio18";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +				regulator-always-on;
>> +				regulator-boot-on;
>> +			};
>> +
>> +			mt6323_vcamd_reg: ldo_vcamd{
>> +				regulator-name = "vcamd";
>> +				regulator-min-microvolt = <1200000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +
>> +			mt6323_vcamio_reg: ldo_vcamio{
>> +				regulator-name = "vcamio";
>> +				regulator-min-microvolt = <1800000>;
>> +				regulator-max-microvolt = <1800000>;
>> +				regulator-enable-ramp-delay = <216>;
>> +			};
>> +		};
>> +	};
>> -- 
>> 1.7.10.4
>> --
>> 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
> 
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek
>
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
new file mode 100644
index 0000000..9fd95e7
--- /dev/null
+++ b/Documentation/devicetree/bindings/regulator/mt6323-regulator.txt
@@ -0,0 +1,239 @@ 
+Mediatek MT6323 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_vproc, buck_vsys, buck_vpa
+LDO:
+  ldo_vtcxo, ldo_vcn28, ldo_vcn33_bt, ldo_vcn33_wifi, ldo_va, ldo_vcama,
+  ldo_vio28, ldo_vusb, ldo_vmc, ldo_vmch, ldo_vemc3v3, ldo_vgp1, ldo_vgp2,
+  ldo_vgp3, ldo_vcn18, ldo_vsim1, ldo_vsim2, ldo_vrtc, ldo_vcamaf, ldo_vibr,
+  ldo_vrf18, ldo_vm, ldo_vio18, ldo_vcamd, ldo_vcamio
+
+Example:
+
+	pmic: mt6323 {
+		compatible = "mediatek,mt6323";
+
+		mt6323regulator: regulators {
+			mt6323_vproc_reg: buck_vproc{
+				regulator-name = "vproc";
+				regulator-min-microvolt = < 700000>;
+				regulator-max-microvolt = <1350000>;
+				regulator-ramp-delay = <12500>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vsys_reg: buck_vsys{
+				regulator-name = "vsys";
+				regulator-min-microvolt = <1400000>;
+				regulator-max-microvolt = <2987500>;
+				regulator-ramp-delay = <25000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vpa_reg: buck_vpa{
+				regulator-name = "vpa";
+				regulator-min-microvolt = < 500000>;
+				regulator-max-microvolt = <3650000>;
+			};
+
+			mt6323_vtcxo_reg: ldo_vtcxo{
+				regulator-name = "vtcxo";
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
+				regulator-enable-ramp-delay = <90>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vcn28_reg: ldo_vcn28{
+				regulator-name = "vcn28";
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
+				regulator-enable-ramp-delay = <185>;
+			};
+
+			mt6323_vcn33_bt_reg: ldo_vcn33_bt{
+				regulator-name = "vcn33_bt";
+				regulator-min-microvolt = <3300000>;
+				regulator-max-microvolt = <3600000>;
+				regulator-enable-ramp-delay = <185>;
+			};
+
+			mt6323_vcn33_wifi_reg: ldo_vcn33_wifi{
+				regulator-name = "vcn33_wifi";
+				regulator-min-microvolt = <3300000>;
+				regulator-max-microvolt = <3600000>;
+				regulator-enable-ramp-delay = <185>;
+			};
+
+			mt6323_va_reg: ldo_va{
+				regulator-name = "va";
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
+				regulator-enable-ramp-delay = <216>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vcama_reg: ldo_vcama{
+				regulator-name = "vcama";
+				regulator-min-microvolt = <1500000>;
+				regulator-max-microvolt = <2800000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vio28_reg: ldo_vio28{
+				regulator-name = "vio28";
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
+				regulator-enable-ramp-delay = <216>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vusb_reg: ldo_vusb{
+				regulator-name = "vusb";
+				regulator-min-microvolt = <3300000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <216>;
+				regulator-boot-on;
+			};
+
+			mt6323_vmc_reg: ldo_vmc{
+				regulator-name = "vmc";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <36>;
+				regulator-boot-on;
+			};
+
+			mt6323_vmch_reg: ldo_vmch{
+				regulator-name = "vmch";
+				regulator-min-microvolt = <3000000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <36>;
+				regulator-boot-on;
+			};
+
+			mt6323_vemc3v3_reg: ldo_vemc3v3{
+				regulator-name = "vemc3v3";
+				regulator-min-microvolt = <3000000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <36>;
+				regulator-boot-on;
+			};
+
+			mt6323_vgp1_reg: ldo_vgp1{
+				regulator-name = "vgp1";
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vgp2_reg: ldo_vgp2{
+				regulator-name = "vgp2";
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <3000000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vgp3_reg: ldo_vgp3{
+				regulator-name = "vgp3";
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vcn18_reg: ldo_vcn18{
+				regulator-name = "vcn18";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vsim1_reg: ldo_vsim1{
+				regulator-name = "vsim1";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3000000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vsim2_reg: ldo_vsim2{
+				regulator-name = "vsim2";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3000000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vrtc_reg: ldo_vrtc{
+				regulator-name = "vrtc";
+				regulator-min-microvolt = <2800000>;
+				regulator-max-microvolt = <2800000>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vcamaf_reg: ldo_vcamaf{
+				regulator-name = "vcamaf";
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vibr_reg: ldo_vibr{
+				regulator-name = "vibr";
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-enable-ramp-delay = <36>;
+			};
+
+			mt6323_vrf18_reg: ldo_vrf18{
+				regulator-name = "vrf18";
+				regulator-min-microvolt = <1825000>;
+				regulator-max-microvolt = <1825000>;
+				regulator-enable-ramp-delay = <187>;
+			};
+
+			mt6323_vm_reg: ldo_vm{
+				regulator-name = "vm";
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-enable-ramp-delay = <216>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vio18_reg: ldo_vio18{
+				regulator-name = "vio18";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-enable-ramp-delay = <216>;
+				regulator-always-on;
+				regulator-boot-on;
+			};
+
+			mt6323_vcamd_reg: ldo_vcamd{
+				regulator-name = "vcamd";
+				regulator-min-microvolt = <1200000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+
+			mt6323_vcamio_reg: ldo_vcamio{
+				regulator-name = "vcamio";
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <1800000>;
+				regulator-enable-ramp-delay = <216>;
+			};
+		};
+	};