arm64: dts: rockchip: Add CPU/memory regulator coupling for 2 RK3588 boards

RK3588 chips allow for their CPU cores to be powered by a different
supply vs. their corresponding memory interfaces, and two of the
boards currently upstream do that (EVB1 and QuartzPro64).

The voltage of the memory interface though has to match that of the
CPU cores that use it, which downstream kernels achieve by the means
of a custom cpufreq driver which adjusts both at the same time.

It seems that regulator coupling is a more appropriate generic
interface for it, so this patch introduces coupling to affected
device trees to ensure that memory interface voltage is also updated
whenever cpufreq switches between CPU OPPs.

Note that other boards, such as Radxa Rock 5B, define both the CPU
and memory interface regulators as aliases to the same DT node, so
this doesn't apply there.

Signed-off-by: Alexey Charkov <alchark@gmail.com>
Link: https://lore.kernel.org/r/20240617-rk-dts-additions-v5-5-c1f5f3267f1e@gmail.com
Signed-off-by: Heiko Stuebner <heiko@sntech.de>

diff --git a/arch/arm64/boot/dts/rockchip/rk3588-evb1-v10.dts b/arch/arm64/boot/dts/rockchip/rk3588-evb1-v10.dts
index 7c3696a3ad3ad105f806201b8bd8e633ed037517..00f660d50127f75e9f4952fd0ea80bb0680916fe 100644 (file)
--- a/arch/arm64/boot/dts/rockchip/rk3588-evb1-v10.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-evb1-v10.dts
@@ -878,6 +878,8 @@
                        vdd_cpu_big1_s0: dcdc-reg1 {
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big1_mem_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <550000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -890,6 +892,8 @@
                        vdd_cpu_big0_s0: dcdc-reg2 {
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big0_mem_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <550000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -902,6 +906,8 @@
                        vdd_cpu_lit_s0: dcdc-reg3 {
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_lit_mem_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <550000>;
                                regulator-max-microvolt = <950000>;
                                regulator-ramp-delay = <12500>;
@@ -926,6 +932,8 @@
                        vdd_cpu_big1_mem_s0: dcdc-reg5 {
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big1_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <675000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -939,6 +947,8 @@
                        vdd_cpu_big0_mem_s0: dcdc-reg6 {
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big0_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <675000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -963,6 +973,8 @@
                        vdd_cpu_lit_mem_s0: dcdc-reg8 {
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_lit_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <675000>;
                                regulator-max-microvolt = <950000>;
                                regulator-ramp-delay = <12500>;

diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
index b4f22d95ac0e1c0a2ac2dbf6f3d154f887275870..baeb08d665c78392a2cac97558950c2b97d61466 100644 (file)
--- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
@@ -832,6 +832,8 @@
                                regulator-name = "vdd_cpu_big1_s0";
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big1_mem_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <550000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -845,6 +847,8 @@
                                regulator-name = "vdd_cpu_big0_s0";
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big0_mem_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <550000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -858,6 +862,8 @@
                                regulator-name = "vdd_cpu_lit_s0";
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_lit_mem_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <550000>;
                                regulator-max-microvolt = <950000>;
                                regulator-ramp-delay = <12500>;
@@ -884,6 +890,8 @@
                                regulator-name = "vdd_cpu_big1_mem_s0";
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big1_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <675000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -898,6 +906,8 @@
                                regulator-name = "vdd_cpu_big0_mem_s0";
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_big0_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <675000>;
                                regulator-max-microvolt = <1050000>;
                                regulator-ramp-delay = <12500>;
@@ -924,6 +934,8 @@
                                regulator-name = "vdd_cpu_lit_mem_s0";
                                regulator-always-on;
                                regulator-boot-on;
+                               regulator-coupled-with = <&vdd_cpu_lit_s0>;
+                               regulator-coupled-max-spread = <10000>;
                                regulator-min-microvolt = <675000>;
                                regulator-max-microvolt = <950000>;
                                regulator-ramp-delay = <12500>;