drm/amd/display: Do DC mode-change check after stream creation

Do DC level mode change checks (via dc_stream_state) only when creating
a new stream, as this check is uneccessary without a new dc_stream_state
anyways. Doing so better demonstrates the intent of this mode-change
check, in comparison to guarding it with the 'enable' flag.

Signed-off-by: Leo (Sunpeng) Li <sunpeng.li@amd.com>
Reviewed-by: Harry Wentland <Harry.Wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index 1cf51bcbbb4a3a0351f3b93ee6fe8c2f67e7f9f1..232c7ea7baa8151a8f9404a44c54cbc7466db52b 100644 (file)
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -4513,18 +4513,15 @@ static int dm_update_crtcs_state(struct dc *dc,
                                                __func__, acrtc->base.base.id);
                                break;
                        }
-               }
-
-               if (enable && dc_is_stream_unchanged(new_stream, dm_old_crtc_state->stream) &&
-                               dc_is_stream_scaling_unchanged(new_stream, dm_old_crtc_state->stream)) {
-
-                       new_crtc_state->mode_changed = false;
 
-                       DRM_DEBUG_DRIVER("Mode change not required, setting mode_changed to %d",
-                                        new_crtc_state->mode_changed);
+                       if (dc_is_stream_unchanged(new_stream, dm_old_crtc_state->stream) &&
+                           dc_is_stream_scaling_unchanged(new_stream, dm_old_crtc_state->stream)) {
+                               new_crtc_state->mode_changed = false;
+                               DRM_DEBUG_DRIVER("Mode change not required, setting mode_changed to %d",
+                                                new_crtc_state->mode_changed);
+                       }
                }
 
-
                if (!drm_atomic_crtc_needs_modeset(new_crtc_state))
                        goto next_crtc;