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��������^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�