]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/amd/display: Remove unnecessary divider update flag
authorCruise Hung <Cruise.Hung@amd.com>
Fri, 5 Dec 2025 13:22:44 +0000 (21:22 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 5 Jan 2026 21:59:59 +0000 (16:59 -0500)
[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: Michael Strauss <michael.strauss@amd.com>
Reviewed-by: Wenjing Liu <wenjing.liu@amd.com>
Signed-off-by: Cruise Hung <Cruise.Hung@amd.com>
Signed-off-by: Chenyu Chen <chen-yu.chen@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/link/accessories/link_dp_cts.c

index 1045c268672e2f58bcf6c02db97acb67df808db1..5a547d41d4a1bbbfe0396b682f92a6f6d2826191 100644 (file)
@@ -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]);