ARM: dts: qcom-apq8064: Fix opp table child name

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

Posted by Konrad Dybcio 2 years, 6 months ago

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 <konrad.dybcio@linaro.org>
---
 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

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

Posted by Bjorn Andersson 2 years, 5 months ago

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 <andersson@kernel.org>

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

Posted by David Heidelberg 2 years, 6 months ago

Thank you!

Reviewed-by: David Heidelberg <david@ixit.cz>

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 <konrad.dybcio@linaro.org>
> ---
>   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