drm/amd/display: Remove optimization for VRR updates

Alvin Lee · gregkh · commit dd6d6f9d47ae · 2023-07-05T18:27:37.000+01:00
commit 3442f4e upstream. Optimization caused unexpected regression, so remove for now. Tested-by: Mark Broadworth <mark.broadworth@amd.com> Reviewed-by: Aric Cyr <Aric.Cyr@amd.com> Acked-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> Signed-off-by: Alvin Lee <Alvin.Lee2@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Cc: Mario Limonciello <mario.limonciello@amd.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -401,9 +401,6 @@ bool dc_stream_adjust_vmin_vmax(struct dc *dc,
 {
 	int i;
 
-	if (memcmp(adjust, &stream->adjust, sizeof(struct dc_crtc_timing_adjust)) == 0)
-		return true;
-
 	stream->adjust.v_total_max = adjust->v_total_max;
 	stream->adjust.v_total_mid = adjust->v_total_mid;
 	stream->adjust.v_total_mid_frame_num = adjust->v_total_mid_frame_num;

Original file line number	Diff line number	Diff line change
`@@ -401,9 +401,6 @@ bool dc_stream_adjust_vmin_vmax(struct dc *dc,`
`401`	`401`	`{`
`402`	`402`	`int i;`
`403`	`403`
`404`		`- if (memcmp(adjust, &stream->adjust, sizeof(struct dc_crtc_timing_adjust)) == 0)`
`405`		`- return true;`
`406`		`-`
`407`	`404`	`stream->adjust.v_total_max = adjust->v_total_max;`
`408`	`405`	`stream->adjust.v_total_mid = adjust->v_total_mid;`
`409`	`406`	`stream->adjust.v_total_mid_frame_num = adjust->v_total_mid_frame_num;`