[PATCH v2] drm/amd/display: Don't show stack trace for missing eDP

[Hamza Mahfooz]

On 7/31/23 14:02, Mario Limonciello wrote:
> Some systems are only connected by HDMI or DP, so warning related to
> missing eDP is unnecessary.  Downgrade to debug instead.
> 
> Cc: Hamza Mahfooz <hamza.mahfooz at amd.com>
> Fixes: 6d9b6dceaa51 ("drm/amd/display: only warn once in dce110_edp_wait_for_hpd_ready()")
> Reported-by: Mastan.Katragadda at amd.com
> Signed-off-by: Mario Limonciello <mario.limonciello at amd.com>
> ---

Reviewed-by: Hamza Mahfooz <hamza.mahfooz at amd.com>

> v1->v2:
>   * Use DC_LOG_DC instead
> ---
>   drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
> index 20d4d08a6a2f..6966420dfbac 100644
> --- a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
> +++ b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
> @@ -777,7 +777,8 @@ void dce110_edp_wait_for_hpd_ready(
>   	dal_gpio_destroy_irq(&hpd);
>   
>   	/* ensure that the panel is detected */
> -	ASSERT(edp_hpd_high);
> +	if (!edp_hpd_high)
> +		DC_LOG_DC("%s: wait timed out!\n", __func__);
>   }
>   
>   void dce110_edp_power_control(
-- 
Hamza