Hi, Fabio Anson Huang Best Regards! > -----Original Message----- > From: Fabio Estevam [mailto:festevam@xxxxxxxxx] > Sent: Sunday, March 18, 2018 8:55 PM > To: Anson Huang <anson.huang@xxxxxxx> > Cc: Robin Gong <yibin.gong@xxxxxxx>; Shawn Guo <shawnguo@xxxxxxxxxx>; > Sascha Hauer <kernel@xxxxxxxxxxxxxx>; Fabio Estevam > <fabio.estevam@xxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland > <mark.rutland@xxxxxxx>; Russell King - ARM Linux <linux@xxxxxxxxxxxxxxx>; > Liam Girdwood <lgirdwood@xxxxxxxxx>; Mark Brown <broonie@xxxxxxxxxx>; > dl-linux-imx <linux-imx@xxxxxxx>; moderated list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > <devicetree@xxxxxxxxxxxxxxx>; linux-kernel <linux-kernel@xxxxxxxxxxxxxxx> > Subject: Re: [PATCH 1/2] ARM: dts: pfuze3000: update sw1a/vldo4 voltage range > > On Sat, Mar 17, 2018 at 11:23 PM, Anson Huang <anson.huang@xxxxxxx> > wrote: > > > I thought the max value here means the capability of regulator itself, like the > internal > > No it is not. > > Please check Documentation/devicetree/bindings/regulator/regulator.txt: > > "- regulator-max-microvolt: largest voltage consumers may set" OK, so I should change the max value of SW1A according to board design, such as for MX7D, it is 1.075V for max cpu-freq's voltage, set it to 1.075 in dts? Also, according to the binding doc, do you think the anatop's vdd_arm and vdd_soc/pu's max value also need to be adjusted to match the value define in each soc's datasheet? Anson. ��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f