diff mbox series

arm64: dts: imx8mp-verdin: add cpu-supply

Message ID 20220822075342.2611279-1-max.oss.09@gmail.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: imx8mp-verdin: add cpu-supply | expand

Commit Message

Max Krummenacher Aug. 22, 2022, 7:53 a.m. UTC 
From: Max Krummenacher <max.krummenacher@toradex.com>

Add the cpu-supply property to all CPU nodes to enable the cpufreq
driver.

Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>

---

 .../boot/dts/freescale/imx8mp-verdin.dtsi      | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

Comments

Laurent Pinchart Aug. 22, 2022, 9:22 a.m. UTC | #1 
Hi Max,

Thank you for the patch.

On Mon, Aug 22, 2022 at 09:53:42AM +0200, Max Krummenacher wrote:
> From: Max Krummenacher <max.krummenacher@toradex.com>
> 
> Add the cpu-supply property to all CPU nodes to enable the cpufreq
> driver.
> 
> Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> 
>  .../boot/dts/freescale/imx8mp-verdin.dtsi      | 18 +++++++++++++++++-
>  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> index f062fdbb2719..dbc22b672fb3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> @@ -146,6 +146,22 @@ reserved-memory {
>  	};
>  };
>  
> +&A53_0 {
> +	cpu-supply = <&reg_vdd_arm>;
> +};
> +
> +&A53_1 {
> +	cpu-supply = <&reg_vdd_arm>;
> +};
> +
> +&A53_2 {
> +	cpu-supply = <&reg_vdd_arm>;
> +};
> +
> +&A53_3 {
> +	cpu-supply = <&reg_vdd_arm>;
> +};
> +
>  &cpu_alert0 {
>  	temperature = <95000>;
>  };
> @@ -453,7 +469,7 @@ BUCK1 {
>  				regulator-ramp-delay = <3125>;
>  			};
>  
> -			BUCK2 {
> +			reg_vdd_arm: BUCK2 {
>  				nxp,dvs-run-voltage = <950000>;
>  				nxp,dvs-standby-voltage = <850000>;
>  				regulator-always-on;
Marcel Ziswiler Aug. 22, 2022, 9:33 a.m. UTC | #2 
On Mon, 2022-08-22 at 09:53 +0200, Max Krummenacher wrote:
> From: Max Krummenacher <max.krummenacher@toradex.com>
> 
> Add the cpu-supply property to all CPU nodes to enable the cpufreq
> driver.
> 
> Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>

Acked-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>

> ---
> 
>  .../boot/dts/freescale/imx8mp-verdin.dtsi      | 18 +++++++++++++++++-
>  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-
> verdin.dtsi
> index f062fdbb2719..dbc22b672fb3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> @@ -146,6 +146,22 @@ reserved-memory {
>         };
>  };
>  
> +&A53_0 {
> +       cpu-supply = <&reg_vdd_arm>;
> +};
> +
> +&A53_1 {
> +       cpu-supply = <&reg_vdd_arm>;
> +};
> +
> +&A53_2 {
> +       cpu-supply = <&reg_vdd_arm>;
> +};
> +
> +&A53_3 {
> +       cpu-supply = <&reg_vdd_arm>;
> +};
> +
>  &cpu_alert0 {
>         temperature = <95000>;
>  };
> @@ -453,7 +469,7 @@ BUCK1 {
>                                 regulator-ramp-delay = <3125>;
>                         };
>  
> -                       BUCK2 {
> +                       reg_vdd_arm: BUCK2 {
>                                 nxp,dvs-run-voltage = <950000>;
>                                 nxp,dvs-standby-voltage = <850000>;
>                                 regulator-always-on;
Francesco Dolcini Aug. 29, 2022, 6:21 p.m. UTC | #3 
On Mon, Aug 22, 2022 at 09:53:42AM +0200, Max Krummenacher wrote:
> From: Max Krummenacher <max.krummenacher@toradex.com>
> 
> Add the cpu-supply property to all CPU nodes to enable the cpufreq
> driver.
> 
> Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>

This would need a fixes tag IMO:

Fixes: a39ed23bdf6e ("arm64: dts: freescale: add initial support for verdin imx8m plus")

Francesco
Shawn Guo Sept. 3, 2022, 1:49 a.m. UTC | #4 
On Mon, Aug 22, 2022 at 09:53:42AM +0200, Max Krummenacher wrote:
> From: Max Krummenacher <max.krummenacher@toradex.com>
> 
> Add the cpu-supply property to all CPU nodes to enable the cpufreq
> driver.
> 
> Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>

Applied, thanks!
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
index f062fdbb2719..dbc22b672fb3 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
@@ -146,6 +146,22 @@  reserved-memory {
 	};
 };
 
+&A53_0 {
+	cpu-supply = <&reg_vdd_arm>;
+};
+
+&A53_1 {
+	cpu-supply = <&reg_vdd_arm>;
+};
+
+&A53_2 {
+	cpu-supply = <&reg_vdd_arm>;
+};
+
+&A53_3 {
+	cpu-supply = <&reg_vdd_arm>;
+};
+
 &cpu_alert0 {
 	temperature = <95000>;
 };
@@ -453,7 +469,7 @@  BUCK1 {
 				regulator-ramp-delay = <3125>;
 			};
 
-			BUCK2 {
+			reg_vdd_arm: BUCK2 {
 				nxp,dvs-run-voltage = <950000>;
 				nxp,dvs-standby-voltage = <850000>;
 				regulator-always-on;