Re: [PATCH] drm/i915/gen9+: Rely on VBT DDI port info for eDP detection

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Tue, Dec 20, 2016 at 05:12:49PM +0200, Imre Deak wrote:
> There is at least one APL based system using port A in DP mode
> (connecting to an on-board DP->VGA adaptor). Atm we'll configure port A
> unconditionally as eDP which is incorrect in this case. Fix this by
> relying on the VBT DDI port 'internal port' flag instead on all ports
> GEN9 onwards. For now chicken out from doing this on earlier DDI
> platforms too.
> 
> Signed-off-by: Imre Deak <imre.deak@intel.com>

Yeah, I think we should try to move towards using the VBT more
consistently here. Eventually we migth even want to unify everything to
using the pre-parsed port infromation, but going in babt steps is a
little less likely to regress old machines.

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

At one point when I was monkeying around with this VBT stuff Paulo
suggested that we should just use the pre-parsed stuff more, so presumably
he should be happy with this direction as well. Although I think that
specific case might have been about the DP++ stuff.

> ---
>  drivers/gpu/drm/i915/i915_drv.h   | 1 +
>  drivers/gpu/drm/i915/intel_bios.c | 1 +
>  drivers/gpu/drm/i915/intel_dp.c   | 3 +++
>  3 files changed, 5 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 7e94e506..42fb7e9 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -1669,6 +1669,7 @@ struct ddi_vbt_port_info {
>  	uint8_t supports_dvi:1;
>  	uint8_t supports_hdmi:1;
>  	uint8_t supports_dp:1;
> +	uint8_t supports_edp:1;
>  
>  	uint8_t alternate_aux_channel;
>  	uint8_t alternate_ddc_pin;
> diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c
> index 1cf2fa6..60fdd31 100644
> --- a/drivers/gpu/drm/i915/intel_bios.c
> +++ b/drivers/gpu/drm/i915/intel_bios.c
> @@ -1161,6 +1161,7 @@ static void parse_ddi_port(struct drm_i915_private *dev_priv, enum port port,
>  	info->supports_dvi = is_dvi;
>  	info->supports_hdmi = is_hdmi;
>  	info->supports_dp = is_dp;
> +	info->supports_edp = is_edp;
>  
>  	DRM_DEBUG_KMS("Port %c VBT info: DP:%d HDMI:%d DVI:%d EDP:%d CRT:%d\n",
>  		      port_name(port), is_dp, is_hdmi, is_dvi, is_edp, is_crt);
> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
> index d37cb00..7f3f4fa 100644
> --- a/drivers/gpu/drm/i915/intel_dp.c
> +++ b/drivers/gpu/drm/i915/intel_dp.c
> @@ -4983,6 +4983,9 @@ bool intel_dp_is_edp(struct drm_i915_private *dev_priv, enum port port)
>  	if (INTEL_GEN(dev_priv) < 5)
>  		return false;
>  
> +	if (INTEL_GEN(dev_priv) >= 9)
> +		return dev_priv->vbt.ddi_port_info[port].supports_edp;
> +
>  	if (port == PORT_A)
>  		return true;
>  
> -- 
> 2.5.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx