2022-08-22 08:24:25

by Max Krummenacher

Subject: [PATCH] arm64: dts: imx8mp-verdin: add cpu-supply

From: Max Krummenacher <[email protected]>

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

Signed-off-by: Max Krummenacher <[email protected]>

---

.../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;
--
2.35.3

2022-08-22 09:34:19

by Laurent Pinchart

Subject: Re: [PATCH] arm64: dts: imx8mp-verdin: add cpu-supply

Hi Max,

Thank you for the patch.

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

Reviewed-by: Laurent Pinchart <[email protected]>

> ---
>
> .../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;

--
Regards,

Laurent Pinchart

2022-09-03 02:05:28

by Shawn Guo

Subject: Re: [PATCH] arm64: dts: imx8mp-verdin: add cpu-supply

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

Applied, thanks!