Re: [PATCH v2 1/2] drm/i915: Use port clock to set correct N value

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

On Thu, Jun 27, 2019 at 03:07:07PM -0700, Aditya Swarup wrote:
> Use port_clock to check the clock values in n/cts lookup table instead
> of crtc_clock. As port_clock is already adjusted based on color mode set
> (8 bit or deep color), this will help in checking clock values for deep
> color modes from n/cts lookup table.
> 
> Signed-off-by: Aditya Swarup <aditya.swarup@intel.com>
> Cc: Clint Taylor <Clinton.A.Taylor@intel.com>
> Cc: Jani Nikula <jani.nikula@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>

Pushed the series to dinq. Thanks.

> ---
>  drivers/gpu/drm/i915/display/intel_audio.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_audio.c b/drivers/gpu/drm/i915/display/intel_audio.c
> index 840daff12246..156d524c29d9 100644
> --- a/drivers/gpu/drm/i915/display/intel_audio.c
> +++ b/drivers/gpu/drm/i915/display/intel_audio.c
> @@ -212,13 +212,11 @@ static u32 audio_config_hdmi_pixel_clock(const struct intel_crtc_state *crtc_sta
>  static int audio_config_hdmi_get_n(const struct intel_crtc_state *crtc_state,
>  				   int rate)
>  {
> -	const struct drm_display_mode *adjusted_mode =
> -		&crtc_state->base.adjusted_mode;
>  	int i;
>  
>  	for (i = 0; i < ARRAY_SIZE(hdmi_aud_ncts); i++) {
>  		if (rate == hdmi_aud_ncts[i].sample_rate &&
> -		    adjusted_mode->crtc_clock == hdmi_aud_ncts[i].clock) {
> +		    crtc_state->port_clock == hdmi_aud_ncts[i].clock) {
>  			return hdmi_aud_ncts[i].n;
>  		}
>  	}
> -- 
> 2.17.1

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