| From bf13db128d25d79fb87a404590811433f6fe8cbd Mon Sep 17 00:00:00 2001 |
| From: James Ausmus <james.ausmus@intel.com> |
| Date: Mon, 23 Sep 2013 16:49:38 -0700 |
| Subject: drm/i915: hw state readout support for fdi m/n |
| |
| We want to use the fdi m/n values to easily compute the adjusted mode |
| dotclock on pch ports. Hence make sure the values stored in the pipe |
| config are always reliable. |
| |
| v2: Fixup FDI TU readout. |
| |
| v3: Rebase on top of moved cpu_transcoder. |
| |
| Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com> |
| Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> |
| (cherry picked from commit 72419203cab9acf173956f5564639b0012cd2604) |
| |
| Conflicts: |
| drivers/gpu/drm/i915/i915_reg.h |
| Signed-off-by: Darren Hart <dvhart@linux.intel.com> |
| --- |
| drivers/gpu/drm/i915/i915_reg.h | 1 + |
| drivers/gpu/drm/i915/intel_display.c | 25 +++++++++++++++++++++++++ |
| 2 files changed, 26 insertions(+) |
| |
| --- a/drivers/gpu/drm/i915/i915_reg.h |
| +++ b/drivers/gpu/drm/i915/i915_reg.h |
| @@ -2785,6 +2785,7 @@ |
| |
| /* Transfer unit size for display port - 1, default is 0x3f (for TU size 64) */ |
| #define TU_SIZE(x) (((x)-1) << 25) /* default size 64 */ |
| +#define TU_SIZE_SHIFT 25 |
| #define TU_SIZE_MASK (0x3f << 25) |
| |
| #define DATA_LINK_M_N_MASK (0xffffff) |
| --- a/drivers/gpu/drm/i915/intel_display.c |
| +++ b/drivers/gpu/drm/i915/intel_display.c |
| @@ -5829,6 +5829,22 @@ static int ironlake_crtc_mode_set(struct |
| return ret; |
| } |
| |
| +static void ironlake_get_fdi_m_n_config(struct intel_crtc *crtc, |
| + struct intel_crtc_config *pipe_config) |
| +{ |
| + struct drm_device *dev = crtc->base.dev; |
| + struct drm_i915_private *dev_priv = dev->dev_private; |
| + enum transcoder transcoder = pipe_config->cpu_transcoder; |
| + |
| + pipe_config->fdi_m_n.link_m = I915_READ(PIPE_LINK_M1(transcoder)); |
| + pipe_config->fdi_m_n.link_n = I915_READ(PIPE_LINK_N1(transcoder)); |
| + pipe_config->fdi_m_n.gmch_m = I915_READ(PIPE_DATA_M1(transcoder)) |
| + & ~TU_SIZE_MASK; |
| + pipe_config->fdi_m_n.gmch_n = I915_READ(PIPE_DATA_N1(transcoder)); |
| + pipe_config->fdi_m_n.tu = ((I915_READ(PIPE_DATA_M1(transcoder)) |
| + & TU_SIZE_MASK) >> TU_SIZE_SHIFT) + 1; |
| +} |
| + |
| static bool ironlake_get_pipe_config(struct intel_crtc *crtc, |
| struct intel_crtc_config *pipe_config) |
| { |
| @@ -5846,6 +5862,8 @@ static bool ironlake_get_pipe_config(str |
| tmp = I915_READ(FDI_RX_CTL(crtc->pipe)); |
| pipe_config->fdi_lanes = ((FDI_DP_PORT_WIDTH_MASK & tmp) >> |
| FDI_DP_PORT_WIDTH_SHIFT) + 1; |
| + |
| + ironlake_get_fdi_m_n_config(crtc, pipe_config); |
| } |
| |
| return true; |
| @@ -5991,6 +6009,8 @@ static bool haswell_get_pipe_config(stru |
| tmp = I915_READ(FDI_RX_CTL(PIPE_A)); |
| pipe_config->fdi_lanes = ((FDI_DP_PORT_WIDTH_MASK & tmp) >> |
| FDI_DP_PORT_WIDTH_SHIFT) + 1; |
| + |
| + ironlake_get_fdi_m_n_config(crtc, pipe_config); |
| } |
| |
| return true; |
| @@ -7976,6 +7996,11 @@ intel_pipe_config_compare(struct intel_c |
| |
| PIPE_CONF_CHECK_I(has_pch_encoder); |
| PIPE_CONF_CHECK_I(fdi_lanes); |
| + PIPE_CONF_CHECK_I(fdi_m_n.gmch_m); |
| + PIPE_CONF_CHECK_I(fdi_m_n.gmch_n); |
| + PIPE_CONF_CHECK_I(fdi_m_n.link_m); |
| + PIPE_CONF_CHECK_I(fdi_m_n.link_n); |
| + PIPE_CONF_CHECK_I(fdi_m_n.tu); |
| |
| #undef PIPE_CONF_CHECK_I |
| |