drm/amd/display: Skip SST ACT polling when sink_count is 0

Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git

kernel os linux

[Why]
Upon disconnecting a UHBR SST display, the disconnection and the
SetTimings to disable the display can occur such that link_set_dpms_off
occurs after the disconnection has already processed
(link->type is dc_connection_none). Thus, the AUX related
operations should be skipped when clearing the payload allocation table.

Reviewed-by: Wenjing Liu <wenjing.liu@amd.com>
Acked-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: George Shen <george.shen@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

authored by

George Shen and committed by

Alex Deucher 2 years ago b01d98ca 8acb8368

+4 -2

1 changed file

expand all

drivers

gpu

drm

amd

display

link

link_dpms.c

+4 -2

drivers/gpu/drm/amd/display/dc/link/link_dpms.c

··· 1158 1158 int i; 1159 1159 const int act_retries = 30; 1160 1160 enum act_return_status result = ACT_FAILED; 1161 + enum dc_connection_type display_connected = (link->type != dc_connection_none); 1161 1162 union payload_table_update_status update_status = {0}; 1162 1163 union lane_status dpcd_lane_status[LANE_COUNT_DP_MAX]; 1163 1164 union lane_align_status_updated lane_status_updated; 1164 1165 DC_LOGGER_INIT(link->ctx->logger); 1165 1166 1166 - if (link->aux_access_disabled) 1167 + if (!display_connected || link->aux_access_disabled) 1167 1168 return true; 1168 1169 for (i = 0; i < act_retries; i++) { 1169 1170 get_lane_status(link, link->cur_link_settings.lane_count, dpcd_lane_status, &lane_status_updated); ··· 1513 1512 union payload_table_update_status update_status = { 0 }; 1514 1513 const uint32_t max_retries = 30; 1515 1514 uint32_t retries = 0; 1515 + enum dc_connection_type display_connected = (link->type != dc_connection_none); 1516 1516 DC_LOGGER_INIT(link->ctx->logger); 1517 1517 1518 1518 if (allocate) { ··· 1531 1529 proposed_table->stream_allocations[0].slot_count = req_slot_count; 1532 1530 proposed_table->stream_allocations[0].vcp_id = vc_id; 1533 1531 1534 - if (link->aux_access_disabled) 1532 + if (!display_connected || link->aux_access_disabled) 1535 1533 return true; 1536 1534 1537 1535 /// Write DPCD 2C0 = 1 to start updating

Configure Feed

Configure Feed