Commit 73d08a87 authored by Cruise Hung's avatar Cruise Hung Committed by Alex Deucher
Browse files

drm/amd/display: Remove unnecessary divider update flag



[Why]
When transitioning from 640x480 at RBRx1 to HBR3x1,
both output pixel mode and pixel rate divider should update.
The needs_divider_update flag was only for 8b10b and 128b132b transition.

[How]
Remove needs_divider_update flag.

Reviewed-by: default avatarMichael Strauss <michael.strauss@amd.com>
Reviewed-by: default avatarWenjing Liu <wenjing.liu@amd.com>
Signed-off-by: default avatarCruise Hung <Cruise.Hung@amd.com>
Signed-off-by: default avatarChenyu Chen <chen-yu.chen@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 878a4b73
Loading
Loading
Loading
Loading
+1 −5
Original line number Diff line number Diff line
@@ -70,7 +70,6 @@ static void dp_retrain_link_dp_test(struct dc_link *link,
	struct dc_state *state = link->dc->current_state;
	struct dc_stream_update stream_update = { 0 };
	bool dpms_off = false;
	bool needs_divider_update = false;
	bool was_hpo_acquired = resource_is_hpo_acquired(link->dc->current_state);
	bool is_hpo_acquired;
	uint8_t count;
@@ -80,9 +79,6 @@ static void dp_retrain_link_dp_test(struct dc_link *link,
	int num_streams_on_link = 0;
	struct dc *dc = (struct dc *)link->dc;

	needs_divider_update = (link->dc->link_srv->dp_get_encoding_format(link_setting) !=
	link->dc->link_srv->dp_get_encoding_format((const struct dc_link_settings *) &link->cur_link_settings));

	udelay(100);

	link_get_master_pipes_with_dpms_on(link, state, &count, pipes);
@@ -99,7 +95,7 @@ static void dp_retrain_link_dp_test(struct dc_link *link,
		pipes[i]->stream_res.tg->funcs->disable_crtc(pipes[i]->stream_res.tg);
	}

	if (needs_divider_update && link->dc->res_pool->funcs->update_dc_state_for_encoder_switch) {
	if (link->dc->res_pool->funcs->update_dc_state_for_encoder_switch) {
		link->dc->res_pool->funcs->update_dc_state_for_encoder_switch(link,
				link_setting, count,
				*pipes, &audio_output[0]);