[PATCH] clk: qcom: gpucc-sm8150: Update the gpu_cc_pll1 config

[PATCH] clk: qcom: gpucc-sm8150: Update the gpu_cc_pll1 config

[Satya Priya Kakitapalli]

Update the test_ctl_hi_val and test_ctl_hi1_val of gpu_cc_pll1
as per latest HW recommendation.

Fixes: 0cef71f2ccc8 ("clk: qcom: Add graphics clock controller driver for SM8150")
Signed-off-by: Satya Priya Kakitapalli <quic_skakitap@quicinc.com>
---
 drivers/clk/qcom/gpucc-sm8150.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/clk/qcom/gpucc-sm8150.c b/drivers/clk/qcom/gpucc-sm8150.c
index 8422fd047493..c89a5b59ddb7 100644
--- a/drivers/clk/qcom/gpucc-sm8150.c
+++ b/drivers/clk/qcom/gpucc-sm8150.c
@@ -37,8 +37,8 @@ static struct alpha_pll_config gpu_cc_pll1_config = {
 	.config_ctl_hi_val = 0x00002267,
 	.config_ctl_hi1_val = 0x00000024,
 	.test_ctl_val = 0x00000000,
-	.test_ctl_hi_val = 0x00000002,
-	.test_ctl_hi1_val = 0x00000000,
+	.test_ctl_hi_val = 0x00000000,
+	.test_ctl_hi1_val = 0x00000020,
 	.user_ctl_val = 0x00000000,
 	.user_ctl_hi_val = 0x00000805,
 	.user_ctl_hi1_val = 0x000000d0,
-- 
2.25.1

Re: [PATCH] clk: qcom: gpucc-sm8150: Update the gpu_cc_pll1 config

[Konrad Dybcio]

On 11/22/23 05:28, Satya Priya Kakitapalli wrote:
> Update the test_ctl_hi_val and test_ctl_hi1_val of gpu_cc_pll1
> as per latest HW recommendation.
IIRC there is SM8150 v2.1 that's there on most devices and v2.2
that was used on the Microsoft Duo (or v2.0 and v2.1 respectively,
not sure, don't have any device on hand to read it back), do these
settings apply to both? Are they different for mobile vs auto?

Konrad

Re: [PATCH] clk: qcom: gpucc-sm8150: Update the gpu_cc_pll1 config

[Satya Priya Kakitapalli (Temp)]

On 11/22/2023 9:32 PM, Konrad Dybcio wrote:
>
>
> On 11/22/23 05:28, Satya Priya Kakitapalli wrote:
>> Update the test_ctl_hi_val and test_ctl_hi1_val of gpu_cc_pll1
>> as per latest HW recommendation.
> IIRC there is SM8150 v2.1 that's there on most devices and v2.2
> that was used on the Microsoft Duo (or v2.0 and v2.1 respectively,
> not sure, don't have any device on hand to read it back), do these
> settings apply to both? Are they different for mobile vs auto?
>

Yes these settings apply to all v2.x devices, also they are same for 
auto and mobile.


> Konrad

Re: [PATCH] clk: qcom: gpucc-sm8150: Update the gpu_cc_pll1 config

[Konrad Dybcio]

On 24.11.2023 12:24, Satya Priya Kakitapalli (Temp) wrote:
> 
> On 11/22/2023 9:32 PM, Konrad Dybcio wrote:
>>
>>
>> On 11/22/23 05:28, Satya Priya Kakitapalli wrote:
>>> Update the test_ctl_hi_val and test_ctl_hi1_val of gpu_cc_pll1
>>> as per latest HW recommendation.
>> IIRC there is SM8150 v2.1 that's there on most devices and v2.2
>> that was used on the Microsoft Duo (or v2.0 and v2.1 respectively,
>> not sure, don't have any device on hand to read it back), do these
>> settings apply to both? Are they different for mobile vs auto?
>>
> 
> Yes these settings apply to all v2.x devices, also they are same for auto and mobile.
Thanks for confirming!

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad

Re: [PATCH] clk: qcom: gpucc-sm8150: Update the gpu_cc_pll1 config

[Bjorn Andersson]

On Wed, 22 Nov 2023 09:58:14 +0530, Satya Priya Kakitapalli wrote:
> Update the test_ctl_hi_val and test_ctl_hi1_val of gpu_cc_pll1
> as per latest HW recommendation.
> 
> 

Applied, thanks!

[1/1] clk: qcom: gpucc-sm8150: Update the gpu_cc_pll1 config
      commit: 6ebd9a4f8b8d2b35cf965a04849c4ba763722f13

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