diff mbox series

ARM: dts: imx6ull: update vdd_soc voltage for 900MHz operating point

Message ID 1536740009-32735-1-git-send-email-Anson.Huang@nxp.com (mailing list archive)
State New, archived
Headers show
Series ARM: dts: imx6ull: update vdd_soc voltage for 900MHz operating point | expand

Commit Message

Anson Huang Sept. 12, 2018, 8:13 a.m. UTC
Update VDD_SOC voltage to 1.25V for 900MHz operating point
according to datasheet Rev. 1.3, 08/2018, 25mV is added to
the minimum allowed values to cover power supply ripple.

Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
 arch/arm/boot/dts/imx6ull.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stefan Wahren Sept. 12, 2018, 8:28 a.m. UTC | #1
Hi Anson,

Am 12.09.2018 um 10:13 schrieb Anson Huang:
> Update VDD_SOC voltage to 1.25V for 900MHz operating point
> according to datasheet Rev. 1.3, 08/2018, 25mV is added to
> the minimum allowed values to cover power supply ripple.

is there a reason why the new datasheet isn't published yet?

Regards
Stefan
Anson Huang Sept. 12, 2018, 8:30 a.m. UTC | #2
Hi, Stefan

Anson Huang
Best Regards!


> -----Original Message-----
> From: Stefan Wahren <stefan.wahren@i2se.com>
> Sent: Wednesday, September 12, 2018 4:28 PM
> To: Anson Huang <anson.huang@nxp.com>; shawnguo@kernel.org;
> s.hauer@pengutronix.de; kernel@pengutronix.de; Fabio Estevam
> <fabio.estevam@nxp.com>; robh+dt@kernel.org; mark.rutland@arm.com;
> linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org
> Cc: dl-linux-imx <linux-imx@nxp.com>
> Subject: Re: [PATCH] ARM: dts: imx6ull: update vdd_soc voltage for 900MHz
> operating point
> 
> Hi Anson,
> 
> Am 12.09.2018 um 10:13 schrieb Anson Huang:
> > Update VDD_SOC voltage to 1.25V for 900MHz operating point according
> > to datasheet Rev. 1.3, 08/2018, 25mV is added to the minimum allowed
> > values to cover power supply ripple.
> 
> is there a reason why the new datasheet isn't published yet?
 
We just got this new datasheet recently, it will be published soon.

Anson.

> 
> Regards
> Stefan
Anson Huang Sept. 18, 2018, 1:15 a.m. UTC | #3
Hi, Stefan

Anson Huang
Best Regards!


> -----Original Message-----
> From: Anson Huang
> Sent: Wednesday, September 12, 2018 4:30 PM
> To: Stefan Wahren <stefan.wahren@i2se.com>; shawnguo@kernel.org;
> s.hauer@pengutronix.de; kernel@pengutronix.de; Fabio Estevam
> <fabio.estevam@nxp.com>; robh+dt@kernel.org; mark.rutland@arm.com;
> linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org;
> linux-kernel@vger.kernel.org
> Cc: dl-linux-imx <linux-imx@nxp.com>
> Subject: RE: [PATCH] ARM: dts: imx6ull: update vdd_soc voltage for 900MHz
> operating point
> 
> Hi, Stefan
> 
> Anson Huang
> Best Regards!
> 
> 
> > -----Original Message-----
> > From: Stefan Wahren <stefan.wahren@i2se.com>
> > Sent: Wednesday, September 12, 2018 4:28 PM
> > To: Anson Huang <anson.huang@nxp.com>; shawnguo@kernel.org;
> > s.hauer@pengutronix.de; kernel@pengutronix.de; Fabio Estevam
> > <fabio.estevam@nxp.com>; robh+dt@kernel.org; mark.rutland@arm.com;
> > linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org;
> > linux-kernel@vger.kernel.org
> > Cc: dl-linux-imx <linux-imx@nxp.com>
> > Subject: Re: [PATCH] ARM: dts: imx6ull: update vdd_soc voltage for
> > 900MHz operating point
> >
> > Hi Anson,
> >
> > Am 12.09.2018 um 10:13 schrieb Anson Huang:
> > > Update VDD_SOC voltage to 1.25V for 900MHz operating point according
> > > to datasheet Rev. 1.3, 08/2018, 25mV is added to the minimum allowed
> > > values to cover power supply ripple.
> >
> > is there a reason why the new datasheet isn't published yet?
> 
> We just got this new datasheet recently, it will be published soon.
> 
> Anson 

The v1.3 datasheet has been published. Please check it from NXP website, thanks.

Anson.

> 
> >
> > Regards
> > Stefan
Sébastien Szymanski Sept. 18, 2018, 7:57 a.m. UTC | #4
On 09/12/2018 10:13 AM, Anson Huang wrote:
> Update VDD_SOC voltage to 1.25V for 900MHz operating point
> according to datasheet Rev. 1.3, 08/2018, 25mV is added to
> the minimum allowed values to cover power supply ripple.
> 
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> ---
>  arch/arm/boot/dts/imx6ull.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/imx6ull.dtsi b/arch/arm/boot/dts/imx6ull.dtsi
> index cd1776a..796ed35 100644
> --- a/arch/arm/boot/dts/imx6ull.dtsi
> +++ b/arch/arm/boot/dts/imx6ull.dtsi
> @@ -22,7 +22,7 @@
>  	>;
>  	fsl,soc-operating-points = <
>  		/* KHz	uV */
> -		900000	1175000
> +		900000	1250000
>  		792000	1175000
>  		528000	1175000
>  		396000	1175000
> 

Reviewed-by: Sébastien Szymanski <sebastien.szymanski@armadeus.com>
Shawn Guo Sept. 25, 2018, 3:33 p.m. UTC | #5
On Wed, Sep 12, 2018 at 04:13:29PM +0800, Anson Huang wrote:
> Update VDD_SOC voltage to 1.25V for 900MHz operating point
> according to datasheet Rev. 1.3, 08/2018, 25mV is added to
> the minimum allowed values to cover power supply ripple.
> 
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>

Applied, thanks.
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/imx6ull.dtsi b/arch/arm/boot/dts/imx6ull.dtsi
index cd1776a..796ed35 100644
--- a/arch/arm/boot/dts/imx6ull.dtsi
+++ b/arch/arm/boot/dts/imx6ull.dtsi
@@ -22,7 +22,7 @@ 
 	>;
 	fsl,soc-operating-points = <
 		/* KHz	uV */
-		900000	1175000
+		900000	1250000
 		792000	1175000
 		528000	1175000
 		396000	1175000