diff mbox series

[RESEND] arm64: dts: sdm850: Add OPP tables for 2.84 and 2.96GHz

Message ID 20210112090640.20062-1-shawn.guo@linaro.org
State Superseded
Headers show
Series [RESEND] arm64: dts: sdm850: Add OPP tables for 2.84 and 2.96GHz | expand

Commit Message

Shawn Guo Jan. 12, 2021, 9:06 a.m. UTC
From: Steev Klimaszewski <steev@kali.org>


Running cpufreq-hw driver on Lenovo Yoga C630 laptop, the following
warning messages will be seen.

[    3.415340] cpu cpu4: Voltage update failed freq=2841600
[    3.418755] cpu cpu4: failed to update OPP for freq=2841600
[    3.422949] cpu cpu4: Voltage update failed freq=2956800
[    3.427086] cpu cpu4: failed to update OPP for freq=2956800

This is because the cpufreq-hw lookup table of SDM850 provides these two
set-points, but they are missing from OPP table in DT.  Let's create
sdm850.dtsi to add the OPP for them, so that the warning will be gone.

Signed-off-by: Steev Klimaszewski <steev@kali.org>

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>

---
 .../boot/dts/qcom/sdm850-lenovo-yoga-c630.dts |  2 +-
 arch/arm64/boot/dts/qcom/sdm850.dtsi          | 21 +++++++++++++++++++
 2 files changed, 22 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm64/boot/dts/qcom/sdm850.dtsi

-- 
2.17.1

Comments

patchwork-bot+linux-arm-msm@kernel.org Jan. 15, 2021, 3 p.m. UTC | #1
Hello:

This patch was applied to qcom/linux.git (refs/heads/for-next):

On Tue, 12 Jan 2021 17:06:40 +0800 you wrote:
> From: Steev Klimaszewski <steev@kali.org>

> 

> Running cpufreq-hw driver on Lenovo Yoga C630 laptop, the following

> warning messages will be seen.

> 

> [    3.415340] cpu cpu4: Voltage update failed freq=2841600

> [    3.418755] cpu cpu4: failed to update OPP for freq=2841600

> [    3.422949] cpu cpu4: Voltage update failed freq=2956800

> [    3.427086] cpu cpu4: failed to update OPP for freq=2956800

> 

> [...]


Here is the summary with links:
  - [RESEND] arm64: dts: sdm850: Add OPP tables for 2.84 and 2.96GHz
    https://git.kernel.org/qcom/c/6be4ba5467be

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
index 7929665aeaee..94390ae630f3 100644
--- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
+++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
@@ -13,7 +13,7 @@ 
 #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
 #include <dt-bindings/sound/qcom,q6afe.h>
 #include <dt-bindings/sound/qcom,q6asm.h>
-#include "sdm845.dtsi"
+#include "sdm850.dtsi"
 #include "pm8998.dtsi"
 
 / {
diff --git a/arch/arm64/boot/dts/qcom/sdm850.dtsi b/arch/arm64/boot/dts/qcom/sdm850.dtsi
new file mode 100644
index 000000000000..b1c2cf566c7a
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/sdm850.dtsi
@@ -0,0 +1,21 @@ 
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * SDM850 SoC device tree source
+ *
+ * Copyright (c) 2020, The Linux Foundation. All rights reserved.
+ */
+
+#include "sdm845.dtsi"
+
+&cpu4_opp_table {
+	cpu4_opp33: opp-2841600000 {
+		opp-hz = /bits/ 64 <2841600000>;
+		opp-peak-kBps = <7216000 25497600>;
+	};
+
+	cpu4_opp34: opp-2956800000 {
+		opp-hz = /bits/ 64 <2956800000>;
+		opp-peak-kBps = <7216000 25497600>;
+		turbo-mode;
+	};
+};