Skip to content

Commit a934f9d

Browse files
Guchun Chenalexdeucher
authored andcommitted
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 <[email protected]> Signed-off-by: Guchun Chen <[email protected]> Acked-by: Christian König <[email protected]> Signed-off-by: Alex Deucher <[email protected]>
1 parent 8fab6a2 commit a934f9d

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4374,7 +4374,12 @@ static int gfx_v9_0_ecc_late_init(void *handle)
43744374
struct amdgpu_device *adev = (struct amdgpu_device *)handle;
43754375
int r;
43764376

4377-
/* limit gds clearing operation in cold boot sequence */
4377+
/*
4378+
* Temp workaround to fix the issue that CP firmware fails to
4379+
* update read pointer when CPDMA is writing clearing operation
4380+
* to GDS in suspend/resume sequence on several cards. So just
4381+
* limit this operation in cold boot sequence.
4382+
*/
43784383
if (!adev->in_suspend) {
43794384
r = gfx_v9_0_do_edc_gds_workarounds(adev);
43804385
if (r)

0 commit comments

Comments
 (0)