diff mbox series

[1/2] ARM: dts: imx51: Adjust CPU operating points

Message ID 20191028140545.10218-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series [1/2] ARM: dts: imx51: Adjust CPU operating points | expand

Commit Message

Fabio Estevam Oct. 28, 2019, 2:05 p.m. UTC
Adjust the CPU operating points as per the characterized settings from
the vendor BSP at:

https://source.codeaurora.org/external/imx/linux-imx/tree/arch/arm/mach-mx5/mx51_babbage.c?h=imx_2.6.35_caf#n1343

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/boot/dts/imx51.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Shawn Guo Oct. 28, 2019, 2:29 p.m. UTC | #1
On Mon, Oct 28, 2019 at 11:05:44AM -0300, Fabio Estevam wrote:
> Adjust the CPU operating points as per the characterized settings from
> the vendor BSP at:
> 
> https://source.codeaurora.org/external/imx/linux-imx/tree/arch/arm/mach-mx5/mx51_babbage.c?h=imx_2.6.35_caf#n1343

Does it match what datasheet says?

Shawn

> 
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
>  arch/arm/boot/dts/imx51.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
> index 0a4b9a5d9a9c..eec423785849 100644
> --- a/arch/arm/boot/dts/imx51.dtsi
> +++ b/arch/arm/boot/dts/imx51.dtsi
> @@ -83,8 +83,8 @@
>  			clocks = <&clks IMX5_CLK_CPU_PODF>;
>  			clock-names = "cpu";
>  			operating-points = <
> -				166000	1000000
> -				600000	1050000
> +				166250	850000
> +				400000	1000000
>  				800000	1100000
>  			>;
>  			voltage-tolerance = <5>;
> -- 
> 2.17.1
>
Fabio Estevam Oct. 29, 2019, 7:02 p.m. UTC | #2
Hi Shawn,

On Mon, Oct 28, 2019 at 11:29 AM Shawn Guo <shawnguo@kernel.org> wrote:
>
> On Mon, Oct 28, 2019 at 11:05:44AM -0300, Fabio Estevam wrote:
> > Adjust the CPU operating points as per the characterized settings from
> > the vendor BSP at:
> >
> > https://source.codeaurora.org/external/imx/linux-imx/tree/arch/arm/mach-mx5/mx51_babbage.c?h=imx_2.6.35_caf#n1343
>
> Does it match what datasheet says?

It matches all operational points from the i.MX53 consumer datasheet.

On i.MX51 I noticed one discrepancy, so I will update it in v2.
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
index 0a4b9a5d9a9c..eec423785849 100644
--- a/arch/arm/boot/dts/imx51.dtsi
+++ b/arch/arm/boot/dts/imx51.dtsi
@@ -83,8 +83,8 @@ 
 			clocks = <&clks IMX5_CLK_CPU_PODF>;
 			clock-names = "cpu";
 			operating-points = <
-				166000	1000000
-				600000	1050000
+				166250	850000
+				400000	1000000
 				800000	1100000
 			>;
 			voltage-tolerance = <5>;