From: Jessica Zhang <[email protected]>

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

Signed-off-by: Jessica Zhang <[email protected]>
[DB: dropped link_ready handling]
Tested-by: Val Packett <[email protected]> # x1e80100-dell-latitude-7455
Tested-by: Yongxing Mou <[email protected]> # Hamoa IOT EVK, 
QCS8300 Ride
Signed-off-by: Dmitry Baryshkov <[email protected]>
---
 drivers/gpu/drm/msm/dp/dp_display.c | 30 ++++++++++++++++++------------
 1 file changed, 18 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/msm/dp/dp_display.c 
b/drivers/gpu/drm/msm/dp/dp_display.c
index cdf8e618838a..fc41ac38556d 100644
--- 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;
 }

-- 
2.47.3

Reply via email to