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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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@xxxxxxxxxxxxx; Viresh Kumar;
>devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
>cpufreq@xxxxxxxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx
>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@xxxxxxxxx> 写道:
>>
>> Hi Anson,
>>
>>> On Wed, Dec 18, 2013 at 7:26 PM, Anson Huang <b20788@xxxxxxxxxxxxx> 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
>>
>>
��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux