Revert "drm/amd/display: disable S/G display on DCN 3.1.2/3"
authorAlex Deucher <alexander.deucher@amd.com>
Thu, 9 Feb 2023 14:07:42 +0000 (09:07 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 9 Feb 2023 15:29:28 +0000 (10:29 -0500)
This reverts commit f081cd4ca2658752a8c0e2353d50aec80d07c65f.

Some users have reported flickerng with S/G display.  We've
tried extensively to reproduce and debug the issue on a wide
variety of platform configurations (DRAM bandwidth, etc.) and
a variety of monitors, but so far have not been able to.  We
disabled S/G display on a number of platforms to address this
but that leads to failure to pin framebuffers errors and
blank displays when there is memory pressure or no displays
at all on systems with limited carveout (e.g., Chromebooks).
We have a parameter to disable this as a debugging option as a
way for users to disable this, depending on their use case,
and for us to help debug this further.  Having this enabled
seems like the lesser of to evils.

Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

index c1da9ac02e7ba80c17b3dfca543045a230279541..1a15c3a97491d055a1331b582f471171724a4aa3 100644 (file)
@@ -1514,6 +1514,8 @@ static int amdgpu_dm_init(struct amdgpu_device *adev)
                                init_data.flags.gpu_vm_support = true;
                        break;
                case IP_VERSION(3, 0, 1):
+               case IP_VERSION(3, 1, 2):
+               case IP_VERSION(3, 1, 3):
                case IP_VERSION(3, 1, 4):
                case IP_VERSION(3, 1, 6):
                        init_data.flags.gpu_vm_support = true;