On 4/13/23 12:17, Srinivasan Shanmugam wrote:
Log if we fail to setup sink power states.

Cc: Fangzhi Zuo <jerry....@amd.com>
Cc: Aurabindo Pillai <aurabindo.pil...@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmu...@amd.com>
---
  .../gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c | 3 +++
  1 file changed, 3 insertions(+)

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 c840ef17802e..adf6a67e0e5d 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
@@ -1043,6 +1043,9 @@ static enum dc_status wake_up_aux_channel(struct dc_link 
*link)
                                DP_SET_POWER,
                                &dpcd_power_state,
                                sizeof(dpcd_power_state));
+               if (status < 0)
+                       DC_LOG_HW_LINK_TRAINING("%s: Failed to power up sink: 
%s\n", __func__,
+                                               dpcd_power_state == DP_SET_POWER_D0 ? 
"D0" : "D3");

Isnt this more of a DC_LOG_DC ? Link training related code was moved to a separate file.

                return DC_ERROR_UNEXPECTED;
        }

Reply via email to