Linux ARM-MSM sub-architecture
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: qcom: sm8250: Fix CPU7 opp table
@ 2025-03-08 10:27 Xilin Wu via B4 Relay
  2025-03-08 15:17 ` Konrad Dybcio
  2025-03-14 20:01 ` Bjorn Andersson
  0 siblings, 2 replies; 3+ messages in thread
From: Xilin Wu via B4 Relay @ 2025-03-08 10:27 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Dmitry Baryshkov, Thara Gopinath
  Cc: linux-arm-msm, devicetree, linux-kernel, Xilin Wu

From: Xilin Wu <wuxilin123@gmail.com>

There is a typo in cpu7_opp9. Fix it to get rid of the following
errors.

[    0.198043] cpu cpu7: Voltage update failed freq=1747200
[    0.198052] cpu cpu7: failed to update OPP for freq=1747200

Fixes: 8e0e8016cb79 ("arm64: dts: qcom: sm8250: Add CPU opp tables")
Signed-off-by: Xilin Wu <wuxilin123@gmail.com>
---
 arch/arm64/boot/dts/qcom/sm8250.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
index c2937b4d9f180296733b6d7a7a16a088f1f96b76..68613ea7146c8882150f1b81dbc0f3384d3380ba 100644
--- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
@@ -606,7 +606,7 @@ cpu7_opp8: opp-1632000000 {
 		};
 
 		cpu7_opp9: opp-1747200000 {
-			opp-hz = /bits/ 64 <1708800000>;
+			opp-hz = /bits/ 64 <1747200000>;
 			opp-peak-kBps = <5412000 42393600>;
 		};
 

---
base-commit: 0a2f889128969dab41861b6e40111aa03dc57014
change-id: 20250308-fix-sm8250-cpufreq-d95944b6d1a3

Best regards,
-- 
Xilin Wu <wuxilin123@gmail.com>



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: qcom: sm8250: Fix CPU7 opp table
  2025-03-08 10:27 [PATCH] arm64: dts: qcom: sm8250: Fix CPU7 opp table Xilin Wu via B4 Relay
@ 2025-03-08 15:17 ` Konrad Dybcio
  2025-03-14 20:01 ` Bjorn Andersson
  1 sibling, 0 replies; 3+ messages in thread
From: Konrad Dybcio @ 2025-03-08 15:17 UTC (permalink / raw)
  To: wuxilin123, Bjorn Andersson, Konrad Dybcio, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Dmitry Baryshkov,
	Thara Gopinath
  Cc: linux-arm-msm, devicetree, linux-kernel

On 8.03.2025 11:27 AM, Xilin Wu via B4 Relay wrote:
> From: Xilin Wu <wuxilin123@gmail.com>
> 
> There is a typo in cpu7_opp9. Fix it to get rid of the following
> errors.
> 
> [    0.198043] cpu cpu7: Voltage update failed freq=1747200
> [    0.198052] cpu cpu7: failed to update OPP for freq=1747200
> 
> Fixes: 8e0e8016cb79 ("arm64: dts: qcom: sm8250: Add CPU opp tables")
> Signed-off-by: Xilin Wu <wuxilin123@gmail.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Konrad


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: qcom: sm8250: Fix CPU7 opp table
  2025-03-08 10:27 [PATCH] arm64: dts: qcom: sm8250: Fix CPU7 opp table Xilin Wu via B4 Relay
  2025-03-08 15:17 ` Konrad Dybcio
@ 2025-03-14 20:01 ` Bjorn Andersson
  1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Andersson @ 2025-03-14 20:01 UTC (permalink / raw)
  To: Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Dmitry Baryshkov, Thara Gopinath, Xilin Wu
  Cc: linux-arm-msm, devicetree, linux-kernel


On Sat, 08 Mar 2025 18:27:51 +0800, Xilin Wu wrote:
> There is a typo in cpu7_opp9. Fix it to get rid of the following
> errors.
> 
> [    0.198043] cpu cpu7: Voltage update failed freq=1747200
> [    0.198052] cpu cpu7: failed to update OPP for freq=1747200
> 
> 
> [...]

Applied, thanks!

[1/1] arm64: dts: qcom: sm8250: Fix CPU7 opp table
      commit: 28f997b89967afdc0855d8aa7538b251fb44f654

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-03-14 20:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-08 10:27 [PATCH] arm64: dts: qcom: sm8250: Fix CPU7 opp table Xilin Wu via B4 Relay
2025-03-08 15:17 ` Konrad Dybcio
2025-03-14 20:01 ` Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox