From patchwork Thu Aug 4 07:58:02 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: libin.yang@linux.intel.com X-Patchwork-Id: 9262885 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 8285A60839 for ; Thu, 4 Aug 2016 08:01:20 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 73A7B27F8C for ; Thu, 4 Aug 2016 08:01:20 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 678332832E; Thu, 4 Aug 2016 08:01:20 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A97A627F8C for ; Thu, 4 Aug 2016 08:01:19 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 5212F6E5EE; Thu, 4 Aug 2016 08:01:17 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTP id 4CB746E5E9 for ; Thu, 4 Aug 2016 08:01:14 +0000 (UTC) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga104.fm.intel.com with ESMTP; 04 Aug 2016 01:01:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos; i="5.28,469,1464678000"; d="scan'208"; a="1034875277" Received: from younglee-grantley.sh.intel.com ([10.239.159.22]) by fmsmga002.fm.intel.com with ESMTP; 04 Aug 2016 01:01:12 -0700 From: libin.yang@linux.intel.com To: intel-gfx@lists.freedesktop.org, jani.nikula@linux.intel.com, ville.syrjala@linux.intel.com, daniel.vetter@intel.com, tiwai@suse.de Date: Thu, 4 Aug 2016 15:58:02 +0800 Message-Id: <1470297484-82178-1-git-send-email-libin.yang@linux.intel.com> X-Mailer: git-send-email 1.9.1 Cc: Libin Yang Subject: [Intel-gfx] [PATCH v3 1/3] drm/i915: set proper N/M in modeset X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Virus-Scanned: ClamAV using ClamSMTP From: Libin Yang When modeset occurs and the LS_CLK is set to some special values in DP mode, the N/M need to be set manually if audio is playing. Otherwise the first several seconds may be silent in audio playback. The relationship of Maud and Naud is expressed in the following equation: Maud/Naud = 512 * fs / f_LS_Clk Please refer VESA DisplayPort Standard spec for details. Signed-off-by: Libin Yang --- drivers/gpu/drm/i915/i915_reg.h | 6 ++ drivers/gpu/drm/i915/intel_audio.c | 133 ++++++++++++++++++++++++++++++++----- 2 files changed, 123 insertions(+), 16 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index 8bfde75..2f9d00e 100644 --- a/drivers/gpu/drm/i915/i915_reg.h +++ b/drivers/gpu/drm/i915/i915_reg.h @@ -7351,6 +7351,12 @@ enum { #define _HSW_AUD_CONFIG_B 0x65100 #define HSW_AUD_CFG(pipe) _MMIO_PIPE(pipe, _HSW_AUD_CONFIG_A, _HSW_AUD_CONFIG_B) +#define _HSW_AUD_M_CTS_ENABLE_A 0x65028 +#define _HSW_AUD_M_CTS_ENABLE_B 0x65128 +#define HSW_AUD_M_CTS_ENABLE(pipe) _MMIO_PIPE(pipe, _HSW_AUD_M_CTS_ENABLE_A, _HSW_AUD_M_CTS_ENABLE_B) +#define AUD_M_CTS_M_VALUE_INDEX (1 << 21) +#define AUD_M_CTS_M_PROG_ENABLE (1 << 20) + #define _HSW_AUD_MISC_CTRL_A 0x65010 #define _HSW_AUD_MISC_CTRL_B 0x65110 #define HSW_AUD_MISC_CTRL(pipe) _MMIO_PIPE(pipe, _HSW_AUD_MISC_CTRL_A, _HSW_AUD_MISC_CTRL_B) diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c index 6700a7b..04e7358 100644 --- a/drivers/gpu/drm/i915/intel_audio.c +++ b/drivers/gpu/drm/i915/intel_audio.c @@ -98,6 +98,35 @@ static const struct { { 192000, TMDS_297M, 20480, 247500 }, }; +#define LC_540M 540000 +#define LC_270M 270000 +#define LC_162M 162000 +static const struct { + int sample_rate; + int clock; + u16 n; + u16 m; +} aud_nm[] = { + {192000, LC_540M, 5625, 1024}, + {96000, LC_540M, 5625, 512}, + {88200, LC_540M, 9375, 784}, + {48000, LC_540M, 5625, 256}, + {44100, LC_540M, 9375, 392}, + {32000, LC_540M, 16875, 512}, + {192000, LC_270M, 5625, 2048}, + {96000, LC_270M, 5625, 1024}, + {88200, LC_270M, 9375, 1568}, + {48000, LC_270M, 5625, 512}, + {44100, LC_270M, 9375, 784}, + {32000, LC_270M, 16875, 1024}, + {192000, LC_162M, 3375, 2048}, + {96000, LC_162M, 3375, 1024}, + {88200, LC_162M, 5625, 1568}, + {48000, LC_162M, 3375, 512}, + {44100, LC_162M, 5625, 784}, + {32000, LC_162M, 10125, 1024}, +}; + /* get AUD_CONFIG_PIXEL_CLOCK_HDMI_* value for mode */ static u32 audio_config_hdmi_pixel_clock(const struct drm_display_mode *adjusted_mode) { @@ -121,20 +150,50 @@ static u32 audio_config_hdmi_pixel_clock(const struct drm_display_mode *adjusted return hdmi_audio_clock[i].config; } -static int audio_config_get_n(const struct drm_display_mode *mode, int rate) +static int audio_config_get_n(struct intel_crtc *crtc, + const struct drm_display_mode *adjusted_mode, + int rate) { int i; - for (i = 0; i < ARRAY_SIZE(aud_ncts); i++) { - if ((rate == aud_ncts[i].sample_rate) && - (mode->clock == aud_ncts[i].clock)) { - return aud_ncts[i].n; + if (intel_crtc_has_type(crtc->config, INTEL_OUTPUT_HDMI)) { + for (i = 0; i < ARRAY_SIZE(aud_ncts); i++) { + if ((rate == aud_ncts[i].sample_rate) && + (adjusted_mode->clock == aud_ncts[i].clock)) { + return aud_ncts[i].n; + } } } + + if (intel_crtc_has_type(crtc->config, INTEL_OUTPUT_DP)) { + for (i = 0; i < ARRAY_SIZE(aud_nm); i++) { + if ((rate == aud_nm[i].sample_rate) && + (crtc->config->port_clock == aud_nm[i].clock)) { + return aud_nm[i].n; + } + } + } + return 0; +} + +static int audio_config_get_m(struct intel_crtc *crtc, int rate) +{ + int i; + + if (intel_crtc_has_type(crtc->config, INTEL_OUTPUT_DP)) { + for (i = 0; i < ARRAY_SIZE(aud_nm); i++) { + if ((rate == aud_nm[i].sample_rate) && + (crtc->config->port_clock == aud_nm[i].clock)) { + return aud_nm[i].m; + } + } + } + return 0; } -static uint32_t audio_config_setup_n_reg(int n, uint32_t val) +static uint32_t audio_config_setup_n_reg(struct intel_crtc *crtc, + int n, uint32_t val) { int n_low, n_up; uint32_t tmp = val; @@ -145,17 +204,39 @@ static uint32_t audio_config_setup_n_reg(int n, uint32_t val) tmp |= ((n_up << AUD_CONFIG_UPPER_N_SHIFT) | (n_low << AUD_CONFIG_LOWER_N_SHIFT) | AUD_CONFIG_N_PROG_ENABLE); + if (intel_crtc_has_type(crtc->config, INTEL_OUTPUT_DP)) + tmp |= AUD_CONFIG_N_VALUE_INDEX; + return tmp; +} + +static uint32_t audio_config_setup_m_reg(struct intel_crtc *crtc, + int m, uint32_t val) +{ + uint32_t tmp = val; + + if (!intel_crtc_has_type(crtc->config, INTEL_OUTPUT_DP)) + return 0; + + tmp |= m; + tmp |= AUD_M_CTS_M_VALUE_INDEX; + tmp |= AUD_M_CTS_M_PROG_ENABLE; + return tmp; } /* check whether N/CTS/M need be set manually */ static bool audio_rate_need_prog(struct intel_crtc *crtc, - const struct drm_display_mode *mode) + const struct drm_display_mode *adjusted_mode) { - if (((mode->clock == TMDS_297M) || - (mode->clock == TMDS_296M)) && + if (((adjusted_mode->clock == TMDS_297M) || + (adjusted_mode->clock == TMDS_296M)) && intel_crtc_has_type(crtc->config, INTEL_OUTPUT_HDMI)) return true; + else if (((crtc->config->port_clock == LC_540M) || + (crtc->config->port_clock == LC_270M) || + (crtc->config->port_clock == LC_162M)) && + intel_crtc_has_type(crtc->config, INTEL_OUTPUT_DP)) + return true; else return false; } @@ -287,7 +368,7 @@ static void hsw_audio_codec_enable(struct drm_connector *connector, struct intel_digital_port *intel_dig_port = enc_to_dig_port(&encoder->base); enum port port = intel_dig_port->port; - uint32_t tmp; + uint32_t tmp, m; int len, i; int n, rate; @@ -343,15 +424,25 @@ static void hsw_audio_codec_enable(struct drm_connector *connector, DRM_ERROR("invalid port: %d\n", port); rate = 0; } - n = audio_config_get_n(adjusted_mode, rate); + n = audio_config_get_n(intel_crtc, adjusted_mode, rate); if (n != 0) - tmp = audio_config_setup_n_reg(n, tmp); + tmp = audio_config_setup_n_reg(intel_crtc, n, tmp); else DRM_DEBUG_KMS("no suitable N value is found\n"); } I915_WRITE(HSW_AUD_CFG(pipe), tmp); + /* setup m value for DP */ + if (intel_crtc_has_type(intel_crtc->config, INTEL_OUTPUT_DP)) { + m = audio_config_get_m(intel_crtc, rate); + if (m != 0) { + tmp = I915_READ(HSW_AUD_M_CTS_ENABLE(pipe)); + tmp = audio_config_setup_m_reg(intel_crtc, m, tmp); + I915_WRITE(HSW_AUD_M_CTS_ENABLE(pipe), tmp); + } + } + mutex_unlock(&dev_priv->av_mutex); } @@ -637,7 +728,7 @@ static int i915_audio_component_sync_audio_rate(struct device *dev, struct drm_display_mode *mode; struct i915_audio_component *acomp = dev_priv->audio_component; enum pipe pipe = INVALID_PIPE; - u32 tmp; + u32 tmp, m; int n; int err = 0; @@ -653,7 +744,8 @@ static int i915_audio_component_sync_audio_rate(struct device *dev, intel_encoder = dev_priv->dig_port_map[port]; /* intel_encoder might be NULL for DP MST */ if (!intel_encoder || !intel_encoder->base.crtc || - intel_encoder->type != INTEL_OUTPUT_HDMI) { + ((intel_encoder->type != INTEL_OUTPUT_HDMI) && + (intel_encoder->type != INTEL_OUTPUT_DP))) { DRM_DEBUG_KMS("no valid port %c\n", port_name(port)); err = -ENODEV; goto unlock; @@ -681,7 +773,7 @@ static int i915_audio_component_sync_audio_rate(struct device *dev, goto unlock; } - n = audio_config_get_n(mode, rate); + n = audio_config_get_n(crtc, mode, rate); if (n == 0) { DRM_DEBUG_KMS("Using automatic mode for N value on port %c\n", port_name(port)); @@ -693,8 +785,17 @@ static int i915_audio_component_sync_audio_rate(struct device *dev, /* 3. set the N/CTS/M */ tmp = I915_READ(HSW_AUD_CFG(pipe)); - tmp = audio_config_setup_n_reg(n, tmp); + tmp = audio_config_setup_n_reg(crtc, n, tmp); I915_WRITE(HSW_AUD_CFG(pipe), tmp); + /* setup m value for DP */ + if (intel_crtc_has_type(crtc->config, INTEL_OUTPUT_DP)) { + m = audio_config_get_m(crtc, rate); + if (m == 0) + goto unlock; + tmp = I915_READ(HSW_AUD_M_CTS_ENABLE(pipe)); + tmp = audio_config_setup_m_reg(crtc, m, tmp); + I915_WRITE(HSW_AUD_M_CTS_ENABLE(pipe), tmp); + } unlock: mutex_unlock(&dev_priv->av_mutex);