Skip to content

Commit

Permalink
drm/amd/display: only warn once in dce110_edp_wait_for_hpd_ready()
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Hamza Mahfooz authored and Alex Deucher committed Feb 23, 2023
1 parent d9e1e14 commit 6d9b6dc
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
Original file line number Diff line number Diff line change
Expand Up @@ -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(
Expand Down

0 comments on commit 6d9b6dc

Please sign in to comment.