Commit 34316c1e authored by Nicholas Kazlauskas's avatar Nicholas Kazlauskas Committed by Alex Deucher
Browse files

drm/amd/display: Optimize bandwidth on following fast update 



[Why]
The current call to optimize_bandwidth never occurs because flip is
always pending from the FULL and FAST updates.

[How]
Optimize on the following flip when it's a FAST update and we know we
aren't going to be modifying the clocks again.

Reviewed-by: default avatarBhawanpreet Lakha <bhawanpreet.lakha@amd.com>
Acked-by: default avatarMikita Lipski <mikita.lipski@amd.com>
Signed-off-by: default avatarNicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 2a50edbf

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

+8 −9
Original line number Diff line number Diff line
@@ -1801,6 +1801,11 @@ void dc_post_update_surfaces_to_stream(struct dc *dc) 


	post_surface_trace(dc);



	if (dc->ctx->dce_version >= DCE_VERSION_MAX)

		TRACE_DCN_CLOCK_STATE(&context->bw_ctx.bw.dcn.clk);

	else

		TRACE_DCE_CLOCK_STATE(&context->bw_ctx.bw.dce);



	if (is_flip_pending_in_pipes(dc, context))

		return;
@@ -2986,6 +2991,9 @@ void dc_commit_updates_for_stream(struct dc *dc, 
			if (new_pipe->plane_state && new_pipe->plane_state != old_pipe->plane_state)

				new_pipe->plane_state->force_full_update = true;

		}

	} else if (update_type == UPDATE_TYPE_FAST) {

		/* Previous frame finished and HW is ready for optimization. */

		dc_post_update_surfaces_to_stream(dc);

	}
@@ -3042,15 +3050,6 @@ void dc_commit_updates_for_stream(struct dc *dc, 
				pipe_ctx->plane_state->force_full_update = false;

		}

	}

	/*let's use current_state to update watermark etc*/

	if (update_type >= UPDATE_TYPE_FULL) {

		dc_post_update_surfaces_to_stream(dc);



		if (dc_ctx->dce_version >= DCE_VERSION_MAX)

			TRACE_DCN_CLOCK_STATE(&context->bw_ctx.bw.dcn.clk);

		else

			TRACE_DCE_CLOCK_STATE(&context->bw_ctx.bw.dce);

	}



	return;