[PATCH 18/22] irq

José Roberto de Souza jose.souza at intel.com
Wed Aug 1 20:23:43 UTC 2018


---
 drivers/gpu/drm/i915/i915_irq.c | 147 +++++++++++++++++++++-----------
 1 file changed, 95 insertions(+), 52 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 255e034e7ee4..ad21d7d6c9b9 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -3553,11 +3553,13 @@ static void ironlake_irq_reset(struct drm_device *dev)
 	if (IS_GEN5(dev_priv))
 		I915_WRITE(HWSTAM, 0xffffffff);
 
-	GEN3_IRQ_RESET(DE);
-	if (IS_GEN7(dev_priv))
-		I915_WRITE(GEN7_ERR_INT, 0xffffffff);
+	if (INTEL_INFO(dev_priv)->num_pipes) {
+		GEN3_IRQ_RESET(DE);
+		if (IS_GEN7(dev_priv))
+			I915_WRITE(GEN7_ERR_INT, 0xffffffff);
+	}
 
-	if (IS_HASWELL(dev_priv)) {
+	if (IS_HASWELL(dev_priv) && INTEL_INFO(dev_priv)->num_pipes) {
 		I915_WRITE(EDP_PSR_IMR, 0xffffffff);
 		I915_WRITE(EDP_PSR_IIR, 0xffffffff);
 	}
@@ -3576,6 +3578,9 @@ static void valleyview_irq_reset(struct drm_device *dev)
 
 	gen5_gt_irq_reset(dev_priv);
 
+	if (!INTEL_INFO(dev_priv)->num_pipes)
+		return;
+
 	spin_lock_irq(&dev_priv->irq_lock);
 	if (dev_priv->display_irqs_enabled)
 		vlv_display_irq_reset(dev_priv);
@@ -3600,6 +3605,9 @@ static void gen8_irq_reset(struct drm_device *dev)
 
 	gen8_gt_irq_reset(dev_priv);
 
+	if (!INTEL_INFO(dev_priv)->num_pipes)
+		return;
+
 	I915_WRITE(EDP_PSR_IMR, 0xffffffff);
 	I915_WRITE(EDP_PSR_IIR, 0xffffffff);
 
@@ -3643,6 +3651,9 @@ static void gen11_irq_reset(struct drm_device *dev)
 
 	gen11_gt_irq_reset(dev_priv);
 
+	if (!INTEL_INFO(dev_priv)->num_pipes)
+		return;
+
 	I915_WRITE(GEN11_DISPLAY_INT_CTL, 0);
 
 	for_each_pipe(dev_priv, pipe)
@@ -3711,6 +3722,9 @@ static void cherryview_irq_reset(struct drm_device *dev)
 
 	gen8_gt_irq_reset(dev_priv);
 
+	if (!INTEL_INFO(dev_priv)->num_pipes)
+		return;
+
 	GEN3_IRQ_RESET(GEN8_PCU_);
 
 	spin_lock_irq(&dev_priv->irq_lock);
@@ -4051,7 +4065,7 @@ static int ironlake_irq_postinstall(struct drm_device *dev)
 			      DE_DP_A_HOTPLUG);
 	}
 
-	if (IS_HASWELL(dev_priv)) {
+	if (IS_HASWELL(dev_priv) && INTEL_INFO(dev_priv)->num_pipes) {
 		gen3_assert_iir_is_zero(dev_priv, EDP_PSR_IIR);
 		intel_psr_irq_control(dev_priv, dev_priv->psr.debug);
 		display_mask |= DE_EDP_PSR_INT_HSW;
@@ -4061,15 +4075,17 @@ static int ironlake_irq_postinstall(struct drm_device *dev)
 
 	ibx_irq_pre_postinstall(dev);
 
-	GEN3_IRQ_INIT(DE, dev_priv->irq_mask, display_mask | extra_mask);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		GEN3_IRQ_INIT(DE, dev_priv->irq_mask, display_mask | extra_mask);
 
 	gen5_gt_irq_postinstall(dev);
 
-	ilk_hpd_detection_setup(dev_priv);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		ilk_hpd_detection_setup(dev_priv);
 
 	ibx_irq_postinstall(dev);
 
-	if (IS_IRONLAKE_M(dev_priv)) {
+	if (IS_IRONLAKE_M(dev_priv) && INTEL_INFO(dev_priv)->num_pipes) {
 		/* Enable PCU event interrupts
 		 *
 		 * spinlocking not required here for correctness since interrupt
@@ -4118,10 +4134,12 @@ static int valleyview_irq_postinstall(struct drm_device *dev)
 
 	gen5_gt_irq_postinstall(dev);
 
-	spin_lock_irq(&dev_priv->irq_lock);
-	if (dev_priv->display_irqs_enabled)
-		vlv_display_irq_postinstall(dev_priv);
-	spin_unlock_irq(&dev_priv->irq_lock);
+	if (INTEL_INFO(dev_priv)->num_pipes) {
+		spin_lock_irq(&dev_priv->irq_lock);
+		if (dev_priv->display_irqs_enabled)
+			vlv_display_irq_postinstall(dev_priv);
+		spin_unlock_irq(&dev_priv->irq_lock);
+	}
 
 	I915_WRITE(VLV_MASTER_IER, MASTER_INTERRUPT_ENABLE);
 	POSTING_READ(VLV_MASTER_IER);
@@ -4236,7 +4254,8 @@ static int gen8_irq_postinstall(struct drm_device *dev)
 		ibx_irq_pre_postinstall(dev);
 
 	gen8_gt_irq_postinstall(dev_priv);
-	gen8_de_irq_postinstall(dev_priv);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		gen8_de_irq_postinstall(dev_priv);
 
 	if (HAS_PCH_SPLIT(dev_priv))
 		ibx_irq_postinstall(dev);
@@ -4298,11 +4317,13 @@ static int gen11_irq_postinstall(struct drm_device *dev)
 		icp_irq_postinstall(dev);
 
 	gen11_gt_irq_postinstall(dev_priv);
-	gen8_de_irq_postinstall(dev_priv);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		gen8_de_irq_postinstall(dev_priv);
 
 	GEN3_IRQ_INIT(GEN11_GU_MISC_, ~gu_misc_masked, gu_misc_masked);
 
-	I915_WRITE(GEN11_DISPLAY_INT_CTL, GEN11_DISPLAY_IRQ_ENABLE);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		I915_WRITE(GEN11_DISPLAY_INT_CTL, GEN11_DISPLAY_IRQ_ENABLE);
 
 	I915_WRITE(GEN11_GFX_MSTR_IRQ, GEN11_MASTER_IRQ);
 	POSTING_READ(GEN11_GFX_MSTR_IRQ);
@@ -4316,10 +4337,12 @@ static int cherryview_irq_postinstall(struct drm_device *dev)
 
 	gen8_gt_irq_postinstall(dev_priv);
 
-	spin_lock_irq(&dev_priv->irq_lock);
-	if (dev_priv->display_irqs_enabled)
-		vlv_display_irq_postinstall(dev_priv);
-	spin_unlock_irq(&dev_priv->irq_lock);
+	if (INTEL_INFO(dev_priv)->num_pipes) {
+		spin_lock_irq(&dev_priv->irq_lock);
+		if (dev_priv->display_irqs_enabled)
+			vlv_display_irq_postinstall(dev_priv);
+		spin_unlock_irq(&dev_priv->irq_lock);
+	}
 
 	I915_WRITE(GEN8_MASTER_IRQ, GEN8_MASTER_IRQ_CONTROL);
 	POSTING_READ(GEN8_MASTER_IRQ);
@@ -4331,7 +4354,8 @@ static void i8xx_irq_reset(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
 
-	i9xx_pipestat_irq_reset(dev_priv);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		i9xx_pipestat_irq_reset(dev_priv);
 
 	I915_WRITE16(HWSTAM, 0xffff);
 
@@ -4341,7 +4365,7 @@ static void i8xx_irq_reset(struct drm_device *dev)
 static int i8xx_irq_postinstall(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
-	u16 enable_mask;
+	u16 enable_mask = I915_MASTER_ERROR_INTERRUPT | I915_USER_INTERRUPT;
 
 	I915_WRITE16(EMR, ~(I915_ERROR_PAGE_TABLE |
 			    I915_ERROR_MEMORY_REFRESH));
@@ -4352,14 +4376,15 @@ static int i8xx_irq_postinstall(struct drm_device *dev)
 		  I915_DISPLAY_PIPE_B_EVENT_INTERRUPT |
 		  I915_MASTER_ERROR_INTERRUPT);
 
-	enable_mask =
-		I915_DISPLAY_PIPE_A_EVENT_INTERRUPT |
-		I915_DISPLAY_PIPE_B_EVENT_INTERRUPT |
-		I915_MASTER_ERROR_INTERRUPT |
-		I915_USER_INTERRUPT;
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		enable_mask |= I915_DISPLAY_PIPE_A_EVENT_INTERRUPT |
+			       I915_DISPLAY_PIPE_B_EVENT_INTERRUPT;
 
 	GEN2_IRQ_INIT(, dev_priv->irq_mask, enable_mask);
 
+	if (!INTEL_INFO(dev_priv)->num_pipes)
+		return 0;
+
 	/* Interrupt setup is already guaranteed to be single-threaded, this is
 	 * just to make the assert_spin_locked check happy. */
 	spin_lock_irq(&dev_priv->irq_lock);
@@ -4495,12 +4520,15 @@ static void i915_irq_reset(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
 
-	if (I915_HAS_HOTPLUG(dev_priv)) {
-		i915_hotplug_interrupt_update(dev_priv, 0xffffffff, 0);
-		I915_WRITE(PORT_HOTPLUG_STAT, I915_READ(PORT_HOTPLUG_STAT));
-	}
+	if (INTEL_INFO(dev_priv)->num_pipes) {
+		if (I915_HAS_HOTPLUG(dev_priv)) {
+			i915_hotplug_interrupt_update(dev_priv, 0xffffffff, 0);
+			I915_WRITE(PORT_HOTPLUG_STAT,
+				   I915_READ(PORT_HOTPLUG_STAT));
+		}
 
-	i9xx_pipestat_irq_reset(dev_priv);
+		i9xx_pipestat_irq_reset(dev_priv);
+	}
 
 	I915_WRITE(HWSTAM, 0xffffffff);
 
@@ -4510,7 +4538,8 @@ static void i915_irq_reset(struct drm_device *dev)
 static int i915_irq_postinstall(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
-	u32 enable_mask;
+	u32 enable_mask = I915_MASTER_ERROR_INTERRUPT |
+			  I915_USER_INTERRUPT;
 
 	I915_WRITE(EMR, ~(I915_ERROR_PAGE_TABLE |
 			  I915_ERROR_MEMORY_REFRESH));
@@ -4522,14 +4551,12 @@ static int i915_irq_postinstall(struct drm_device *dev)
 		  I915_DISPLAY_PIPE_B_EVENT_INTERRUPT |
 		  I915_MASTER_ERROR_INTERRUPT);
 
-	enable_mask =
-		I915_ASLE_INTERRUPT |
-		I915_DISPLAY_PIPE_A_EVENT_INTERRUPT |
-		I915_DISPLAY_PIPE_B_EVENT_INTERRUPT |
-		I915_MASTER_ERROR_INTERRUPT |
-		I915_USER_INTERRUPT;
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		enable_mask |= I915_ASLE_INTERRUPT |
+			       I915_DISPLAY_PIPE_A_EVENT_INTERRUPT |
+			       I915_DISPLAY_PIPE_B_EVENT_INTERRUPT;
 
-	if (I915_HAS_HOTPLUG(dev_priv)) {
+	if (INTEL_INFO(dev_priv)->num_pipes && I915_HAS_HOTPLUG(dev_priv)) {
 		/* Enable in IER... */
 		enable_mask |= I915_DISPLAY_PORT_INTERRUPT;
 		/* and unmask in IMR */
@@ -4538,6 +4565,9 @@ static int i915_irq_postinstall(struct drm_device *dev)
 
 	GEN3_IRQ_INIT(, dev_priv->irq_mask, enable_mask);
 
+	if (!INTEL_INFO(dev_priv)->num_pipes)
+		return 0;
+
 	/* Interrupt setup is already guaranteed to be single-threaded, this is
 	 * just to make the assert_spin_locked check happy. */
 	spin_lock_irq(&dev_priv->irq_lock);
@@ -4608,10 +4638,12 @@ static void i965_irq_reset(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
 
-	i915_hotplug_interrupt_update(dev_priv, 0xffffffff, 0);
-	I915_WRITE(PORT_HOTPLUG_STAT, I915_READ(PORT_HOTPLUG_STAT));
+	if (INTEL_INFO(dev_priv)->num_pipes) {
+		i915_hotplug_interrupt_update(dev_priv, 0xffffffff, 0);
+		I915_WRITE(PORT_HOTPLUG_STAT, I915_READ(PORT_HOTPLUG_STAT));
 
-	i9xx_pipestat_irq_reset(dev_priv);
+		i9xx_pipestat_irq_reset(dev_priv);
+	}
 
 	I915_WRITE(HWSTAM, 0xffffffff);
 
@@ -4621,7 +4653,7 @@ static void i965_irq_reset(struct drm_device *dev)
 static int i965_irq_postinstall(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
-	u32 enable_mask;
+	u32 enable_mask = I915_MASTER_ERROR_INTERRUPT | I915_USER_INTERRUPT;
 	u32 error_mask;
 
 	/*
@@ -4647,19 +4679,22 @@ static int i965_irq_postinstall(struct drm_device *dev)
 		  I915_DISPLAY_PIPE_B_EVENT_INTERRUPT |
 		  I915_MASTER_ERROR_INTERRUPT);
 
-	enable_mask =
-		I915_ASLE_INTERRUPT |
-		I915_DISPLAY_PORT_INTERRUPT |
-		I915_DISPLAY_PIPE_A_EVENT_INTERRUPT |
-		I915_DISPLAY_PIPE_B_EVENT_INTERRUPT |
-		I915_MASTER_ERROR_INTERRUPT |
-		I915_USER_INTERRUPT;
+	if (INTEL_INFO(dev_priv)->num_pipes) {
+		enable_mask |=
+			I915_ASLE_INTERRUPT |
+			I915_DISPLAY_PORT_INTERRUPT |
+			I915_DISPLAY_PIPE_A_EVENT_INTERRUPT |
+			I915_DISPLAY_PIPE_B_EVENT_INTERRUPT;
+	}
 
 	if (IS_G4X(dev_priv))
 		enable_mask |= I915_BSD_USER_INTERRUPT;
 
 	GEN3_IRQ_INIT(, dev_priv->irq_mask, enable_mask);
 
+	if (!INTEL_INFO(dev_priv)->num_pipes)
+		return 0;
+
 	/* Interrupt setup is already guaranteed to be single-threaded, this is
 	 * just to make the assert_spin_locked check happy. */
 	spin_lock_irq(&dev_priv->irq_lock);
@@ -4767,7 +4802,8 @@ void intel_irq_init(struct drm_i915_private *dev_priv)
 	struct intel_rps *rps = &dev_priv->gt_pm.rps;
 	int i;
 
-	intel_hpd_init_work(dev_priv);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		intel_hpd_init_work(dev_priv);
 
 	INIT_WORK(&rps->work, gen6_pm_rps_work);
 
@@ -4905,6 +4941,12 @@ void intel_irq_init(struct drm_i915_private *dev_priv)
 		if (I915_HAS_HOTPLUG(dev_priv))
 			dev_priv->display.hpd_irq_setup = i915_hpd_irq_setup;
 	}
+
+	if (!INTEL_INFO(dev_priv)->num_pipes) {
+		dev->driver->enable_vblank = NULL;
+		dev->driver->disable_vblank = NULL;
+		dev_priv->display.hpd_irq_setup = NULL;
+	}
 }
 
 /**
@@ -4954,7 +4996,8 @@ int intel_irq_install(struct drm_i915_private *dev_priv)
 void intel_irq_uninstall(struct drm_i915_private *dev_priv)
 {
 	drm_irq_uninstall(&dev_priv->drm);
-	intel_hpd_cancel_work(dev_priv);
+	if (INTEL_INFO(dev_priv)->num_pipes)
+		intel_hpd_cancel_work(dev_priv);
 	dev_priv->runtime_pm.irqs_enabled = false;
 }
 
-- 
2.18.0



More information about the Intel-gfx-trybot mailing list