2023-02-20 12:08:47

by Konrad Dybcio

Subject: [PATCH] ARM: dts: qcom-apq8064: Fix opp table child name

The opp-320000000 name is rather misleading with the opp-hz value
of 450 MHz. Fix it!

Fixes: 8db0b6c7b636 ("ARM: dts: qcom: apq8064: Convert adreno from legacy gpu-pwrlevels to opp-v2")
Signed-off-by: Konrad Dybcio <[email protected]>
---
arch/arm/boot/dts/qcom-apq8064.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index 92aa2b081901..3aeac0cabb28 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -1260,7 +1260,7 @@ &gfx3d1 30
gpu_opp_table: opp-table {
compatible = "operating-points-v2";

- opp-320000000 {
+ opp-450000000 {
opp-hz = /bits/ 64 <450000000>;
};

--
2.39.2

2023-02-20 12:40:06

by David Heidelberg

Subject: Re: [PATCH] ARM: dts: qcom-apq8064: Fix opp table child name

Thank you!

Reviewed-by: David Heidelberg <[email protected]>

On 20/02/2023 13:08, Konrad Dybcio wrote:
> The opp-320000000 name is rather misleading with the opp-hz value
> of 450 MHz. Fix it!
>
> Fixes: 8db0b6c7b636 ("ARM: dts: qcom: apq8064: Convert adreno from legacy gpu-pwrlevels to opp-v2")
> Signed-off-by: Konrad Dybcio <[email protected]>
> ---
> arch/arm/boot/dts/qcom-apq8064.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index 92aa2b081901..3aeac0cabb28 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -1260,7 +1260,7 @@ &gfx3d1 30
> gpu_opp_table: opp-table {
> compatible = "operating-points-v2";
>
> - opp-320000000 {
> + opp-450000000 {
> opp-hz = /bits/ 64 <450000000>;
> };
>

--
David Heidelberg
Consultant Software Engineer

2023-03-15 23:33:38

by Bjorn Andersson

Subject: Re: [PATCH] ARM: dts: qcom-apq8064: Fix opp table child name

On Mon, 20 Feb 2023 13:08:31 +0100, Konrad Dybcio wrote:
> The opp-320000000 name is rather misleading with the opp-hz value
> of 450 MHz. Fix it!
>
>

Applied, thanks!

[1/1] ARM: dts: qcom-apq8064: Fix opp table child name
commit: b9745c275246a7e43c34d1b3be5ff9a9f3cf9305

Best regards,
--
Bjorn Andersson <[email protected]>