[PATCH] drm/amd/display: Only require EDID read for HDMI and DVI

[Alex Deucher]

On Thu, Aug 2, 2018 at 3:38 PM, Harry Wentland <harry.wentland at amd.com> wrote:
> [Why]
> VGA sometimes has trouble retrieving the EDID on very long cables, KVM
> switches, or old displays.
>
> [How]
> Only require EDID read for HDMI and DVI and exempt other types (DP,
> VGA). We currently don't support VGA but if anyone adds support in the
> future this might get overlooked.
>
> Signed-off-by: Harry Wentland <harry.wentland at amd.com>
> Suggested-by: Michel Dänzer <michel at daenzer.net>

Acked-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/display/dc/core/dc_link.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> index d6086c591f75..71b94e3b472e 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> @@ -753,7 +753,8 @@ bool dc_link_detect(struct dc_link *link, enum dc_detect_reason reason)
>                          * even if we have no EDID in order to go to
>                          * fail-safe mode
>                          */
> -                       if (!dc_is_dp_signal(link->connector_signal))
> +                       if (dc_is_hdmi_signal(link->connector_signal) ||
> +                           dc_is_dvi_signal(link->connector_signal))
>                                 return false;
>                 default:
>                         break;
> --
> 2.17.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx