Revert "drm/amd/display: Reset freesync config before update new state"

leo-sunli1 · alexdeucher · commit dc1000bf463d · 2024-07-11T20:48:42.000-04:00
This change caused PSR SU panels to not read from their remote fb, preventing us from entering self-refresh. It is a regression. This reverts commit 6b8487c. Signed-off-by: Leo Li <sunpeng.li@amd.com> Acked-by: Alex Deucher <alexander.deucher@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
@@ -10048,7 +10048,6 @@ static int dm_update_crtc_state(struct amdgpu_display_manager *dm,
 	}
 
 	/* Update Freesync settings. */
-	reset_freesync_config_for_crtc(dm_new_crtc_state);
 	get_freesync_config_for_crtc(dm_new_crtc_state,
 				     dm_new_conn_state);
 

Original file line number	Diff line number	Diff line change
`@@ -10048,7 +10048,6 @@ static int dm_update_crtc_state(struct amdgpu_display_manager *dm,`
`10048`	`10048`	`}`
`10049`	`10049`
`10050`	`10050`	`/* Update Freesync settings. */`
`10051`		`- reset_freesync_config_for_crtc(dm_new_crtc_state);`
`10052`	`10051`	`get_freesync_config_for_crtc(dm_new_crtc_state,`
`10053`	`10052`	`dm_new_conn_state);`
`10054`	`10053`