[PATCH V3 1/2] ARM: imx: add vddsoc/pu setpoint info into dts

[Anson.Huang at freescale.com]

Best Regards.
Anson huang 黄勇才
 
Freescale Semiconductor Shanghai
上海浦东新区亮景路192号A座2楼
201203
Tel:021-28937058


>-----Original Message-----
>From: Huang Yongcai-B20788
>Sent: Wednesday, December 18, 2013 8:01 PM
>To: Fabio Estevam
>Cc: Huang Yongcai-B20788; Shawn Guo; rjw at rjwysocki.net; Viresh Kumar;
>devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
>cpufreq at vger.kernel.org; linux-pm at vger.kernel.org
>Subject: Re: [PATCH V3 1/2] ARM: imx: add vddsoc/pu setpoint info into dts
>
>
>
>Sent from Anson's iPhone
>
>> 在 2013年12月18日，19:08，"Fabio Estevam" <festevam at gmail.com> 写道：
>>
>> Hi Anson,
>>
>>> On Wed, Dec 18, 2013 at 7:26 PM, Anson Huang <b20788 at freescale.com> wrote:
>>>
>>> +  Documentation/devicetree/bindings/regulator/regulator.txt for details.
>>> +  arm-supply: vddarm input.
>>> +  pu-supply: vddpu input.
>>> +  soc-supply: vddsoc input.
>>
>> I found these messages confusing, as someone may think that you are
>> referring to the VDD_ARM_IN/VDD_ARM23_IN and VDD_SOC_IN, which are
>> really inputs.
>>
>> What about doing this instead?
>>
>> arm-supply: VDDARM LDO output
>> pu-supply:  VDDPU LDO output
>> soc-supply: VDDSOC LDO output
>
>I think they should know these regulators mean the arm, soc/pu's real working
>voltage, as you know, our i.MX6 also support LDO bypass mode, then these
>regulators can be VDDXXX_IN. that is why I use vddxxx input, not vddxxx LDO
>input.Thanks!

After re-consideration, I think below should be better:
arm-supply: regulator node supplying arm.
pu-supply: regulator node supplying pu.
soc-supply: regulator node supplying soc.

Anson 


>>
>> Regards,
>>
>> Fabio Estevam
>>
>>