summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWayne Lin <wayne.lin@amd.com>2024-05-27 09:17:26 +0200
committerAlex Deucher <alexander.deucher@amd.com>2024-06-14 22:17:16 +0200
commit9a573481887e330b808e88005ffa142092c9fdbb (patch)
treea70de0c046ca595460536abcbdc81c986bdc7f95
parentdrm/amd/display: Adjust debug msg for usb4/tbt (diff)
downloadlinux-9a573481887e330b808e88005ffa142092c9fdbb.tar.xz
linux-9a573481887e330b808e88005ffa142092c9fdbb.zip
drm/amd/display: Defer the setting of link hpd status for usb4/tbt
[Why & How] Link hpd status is set during link detection process via dpia_query_hpd_status(), doesn't need to explicitly set it during outbox irq. Remove it. Reviewed-by: Jerry Zuo <jerry.zuo@amd.com> Acked-by: Zaeem Mohamed <zaeem.mohamed@amd.com> Signed-off-by: Wayne Lin <wayne.lin@amd.com> Tested-by: Daniel Wheeler <daniel.wheeler@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r--drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c8
1 files changed, 0 insertions, 8 deletions
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index 43ee0618ebf5..43ac3e44d74f 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -863,7 +863,6 @@ static void dm_dmub_outbox1_low_irq(void *interrupt_params)
struct dmcub_trace_buf_entry entry = { 0 };
u32 count = 0;
struct dmub_hpd_work *dmub_hpd_wrk;
- struct dc_link *plink = NULL;
static const char *const event_type[] = {
"NO_DATA",
"AUX_REPLY",
@@ -919,13 +918,6 @@ static void dm_dmub_outbox1_low_irq(void *interrupt_params)
}
INIT_WORK(&dmub_hpd_wrk->handle_hpd_work, dm_handle_hpd_work);
dmub_hpd_wrk->adev = adev;
- if (notify.type == DMUB_NOTIFICATION_HPD) {
- plink = adev->dm.dc->links[notify.link_index];
- if (plink) {
- plink->hpd_status =
- notify.hpd_status == DP_HPD_PLUG;
- }
- }
queue_work(adev->dm.delayed_hpd_wq, &dmub_hpd_wrk->handle_hpd_work);
} else {
dm->dmub_callback[notify.type](adev, &notify);