drm/msm/dp: Add sink_count to debug logs

Add sink count to the debug logs for [un]plug and HPD IRQ handling.

Signed-off-by: Jessica Zhang <jessica.zhang@oss.qualcomm.com>
[DB: dropped link_ready handling]
Tested-by: Val Packett <val@packett.cool> # x1e80100-dell-latitude-7455
Tested-by: Yongxing Mou <yongxing.mou@oss.qualcomm.com> # Hamoa IOT EVK, QCS8300 Ride
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Patchwork: https://patchwork.freedesktop.org/patch/727620/
Link: https://lore.kernel.org/r/20260524-hpd-refactor-v6-8-cf3ab488dd7b@oss.qualcomm.com

diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c
index 9292c475e4e78bf44cfc6e0bfc238452ee84e6f9..bf31bc3f83770e815f0023f76875cf0df123f35b 100644 (file)
--- a/drivers/gpu/drm/msm/dp/dp_display.c
+++ b/drivers/gpu/drm/msm/dp/dp_display.c
@@ -389,8 +389,9 @@ static int msm_dp_hpd_plug_handle(struct msm_dp_display_private *dp)
        int ret;
        struct platform_device *pdev = dp->msm_dp_display.pdev;
 
-       drm_dbg_dp(dp->drm_dev, "Before, type=%d\n",
-                       dp->msm_dp_display.connector_type);
+       drm_dbg_dp(dp->drm_dev, "Before, type=%d sink_count=%d\n",
+                       dp->msm_dp_display.connector_type,
+                       dp->link->sink_count);
 
        if (dp->msm_dp_display.link_ready)
                return 0;
@@ -412,8 +413,9 @@ static int msm_dp_hpd_plug_handle(struct msm_dp_display_private *dp)
                pm_runtime_put_sync(&pdev->dev);
        }
 
-       drm_dbg_dp(dp->drm_dev, "After, type=%d\n",
-                       dp->msm_dp_display.connector_type);
+       drm_dbg_dp(dp->drm_dev, "After, type=%d sink_count=%d\n",
+                       dp->msm_dp_display.connector_type,
+                       dp->link->sink_count);
 
        /* uevent will complete connection part */
        return 0;
@@ -441,8 +443,9 @@ static int msm_dp_hpd_unplug_handle(struct msm_dp_display_private *dp)
 
        msm_dp_aux_enable_xfers(dp->aux, false);
 
-       drm_dbg_dp(dp->drm_dev, "Before, type=%d\n",
-                       dp->msm_dp_display.connector_type);
+       drm_dbg_dp(dp->drm_dev, "Before, type=%d sink_count=%d\n",
+                       dp->msm_dp_display.connector_type,
+                       dp->link->sink_count);
 
        if (!dp->msm_dp_display.link_ready)
                return 0;
@@ -466,8 +469,9 @@ static int msm_dp_hpd_unplug_handle(struct msm_dp_display_private *dp)
        /* signal the disconnect event early to ensure proper teardown */
        msm_dp_display_handle_plugged_change(&dp->msm_dp_display, false);
 
-       drm_dbg_dp(dp->drm_dev, "After, type=%d\n",
-                       dp->msm_dp_display.connector_type);
+       drm_dbg_dp(dp->drm_dev, "After, type=%d, sink_count=%d\n",
+                       dp->msm_dp_display.connector_type,
+                       dp->link->sink_count);
 
        /* uevent will complete disconnection part */
        pm_runtime_put_sync(&pdev->dev);
@@ -480,8 +484,9 @@ static int msm_dp_irq_hpd_handle(struct msm_dp_display_private *dp)
        int rc = 0;
 
        /* irq_hpd can happen at either connected or disconnected state */
-       drm_dbg_dp(dp->drm_dev, "Before, type=%d\n",
-                       dp->msm_dp_display.connector_type);
+       drm_dbg_dp(dp->drm_dev, "Before, type=%d, sink_count=%d\n",
+                       dp->msm_dp_display.connector_type,
+                       dp->link->sink_count);
 
        /* check for any test request issued by sink */
        rc = msm_dp_link_process_request(dp->link);
@@ -494,8 +499,9 @@ static int msm_dp_irq_hpd_handle(struct msm_dp_display_private *dp)
                        rc = msm_dp_display_handle_irq_hpd(dp);
        }
 
-       drm_dbg_dp(dp->drm_dev, "After, type=%d\n",
-                       dp->msm_dp_display.connector_type);
+       drm_dbg_dp(dp->drm_dev, "After, type=%d, sink_count=%d\n",
+                       dp->msm_dp_display.connector_type,
+                       dp->link->sink_count);
 
        return rc;
 }