From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [PATCH 23/58] drm/i915/dp: implement get_hw_state Date: Tue, 4 Sep 2012 13:26:46 -0700 Message-ID: <20120904132646.170009aa@jbarnes-desktop> References: <1345403595-9678-1-git-send-email-daniel.vetter@ffwll.ch> <1345403595-9678-24-git-send-email-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from oproxy7-pub.bluehost.com (oproxy7-pub.bluehost.com [67.222.55.9]) by gabe.freedesktop.org (Postfix) with SMTP id D96879E936 for ; Tue, 4 Sep 2012 13:26:48 -0700 (PDT) In-Reply-To: <1345403595-9678-24-git-send-email-daniel.vetter@ffwll.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Sun, 19 Aug 2012 21:12:40 +0200 Daniel Vetter wrote: > Also add some macros to make the pipe computation a bit easier. > > v2: I've mixed up the CPT and !CPT PORT_TO_PIPE macro variants ... > > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/i915/i915_reg.h | 2 ++ > drivers/gpu/drm/i915/intel_dp.c | 50 +++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 52 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h > index fd6a26a..1e5f77a 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -4029,6 +4029,8 @@ > #define PORT_TRANS_C_SEL_CPT (2<<29) > #define PORT_TRANS_SEL_MASK (3<<29) > #define PORT_TRANS_SEL_CPT(pipe) ((pipe) << 29) > +#define PORT_TO_PIPE(val) (((val) & (1<<30)) >> 30) > +#define PORT_TO_PIPE_CPT(val) (((val) & PORT_TRANS_SEL_MASK) >> 29) > > #define TRANS_DP_CTL_A 0xe0300 > #define TRANS_DP_CTL_B 0xe1300 > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > index ff993a0..e3928b9 100644 > --- a/drivers/gpu/drm/i915/intel_dp.c > +++ b/drivers/gpu/drm/i915/intel_dp.c > @@ -1250,6 +1250,54 @@ static void intel_dp_sink_dpms(struct intel_dp *intel_dp, int mode) > } > } > > +static bool intel_dp_get_hw_state(struct intel_encoder *encoder, > + enum pipe *pipe) > +{ > + struct intel_dp *intel_dp = enc_to_intel_dp(&encoder->base); > + struct drm_device *dev = encoder->base.dev; > + struct drm_i915_private *dev_priv = dev->dev_private; > + u32 tmp = I915_READ(intel_dp->output_reg); > + > + if (!(tmp & DP_PORT_EN)) > + return false; > + > + if (is_cpu_edp(intel_dp) && IS_GEN7(dev)) { > + *pipe = PORT_TO_PIPE_CPT(tmp); > + } else if (!HAS_PCH_CPT(dev) || is_cpu_edp(intel_dp)) { > + *pipe = PORT_TO_PIPE(tmp); > + } else { > + u32 trans_sel; > + u32 trans_dp; > + int i; > + > + switch (intel_dp->output_reg) { > + case PCH_DP_B: > + trans_sel = TRANS_DP_PORT_SEL_B; > + break; > + case PCH_DP_C: > + trans_sel = TRANS_DP_PORT_SEL_C; > + break; > + case PCH_DP_D: > + trans_sel = TRANS_DP_PORT_SEL_D; > + break; > + default: > + return true; > + } > + > + for_each_pipe(i) { > + trans_dp = I915_READ(TRANS_DP_CTL(i)); > + if ((trans_dp & TRANS_DP_PORT_SEL_MASK) == trans_sel) { > + *pipe = i; > + return true; > + } > + } > + } > + > + DRM_DEBUG_KMS("No pipe for dp port 0x%x found\n", intel_dp->output_reg); > + > + return true; > +} > + > static void intel_disable_dp(struct intel_encoder *encoder) > { > struct intel_dp *intel_dp = enc_to_intel_dp(&encoder->base); > @@ -2486,6 +2534,8 @@ intel_dp_init(struct drm_device *dev, int output_reg, enum port port) > > intel_encoder->enable = intel_enable_dp; > intel_encoder->disable = intel_disable_dp; > + intel_encoder->get_hw_state = intel_dp_get_hw_state; > + intel_connector->get_hw_state = intel_connector_get_hw_state; > > /* Set up the DDC bus. */ > switch (port) { Why did the hw guys change the regs between PCH and CPU, and then change them again... Reviewed-by: Jesse Barnes -- Jesse Barnes, Intel Open Source Technology Center