lkml.org 
[lkml]   [2016]   [Mar]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH V7 1/2] regulator: Add document for MT6323 regulator
From
Date


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
>

\
 
 \ /
  Last update: 2016-03-02 20:01    [W:0.119 / U:0.884 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site