diff mbox

drm/i915/bdw: make sure south port interrupts are enabled properly v2

Message ID 1389388389-7008-1-git-send-email-jbarnes@virtuousgeek.org (mailing list archive)
State New, archived
Headers show

Commit Message

Jesse Barnes Jan. 10, 2014, 9:13 p.m. UTC
We were apparently relying on the defaults on BDW, which resulted in no
hotplug or AUX interrupts.  So be sure to call the ibx_irq_preinstall to
enable all interrupts.

v2: use preinstall instead of redundant SDIER write

References: https://bugs.freedesktop.org/show_bug.cgi?id=72834
References: https://bugs.freedesktop.org/show_bug.cgi?id=72833
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
---
 drivers/gpu/drm/i915/i915_irq.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Paulo Zanoni Jan. 10, 2014, 9:17 p.m. UTC | #1
2014/1/10 Jesse Barnes <jbarnes@virtuousgeek.org>:
> We were apparently relying on the defaults on BDW, which resulted in no
> hotplug or AUX interrupts.  So be sure to call the ibx_irq_preinstall to
> enable all interrupts.
>
> v2: use preinstall instead of redundant SDIER write
>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=72834
> References: https://bugs.freedesktop.org/show_bug.cgi?id=72833
> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

And this should probably go to Kernel 3.13 too, right?

Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>

> ---
>  drivers/gpu/drm/i915/i915_irq.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index 261254a..69b0b71 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -2714,6 +2714,8 @@ static void gen8_irq_preinstall(struct drm_device *dev)
>  #undef GEN8_IRQ_INIT_NDX
>
>         POSTING_READ(GEN8_PCU_IIR);
> +
> +       ibx_irq_preinstall(dev);
>  }
>
>  static void ibx_hpd_irq_setup(struct drm_device *dev)
> --
> 1.8.3.2
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Jesse Barnes Jan. 10, 2014, 9:19 p.m. UTC | #2
On Fri, 10 Jan 2014 19:17:02 -0200
Paulo Zanoni <przanoni@gmail.com> wrote:

> 2014/1/10 Jesse Barnes <jbarnes@virtuousgeek.org>:
> > We were apparently relying on the defaults on BDW, which resulted in no
> > hotplug or AUX interrupts.  So be sure to call the ibx_irq_preinstall to
> > enable all interrupts.
> >
> > v2: use preinstall instead of redundant SDIER write
> >
> > References: https://bugs.freedesktop.org/show_bug.cgi?id=72834
> > References: https://bugs.freedesktop.org/show_bug.cgi?id=72833
> > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
> 
> And this should probably go to Kernel 3.13 too, right?
> 
> Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>

Yeah probably.

Daniel, please cc:stable

Thanks,
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 261254a..69b0b71 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -2714,6 +2714,8 @@  static void gen8_irq_preinstall(struct drm_device *dev)
 #undef GEN8_IRQ_INIT_NDX
 
 	POSTING_READ(GEN8_PCU_IIR);
+
+	ibx_irq_preinstall(dev);
 }
 
 static void ibx_hpd_irq_setup(struct drm_device *dev)