Attachment 59920 Details for Bug 48652 – Separate the PLL from the pipes

[patch] Separate the PLL from the pipes

0001-drm-i915-manage-PCH-PLLs-separately-from-pipes.patch (text/plain), 17.92 KB, created by Chris Wilson on 2012-04-13 10:47:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Wilson

Created: 2012-04-13 10:47:36 UTC

Size: 17.92 KB

patch

obsolete

>From 8626f4287397963e7b42321d708ae5c9d341d7d5 Mon Sep 17 00:00:00 2001
>From: Jesse Barnes <jbarnes@virtuousgeek.org>
>Date: Wed, 11 Apr 2012 14:16:43 -0700
>Subject: [PATCH] drm/i915: manage PCH PLLs separately from pipes
>
>PCH PLLs aren't required for outputs on the CPU, so we shouldn't just
>treat them as part of the pipe.
>
>So split the code out and manage PCH PLLs separately, allocating them
>when needed or trying to re-use existing PCH PLL setups when the timings
>match.
>
>v2: add num_pch_pll field to dev_priv (Daniel)
>    don't NULL the pch_pll pointer in disable or DPMS will fail (Jesse)
>    put register offsets in pll struct (Chris)
>
>v3: Decouple enable/disable of PLLs from get/put.
>
>Fixes https://bugs.freedesktop.org/show_bug.cgi?id=44309
>
>Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
>Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>---
> drivers/gpu/drm/i915/i915_drv.c      |    4 +
> drivers/gpu/drm/i915/i915_drv.h      |    4 +
> drivers/gpu/drm/i915/i915_reg.h      |    6 +-
> drivers/gpu/drm/i915/i915_suspend.c  |    2 +-
> drivers/gpu/drm/i915/intel_display.c |  243 +++++++++++++++++++++++-----------
> drivers/gpu/drm/i915/intel_dp.c      |    1 +
> drivers/gpu/drm/i915/intel_drv.h     |   12 ++-
> 7 files changed, 189 insertions(+), 83 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
>index b66bf5e..504db72 100644
>--- a/drivers/gpu/drm/i915/i915_drv.c
>+++ b/drivers/gpu/drm/i915/i915_drv.c
>@@ -381,16 +381,20 @@ void intel_detect_pch(struct drm_device *dev)
> 
> 			if (id == INTEL_PCH_IBX_DEVICE_ID_TYPE) {
> 				dev_priv->pch_type = PCH_IBX;
>+				dev_priv->num_pch_pll = 2;
> 				DRM_DEBUG_KMS("Found Ibex Peak PCH\n");
> 			} else if (id == INTEL_PCH_CPT_DEVICE_ID_TYPE) {
> 				dev_priv->pch_type = PCH_CPT;
>+				dev_priv->num_pch_pll = 2;
> 				DRM_DEBUG_KMS("Found CougarPoint PCH\n");
> 			} else if (id == INTEL_PCH_PPT_DEVICE_ID_TYPE) {
> 				/* PantherPoint is CPT compatible */
> 				dev_priv->pch_type = PCH_CPT;
>+				dev_priv->num_pch_pll = 2;
> 				DRM_DEBUG_KMS("Found PatherPoint PCH\n");
> 			} else if (id == INTEL_PCH_LPT_DEVICE_ID_TYPE) {
> 				dev_priv->pch_type = PCH_LPT;
>+				dev_priv->num_pch_pll = 0;
> 				DRM_DEBUG_KMS("Found LynxPoint PCH\n");
> 			}
> 		}
>diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
>index 057169a..902f918 100644
>--- a/drivers/gpu/drm/i915/i915_drv.h
>+++ b/drivers/gpu/drm/i915/i915_drv.h
>@@ -233,6 +233,7 @@ struct drm_i915_display_funcs {
> 			     struct drm_display_mode *adjusted_mode,
> 			     int x, int y,
> 			     struct drm_framebuffer *old_fb);
>+	void (*off)(struct drm_crtc *crtc);
> 	void (*write_eld)(struct drm_connector *connector,
> 			  struct drm_crtc *crtc);
> 	void (*fdi_link_train)(struct drm_crtc *crtc);
>@@ -394,6 +395,7 @@ typedef struct drm_i915_private {
> 	unsigned int sr01, adpa, ppcr, dvob, dvoc, lvds;
> 	int vblank_pipe;
> 	int num_pipe;
>+	int num_pch_pll;
> 
> 	/* For hangcheck timer */
> #define DRM_I915_HANGCHECK_PERIOD 1500 /* in ms */
>@@ -749,6 +751,8 @@ typedef struct drm_i915_private {
> 	wait_queue_head_t pending_flip_queue;
> 	bool flip_pending_is_done;
> 
>+	struct intel_pch_pll *pch_plls;
>+
> 	/* Reclocking support */
> 	bool render_reclock_avail;
> 	bool lvds_downclock_avail;
>diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
>index 4c8243e..418aa27 100644
>--- a/drivers/gpu/drm/i915/i915_reg.h
>+++ b/drivers/gpu/drm/i915/i915_reg.h
>@@ -3374,15 +3374,15 @@
> 
> #define _PCH_DPLL_A              0xc6014
> #define _PCH_DPLL_B              0xc6018
>-#define PCH_DPLL(pipe) (pipe == 0 ?  _PCH_DPLL_A : _PCH_DPLL_B)
>+#define _PCH_DPLL(pll) (pll == 0 ? _PCH_DPLL_A : _PCH_DPLL_B)
> 
> #define _PCH_FPA0                0xc6040
> #define  FP_CB_TUNE		(0x3<<22)
> #define _PCH_FPA1                0xc6044
> #define _PCH_FPB0                0xc6048
> #define _PCH_FPB1                0xc604c
>-#define PCH_FP0(pipe) (pipe == 0 ? _PCH_FPA0 : _PCH_FPB0)
>-#define PCH_FP1(pipe) (pipe == 0 ? _PCH_FPA1 : _PCH_FPB1)
>+#define _PCH_FP0(pll) (pll == 0 ? _PCH_FPA0 : _PCH_FPB0)
>+#define _PCH_FP1(pll) (pll == 0 ? _PCH_FPA1 : _PCH_FPB1)
> 
> #define PCH_DPLL_TEST           0xc606c
> 
>diff --git a/drivers/gpu/drm/i915/i915_suspend.c b/drivers/gpu/drm/i915/i915_suspend.c
>index 0c3e3bf..73a5c3c 100644
>--- a/drivers/gpu/drm/i915/i915_suspend.c
>+++ b/drivers/gpu/drm/i915/i915_suspend.c
>@@ -40,7 +40,7 @@ static bool i915_pipe_enabled(struct drm_device *dev, enum pipe pipe)
> 		return false;
> 
> 	if (HAS_PCH_SPLIT(dev))
>-		dpll_reg = PCH_DPLL(pipe);
>+		dpll_reg = _PCH_DPLL(pipe);
> 	else
> 		dpll_reg = (pipe == PIPE_A) ? _DPLL_A : _DPLL_B;
> 
>diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
>index 7091ea1..c4f05f8 100644
>--- a/drivers/gpu/drm/i915/intel_display.c
>+++ b/drivers/gpu/drm/i915/intel_display.c
>@@ -914,26 +914,28 @@ static void assert_pll(struct drm_i915_private *dev_priv,
> 
> /* For ILK+ */
> static void assert_pch_pll(struct drm_i915_private *dev_priv,
>-			   enum pipe pipe, bool state)
>+			   struct intel_crtc *intel_crtc, bool state)
> {
> 	int reg;
> 	u32 val;
> 	bool cur_state;
> 
>+	if (!intel_crtc->pch_pll) {
>+		WARN(1, "asserting PCH PLL enabled with no PLL\n");
>+		return;
>+	}
>+
> 	if (HAS_PCH_CPT(dev_priv->dev)) {
> 		u32 pch_dpll;
> 
> 		pch_dpll = I915_READ(PCH_DPLL_SEL);
> 
> 		/* Make sure the selected PLL is enabled to the transcoder */
>-		WARN(!((pch_dpll >> (4 * pipe)) & 8),
>-		     "transcoder %d PLL not enabled\n", pipe);
>-
>-		/* Convert the transcoder pipe number to a pll pipe number */
>-		pipe = (pch_dpll >> (4 * pipe)) & 1;
>+		WARN(!((pch_dpll >> (4 * intel_crtc->pipe)) & 8),
>+		     "transcoder %d PLL not enabled\n", intel_crtc->pipe);
> 	}
> 
>-	reg = PCH_DPLL(pipe);
>+	reg = intel_crtc->pch_pll->pll_reg;
> 	val = I915_READ(reg);
> 	cur_state = !!(val & DPLL_VCO_ENABLE);
> 	WARN(cur_state != state,
>@@ -1309,22 +1311,24 @@ static void intel_disable_pll(struct drm_i915_private *dev_priv, enum pipe pipe)
>  * The PCH PLL needs to be enabled before the PCH transcoder, since it
>  * drives the transcoder clock.
>  */
>-static void intel_enable_pch_pll(struct drm_i915_private *dev_priv,
>-				 enum pipe pipe)
>+static void intel_enable_pch_pll(struct intel_crtc *intel_crtc)
> {
>+	struct drm_i915_private *dev_priv = intel_crtc->base.dev->dev_private;
>+	struct intel_pch_pll *pll = intel_crtc->pch_pll;
> 	int reg;
> 	u32 val;
> 
>-	if (pipe > 1)
>-		return;
>-
> 	/* PCH only available on ILK+ */
> 	BUG_ON(dev_priv->info->gen < 5);
>+	BUG_ON(pll == NULL);
> 
> 	/* PCH refclock must be enabled first */
> 	assert_pch_refclk_enabled(dev_priv);
> 
>-	reg = PCH_DPLL(pipe);
>+	if (pll->active++)
>+		return;
>+
>+	reg = pll->pll_reg;
> 	val = I915_READ(reg);
> 	val |= DPLL_VCO_ENABLE;
> 	I915_WRITE(reg, val);
>@@ -1332,32 +1336,26 @@ static void intel_enable_pch_pll(struct drm_i915_private *dev_priv,
> 	udelay(200);
> }
> 
>-static void intel_disable_pch_pll(struct drm_i915_private *dev_priv,
>-				  enum pipe pipe)
>+static void intel_disable_pch_pll(struct intel_crtc *intel_crtc)
> {
>+	struct drm_i915_private *dev_priv = intel_crtc->base.dev->dev_private;
>+	struct intel_pch_pll *pll = intel_crtc->pch_pll;
> 	int reg;
>-	u32 val, pll_mask = TRANSC_DPLL_ENABLE | TRANSC_DPLLB_SEL,
>-		pll_sel = TRANSC_DPLL_ENABLE;
>-
>-	if (pipe > 1)
>-		return;
>+	u32 val;
> 
> 	/* PCH only available on ILK+ */
> 	BUG_ON(dev_priv->info->gen < 5);
>+	if (pll == NULL)
>+	       return;
> 
>-	/* Make sure transcoder isn't still depending on us */
>-	assert_transcoder_disabled(dev_priv, pipe);
>-
>-	if (pipe == 0)
>-		pll_sel |= TRANSC_DPLLA_SEL;
>-	else if (pipe == 1)
>-		pll_sel |= TRANSC_DPLLB_SEL;
>-
>-
>-	if ((I915_READ(PCH_DPLL_SEL) & pll_mask) == pll_sel)
>+	BUG_ON(pll->active == 0);
>+	if (--pll->active)
> 		return;
> 
>-	reg = PCH_DPLL(pipe);
>+	/* Make sure transcoder isn't still depending on us */
>+	assert_transcoder_disabled(dev_priv, intel_crtc->pipe);
>+
>+	reg = pll->pll_reg;
> 	val = I915_READ(reg);
> 	val &= ~DPLL_VCO_ENABLE;
> 	I915_WRITE(reg, val);
>@@ -1376,7 +1374,7 @@ static void intel_enable_transcoder(struct drm_i915_private *dev_priv,
> 	BUG_ON(dev_priv->info->gen < 5);
> 
> 	/* Make sure PCH DPLL is enabled */
>-	assert_pch_pll_enabled(dev_priv, pipe);
>+	assert_pch_pll_enabled(dev_priv, to_intel_crtc(crtc));
> 
> 	/* FDI must be feeding us bits for PCH ports */
> 	assert_fdi_tx_enabled(dev_priv, pipe);
>@@ -3060,29 +3058,36 @@ static void ironlake_pch_enable(struct drm_crtc *crtc)
> 	struct drm_i915_private *dev_priv = dev->dev_private;
> 	struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> 	int pipe = intel_crtc->pipe;
>-	u32 reg, temp, transc_sel;
>+	u32 reg, temp;
> 
> 	/* For PCH output, training FDI link */
> 	dev_priv->display.fdi_link_train(crtc);
> 
>-	intel_enable_pch_pll(dev_priv, pipe);
>+	intel_enable_pch_pll(intel_crtc);
> 
> 	if (HAS_PCH_CPT(dev)) {
>-		transc_sel = intel_crtc->use_pll_a ? TRANSC_DPLLA_SEL :
>-			TRANSC_DPLLB_SEL;
>+		u32 sel;
> 
>-		/* Be sure PCH DPLL SEL is set */
> 		temp = I915_READ(PCH_DPLL_SEL);
>-		if (pipe == 0) {
>-			temp &= ~(TRANSA_DPLLB_SEL);
>-			temp |= (TRANSA_DPLL_ENABLE | TRANSA_DPLLA_SEL);
>-		} else if (pipe == 1) {
>-			temp &= ~(TRANSB_DPLLB_SEL);
>-			temp |= (TRANSB_DPLL_ENABLE | TRANSB_DPLLB_SEL);
>-		} else if (pipe == 2) {
>-			temp &= ~(TRANSC_DPLLB_SEL);
>-			temp |= (TRANSC_DPLL_ENABLE | transc_sel);
>+		switch (pipe) {
>+		default:
>+		case 0:
>+			temp |= TRANSA_DPLL_ENABLE;
>+			sel = TRANSA_DPLLB_SEL;
>+			break;
>+		case 1:
>+			temp |= TRANSB_DPLL_ENABLE;
>+			sel = TRANSB_DPLLB_SEL;
>+			break;
>+		case 2:
>+			temp |= TRANSC_DPLL_ENABLE;
>+			sel = TRANSC_DPLLB_SEL;
>+			break;
> 		}
>+		if (intel_crtc->pch_pll->pll_reg == _PCH_DPLL_B)
>+			temp |= sel;
>+		else
>+			temp &= ~sel;
> 		I915_WRITE(PCH_DPLL_SEL, temp);
> 	}
> 
>@@ -3140,6 +3145,59 @@ static void ironlake_pch_enable(struct drm_crtc *crtc)
> 	intel_enable_transcoder(dev_priv, pipe);
> }
> 
>+static void intel_put_pch_pll(struct intel_crtc *intel_crtc)
>+{
>+	struct intel_pch_pll *pll = intel_crtc->pch_pll;
>+
>+	if (pll == NULL)
>+		return;
>+
>+	if (pll->refcount == 0) {
>+		WARN(1, "bad PCH PLL refcount\n");
>+		return;
>+	}
>+
>+	--pll->refcount;
>+	intel_crtc->pch_pll = NULL;
>+}
>+
>+static struct intel_pch_pll *intel_get_pch_pll(struct intel_crtc *intel_crtc, u32 dpll, u32 fp)
>+{
>+	struct drm_i915_private *dev_priv = intel_crtc->base.dev->dev_private;
>+	struct intel_pch_pll *pll;
>+	int i;
>+
>+	if (intel_crtc->pch_pll)
>+		return intel_crtc->pch_pll;
>+
>+	for (i = 0; i < dev_priv->num_pch_pll; i++) {
>+		pll = &dev_priv->pch_plls[i];
>+
>+		/* Only want to check enabled timings first */
>+		if (pll->refcount == 0)
>+			continue;
>+
>+		if (dpll == (I915_READ(pll->pll_reg) & 0x7fffffff) &&
>+		    fp == I915_READ(pll->fp0_reg))
>+			goto found;
>+	}
>+
>+	/* Ok no matching timings, maybe there's a free one? */
>+	for (i = 0; i < dev_priv->num_pch_pll; i++) {
>+		pll = &dev_priv->pch_plls[i];
>+		if (pll->refcount == 0)
>+			goto found;
>+	}
>+
>+	return NULL;
>+
>+found:
>+	intel_crtc->pch_pll = pll;
>+	pll->refcount++;
>+	DRM_DEBUG_DRIVER("using pll %d for pipe %d\n", i, intel_crtc->pipe);
>+	return pll;
>+}
>+
> void intel_cpt_verify_modeset(struct drm_device *dev, int pipe)
> {
> 	struct drm_i915_private *dev_priv = dev->dev_private;
>@@ -3284,8 +3342,7 @@ static void ironlake_crtc_disable(struct drm_crtc *crtc)
> 	}
> 
> 	/* disable PCH DPLL */
>-	if (!intel_crtc->no_pll)
>-		intel_disable_pch_pll(dev_priv, pipe);
>+	intel_disable_pch_pll(intel_crtc);
> 
> 	/* Switch from PCDclk to Rawclk */
> 	reg = FDI_RX_CTL(pipe);
>@@ -3341,6 +3398,12 @@ static void ironlake_crtc_dpms(struct drm_crtc *crtc, int mode)
> 	}
> }
> 
>+static void ironlake_crtc_off(struct drm_crtc *crtc)
>+{
>+	struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
>+	intel_put_pch_pll(intel_crtc);
>+}
>+
> static void intel_crtc_dpms_overlay(struct intel_crtc *intel_crtc, bool enable)
> {
> 	if (!enable && intel_crtc->overlay) {
>@@ -3432,6 +3495,10 @@ static void i9xx_crtc_dpms(struct drm_crtc *crtc, int mode)
> 	}
> }
> 
>+static void i9xx_crtc_off(struct drm_crtc *crtc)
>+{
>+}
>+
> /**
>  * Sets the power management mode of the pipe and plane.
>  */
>@@ -3479,8 +3546,11 @@ static void intel_crtc_disable(struct drm_crtc *crtc)
> {
> 	struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
> 	struct drm_device *dev = crtc->dev;
>+	struct drm_i915_private *dev_priv = dev->dev_private;
> 
> 	crtc_funcs->dpms(crtc, DRM_MODE_DPMS_OFF);
>+	dev_priv->display.off(crtc);
>+
> 	assert_plane_disabled(dev->dev_private, to_intel_crtc(crtc)->plane);
> 	assert_pipe_disabled(dev->dev_private, to_intel_crtc(crtc)->pipe);
> 
>@@ -6208,29 +6278,24 @@ static int ironlake_crtc_mode_set(struct drm_crtc *crtc,
> 	DRM_DEBUG_KMS("Mode for pipe %d:\n", pipe);
> 	drm_mode_debug_printmodeline(mode);
> 
>-	/* PCH eDP needs FDI, but CPU eDP does not */
>-	if (!intel_crtc->no_pll) {
>-		if (!is_cpu_edp) {
>-			I915_WRITE(PCH_FP0(pipe), fp);
>-			I915_WRITE(PCH_DPLL(pipe), dpll & ~DPLL_VCO_ENABLE);
>+	/* CPU eDP is the only output that doesn't need a PCH PLL of its own */
>+	if (!is_cpu_edp) {
>+		struct intel_pch_pll *pll;
> 
>-			POSTING_READ(PCH_DPLL(pipe));
>-			udelay(150);
>-		}
>-	} else {
>-		if (dpll == (I915_READ(PCH_DPLL(0)) & 0x7fffffff) &&
>-		    fp == I915_READ(PCH_FP0(0))) {
>-			intel_crtc->use_pll_a = true;
>-			DRM_DEBUG_KMS("using pipe a dpll\n");
>-		} else if (dpll == (I915_READ(PCH_DPLL(1)) & 0x7fffffff) &&
>-			   fp == I915_READ(PCH_FP0(1))) {
>-			intel_crtc->use_pll_a = false;
>-			DRM_DEBUG_KMS("using pipe b dpll\n");
>-		} else {
>-			DRM_DEBUG_KMS("no matching PLL configuration for pipe 2\n");
>+		pll = intel_get_pch_pll(intel_crtc, dpll, fp);
>+		if (pll == NULL) {
>+			DRM_DEBUG_DRIVER("failed to find PLL for pipe %d\n",
>+					 pipe);
> 			return -EINVAL;
> 		}
>-	}
>+
>+		I915_WRITE(pll->fp0_reg, fp);
>+		I915_WRITE(pll->pll_reg, dpll & ~DPLL_VCO_ENABLE);
>+
>+		POSTING_READ(pll->pll_reg);
>+		udelay(150);
>+	} else
>+		intel_put_pch_pll(intel_crtc);
> 
> 	/* The LVDS pin pair needs to be on before the DPLLs are enabled.
> 	 * This is an exception to the general rule that mode_set doesn't turn
>@@ -6298,11 +6363,11 @@ static int ironlake_crtc_mode_set(struct drm_crtc *crtc,
> 		I915_WRITE(TRANSDPLINK_N1(pipe), 0);
> 	}
> 
>-	if (!intel_crtc->no_pll && (!edp_encoder || is_pch_edp)) {
>-		I915_WRITE(PCH_DPLL(pipe), dpll);
>+	if (intel_crtc->pch_pll) {
>+		I915_WRITE(intel_crtc->pch_pll->pll_reg, dpll);
> 
> 		/* Wait for the clocks to stabilize. */
>-		POSTING_READ(PCH_DPLL(pipe));
>+		POSTING_READ(intel_crtc->pch_pll->pll_reg);
> 		udelay(150);
> 
> 		/* The pixel multiplier can only be updated once the
>@@ -6310,20 +6375,20 @@ static int ironlake_crtc_mode_set(struct drm_crtc *crtc,
> 		 *
> 		 * So write it again.
> 		 */
>-		I915_WRITE(PCH_DPLL(pipe), dpll);
>+		I915_WRITE(intel_crtc->pch_pll->pll_reg, dpll);
> 	}
> 
> 	intel_crtc->lowfreq_avail = false;
>-	if (!intel_crtc->no_pll) {
>+	if (intel_crtc->pch_pll) {
> 		if (is_lvds && has_reduced_clock && i915_powersave) {
>-			I915_WRITE(PCH_FP1(pipe), fp2);
>+			I915_WRITE(intel_crtc->pch_pll->fp1_reg, fp2);
> 			intel_crtc->lowfreq_avail = true;
> 			if (HAS_PIPE_CXSR(dev)) {
> 				DRM_DEBUG_KMS("enabling CxSR downclocking\n");
> 				pipeconf |= PIPECONF_CXSR_DOWNCLOCK;
> 			}
> 		} else {
>-			I915_WRITE(PCH_FP1(pipe), fp);
>+			I915_WRITE(intel_crtc->pch_pll->fp1_reg, fp);
> 			if (HAS_PIPE_CXSR(dev)) {
> 				DRM_DEBUG_KMS("disabling CxSR downclocking\n");
> 				pipeconf &= ~PIPECONF_CXSR_DOWNCLOCK;
>@@ -8061,6 +8126,27 @@ static const struct drm_crtc_funcs intel_crtc_funcs = {
> 	.page_flip = intel_crtc_page_flip,
> };
> 
>+static int intel_pch_pll_init(struct drm_device *dev)
>+{
>+	drm_i915_private_t *dev_priv = dev->dev_private;
>+	struct intel_pch_pll *pch_plls;
>+	int i;
>+
>+	pch_plls = kzalloc(sizeof(struct intel_pch_pll) *
>+			   dev_priv->num_pch_pll, GFP_KERNEL);
>+	if (!pch_plls)
>+		return -ENOMEM;
>+
>+	for (i = 0; i < dev_priv->num_pch_pll; i++) {
>+		pch_plls[i].pll_reg = _PCH_DPLL(i);
>+		pch_plls[i].fp0_reg = _PCH_FP0(i);
>+		pch_plls[i].fp1_reg = _PCH_FP1(i);
>+	}
>+
>+	dev_priv->pch_plls = pch_plls;
>+	return 0;
>+}
>+
> static void intel_crtc_init(struct drm_device *dev, int pipe)
> {
> 	drm_i915_private_t *dev_priv = dev->dev_private;
>@@ -8100,8 +8186,6 @@ static void intel_crtc_init(struct drm_device *dev, int pipe)
> 	intel_crtc->bpp = 24; /* default for pre-Ironlake */
> 
> 	if (HAS_PCH_SPLIT(dev)) {
>-		if (pipe == 2 && IS_IVYBRIDGE(dev))
>-			intel_crtc->no_pll = true;
> 		intel_helper_funcs.prepare = ironlake_crtc_prepare;
> 		intel_helper_funcs.commit = ironlake_crtc_commit;
> 	} else {
>@@ -9304,10 +9388,12 @@ static void intel_init_display(struct drm_device *dev)
> 	if (HAS_PCH_SPLIT(dev)) {
> 		dev_priv->display.dpms = ironlake_crtc_dpms;
> 		dev_priv->display.crtc_mode_set = ironlake_crtc_mode_set;
>+		dev_priv->display.off = ironlake_crtc_off;
> 		dev_priv->display.update_plane = ironlake_update_plane;
> 	} else {
> 		dev_priv->display.dpms = i9xx_crtc_dpms;
> 		dev_priv->display.crtc_mode_set = i9xx_crtc_mode_set;
>+		dev_priv->display.off = i9xx_crtc_off;
> 		dev_priv->display.update_plane = i9xx_update_plane;
> 	}
> 
>@@ -9691,6 +9777,9 @@ void intel_modeset_init(struct drm_device *dev)
> 			DRM_DEBUG_KMS("plane %d init failed: %d\n", i, ret);
> 	}
> 
>+	if (HAS_PCH_SPLIT(dev))
>+		intel_pch_pll_init(dev);
>+
> 	/* Just disable it once at startup */
> 	i915_disable_vga(dev);
> 	intel_setup_outputs(dev);
>diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
>index 6346b29..f07652b 100644
>--- a/drivers/gpu/drm/i915/intel_dp.c
>+++ b/drivers/gpu/drm/i915/intel_dp.c
>@@ -2417,6 +2417,7 @@ intel_dp_init(struct drm_device *dev, int output_reg)
> 	}
> 
> 	intel_encoder->crtc_mask = (1 << 0) | (1 << 1) | (1 << 2);
>+
> 	connector->interlace_allowed = true;
> 	connector->doublescan_allowed = 0;
> 
>diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
>index 1f073f0..bcc7da5 100644
>--- a/drivers/gpu/drm/i915/intel_drv.h
>+++ b/drivers/gpu/drm/i915/intel_drv.h
>@@ -158,6 +158,15 @@ struct intel_connector {
> 	struct intel_encoder *encoder;
> };
> 
>+/* We can share PLLs across outputs if the timings match */
>+struct intel_pch_pll {
>+	int refcount;
>+	int active;
>+	int pll_reg;
>+	int fp0_reg;
>+	int fp1_reg;
>+};
>+
> struct intel_crtc {
> 	struct drm_crtc base;
> 	enum pipe pipe;
>@@ -182,8 +191,7 @@ struct intel_crtc {
> 	bool cursor_visible;
> 	unsigned int bpp;
> 
>-	bool no_pll; /* tertiary pipe for IVB */
>-	bool use_pll_a;
>+	struct intel_pch_pll *pch_pll; /* If any */
> };
> 
> struct intel_plane {
>-- 
>1.7.9.1
>

Actions: View | Splinter Review

Attachments on bug 48652: 59911 | 59912 | 59913 | 59914 | 59915 | 59916 | 59917 | 59920 | 59957 | 60147 | 60168 | 60171 | 60444 | 62393 | 62395 | 62396 | 66158 | 66159 | 71397 | 81446 | 82263 | 82344 | 87402 | 87403 | 87441 | 87451

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.