Commit 9a5d9c48 authored by Leo (Sunpeng) Li's avatar Leo (Sunpeng) Li Committed by Alex Deucher
Browse files

drm/amd/display: Report stream as different on scaling change 



When scaling is enabled, our preference is to scale up to the prefered
(native) mode. This means that hardware timings will be the same across
a modeset.

Therefore, also report mode as changed if source or destination
rectangle is different.

Signed-off-by: default avatarLeo (Sunpeng) Li <sunpeng.li@amd.com>
Reviewed-by: default avatarHarry Wentland <Harry.Wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 199e458a

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

+3 −2
Original line number Diff line number Diff line
@@ -4428,7 +4428,8 @@ static int dm_update_crtcs_state(struct dc *dc, 
			}

		}



		if (dc_is_stream_unchanged(new_stream, dm_old_crtc_state->stream)) {

		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;

drivers/gpu/drm/amd/display/dc/core/dc_resource.c

+22 −0
Original line number Diff line number Diff line
@@ -1335,6 +1335,28 @@ bool dc_is_stream_unchanged( 
	return true;

}



bool dc_is_stream_scaling_unchanged(

	struct dc_stream_state *old_stream, struct dc_stream_state *stream)

{

	if (old_stream == stream)

		return true;



	if (old_stream == NULL || stream == NULL)

		return false;



	if (memcmp(&old_stream->src,

			&stream->src,

			sizeof(struct rect)) != 0)

		return false;



	if (memcmp(&old_stream->dst,

			&stream->dst,

			sizeof(struct rect)) != 0)

		return false;



	return true;

}



/* Maximum TMDS single link pixel clock 165MHz */

#define TMDS_MAX_PIXEL_CLOCK_IN_KHZ 165000

drivers/gpu/drm/amd/display/dc/dc.h

+2 −0
Original line number Diff line number Diff line
@@ -627,6 +627,8 @@ struct dc_stream_update { 


bool dc_is_stream_unchanged(

	struct dc_stream_state *old_stream, struct dc_stream_state *stream);

bool dc_is_stream_scaling_unchanged(

	struct dc_stream_state *old_stream, struct dc_stream_state *stream);



/*

 * Set up surface attributes and associate to a stream