Re: [PATCH 3/3] drm/msm/a8xx: Enable CLX feature on Adreno X2-85

[Akhil P Oommen <akhilpo@oss.qualcomm.com>]

On 10/20/2026 2:27 PM, Oleg Keri wrote:
> I guess this should be investigated further. Yoga Slim 7x Gen 11, no modules,
> firmware built in, so the GMU comes up at 0.6 s.
> 
> It's rpmhpd. Before its sync_state, rpmhpd_aggregate_corner() pushes every
> vote to the highest corner. The gxclkctl probe does a runtime resume/suspend
> of its GFX and GMXC domains at 0.18 s (qcom_cc_probe -> pm_runtime -> genpd
> -> rpmhpd_power_on), and from that point both rails are stuck at max until
> rpmhpd syncs, which is at 2.8 s here. The GMU then does its CLX DCVS
> calibration on a rail that won't move.
> 
> The GMU log buffer (fw v5.2.38) agrees: the calibration trace of the bad
> boot matches the first good one step for step up to the ninth level. There
> it stops, and the firmware puts HFI_F2H_MSG_ERROR 0x007306c7
> "ResMgrCalibrateDcvsLvl" on the debug queue. The GX_BW_PERF_VOTE timeout is
> simply the next message we send.
> 
> With gfx and gmxc taken out of the clamp (hack below) the first GPU init at
> 0.6 s just works, CLX tables included. Three boots so far.
> 
> The wedged GMU after a failed attempt is a different story. Retries after
> rpmhpd's sync_state still die with "GMU firmware initialization timed out"
> until gpucc's sync_state (fw_devlink timeout, 13 s) collapses CX. That's
> what the gpu_cc_cx_gdsc/NO_STAY_ON thing in my earlier mail was about: the
> recovery, not the cause.
> 
> The RSC side is not it. On A8xx the GMU firmware loads the RSC sequencer
> code and TCS slots on its own; preloading them before the first boot made
> no difference.
> 
> Abel's CRD failing even with msm loaded at 90 s doesn't match this, unless
> something keeps gfx.lvl high on the CRD after sync_state. Could be a second
> problem.
> 
> --- a/drivers/pmdomain/qcom/rpmhpd.c
> +++ b/drivers/pmdomain/qcom/rpmhpd.c
> @@ -1009,7 +1009,7 @@ static int rpmhpd_aggregate_corner(struct rpmhpd *pd, unsigned int corner)
> -	if (pd->state_synced) {
> +	if (pd->state_synced || pd == &gfx || pd == &gmxc) {
>  		to_active_sleep(pd, corner, &this_active_corner, &this_sleep_corner);
>  	} else {
>  		/* Clamp to highest corner if sync_state hasn't happened */
> 

Acked-by: Akhil P Oommen <akhilpo@oss.qualcomm.com>

Could you please send this out as a fix for v7.4? I will respin the gmu driver series and revert this
bit there. But that would be v7.5 stuff.

-Akhil.

> Oleg