drm/amd/display: Fix incorrect DSC recompute trigger
authorFangzhi Zuo <Jerry.Zuo@amd.com>
Wed, 4 Sep 2024 20:56:45 +0000 (16:56 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 1 Oct 2024 21:31:10 +0000 (17:31 -0400)
A stream without dsc_aux should not be eliminated from
the dsc determination. Whether it needs a dsc recompute depends on
whether its mode has changed or not. Eliminating such a no-dsc stream
from the dsc determination policy will end up with inconsistencies
in the new dc_state when compared to the current dc_state,
triggering a dsc recompute that should not have happened.

Reviewed-by: Rodrigo Siqueira <rodrigo.siqueira@amd.com>
Signed-off-by: Fangzhi Zuo <Jerry.Zuo@amd.com>
Signed-off-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c

index a08e8a0..68bd8e6 100644 (file)
@@ -1302,7 +1302,7 @@ static bool is_dsc_need_re_compute(
                        continue;
 
                aconnector = (struct amdgpu_dm_connector *) stream->dm_stream_context;
-               if (!aconnector || !aconnector->dsc_aux)
+               if (!aconnector)
                        continue;
 
                stream_on_link[new_stream_on_link_num] = aconnector;