From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: Dave Airlie <airlied@redhat.com>
Subject: Re: [Intel-gfx] [PATCH 03/11] drm/i915: split fdi code out from intel_display.c
Date: Sat, 16 Jan 2021 00:23:27 +0200 [thread overview]
Message-ID: <87r1mlon4w.fsf@intel.com> (raw)
In-Reply-To: <f9d52c3d91f0973af308ede16e266fc9b753ecf9.1610622609.git.jani.nikula@intel.com>
On Thu, 14 Jan 2021, Jani Nikula <jani.nikula@intel.com> wrote:
> From: Dave Airlie <airlied@redhat.com>
>
> This just refactors out the fdi code to a separate file.
>
> Signed-off-by: Dave Airlie <airlied@redhat.com>
> [Jani: cleaned up intel_fdi.h a bit.]
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Thanks for the patches and review, pushed the first three for starters.
BR,
Jani.
> ---
> drivers/gpu/drm/i915/Makefile | 1 +
> drivers/gpu/drm/i915/display/intel_display.c | 685 +-----------------
> .../drm/i915/display/intel_display_types.h | 9 +
> drivers/gpu/drm/i915/display/intel_fdi.c | 683 +++++++++++++++++
> drivers/gpu/drm/i915/display/intel_fdi.h | 22 +
> 5 files changed, 718 insertions(+), 682 deletions(-)
> create mode 100644 drivers/gpu/drm/i915/display/intel_fdi.c
> create mode 100644 drivers/gpu/drm/i915/display/intel_fdi.h
>
> diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
> index 2479ab2ffb09..e245a036613e 100644
> --- a/drivers/gpu/drm/i915/Makefile
> +++ b/drivers/gpu/drm/i915/Makefile
> @@ -210,6 +210,7 @@ i915-y += \
> display/intel_dpll_mgr.o \
> display/intel_dsb.o \
> display/intel_fbc.o \
> + display/intel_fdi.o \
> display/intel_fifo_underrun.o \
> display/intel_frontbuffer.o \
> display/intel_global_state.o \
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index f4d9ccaafcb3..a2fa7a219c4a 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -74,6 +74,7 @@
> #include "intel_display_types.h"
> #include "intel_dp_link_training.h"
> #include "intel_fbc.h"
> +#include "intel_fdi.h"
> #include "intel_fbdev.h"
> #include "intel_fifo_underrun.h"
> #include "intel_frontbuffer.h"
> @@ -173,16 +174,6 @@ static void intel_update_czclk(struct drm_i915_private *dev_priv)
> dev_priv->czclk_freq);
> }
>
> -/* units of 100MHz */
> -static u32 intel_fdi_link_freq(struct drm_i915_private *dev_priv,
> - const struct intel_crtc_state *pipe_config)
> -{
> - if (HAS_DDI(dev_priv))
> - return pipe_config->port_clock; /* SPLL */
> - else
> - return dev_priv->fdi_pll_freq;
> -}
> -
> /* WA Display #0827: Gen9:all */
> static void
> skl_wa_827(struct drm_i915_private *dev_priv, enum pipe pipe, bool enable)
> @@ -3727,532 +3718,6 @@ static void icl_set_pipe_chicken(struct intel_crtc *crtc)
> intel_de_write(dev_priv, PIPE_CHICKEN(pipe), tmp);
> }
>
> -static void intel_fdi_normal_train(struct intel_crtc *crtc)
> -{
> - struct drm_device *dev = crtc->base.dev;
> - struct drm_i915_private *dev_priv = to_i915(dev);
> - enum pipe pipe = crtc->pipe;
> - i915_reg_t reg;
> - u32 temp;
> -
> - /* enable normal train */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - if (IS_IVYBRIDGE(dev_priv)) {
> - temp &= ~FDI_LINK_TRAIN_NONE_IVB;
> - temp |= FDI_LINK_TRAIN_NONE_IVB | FDI_TX_ENHANCE_FRAME_ENABLE;
> - } else {
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_NONE | FDI_TX_ENHANCE_FRAME_ENABLE;
> - }
> - intel_de_write(dev_priv, reg, temp);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - if (HAS_PCH_CPT(dev_priv)) {
> - temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> - temp |= FDI_LINK_TRAIN_NORMAL_CPT;
> - } else {
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_NONE;
> - }
> - intel_de_write(dev_priv, reg, temp | FDI_RX_ENHANCE_FRAME_ENABLE);
> -
> - /* wait one idle pattern time */
> - intel_de_posting_read(dev_priv, reg);
> - udelay(1000);
> -
> - /* IVB wants error correction enabled */
> - if (IS_IVYBRIDGE(dev_priv))
> - intel_de_write(dev_priv, reg,
> - intel_de_read(dev_priv, reg) | FDI_FS_ERRC_ENABLE | FDI_FE_ERRC_ENABLE);
> -}
> -
> -/* The FDI link training functions for ILK/Ibexpeak. */
> -static void ilk_fdi_link_train(struct intel_crtc *crtc,
> - const struct intel_crtc_state *crtc_state)
> -{
> - struct drm_device *dev = crtc->base.dev;
> - struct drm_i915_private *dev_priv = to_i915(dev);
> - enum pipe pipe = crtc->pipe;
> - i915_reg_t reg;
> - u32 temp, tries;
> -
> - /* FDI needs bits from pipe first */
> - assert_pipe_enabled(dev_priv, crtc_state->cpu_transcoder);
> -
> - /* Train 1: umask FDI RX Interrupt symbol_lock and bit_lock bit
> - for train result */
> - reg = FDI_RX_IMR(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_RX_SYMBOL_LOCK;
> - temp &= ~FDI_RX_BIT_LOCK;
> - intel_de_write(dev_priv, reg, temp);
> - intel_de_read(dev_priv, reg);
> - udelay(150);
> -
> - /* enable CPU FDI TX and PCH FDI RX */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_DP_PORT_WIDTH_MASK;
> - temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_1;
> - intel_de_write(dev_priv, reg, temp | FDI_TX_ENABLE);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_1;
> - intel_de_write(dev_priv, reg, temp | FDI_RX_ENABLE);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(150);
> -
> - /* Ironlake workaround, enable clock pointer after FDI enable*/
> - intel_de_write(dev_priv, FDI_RX_CHICKEN(pipe),
> - FDI_RX_PHASE_SYNC_POINTER_OVR);
> - intel_de_write(dev_priv, FDI_RX_CHICKEN(pipe),
> - FDI_RX_PHASE_SYNC_POINTER_OVR | FDI_RX_PHASE_SYNC_POINTER_EN);
> -
> - reg = FDI_RX_IIR(pipe);
> - for (tries = 0; tries < 5; tries++) {
> - temp = intel_de_read(dev_priv, reg);
> - drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> -
> - if ((temp & FDI_RX_BIT_LOCK)) {
> - drm_dbg_kms(&dev_priv->drm, "FDI train 1 done.\n");
> - intel_de_write(dev_priv, reg, temp | FDI_RX_BIT_LOCK);
> - break;
> - }
> - }
> - if (tries == 5)
> - drm_err(&dev_priv->drm, "FDI train 1 fail!\n");
> -
> - /* Train 2 */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_2;
> - intel_de_write(dev_priv, reg, temp);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_2;
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(150);
> -
> - reg = FDI_RX_IIR(pipe);
> - for (tries = 0; tries < 5; tries++) {
> - temp = intel_de_read(dev_priv, reg);
> - drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> -
> - if (temp & FDI_RX_SYMBOL_LOCK) {
> - intel_de_write(dev_priv, reg,
> - temp | FDI_RX_SYMBOL_LOCK);
> - drm_dbg_kms(&dev_priv->drm, "FDI train 2 done.\n");
> - break;
> - }
> - }
> - if (tries == 5)
> - drm_err(&dev_priv->drm, "FDI train 2 fail!\n");
> -
> - drm_dbg_kms(&dev_priv->drm, "FDI train done\n");
> -
> -}
> -
> -static const int snb_b_fdi_train_param[] = {
> - FDI_LINK_TRAIN_400MV_0DB_SNB_B,
> - FDI_LINK_TRAIN_400MV_6DB_SNB_B,
> - FDI_LINK_TRAIN_600MV_3_5DB_SNB_B,
> - FDI_LINK_TRAIN_800MV_0DB_SNB_B,
> -};
> -
> -/* The FDI link training functions for SNB/Cougarpoint. */
> -static void gen6_fdi_link_train(struct intel_crtc *crtc,
> - const struct intel_crtc_state *crtc_state)
> -{
> - struct drm_device *dev = crtc->base.dev;
> - struct drm_i915_private *dev_priv = to_i915(dev);
> - enum pipe pipe = crtc->pipe;
> - i915_reg_t reg;
> - u32 temp, i, retry;
> -
> - /* Train 1: umask FDI RX Interrupt symbol_lock and bit_lock bit
> - for train result */
> - reg = FDI_RX_IMR(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_RX_SYMBOL_LOCK;
> - temp &= ~FDI_RX_BIT_LOCK;
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(150);
> -
> - /* enable CPU FDI TX and PCH FDI RX */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_DP_PORT_WIDTH_MASK;
> - temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_1;
> - temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> - /* SNB-B */
> - temp |= FDI_LINK_TRAIN_400MV_0DB_SNB_B;
> - intel_de_write(dev_priv, reg, temp | FDI_TX_ENABLE);
> -
> - intel_de_write(dev_priv, FDI_RX_MISC(pipe),
> - FDI_RX_TP1_TO_TP2_48 | FDI_RX_FDI_DELAY_90);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - if (HAS_PCH_CPT(dev_priv)) {
> - temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> - temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
> - } else {
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_1;
> - }
> - intel_de_write(dev_priv, reg, temp | FDI_RX_ENABLE);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(150);
> -
> - for (i = 0; i < 4; i++) {
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> - temp |= snb_b_fdi_train_param[i];
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(500);
> -
> - for (retry = 0; retry < 5; retry++) {
> - reg = FDI_RX_IIR(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> - if (temp & FDI_RX_BIT_LOCK) {
> - intel_de_write(dev_priv, reg,
> - temp | FDI_RX_BIT_LOCK);
> - drm_dbg_kms(&dev_priv->drm,
> - "FDI train 1 done.\n");
> - break;
> - }
> - udelay(50);
> - }
> - if (retry < 5)
> - break;
> - }
> - if (i == 4)
> - drm_err(&dev_priv->drm, "FDI train 1 fail!\n");
> -
> - /* Train 2 */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_2;
> - if (IS_GEN(dev_priv, 6)) {
> - temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> - /* SNB-B */
> - temp |= FDI_LINK_TRAIN_400MV_0DB_SNB_B;
> - }
> - intel_de_write(dev_priv, reg, temp);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - if (HAS_PCH_CPT(dev_priv)) {
> - temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> - temp |= FDI_LINK_TRAIN_PATTERN_2_CPT;
> - } else {
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_2;
> - }
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(150);
> -
> - for (i = 0; i < 4; i++) {
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> - temp |= snb_b_fdi_train_param[i];
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(500);
> -
> - for (retry = 0; retry < 5; retry++) {
> - reg = FDI_RX_IIR(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> - if (temp & FDI_RX_SYMBOL_LOCK) {
> - intel_de_write(dev_priv, reg,
> - temp | FDI_RX_SYMBOL_LOCK);
> - drm_dbg_kms(&dev_priv->drm,
> - "FDI train 2 done.\n");
> - break;
> - }
> - udelay(50);
> - }
> - if (retry < 5)
> - break;
> - }
> - if (i == 4)
> - drm_err(&dev_priv->drm, "FDI train 2 fail!\n");
> -
> - drm_dbg_kms(&dev_priv->drm, "FDI train done.\n");
> -}
> -
> -/* Manual link training for Ivy Bridge A0 parts */
> -static void ivb_manual_fdi_link_train(struct intel_crtc *crtc,
> - const struct intel_crtc_state *crtc_state)
> -{
> - struct drm_device *dev = crtc->base.dev;
> - struct drm_i915_private *dev_priv = to_i915(dev);
> - enum pipe pipe = crtc->pipe;
> - i915_reg_t reg;
> - u32 temp, i, j;
> -
> - /* Train 1: umask FDI RX Interrupt symbol_lock and bit_lock bit
> - for train result */
> - reg = FDI_RX_IMR(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_RX_SYMBOL_LOCK;
> - temp &= ~FDI_RX_BIT_LOCK;
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(150);
> -
> - drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR before link train 0x%x\n",
> - intel_de_read(dev_priv, FDI_RX_IIR(pipe)));
> -
> - /* Try each vswing and preemphasis setting twice before moving on */
> - for (j = 0; j < ARRAY_SIZE(snb_b_fdi_train_param) * 2; j++) {
> - /* disable first in case we need to retry */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~(FDI_LINK_TRAIN_AUTO | FDI_LINK_TRAIN_NONE_IVB);
> - temp &= ~FDI_TX_ENABLE;
> - intel_de_write(dev_priv, reg, temp);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_AUTO;
> - temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> - temp &= ~FDI_RX_ENABLE;
> - intel_de_write(dev_priv, reg, temp);
> -
> - /* enable CPU FDI TX and PCH FDI RX */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_DP_PORT_WIDTH_MASK;
> - temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> - temp |= FDI_LINK_TRAIN_PATTERN_1_IVB;
> - temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> - temp |= snb_b_fdi_train_param[j/2];
> - temp |= FDI_COMPOSITE_SYNC;
> - intel_de_write(dev_priv, reg, temp | FDI_TX_ENABLE);
> -
> - intel_de_write(dev_priv, FDI_RX_MISC(pipe),
> - FDI_RX_TP1_TO_TP2_48 | FDI_RX_FDI_DELAY_90);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
> - temp |= FDI_COMPOSITE_SYNC;
> - intel_de_write(dev_priv, reg, temp | FDI_RX_ENABLE);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(1); /* should be 0.5us */
> -
> - for (i = 0; i < 4; i++) {
> - reg = FDI_RX_IIR(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> -
> - if (temp & FDI_RX_BIT_LOCK ||
> - (intel_de_read(dev_priv, reg) & FDI_RX_BIT_LOCK)) {
> - intel_de_write(dev_priv, reg,
> - temp | FDI_RX_BIT_LOCK);
> - drm_dbg_kms(&dev_priv->drm,
> - "FDI train 1 done, level %i.\n",
> - i);
> - break;
> - }
> - udelay(1); /* should be 0.5us */
> - }
> - if (i == 4) {
> - drm_dbg_kms(&dev_priv->drm,
> - "FDI train 1 fail on vswing %d\n", j / 2);
> - continue;
> - }
> -
> - /* Train 2 */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_NONE_IVB;
> - temp |= FDI_LINK_TRAIN_PATTERN_2_IVB;
> - intel_de_write(dev_priv, reg, temp);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> - temp |= FDI_LINK_TRAIN_PATTERN_2_CPT;
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(2); /* should be 1.5us */
> -
> - for (i = 0; i < 4; i++) {
> - reg = FDI_RX_IIR(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> -
> - if (temp & FDI_RX_SYMBOL_LOCK ||
> - (intel_de_read(dev_priv, reg) & FDI_RX_SYMBOL_LOCK)) {
> - intel_de_write(dev_priv, reg,
> - temp | FDI_RX_SYMBOL_LOCK);
> - drm_dbg_kms(&dev_priv->drm,
> - "FDI train 2 done, level %i.\n",
> - i);
> - goto train_done;
> - }
> - udelay(2); /* should be 1.5us */
> - }
> - if (i == 4)
> - drm_dbg_kms(&dev_priv->drm,
> - "FDI train 2 fail on vswing %d\n", j / 2);
> - }
> -
> -train_done:
> - drm_dbg_kms(&dev_priv->drm, "FDI train done.\n");
> -}
> -
> -static void ilk_fdi_pll_enable(const struct intel_crtc_state *crtc_state)
> -{
> - struct intel_crtc *intel_crtc = to_intel_crtc(crtc_state->uapi.crtc);
> - struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev);
> - enum pipe pipe = intel_crtc->pipe;
> - i915_reg_t reg;
> - u32 temp;
> -
> - /* enable PCH FDI RX PLL, wait warmup plus DMI latency */
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~(FDI_DP_PORT_WIDTH_MASK | (0x7 << 16));
> - temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> - temp |= (intel_de_read(dev_priv, PIPECONF(pipe)) & PIPECONF_BPC_MASK) << 11;
> - intel_de_write(dev_priv, reg, temp | FDI_RX_PLL_ENABLE);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(200);
> -
> - /* Switch from Rawclk to PCDclk */
> - temp = intel_de_read(dev_priv, reg);
> - intel_de_write(dev_priv, reg, temp | FDI_PCDCLK);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(200);
> -
> - /* Enable CPU FDI TX PLL, always on for Ironlake */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - if ((temp & FDI_TX_PLL_ENABLE) == 0) {
> - intel_de_write(dev_priv, reg, temp | FDI_TX_PLL_ENABLE);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(100);
> - }
> -}
> -
> -static void ilk_fdi_pll_disable(struct intel_crtc *intel_crtc)
> -{
> - struct drm_device *dev = intel_crtc->base.dev;
> - struct drm_i915_private *dev_priv = to_i915(dev);
> - enum pipe pipe = intel_crtc->pipe;
> - i915_reg_t reg;
> - u32 temp;
> -
> - /* Switch from PCDclk to Rawclk */
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - intel_de_write(dev_priv, reg, temp & ~FDI_PCDCLK);
> -
> - /* Disable CPU FDI TX PLL */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - intel_de_write(dev_priv, reg, temp & ~FDI_TX_PLL_ENABLE);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(100);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - intel_de_write(dev_priv, reg, temp & ~FDI_RX_PLL_ENABLE);
> -
> - /* Wait for the clocks to turn off. */
> - intel_de_posting_read(dev_priv, reg);
> - udelay(100);
> -}
> -
> -static void ilk_fdi_disable(struct intel_crtc *crtc)
> -{
> - struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
> - enum pipe pipe = crtc->pipe;
> - i915_reg_t reg;
> - u32 temp;
> -
> - /* disable CPU FDI tx and PCH FDI rx */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - intel_de_write(dev_priv, reg, temp & ~FDI_TX_ENABLE);
> - intel_de_posting_read(dev_priv, reg);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~(0x7 << 16);
> - temp |= (intel_de_read(dev_priv, PIPECONF(pipe)) & PIPECONF_BPC_MASK) << 11;
> - intel_de_write(dev_priv, reg, temp & ~FDI_RX_ENABLE);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(100);
> -
> - /* Ironlake workaround, disable clock pointer after downing FDI */
> - if (HAS_PCH_IBX(dev_priv))
> - intel_de_write(dev_priv, FDI_RX_CHICKEN(pipe),
> - FDI_RX_PHASE_SYNC_POINTER_OVR);
> -
> - /* still set train pattern 1 */
> - reg = FDI_TX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_1;
> - intel_de_write(dev_priv, reg, temp);
> -
> - reg = FDI_RX_CTL(pipe);
> - temp = intel_de_read(dev_priv, reg);
> - if (HAS_PCH_CPT(dev_priv)) {
> - temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> - temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
> - } else {
> - temp &= ~FDI_LINK_TRAIN_NONE;
> - temp |= FDI_LINK_TRAIN_PATTERN_1;
> - }
> - /* BPC in FDI rx is consistent with that in PIPECONF */
> - temp &= ~(0x07 << 16);
> - temp |= (intel_de_read(dev_priv, PIPECONF(pipe)) & PIPECONF_BPC_MASK) << 11;
> - intel_de_write(dev_priv, reg, temp);
> -
> - intel_de_posting_read(dev_priv, reg);
> - udelay(100);
> -}
> -
> bool intel_has_pending_fb_unpin(struct drm_i915_private *dev_priv)
> {
> struct drm_crtc *crtc;
> @@ -6659,143 +6124,6 @@ static void intel_connector_verify_state(struct intel_crtc_state *crtc_state,
> }
> }
>
> -static int pipe_required_fdi_lanes(struct intel_crtc_state *crtc_state)
> -{
> - if (crtc_state->hw.enable && crtc_state->has_pch_encoder)
> - return crtc_state->fdi_lanes;
> -
> - return 0;
> -}
> -
> -static int ilk_check_fdi_lanes(struct drm_device *dev, enum pipe pipe,
> - struct intel_crtc_state *pipe_config)
> -{
> - struct drm_i915_private *dev_priv = to_i915(dev);
> - struct drm_atomic_state *state = pipe_config->uapi.state;
> - struct intel_crtc *other_crtc;
> - struct intel_crtc_state *other_crtc_state;
> -
> - drm_dbg_kms(&dev_priv->drm,
> - "checking fdi config on pipe %c, lanes %i\n",
> - pipe_name(pipe), pipe_config->fdi_lanes);
> - if (pipe_config->fdi_lanes > 4) {
> - drm_dbg_kms(&dev_priv->drm,
> - "invalid fdi lane config on pipe %c: %i lanes\n",
> - pipe_name(pipe), pipe_config->fdi_lanes);
> - return -EINVAL;
> - }
> -
> - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) {
> - if (pipe_config->fdi_lanes > 2) {
> - drm_dbg_kms(&dev_priv->drm,
> - "only 2 lanes on haswell, required: %i lanes\n",
> - pipe_config->fdi_lanes);
> - return -EINVAL;
> - } else {
> - return 0;
> - }
> - }
> -
> - if (INTEL_NUM_PIPES(dev_priv) == 2)
> - return 0;
> -
> - /* Ivybridge 3 pipe is really complicated */
> - switch (pipe) {
> - case PIPE_A:
> - return 0;
> - case PIPE_B:
> - if (pipe_config->fdi_lanes <= 2)
> - return 0;
> -
> - other_crtc = intel_get_crtc_for_pipe(dev_priv, PIPE_C);
> - other_crtc_state =
> - intel_atomic_get_crtc_state(state, other_crtc);
> - if (IS_ERR(other_crtc_state))
> - return PTR_ERR(other_crtc_state);
> -
> - if (pipe_required_fdi_lanes(other_crtc_state) > 0) {
> - drm_dbg_kms(&dev_priv->drm,
> - "invalid shared fdi lane config on pipe %c: %i lanes\n",
> - pipe_name(pipe), pipe_config->fdi_lanes);
> - return -EINVAL;
> - }
> - return 0;
> - case PIPE_C:
> - if (pipe_config->fdi_lanes > 2) {
> - drm_dbg_kms(&dev_priv->drm,
> - "only 2 lanes on pipe %c: required %i lanes\n",
> - pipe_name(pipe), pipe_config->fdi_lanes);
> - return -EINVAL;
> - }
> -
> - other_crtc = intel_get_crtc_for_pipe(dev_priv, PIPE_B);
> - other_crtc_state =
> - intel_atomic_get_crtc_state(state, other_crtc);
> - if (IS_ERR(other_crtc_state))
> - return PTR_ERR(other_crtc_state);
> -
> - if (pipe_required_fdi_lanes(other_crtc_state) > 2) {
> - drm_dbg_kms(&dev_priv->drm,
> - "fdi link B uses too many lanes to enable link C\n");
> - return -EINVAL;
> - }
> - return 0;
> - default:
> - BUG();
> - }
> -}
> -
> -#define RETRY 1
> -static int ilk_fdi_compute_config(struct intel_crtc *intel_crtc,
> - struct intel_crtc_state *pipe_config)
> -{
> - struct drm_device *dev = intel_crtc->base.dev;
> - struct drm_i915_private *i915 = to_i915(dev);
> - const struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
> - int lane, link_bw, fdi_dotclock, ret;
> - bool needs_recompute = false;
> -
> -retry:
> - /* FDI is a binary signal running at ~2.7GHz, encoding
> - * each output octet as 10 bits. The actual frequency
> - * is stored as a divider into a 100MHz clock, and the
> - * mode pixel clock is stored in units of 1KHz.
> - * Hence the bw of each lane in terms of the mode signal
> - * is:
> - */
> - link_bw = intel_fdi_link_freq(i915, pipe_config);
> -
> - fdi_dotclock = adjusted_mode->crtc_clock;
> -
> - lane = ilk_get_lanes_required(fdi_dotclock, link_bw,
> - pipe_config->pipe_bpp);
> -
> - pipe_config->fdi_lanes = lane;
> -
> - intel_link_compute_m_n(pipe_config->pipe_bpp, lane, fdi_dotclock,
> - link_bw, &pipe_config->fdi_m_n, false, false);
> -
> - ret = ilk_check_fdi_lanes(dev, intel_crtc->pipe, pipe_config);
> - if (ret == -EDEADLK)
> - return ret;
> -
> - if (ret == -EINVAL && pipe_config->pipe_bpp > 6*3) {
> - pipe_config->pipe_bpp -= 2*3;
> - drm_dbg_kms(&i915->drm,
> - "fdi link bw constraint, reducing pipe bpp to %i\n",
> - pipe_config->pipe_bpp);
> - needs_recompute = true;
> - pipe_config->bw_constrained = true;
> -
> - goto retry;
> - }
> -
> - if (needs_recompute)
> - return RETRY;
> -
> - return ret;
> -}
> -
> bool hsw_crtc_state_ips_capable(const struct intel_crtc_state *crtc_state)
> {
> struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
> @@ -11322,7 +10650,7 @@ intel_modeset_pipe_config(struct intel_atomic_state *state,
> return ret;
> }
>
> - if (ret == RETRY) {
> + if (ret == I915_DISPLAY_CONFIG_RETRY) {
> if (drm_WARN(&i915->drm, !retry,
> "loop in pipe configuration computation\n"))
> return -EINVAL;
> @@ -15121,14 +14449,7 @@ void intel_init_display_hooks(struct drm_i915_private *dev_priv)
> dev_priv->display.crtc_disable = i9xx_crtc_disable;
> }
>
> - if (IS_GEN(dev_priv, 5)) {
> - dev_priv->display.fdi_link_train = ilk_fdi_link_train;
> - } else if (IS_GEN(dev_priv, 6)) {
> - dev_priv->display.fdi_link_train = gen6_fdi_link_train;
> - } else if (IS_IVYBRIDGE(dev_priv)) {
> - /* FIXME: detect B0+ stepping and use auto training */
> - dev_priv->display.fdi_link_train = ivb_manual_fdi_link_train;
> - }
> + intel_fdi_init_hook(dev_priv);
>
> if (INTEL_GEN(dev_priv) >= 9) {
> dev_priv->display.commit_modeset_enables = skl_commit_modeset_enables;
> diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h
> index ed129f805a21..7cdcf11b28db 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_types.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_types.h
> @@ -1852,4 +1852,13 @@ static inline u32 i9xx_dpll_compute_fp(struct dpll *dpll)
> return dpll->n << 16 | dpll->m1 << 8 | dpll->m2;
> }
>
> +static inline u32 intel_fdi_link_freq(struct drm_i915_private *dev_priv,
> + const struct intel_crtc_state *pipe_config)
> +{
> + if (HAS_DDI(dev_priv))
> + return pipe_config->port_clock; /* SPLL */
> + else
> + return dev_priv->fdi_pll_freq;
> +}
> +
> #endif /* __INTEL_DISPLAY_TYPES_H__ */
> diff --git a/drivers/gpu/drm/i915/display/intel_fdi.c b/drivers/gpu/drm/i915/display/intel_fdi.c
> new file mode 100644
> index 000000000000..b2eb96ae10a2
> --- /dev/null
> +++ b/drivers/gpu/drm/i915/display/intel_fdi.c
> @@ -0,0 +1,683 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * Copyright © 2020 Intel Corporation
> + */
> +#include "intel_atomic.h"
> +#include "intel_display_types.h"
> +#include "intel_fdi.h"
> +
> +/* units of 100MHz */
> +static int pipe_required_fdi_lanes(struct intel_crtc_state *crtc_state)
> +{
> + if (crtc_state->hw.enable && crtc_state->has_pch_encoder)
> + return crtc_state->fdi_lanes;
> +
> + return 0;
> +}
> +
> +static int ilk_check_fdi_lanes(struct drm_device *dev, enum pipe pipe,
> + struct intel_crtc_state *pipe_config)
> +{
> + struct drm_i915_private *dev_priv = to_i915(dev);
> + struct drm_atomic_state *state = pipe_config->uapi.state;
> + struct intel_crtc *other_crtc;
> + struct intel_crtc_state *other_crtc_state;
> +
> + drm_dbg_kms(&dev_priv->drm,
> + "checking fdi config on pipe %c, lanes %i\n",
> + pipe_name(pipe), pipe_config->fdi_lanes);
> + if (pipe_config->fdi_lanes > 4) {
> + drm_dbg_kms(&dev_priv->drm,
> + "invalid fdi lane config on pipe %c: %i lanes\n",
> + pipe_name(pipe), pipe_config->fdi_lanes);
> + return -EINVAL;
> + }
> +
> + if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) {
> + if (pipe_config->fdi_lanes > 2) {
> + drm_dbg_kms(&dev_priv->drm,
> + "only 2 lanes on haswell, required: %i lanes\n",
> + pipe_config->fdi_lanes);
> + return -EINVAL;
> + } else {
> + return 0;
> + }
> + }
> +
> + if (INTEL_NUM_PIPES(dev_priv) == 2)
> + return 0;
> +
> + /* Ivybridge 3 pipe is really complicated */
> + switch (pipe) {
> + case PIPE_A:
> + return 0;
> + case PIPE_B:
> + if (pipe_config->fdi_lanes <= 2)
> + return 0;
> +
> + other_crtc = intel_get_crtc_for_pipe(dev_priv, PIPE_C);
> + other_crtc_state =
> + intel_atomic_get_crtc_state(state, other_crtc);
> + if (IS_ERR(other_crtc_state))
> + return PTR_ERR(other_crtc_state);
> +
> + if (pipe_required_fdi_lanes(other_crtc_state) > 0) {
> + drm_dbg_kms(&dev_priv->drm,
> + "invalid shared fdi lane config on pipe %c: %i lanes\n",
> + pipe_name(pipe), pipe_config->fdi_lanes);
> + return -EINVAL;
> + }
> + return 0;
> + case PIPE_C:
> + if (pipe_config->fdi_lanes > 2) {
> + drm_dbg_kms(&dev_priv->drm,
> + "only 2 lanes on pipe %c: required %i lanes\n",
> + pipe_name(pipe), pipe_config->fdi_lanes);
> + return -EINVAL;
> + }
> +
> + other_crtc = intel_get_crtc_for_pipe(dev_priv, PIPE_B);
> + other_crtc_state =
> + intel_atomic_get_crtc_state(state, other_crtc);
> + if (IS_ERR(other_crtc_state))
> + return PTR_ERR(other_crtc_state);
> +
> + if (pipe_required_fdi_lanes(other_crtc_state) > 2) {
> + drm_dbg_kms(&dev_priv->drm,
> + "fdi link B uses too many lanes to enable link C\n");
> + return -EINVAL;
> + }
> + return 0;
> + default:
> + BUG();
> + }
> +}
> +
> +int ilk_fdi_compute_config(struct intel_crtc *intel_crtc,
> + struct intel_crtc_state *pipe_config)
> +{
> + struct drm_device *dev = intel_crtc->base.dev;
> + struct drm_i915_private *i915 = to_i915(dev);
> + const struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
> + int lane, link_bw, fdi_dotclock, ret;
> + bool needs_recompute = false;
> +
> +retry:
> + /* FDI is a binary signal running at ~2.7GHz, encoding
> + * each output octet as 10 bits. The actual frequency
> + * is stored as a divider into a 100MHz clock, and the
> + * mode pixel clock is stored in units of 1KHz.
> + * Hence the bw of each lane in terms of the mode signal
> + * is:
> + */
> + link_bw = intel_fdi_link_freq(i915, pipe_config);
> +
> + fdi_dotclock = adjusted_mode->crtc_clock;
> +
> + lane = ilk_get_lanes_required(fdi_dotclock, link_bw,
> + pipe_config->pipe_bpp);
> +
> + pipe_config->fdi_lanes = lane;
> +
> + intel_link_compute_m_n(pipe_config->pipe_bpp, lane, fdi_dotclock,
> + link_bw, &pipe_config->fdi_m_n, false, false);
> +
> + ret = ilk_check_fdi_lanes(dev, intel_crtc->pipe, pipe_config);
> + if (ret == -EDEADLK)
> + return ret;
> +
> + if (ret == -EINVAL && pipe_config->pipe_bpp > 6*3) {
> + pipe_config->pipe_bpp -= 2*3;
> + drm_dbg_kms(&i915->drm,
> + "fdi link bw constraint, reducing pipe bpp to %i\n",
> + pipe_config->pipe_bpp);
> + needs_recompute = true;
> + pipe_config->bw_constrained = true;
> +
> + goto retry;
> + }
> +
> + if (needs_recompute)
> + return I915_DISPLAY_CONFIG_RETRY;
> +
> + return ret;
> +}
> +
> +void intel_fdi_normal_train(struct intel_crtc *crtc)
> +{
> + struct drm_device *dev = crtc->base.dev;
> + struct drm_i915_private *dev_priv = to_i915(dev);
> + enum pipe pipe = crtc->pipe;
> + i915_reg_t reg;
> + u32 temp;
> +
> + /* enable normal train */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + if (IS_IVYBRIDGE(dev_priv)) {
> + temp &= ~FDI_LINK_TRAIN_NONE_IVB;
> + temp |= FDI_LINK_TRAIN_NONE_IVB | FDI_TX_ENHANCE_FRAME_ENABLE;
> + } else {
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_NONE | FDI_TX_ENHANCE_FRAME_ENABLE;
> + }
> + intel_de_write(dev_priv, reg, temp);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + if (HAS_PCH_CPT(dev_priv)) {
> + temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> + temp |= FDI_LINK_TRAIN_NORMAL_CPT;
> + } else {
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_NONE;
> + }
> + intel_de_write(dev_priv, reg, temp | FDI_RX_ENHANCE_FRAME_ENABLE);
> +
> + /* wait one idle pattern time */
> + intel_de_posting_read(dev_priv, reg);
> + udelay(1000);
> +
> + /* IVB wants error correction enabled */
> + if (IS_IVYBRIDGE(dev_priv))
> + intel_de_write(dev_priv, reg,
> + intel_de_read(dev_priv, reg) | FDI_FS_ERRC_ENABLE | FDI_FE_ERRC_ENABLE);
> +}
> +
> +/* The FDI link training functions for ILK/Ibexpeak. */
> +static void ilk_fdi_link_train(struct intel_crtc *crtc,
> + const struct intel_crtc_state *crtc_state)
> +{
> + struct drm_device *dev = crtc->base.dev;
> + struct drm_i915_private *dev_priv = to_i915(dev);
> + enum pipe pipe = crtc->pipe;
> + i915_reg_t reg;
> + u32 temp, tries;
> +
> + /* FDI needs bits from pipe first */
> + assert_pipe_enabled(dev_priv, crtc_state->cpu_transcoder);
> +
> + /* Train 1: umask FDI RX Interrupt symbol_lock and bit_lock bit
> + for train result */
> + reg = FDI_RX_IMR(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_RX_SYMBOL_LOCK;
> + temp &= ~FDI_RX_BIT_LOCK;
> + intel_de_write(dev_priv, reg, temp);
> + intel_de_read(dev_priv, reg);
> + udelay(150);
> +
> + /* enable CPU FDI TX and PCH FDI RX */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_DP_PORT_WIDTH_MASK;
> + temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_1;
> + intel_de_write(dev_priv, reg, temp | FDI_TX_ENABLE);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_1;
> + intel_de_write(dev_priv, reg, temp | FDI_RX_ENABLE);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(150);
> +
> + /* Ironlake workaround, enable clock pointer after FDI enable*/
> + intel_de_write(dev_priv, FDI_RX_CHICKEN(pipe),
> + FDI_RX_PHASE_SYNC_POINTER_OVR);
> + intel_de_write(dev_priv, FDI_RX_CHICKEN(pipe),
> + FDI_RX_PHASE_SYNC_POINTER_OVR | FDI_RX_PHASE_SYNC_POINTER_EN);
> +
> + reg = FDI_RX_IIR(pipe);
> + for (tries = 0; tries < 5; tries++) {
> + temp = intel_de_read(dev_priv, reg);
> + drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> +
> + if ((temp & FDI_RX_BIT_LOCK)) {
> + drm_dbg_kms(&dev_priv->drm, "FDI train 1 done.\n");
> + intel_de_write(dev_priv, reg, temp | FDI_RX_BIT_LOCK);
> + break;
> + }
> + }
> + if (tries == 5)
> + drm_err(&dev_priv->drm, "FDI train 1 fail!\n");
> +
> + /* Train 2 */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_2;
> + intel_de_write(dev_priv, reg, temp);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_2;
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(150);
> +
> + reg = FDI_RX_IIR(pipe);
> + for (tries = 0; tries < 5; tries++) {
> + temp = intel_de_read(dev_priv, reg);
> + drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> +
> + if (temp & FDI_RX_SYMBOL_LOCK) {
> + intel_de_write(dev_priv, reg,
> + temp | FDI_RX_SYMBOL_LOCK);
> + drm_dbg_kms(&dev_priv->drm, "FDI train 2 done.\n");
> + break;
> + }
> + }
> + if (tries == 5)
> + drm_err(&dev_priv->drm, "FDI train 2 fail!\n");
> +
> + drm_dbg_kms(&dev_priv->drm, "FDI train done\n");
> +
> +}
> +
> +static const int snb_b_fdi_train_param[] = {
> + FDI_LINK_TRAIN_400MV_0DB_SNB_B,
> + FDI_LINK_TRAIN_400MV_6DB_SNB_B,
> + FDI_LINK_TRAIN_600MV_3_5DB_SNB_B,
> + FDI_LINK_TRAIN_800MV_0DB_SNB_B,
> +};
> +
> +/* The FDI link training functions for SNB/Cougarpoint. */
> +static void gen6_fdi_link_train(struct intel_crtc *crtc,
> + const struct intel_crtc_state *crtc_state)
> +{
> + struct drm_device *dev = crtc->base.dev;
> + struct drm_i915_private *dev_priv = to_i915(dev);
> + enum pipe pipe = crtc->pipe;
> + i915_reg_t reg;
> + u32 temp, i, retry;
> +
> + /* Train 1: umask FDI RX Interrupt symbol_lock and bit_lock bit
> + for train result */
> + reg = FDI_RX_IMR(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_RX_SYMBOL_LOCK;
> + temp &= ~FDI_RX_BIT_LOCK;
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(150);
> +
> + /* enable CPU FDI TX and PCH FDI RX */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_DP_PORT_WIDTH_MASK;
> + temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_1;
> + temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> + /* SNB-B */
> + temp |= FDI_LINK_TRAIN_400MV_0DB_SNB_B;
> + intel_de_write(dev_priv, reg, temp | FDI_TX_ENABLE);
> +
> + intel_de_write(dev_priv, FDI_RX_MISC(pipe),
> + FDI_RX_TP1_TO_TP2_48 | FDI_RX_FDI_DELAY_90);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + if (HAS_PCH_CPT(dev_priv)) {
> + temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> + temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
> + } else {
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_1;
> + }
> + intel_de_write(dev_priv, reg, temp | FDI_RX_ENABLE);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(150);
> +
> + for (i = 0; i < 4; i++) {
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> + temp |= snb_b_fdi_train_param[i];
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(500);
> +
> + for (retry = 0; retry < 5; retry++) {
> + reg = FDI_RX_IIR(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> + if (temp & FDI_RX_BIT_LOCK) {
> + intel_de_write(dev_priv, reg,
> + temp | FDI_RX_BIT_LOCK);
> + drm_dbg_kms(&dev_priv->drm,
> + "FDI train 1 done.\n");
> + break;
> + }
> + udelay(50);
> + }
> + if (retry < 5)
> + break;
> + }
> + if (i == 4)
> + drm_err(&dev_priv->drm, "FDI train 1 fail!\n");
> +
> + /* Train 2 */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_2;
> + if (IS_GEN(dev_priv, 6)) {
> + temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> + /* SNB-B */
> + temp |= FDI_LINK_TRAIN_400MV_0DB_SNB_B;
> + }
> + intel_de_write(dev_priv, reg, temp);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + if (HAS_PCH_CPT(dev_priv)) {
> + temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> + temp |= FDI_LINK_TRAIN_PATTERN_2_CPT;
> + } else {
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_2;
> + }
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(150);
> +
> + for (i = 0; i < 4; i++) {
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> + temp |= snb_b_fdi_train_param[i];
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(500);
> +
> + for (retry = 0; retry < 5; retry++) {
> + reg = FDI_RX_IIR(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> + if (temp & FDI_RX_SYMBOL_LOCK) {
> + intel_de_write(dev_priv, reg,
> + temp | FDI_RX_SYMBOL_LOCK);
> + drm_dbg_kms(&dev_priv->drm,
> + "FDI train 2 done.\n");
> + break;
> + }
> + udelay(50);
> + }
> + if (retry < 5)
> + break;
> + }
> + if (i == 4)
> + drm_err(&dev_priv->drm, "FDI train 2 fail!\n");
> +
> + drm_dbg_kms(&dev_priv->drm, "FDI train done.\n");
> +}
> +
> +/* Manual link training for Ivy Bridge A0 parts */
> +static void ivb_manual_fdi_link_train(struct intel_crtc *crtc,
> + const struct intel_crtc_state *crtc_state)
> +{
> + struct drm_device *dev = crtc->base.dev;
> + struct drm_i915_private *dev_priv = to_i915(dev);
> + enum pipe pipe = crtc->pipe;
> + i915_reg_t reg;
> + u32 temp, i, j;
> +
> + /* Train 1: umask FDI RX Interrupt symbol_lock and bit_lock bit
> + for train result */
> + reg = FDI_RX_IMR(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_RX_SYMBOL_LOCK;
> + temp &= ~FDI_RX_BIT_LOCK;
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(150);
> +
> + drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR before link train 0x%x\n",
> + intel_de_read(dev_priv, FDI_RX_IIR(pipe)));
> +
> + /* Try each vswing and preemphasis setting twice before moving on */
> + for (j = 0; j < ARRAY_SIZE(snb_b_fdi_train_param) * 2; j++) {
> + /* disable first in case we need to retry */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~(FDI_LINK_TRAIN_AUTO | FDI_LINK_TRAIN_NONE_IVB);
> + temp &= ~FDI_TX_ENABLE;
> + intel_de_write(dev_priv, reg, temp);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_AUTO;
> + temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> + temp &= ~FDI_RX_ENABLE;
> + intel_de_write(dev_priv, reg, temp);
> +
> + /* enable CPU FDI TX and PCH FDI RX */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_DP_PORT_WIDTH_MASK;
> + temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> + temp |= FDI_LINK_TRAIN_PATTERN_1_IVB;
> + temp &= ~FDI_LINK_TRAIN_VOL_EMP_MASK;
> + temp |= snb_b_fdi_train_param[j/2];
> + temp |= FDI_COMPOSITE_SYNC;
> + intel_de_write(dev_priv, reg, temp | FDI_TX_ENABLE);
> +
> + intel_de_write(dev_priv, FDI_RX_MISC(pipe),
> + FDI_RX_TP1_TO_TP2_48 | FDI_RX_FDI_DELAY_90);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
> + temp |= FDI_COMPOSITE_SYNC;
> + intel_de_write(dev_priv, reg, temp | FDI_RX_ENABLE);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(1); /* should be 0.5us */
> +
> + for (i = 0; i < 4; i++) {
> + reg = FDI_RX_IIR(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> +
> + if (temp & FDI_RX_BIT_LOCK ||
> + (intel_de_read(dev_priv, reg) & FDI_RX_BIT_LOCK)) {
> + intel_de_write(dev_priv, reg,
> + temp | FDI_RX_BIT_LOCK);
> + drm_dbg_kms(&dev_priv->drm,
> + "FDI train 1 done, level %i.\n",
> + i);
> + break;
> + }
> + udelay(1); /* should be 0.5us */
> + }
> + if (i == 4) {
> + drm_dbg_kms(&dev_priv->drm,
> + "FDI train 1 fail on vswing %d\n", j / 2);
> + continue;
> + }
> +
> + /* Train 2 */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_NONE_IVB;
> + temp |= FDI_LINK_TRAIN_PATTERN_2_IVB;
> + intel_de_write(dev_priv, reg, temp);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> + temp |= FDI_LINK_TRAIN_PATTERN_2_CPT;
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(2); /* should be 1.5us */
> +
> + for (i = 0; i < 4; i++) {
> + reg = FDI_RX_IIR(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + drm_dbg_kms(&dev_priv->drm, "FDI_RX_IIR 0x%x\n", temp);
> +
> + if (temp & FDI_RX_SYMBOL_LOCK ||
> + (intel_de_read(dev_priv, reg) & FDI_RX_SYMBOL_LOCK)) {
> + intel_de_write(dev_priv, reg,
> + temp | FDI_RX_SYMBOL_LOCK);
> + drm_dbg_kms(&dev_priv->drm,
> + "FDI train 2 done, level %i.\n",
> + i);
> + goto train_done;
> + }
> + udelay(2); /* should be 1.5us */
> + }
> + if (i == 4)
> + drm_dbg_kms(&dev_priv->drm,
> + "FDI train 2 fail on vswing %d\n", j / 2);
> + }
> +
> +train_done:
> + drm_dbg_kms(&dev_priv->drm, "FDI train done.\n");
> +}
> +
> +void ilk_fdi_pll_enable(const struct intel_crtc_state *crtc_state)
> +{
> + struct intel_crtc *intel_crtc = to_intel_crtc(crtc_state->uapi.crtc);
> + struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev);
> + enum pipe pipe = intel_crtc->pipe;
> + i915_reg_t reg;
> + u32 temp;
> +
> + /* enable PCH FDI RX PLL, wait warmup plus DMI latency */
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~(FDI_DP_PORT_WIDTH_MASK | (0x7 << 16));
> + temp |= FDI_DP_PORT_WIDTH(crtc_state->fdi_lanes);
> + temp |= (intel_de_read(dev_priv, PIPECONF(pipe)) & PIPECONF_BPC_MASK) << 11;
> + intel_de_write(dev_priv, reg, temp | FDI_RX_PLL_ENABLE);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(200);
> +
> + /* Switch from Rawclk to PCDclk */
> + temp = intel_de_read(dev_priv, reg);
> + intel_de_write(dev_priv, reg, temp | FDI_PCDCLK);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(200);
> +
> + /* Enable CPU FDI TX PLL, always on for Ironlake */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + if ((temp & FDI_TX_PLL_ENABLE) == 0) {
> + intel_de_write(dev_priv, reg, temp | FDI_TX_PLL_ENABLE);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(100);
> + }
> +}
> +
> +void ilk_fdi_pll_disable(struct intel_crtc *intel_crtc)
> +{
> + struct drm_device *dev = intel_crtc->base.dev;
> + struct drm_i915_private *dev_priv = to_i915(dev);
> + enum pipe pipe = intel_crtc->pipe;
> + i915_reg_t reg;
> + u32 temp;
> +
> + /* Switch from PCDclk to Rawclk */
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + intel_de_write(dev_priv, reg, temp & ~FDI_PCDCLK);
> +
> + /* Disable CPU FDI TX PLL */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + intel_de_write(dev_priv, reg, temp & ~FDI_TX_PLL_ENABLE);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(100);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + intel_de_write(dev_priv, reg, temp & ~FDI_RX_PLL_ENABLE);
> +
> + /* Wait for the clocks to turn off. */
> + intel_de_posting_read(dev_priv, reg);
> + udelay(100);
> +}
> +
> +void ilk_fdi_disable(struct intel_crtc *crtc)
> +{
> + struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
> + enum pipe pipe = crtc->pipe;
> + i915_reg_t reg;
> + u32 temp;
> +
> + /* disable CPU FDI tx and PCH FDI rx */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + intel_de_write(dev_priv, reg, temp & ~FDI_TX_ENABLE);
> + intel_de_posting_read(dev_priv, reg);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~(0x7 << 16);
> + temp |= (intel_de_read(dev_priv, PIPECONF(pipe)) & PIPECONF_BPC_MASK) << 11;
> + intel_de_write(dev_priv, reg, temp & ~FDI_RX_ENABLE);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(100);
> +
> + /* Ironlake workaround, disable clock pointer after downing FDI */
> + if (HAS_PCH_IBX(dev_priv))
> + intel_de_write(dev_priv, FDI_RX_CHICKEN(pipe),
> + FDI_RX_PHASE_SYNC_POINTER_OVR);
> +
> + /* still set train pattern 1 */
> + reg = FDI_TX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_1;
> + intel_de_write(dev_priv, reg, temp);
> +
> + reg = FDI_RX_CTL(pipe);
> + temp = intel_de_read(dev_priv, reg);
> + if (HAS_PCH_CPT(dev_priv)) {
> + temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
> + temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
> + } else {
> + temp &= ~FDI_LINK_TRAIN_NONE;
> + temp |= FDI_LINK_TRAIN_PATTERN_1;
> + }
> + /* BPC in FDI rx is consistent with that in PIPECONF */
> + temp &= ~(0x07 << 16);
> + temp |= (intel_de_read(dev_priv, PIPECONF(pipe)) & PIPECONF_BPC_MASK) << 11;
> + intel_de_write(dev_priv, reg, temp);
> +
> + intel_de_posting_read(dev_priv, reg);
> + udelay(100);
> +}
> +
> +void
> +intel_fdi_init_hook(struct drm_i915_private *dev_priv)
> +{
> + if (IS_GEN(dev_priv, 5)) {
> + dev_priv->display.fdi_link_train = ilk_fdi_link_train;
> + } else if (IS_GEN(dev_priv, 6)) {
> + dev_priv->display.fdi_link_train = gen6_fdi_link_train;
> + } else if (IS_IVYBRIDGE(dev_priv)) {
> + /* FIXME: detect B0+ stepping and use auto training */
> + dev_priv->display.fdi_link_train = ivb_manual_fdi_link_train;
> + }
> +}
> diff --git a/drivers/gpu/drm/i915/display/intel_fdi.h b/drivers/gpu/drm/i915/display/intel_fdi.h
> new file mode 100644
> index 000000000000..a9cd21663eb8
> --- /dev/null
> +++ b/drivers/gpu/drm/i915/display/intel_fdi.h
> @@ -0,0 +1,22 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * Copyright © 2020 Intel Corporation
> + */
> +
> +#ifndef _INTEL_FDI_H_
> +#define _INTEL_FDI_H_
> +
> +struct drm_i915_private;
> +struct intel_crtc;
> +struct intel_crtc_state;
> +
> +#define I915_DISPLAY_CONFIG_RETRY 1
> +int ilk_fdi_compute_config(struct intel_crtc *intel_crtc,
> + struct intel_crtc_state *pipe_config);
> +void intel_fdi_normal_train(struct intel_crtc *crtc);
> +void ilk_fdi_disable(struct intel_crtc *crtc);
> +void ilk_fdi_pll_disable(struct intel_crtc *intel_crtc);
> +void ilk_fdi_pll_enable(const struct intel_crtc_state *crtc_state);
> +void intel_fdi_init_hook(struct drm_i915_private *dev_priv);
> +
> +#endif
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2021-01-15 22:23 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-14 11:13 [Intel-gfx] [PATCH 00/11] [rfc v7.1] rebased refactor of intel_display Jani Nikula
2021-01-14 11:13 ` [Intel-gfx] [PATCH 01/11] drm/i915: refactor some crtc code out of intel display. (v2) Jani Nikula
2021-01-14 11:13 ` [Intel-gfx] [PATCH 02/11] drm/i915: refactor pll code out into intel_dpll.c Jani Nikula
2021-01-14 11:13 ` [Intel-gfx] [PATCH 03/11] drm/i915: split fdi code out from intel_display.c Jani Nikula
2021-01-15 22:23 ` Jani Nikula [this message]
2021-01-14 11:13 ` [Intel-gfx] [PATCH 04/11] drm/i915: refactor ddi translations into a separate file Jani Nikula
2021-01-14 15:41 ` Ville Syrjälä
2021-01-14 11:13 ` [Intel-gfx] [PATCH 05/11] drm/i915: migrate hsw fdi code to new file Jani Nikula
2021-01-14 15:45 ` Ville Syrjälä
2021-01-14 11:13 ` [Intel-gfx] [PATCH 06/11] drm/i915: migrate skl planes code new file (v3) Jani Nikula
2021-01-14 15:56 ` Ville Syrjälä
2021-01-21 3:37 ` Dave Airlie
2021-01-14 11:13 ` [Intel-gfx] [PATCH 07/11] drm/i915: move pipe update code into crtc Jani Nikula
2021-01-14 11:13 ` [Intel-gfx] [PATCH 08/11] drm/i915: split fb scalable checks into g4x and skl versions Jani Nikula
2021-01-14 11:13 ` [Intel-gfx] [PATCH 09/11] drm/i915: move is_ccs_modifier to an inline Jani Nikula
2021-01-14 15:58 ` Ville Syrjälä
2021-01-14 11:13 ` [Intel-gfx] [PATCH 10/11] drm/i915: migrate pll enable/disable code to intel_dpll.[ch] Jani Nikula
2021-01-14 15:59 ` Ville Syrjälä
2021-01-14 11:13 ` [Intel-gfx] [PATCH 11/11] drm/i915: migrate i9xx plane get config Jani Nikula
2021-01-14 16:00 ` Ville Syrjälä
2021-01-14 13:43 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for rebased refactor of intel_display Patchwork
2021-01-14 13:45 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-01-14 14:13 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
2021-01-15 13:51 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for rebased refactor of intel_display (rev2) Patchwork
2021-01-15 13:52 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-01-15 14:23 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-01-15 19:27 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
-- strict thread matches above, loose matches on Subject: below --
2021-01-14 9:05 [Intel-gfx] [rfc v7] rebased refactor of intel_display Dave Airlie
2021-01-14 9:05 ` [Intel-gfx] [PATCH 03/11] drm/i915: split fdi code out from intel_display.c Dave Airlie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87r1mlon4w.fsf@intel.com \
--to=jani.nikula@intel.com \
--cc=airlied@redhat.com \
--cc=intel-gfx@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.