2018-09-12 08:20:29

by Anson Huang

[permalink] [raw]
Subject: [PATCH] ARM: dts: imx6ull: update vdd_soc voltage for 900MHz operating point

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 <[email protected]>
---
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
--
2.7.4



2018-09-12 08:30:29

by Stefan Wahren

[permalink] [raw]
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?

Regards
Stefan


2018-09-12 08:31:34

by Anson Huang

[permalink] [raw]
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 <[email protected]>
> Sent: Wednesday, September 12, 2018 4:28 PM
> To: Anson Huang <[email protected]>; [email protected];
> [email protected]; [email protected]; Fabio Estevam
> <[email protected]>; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]
> Cc: dl-linux-imx <[email protected]>
> 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

2018-09-18 01:15:43

by Anson Huang

[permalink] [raw]
Subject: RE: [PATCH] ARM: dts: imx6ull: update vdd_soc voltage for 900MHz operating point

Hi, Stefan

Anson Huang
Best Regards!


> -----Original Message-----
> From: Anson Huang
> Sent: Wednesday, September 12, 2018 4:30 PM
> To: Stefan Wahren <[email protected]>; [email protected];
> [email protected]; [email protected]; Fabio Estevam
> <[email protected]>; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]
> Cc: dl-linux-imx <[email protected]>
> 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 <[email protected]>
> > Sent: Wednesday, September 12, 2018 4:28 PM
> > To: Anson Huang <[email protected]>; [email protected];
> > [email protected]; [email protected]; Fabio Estevam
> > <[email protected]>; [email protected]; [email protected];
> > [email protected]; [email protected];
> > [email protected]
> > Cc: dl-linux-imx <[email protected]>
> > 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

2018-09-18 12:33:44

by Sébastien Szymanski

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: imx6ull: update vdd_soc voltage for 900MHz operating point

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 <[email protected]>
> ---
> 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 <[email protected]>

--
Sébastien Szymanski
Software engineer, Armadeus Systems
Tel: +33 (0)9 72 29 41 44
Fax: +33 (0)9 72 28 79 26

2018-09-25 15:35:54

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: imx6ull: update vdd_soc voltage for 900MHz operating point

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 <[email protected]>

Applied, thanks.