On 10/2/2026 8:08 PM, Bjorn Andersson wrote:
> On Sat, May 16, 2026 at 01:37:51AM +0530, Akhil P Oommen wrote:
>> Add the CLX domain table and the IFF/PCLX limits tables to the catalog to
>> enable CLX feature support on Adreno X2-85 GPU present in the Glymur
>> chipset.
>>
> 
> This patch landed in next-20261001 and while it works on my Glymur CRD
> (COB), on the Asus Zenbook A14 I see the following failure during boot:
> 
> [    4.843643] platform 3d6c000.gmu: [drm:a6xx_hfi_wait_for_msg_interrupt 
> [msm]] *ERROR* Message HFI_H2F_MSG_GX_BW_PERF_VOTE id 11 timed out waiting 
> for response
> [    4.853804] platform 3d6c000.gmu: [drm:a6xx_gmu_set_oob [msm]] *ERROR* 
> Timeout waiting for GMU OOB set GPU_SET: 0x4
> [    4.853878] msm_dpu ae01000.display-controller: [drm:adreno_load_gpu 
> [msm]] *ERROR* gpu hw init failed: -110
> [    4.863980] platform 3d6c000.gmu: [drm:a6xx_gmu_set_oob [msm]] *ERROR* 
> Timeout waiting for GMU OOB set GPU_SET: 0x4
> [    4.864038] platform 3d6c000.gmu: [drm:a6xx_hfi_stop [msm]] *ERROR* HFI 
> queue 2 is not empty
> [    5.128548] ------------[ cut here ]------------
> 
> And then as one can expect there's no more GPU.
> 
> Reverting this single patch makes next-20261001 work again.
> 
> #regzb introduced: 93b6c8d10fad ("drm/msm/a8xx: Enable CLX feature on Adreno 
> X2-85")

Could you please try this fix?

diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c 
b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c
index 662f09bec9d5..52cd60272870 100644
--- a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c
+++ b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c
@@ -1241,8 +1241,12 @@ static int a6xx_gmu_secure_init(struct a6xx_gpu 
*a6xx_gpu)
                        goto done;
                }

-               ret = qcom_scm_gpu_init_regs(QCOM_SCM_GPU_ALWAYS_EN_REQ |
-                                            QCOM_SCM_GPU_TSENSE_EN_REQ);
+               u32 gpu_req = QCOM_SCM_GPU_ALWAYS_EN_REQ | 
QCOM_SCM_GPU_TSENSE_EN_REQ;
+
+               if (adreno_gpu->info->a6xx->clx_tbl)
+                       gpu_req |= QCOM_SCM_GPU_CLX_EN_REQ;
+
+               ret = qcom_scm_gpu_init_regs(gpu_req);
                if (ret) {
                        dev_warn_once(gpu->dev->dev,
                                "SCM call failed\n");
> 
> Regards,
> Bjorn
> 
>> Signed-off-by: Akhil P Oommen <[email protected]>
>> ---
>>  drivers/gpu/drm/msm/adreno/a6xx_catalog.c | 86 
>> +++++++++++++++++++++++++++++++
>>  1 file changed, 86 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c 
>> b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
>> index c503912a61c7..9ad13eb173de 100644
>> --- a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
>> +++ b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
>> @@ -1799,6 +1799,90 @@ static const struct adreno_reglist_pipe 
>> x285_dyn_pwrup_reglist_regs[] = {
>>  };
>>  DECLARE_ADRENO_REGLIST_PIPE_LIST(x285_dyn_pwrup_reglist);
>>  
>> +static const struct a6xx_hfi_clx_table_v2_cmd x285_clx_tbl = {
>> +    .version = (2 << 16) | 1,
>> +    .domain = {
>> +            /* GX DOMAIN */
>> +            {
>> +                    .data = CLX_DATA(60, 11, 0, 1),
>> +                    .clxt = 0,
>> +                    .clxh = 0,
>> +                    .urg_mode = 1,
>> +                    .lkg_en = 0,
>> +                    .curr_budget = 100,
>> +            },
>> +            /* MXG DOMAIN */
>> +            {
>> +                    .data = CLX_DATA(60, 3, 1, 1),
>> +                    .clxt = 0,
>> +                    .clxh = 0,
>> +                    .urg_mode = 1,
>> +                    .lkg_en = 0,
>> +                    .curr_budget = 100,
>> +            },
>> +    },
>> +};
>> +
>> +struct a6xx_hfi_limits_tbl x285_limits_tbl[] = {
>> +    {
>> +            .feature_id = GMU_MIT_IFF,
>> +            .domain = GMU_GX_DOMAIN,
>> +            .feature_rev = 0,
>> +            .cfg = {
>> +                    .enable = 1,
>> +                    .msg_path = 0,
>> +                    .lkg_en = 0,
>> +                    .mode = 0,
>> +                    .sid = 12,
>> +                    .mit_time = 5,
>> +                    .curr_limit = 18000,
>> +            },
>> +    },
>> +    {
>> +            .feature_id = GMU_MIT_IFF,
>> +            .domain = GMU_MX_DOMAIN,
>> +            .feature_rev = 0,
>> +            .cfg = {
>> +                    .enable = 1,
>> +                    .msg_path = 0,
>> +                    .lkg_en = 0,
>> +                    .mode = 0,
>> +                    .sid = 18,
>> +                    .mit_time = 2000,
>> +                    .curr_limit = 6000,
>> +            },
>> +    },
>> +    {
>> +            .feature_id = GMU_MIT_PCLX,
>> +            .domain = GMU_GX_DOMAIN,
>> +            .feature_rev = 0,
>> +            .cfg = {
>> +                    .enable = 1,
>> +                    .msg_path = 0,
>> +                    .lkg_en = 0,
>> +                    .mode = 0,
>> +                    .sid = 6,
>> +                    .mit_time = 3,
>> +                    .curr_limit = 66000,
>> +            },
>> +    },
>> +    {
>> +            .feature_id = GMU_MIT_PCLX,
>> +            .domain = GMU_MX_DOMAIN,
>> +            .feature_rev = 0,
>> +            .cfg = {
>> +                    .enable = 0,
>> +                    .msg_path = 0,
>> +                    .lkg_en = 0,
>> +                    .mode = 0,
>> +                    .sid = 6,
>> +                    .mit_time = 3,
>> +                    .curr_limit = 18000,
>> +            },
>> +    },
>> +};
>> +DECLARE_ADRENO_LIMITS_TABLE(x285_limits);
>> +
>>  static const struct adreno_reglist_pipe a840_nonctxt_regs[] = {
>>      { REG_A8XX_CP_SMMU_STREAM_ID_LPAC, 0x00000101, BIT(PIPE_NONE) },
>>      { REG_A8XX_GRAS_DBG_ECO_CNTL, 0x00000800, BIT(PIPE_BV) | BIT(PIPE_BR) },
>> @@ -2227,6 +2311,8 @@ static const struct adreno_info a8xx_gpus[] = {
>>                              },
>>                              { /* sentinel */ },
>>                      },
>> +                    .clx_tbl = &x285_clx_tbl,
>> +                    .limits_tbl = &x285_limits,
>>              },
>>              .speedbins = ADRENO_SPEEDBINS(
>>                      { 0,   0 },
>>
>> -- 
>> 2.51.0
>>
>>

-Akhil

Reply via email to