drm/amd/display: only warn once in dce110_edp_wait_for_hpd_ready()
authorHamza Mahfooz <hamza.mahfooz@amd.com>
Tue, 14 Feb 2023 18:51:08 +0000 (13:51 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 23 Feb 2023 22:35:58 +0000 (17:35 -0500)
Since, hot plugging eDP displays isn't supported, it is sufficient for
us to warn about the lack of a connected display once. So, use ASSERT()
in dce110_edp_wait_for_hpd_ready() instead of DC_LOG_WARNING().

Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Hamza Mahfooz <hamza.mahfooz@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c

index fb3fd5b7c78b995ee7f0eaf411aa30e58063be41..0d4d3d586166d54c4f6559d999deeeeb3ba9559e 100644 (file)
@@ -779,10 +779,8 @@ void dce110_edp_wait_for_hpd_ready(
 
        dal_gpio_destroy_irq(&hpd);
 
-       if (false == edp_hpd_high) {
-               DC_LOG_WARNING(
-                               "%s: wait timed out!\n", __func__);
-       }
+       /* ensure that the panel is detected */
+       ASSERT(edp_hpd_high);
 }
 
 void dce110_edp_power_control(