[Intel-gfx] [PATCH 47/49] drm/i915: Don't write the HDMI buffer translation entry when not needed

[Sivakumar Thulasimani]

Reviewed-by: Sivakumar Thulasimani <sivakumar.thulasimani at intel.com>

On 3/17/2015 3:10 PM, Imre Deak wrote:
> From: Damien Lespiau <damien.lespiau at intel.com>
>
> We don't actually need to write the HDMI entry on DDIs that have no
> chance to be used as HDMI ports.
>
> While this patch shouldn't change the current behaviour, it makes
> further enabling work easier as we'll have an eDP table filling the full
> 10 entries.
>
> v2: Rely on the logic from intel_ddi_init() to figure out if the DDI port
>      supports HDMI or not (Paulo).
>
> Suggested-by: Satheeshakrishna M <satheeshakrishna.m at intel.com>
> Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
> ---
>   drivers/gpu/drm/i915/intel_ddi.c | 9 +++++++++
>   1 file changed, 9 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
> index 5c18018..5aa4dab 100644
> --- a/drivers/gpu/drm/i915/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/intel_ddi.c
> @@ -182,6 +182,12 @@ enum port intel_ddi_get_encoder_port(struct intel_encoder *intel_encoder)
>   	}
>   }
>   
> +static bool
> +intel_dig_port_supports_hdmi(const struct intel_digital_port *intel_dig_port)
> +{
> +	return intel_dig_port->hdmi.hdmi_reg;
> +}
> +
>   /*
>    * Starting with Haswell, DDI port buffers must be programmed with correct
>    * values in advance. The buffer values are different for FDI and DP modes,
> @@ -292,6 +298,9 @@ static void intel_prepare_ddi_buffers(struct drm_device *dev,
>   		reg += 4;
>   	}
>   
> +	if (!intel_dig_port_supports_hdmi(intel_dig_port))
> +		return;
> +
>   	/* Choose a good default if VBT is badly populated */
>   	if (hdmi_level == HDMI_LEVEL_SHIFT_UNKNOWN ||
>   	    hdmi_level >= n_hdmi_entries)