From: Aurabindo Pillai <aurabindo.pil...@amd.com>

Unconditionally reading DP tunneling support results in extraneous
errors messages on certain devices. Fix this by guarding the DPCD read
for DP tunneling support for USB4 DPIA endpoints.

Reviewed-by: Meenakshikumar Somasundaram <meenakshikumar.somasunda...@amd.com>
Signed-off-by: Aurabindo Pillai <aurabindo.pil...@amd.com>
Signed-off-by: Hamza Mahfooz <hamza.mahf...@amd.com>
---
 .../amd/display/dc/link/protocols/link_dp_capability.c    | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c 
b/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
index 72ef0c3a7ebd..9dabaf682171 100644
--- a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
+++ b/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
@@ -1633,9 +1633,11 @@ static bool retrieve_link_cap(struct dc_link *link)
        }
 
        /* Read DP tunneling information. */
-       status = dpcd_get_tunneling_device_data(link);
-       if (status != DC_OK)
-               dm_error("%s: Read tunneling device data failed.\n", __func__);
+       if (link->ep_type == DISPLAY_ENDPOINT_USB4_DPIA) {
+               status = dpcd_get_tunneling_device_data(link);
+               if (status != DC_OK)
+                       dm_error("%s: Read tunneling device data failed.\n", 
__func__);
+       }
 
        dpcd_set_source_specific_data(link);
        /* Sink may need to configure internals based on vendor, so allow some
-- 
2.46.1

Reply via email to