[Intel-gfx] [PATCH] drm/i915/tgl: allow DVI/HDMI on port A

[Jani Nikula]

On Tue, 12 Nov 2019, Lucas De Marchi <lucas.demarchi at intel.com> wrote:
> Tiger Lake supports HDMI on port A. For other platforms we ignore what the VBT
> says regarding HDMI to workaround broken VBTs, see commit
> 2ba7d7e04371 ("drm/i915/bios: ignore HDMI on port A"). Make this apply
> gen12+ so they inherit the TGL behavior.
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>

Do *NOT* merge patches without full IGT results. BAT is *NOT* enough,
and this did not even pass BAT.

BR,
Jani.


> ---
>  drivers/gpu/drm/i915/display/intel_bios.c | 2 +-
>  drivers/gpu/drm/i915/display/intel_hdmi.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_bios.c b/drivers/gpu/drm/i915/display/intel_bios.c
> index 7c0ca733bef8..e6da98729e1f 100644
> --- a/drivers/gpu/drm/i915/display/intel_bios.c
> +++ b/drivers/gpu/drm/i915/display/intel_bios.c
> @@ -1450,7 +1450,7 @@ static void parse_ddi_port(struct drm_i915_private *dev_priv,
>  	is_hdmi = is_dvi && (child->device_type & DEVICE_TYPE_NOT_HDMI_OUTPUT) == 0;
>  	is_edp = is_dp && (child->device_type & DEVICE_TYPE_INTERNAL_CONNECTOR);
>  
> -	if (port == PORT_A && is_dvi) {
> +	if (port == PORT_A && is_dvi && INTEL_GEN(dev_priv) < 12) {
>  		DRM_DEBUG_KMS("VBT claims port A supports DVI%s, ignoring\n",
>  			      is_hdmi ? "/HDMI" : "");
>  		is_dvi = false;
> diff --git a/drivers/gpu/drm/i915/display/intel_hdmi.c b/drivers/gpu/drm/i915/display/intel_hdmi.c
> index ed4a68fb351f..659a03b08849 100644
> --- a/drivers/gpu/drm/i915/display/intel_hdmi.c
> +++ b/drivers/gpu/drm/i915/display/intel_hdmi.c
> @@ -3140,7 +3140,7 @@ void intel_hdmi_init_connector(struct intel_digital_port *intel_dig_port,
>  	DRM_DEBUG_KMS("Adding HDMI connector on [ENCODER:%d:%s]\n",
>  		      intel_encoder->base.base.id, intel_encoder->base.name);
>  
> -	if (WARN_ON(port == PORT_A))
> +	if (INTEL_GEN(dev_priv) < 12 && WARN_ON(port == PORT_A))
>  		return;
>  
>  	if (WARN(intel_dig_port->max_lanes < 4,

-- 
Jani Nikula, Intel Open Source Graphics Center