drm/amd/powerplay: show gfxclk=0 in gfxoff state
authorKenneth Feng <kenneth.feng@amd.com>
Thu, 7 May 2020 06:33:06 +0000 (14:33 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 1 Jul 2020 05:59:13 +0000 (01:59 -0400)
The instant retrieved gfxclk value should be 0 in gfxoff state.
This can be fetched with gfxoff enabled.

Signed-off-by: Kenneth Feng <kenneth.feng@amd.com>
Reviewed-by: Likun Gao <Likun.Gao@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/powerplay/sienna_cichlid_ppt.c

index 1011413..c924ecf 100644 (file)
@@ -709,9 +709,6 @@ static int sienna_cichlid_print_clk_levels(struct smu_context *smu,
        uint32_t mark_index = 0;
        uint32_t gen_speed, lane_width;
 
-       if ((clk_type == SMU_GFXCLK) || (clk_type == SMU_SCLK))
-               amdgpu_gfx_off_ctrl(adev, false);
-
        switch (clk_type) {
        case SMU_GFXCLK:
        case SMU_SCLK:
@@ -727,6 +724,10 @@ static int sienna_cichlid_print_clk_levels(struct smu_context *smu,
                /* 10KHz -> MHz */
                cur_value = cur_value / 100;
 
+               /* no need to disable gfxoff when retrieving the current gfxclk */
+               if ((clk_type == SMU_GFXCLK) || (clk_type == SMU_SCLK))
+                       amdgpu_gfx_off_ctrl(adev, false);
+
                ret = smu_get_dpm_level_count(smu, clk_type, &count);
                if (ret)
                        goto print_clk_out;