From: Yixun Lan <dlan@kernel.org>
To: Vincent Legoll <legoll@online.fr>
Cc: Shuwei Wu <shuwei.wu@mailbox.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Viresh Kumar <viresh.kumar@linaro.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Paul Walmsley <pjw@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Albert Ou <aou@eecs.berkeley.edu>,
Alexandre Ghiti <alex@ghiti.fr>,
linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-riscv@lists.infradead.org, spacemit@lists.linux.dev,
devicetree@vger.kernel.org
Subject: Re: [PATCH] riscv: dts: spacemit: orangepi-rv2: Add cpu scaling for K1, SoC
Date: Mon, 15 Jun 2026 00:41:08 +0000 [thread overview]
Message-ID: <20260615004108-GKA1002079@kernel.org> (raw)
In-Reply-To: <f031e13e-d9f5-414a-b90e-a3d5a60b8d05@online.fr>
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 <vincent.legoll@gmail.com> # 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 <vincent.legoll@gmail.com>
> 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 <vincent.legoll@gmail.com>
> ---
> .../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)
WARNING: multiple messages have this Message-ID (diff)
From: Yixun Lan <dlan@kernel.org>
To: Vincent Legoll <legoll@online.fr>
Cc: Shuwei Wu <shuwei.wu@mailbox.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Viresh Kumar <viresh.kumar@linaro.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Paul Walmsley <pjw@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Albert Ou <aou@eecs.berkeley.edu>,
Alexandre Ghiti <alex@ghiti.fr>,
linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-riscv@lists.infradead.org, spacemit@lists.linux.dev,
devicetree@vger.kernel.org
Subject: Re: [PATCH] riscv: dts: spacemit: orangepi-rv2: Add cpu scaling for K1, SoC
Date: Mon, 15 Jun 2026 00:41:08 +0000 [thread overview]
Message-ID: <20260615004108-GKA1002079@kernel.org> (raw)
In-Reply-To: <f031e13e-d9f5-414a-b90e-a3d5a60b8d05@online.fr>
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 <vincent.legoll@gmail.com> # 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 <vincent.legoll@gmail.com>
> 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 <vincent.legoll@gmail.com>
> ---
> .../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)
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2026-06-15 0:41 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-12 9:51 [PATCH v3 0/2] cpufreq: spacemit: Add cpufreq support for K1 SoC Shuwei Wu
2026-06-12 9:51 ` Shuwei Wu
2026-06-12 9:51 ` [PATCH v3 1/2] cpufreq: spacemit: Add K1 cpufreq driver Shuwei Wu
2026-06-12 9:51 ` Shuwei Wu
2026-06-12 10:05 ` sashiko-bot
2026-06-15 5:24 ` Viresh Kumar
2026-06-15 5:24 ` Viresh Kumar
2026-06-15 6:12 ` Shuwei Wu
2026-06-15 6:12 ` Shuwei Wu
2026-06-15 6:34 ` Viresh Kumar
2026-06-15 6:34 ` Viresh Kumar
2026-06-15 12:11 ` Shuwei Wu
2026-06-15 12:11 ` Shuwei Wu
2026-06-12 9:51 ` [PATCH v3 2/2] riscv: dts: spacemit: Add cpu scaling for K1 SoC Shuwei Wu
2026-06-12 9:51 ` Shuwei Wu
2026-06-14 6:50 ` [PATCH] riscv: dts: spacemit: orangepi-rv2: Add cpu scaling for K1, SoC Vincent Legoll
2026-06-14 6:50 ` Vincent Legoll
2026-06-15 0:41 ` Yixun Lan [this message]
2026-06-15 0:41 ` Yixun Lan
2026-06-14 12:28 ` [PATCH] riscv: dts: spacemit: k1-musepi-pro: add cpu scaling Andre Heider
2026-06-14 12:28 ` Andre Heider
2026-06-14 12:32 ` sashiko-bot
2026-06-15 5:07 ` [PATCH v3 0/2] cpufreq: spacemit: Add cpufreq support for K1 SoC Viresh Kumar
2026-06-15 5:07 ` Viresh Kumar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260615004108-GKA1002079@kernel.org \
--to=dlan@kernel.org \
--cc=alex@ghiti.fr \
--cc=aou@eecs.berkeley.edu \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=legoll@online.fr \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=palmer@dabbelt.com \
--cc=pjw@kernel.org \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=shuwei.wu@mailbox.org \
--cc=spacemit@lists.linux.dev \
--cc=viresh.kumar@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.