summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGuchun Chen <guchun.chen@amd.com>2020-02-11 05:10:44 +0100
committerAlex Deucher <alexander.deucher@amd.com>2020-02-11 21:40:12 +0100
commita934f9d866598ec777174d449a0dd903ea3db817 (patch)
tree4290b16ea08be6ea1f154879ad743fa8cd26c993
parentdrm/amd/display: DCN2.x Do not program DPPCLK if same value (diff)
downloadlinux-a934f9d866598ec777174d449a0dd903ea3db817.tar.xz
linux-a934f9d866598ec777174d449a0dd903ea3db817.zip
drm/amdgpu: correct comment to clear up the confusion
Former comment looks to be one intended behavior in code, actually it's not. So correct it. Suggested-by: Christian König <christian.koenig@amd.com> Signed-off-by: Guchun Chen <guchun.chen@amd.com> Acked-by: Christian König <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r--drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index be289f0fff37..b33a4eb39193 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
@@ -4374,7 +4374,12 @@ static int gfx_v9_0_ecc_late_init(void *handle)
struct amdgpu_device *adev = (struct amdgpu_device *)handle;
int r;
- /* limit gds clearing operation in cold boot sequence */
+ /*
+ * Temp workaround to fix the issue that CP firmware fails to
+ * update read pointer when CPDMA is writing clearing operation
+ * to GDS in suspend/resume sequence on several cards. So just
+ * limit this operation in cold boot sequence.
+ */
if (!adev->in_suspend) {
r = gfx_v9_0_do_edc_gds_workarounds(adev);
if (r)