Re: [PATCH] riscv: dts: spacemit: orangepi-rv2: Add cpu scaling for K1, SoC
Yixun Lan <[email protected]>
| Newsgroups | dev.linux.lists.spacemit,org.infradead.lists.linux-riscv,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-pm |
|---|---|
| Message-ID | <[email protected]> |
Hi Vincent, Please send the patch directly instead of attached file, it's more easy for people to review and apply, suggest to use b4, or "git send-email" (old way) On 08:50 Sun 14 Jun , Vincent Legoll wrote: > [RESEND] as I cannot see yesterday's > > > Hello, > > > > I tested (on OrangePi RV2) the attached patch applied over > > spacemit/for-next + Shuwei Wu's V3 series. > > > This is a copy/paste of Shuwei Wu's work, so I don't know > > if I can submit this with my SoB or if it should be done > It's ok to add your SoB while adding support for new board > differently, please advise. > > > Anyways, you can add my: > > Tested-by: Vincent Legoll <[email protected]> # OrangePi-RV2 > > > To the relevant patches from your series, if that's useful, > > because it looks like it is working properly: > > > On Void linux (musl libc): > > > uname -a > Linux opirv2 7.1.0-rc1-00043-gb860bca13be4 #14 SMP PREEMPT Sat Jun 13 > 11:02:13 CEST 2026 riscv64 GNU/Linux > > awk --version | head -1 > GNU Awk 5.3.2, API 4.0 > > echo userspace > /sys/devices/system/cpu/cpufreq/policy0/scaling_governor > > echo 1600000 > /sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed > time awk 'BEGIN{for(i=0;i<1000000;i++){}}' > real 0m0.300s > user 0m0.299s > sys 0m0.001s > > echo 1228800 > /sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed > time awk 'BEGIN{for(i=0;i<1000000;i++){}}' > real 0m0.432s > user 0m0.429s > sys 0m0.004s > > echo 1000000 > /sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed > time awk 'BEGIN{for(i=0;i<1000000;i++){}}' > real 0m0.476s > user 0m0.476s > sys 0m0.001s > > echo 819000 > /sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed > time awk 'BEGIN{for(i=0;i<1000000;i++){}}' > real 0m0.582s > user 0m0.581s > sys 0m0.001s > > echo 614400 > /sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed > time awk 'BEGIN{for(i=0;i<1000000;i++){}}' > real 0m0.778s > user 0m0.773s > sys 0m0.005s > > -- > > Vincent Legoll > > From 9924789655ae5f4f98e7a5cdc3df3f34e7bfb659 Mon Sep 17 00:00:00 2001 > From: Vincent Legoll <[email protected]> > Date: Sat, 13 Jun 2026 10:49:34 +0200 > Subject: [PATCH] riscv: dts: spacemit: orangepi-rv2: Add cpu scaling for K1 > SoC > > Enable CPU DVFS on OrangePi RV2 > .. > This is a copy/paste from Shuwei Wu's BPI-F3 DTS modifications > It's quite common that people do copy/paste for additional board support, no need to mention it in commit message which I think it isn't really useful > Signed-off-by: Vincent Legoll <[email protected]> > --- > .../boot/dts/spacemit/k1-orangepi-rv2.dts | 35 ++++++++++++++++++- > 1 file changed, 34 insertions(+), 1 deletion(-) > > diff --git a/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts b/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts > index 7c49bce427f3..62d25f579af6 100644 > --- a/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts > +++ b/arch/riscv/boot/dts/spacemit/k1-orangepi-rv2.dts > @@ -8,6 +8,7 @@ > > #include "k1.dtsi" > #include "k1-pinctrl.dtsi" > +#include "k1-opp.dtsi" keep alphabet sorted > > / { > model = "OrangePi RV2"; > @@ -80,6 +81,38 @@ &combo_phy { > status = "okay"; > }; > > +&cpu_0 { > + cpu-supply = <&buck1_0v9>; > +}; > + > +&cpu_1 { > + cpu-supply = <&buck1_0v9>; > +}; > + > +&cpu_2 { > + cpu-supply = <&buck1_0v9>; > +}; > + > +&cpu_3 { > + cpu-supply = <&buck1_0v9>; > +}; > + > +&cpu_4 { > + cpu-supply = <&buck1_0v9>; > +}; > + > +&cpu_5 { > + cpu-supply = <&buck1_0v9>; > +}; > + > +&cpu_6 { > + cpu-supply = <&buck1_0v9>; > +}; > + > +&cpu_7 { > + cpu-supply = <&buck1_0v9>; > +}; > + > &emmc { > bus-width = <8>; > mmc-hs400-1_8v; > @@ -162,7 +195,7 @@ pmic@41 { > dldoin2-supply = <&buck5>; > > regulators { > - buck1 { > + buck1_0v9: buck1 { > regulator-min-microvolt = <500000>; > regulator-max-microvolt = <3450000>; > regulator-ramp-delay = <5000>; > -- > 2.54.0 > -- Yixun Lan (dlan)