From: Paulo Zanoni <paulo.r.zanoni at intel.com> This is something that have always bothered me: we have two functions that do almost the same thing, but their implementations are quite different. On this series I try to reorganize the two irq handlers so their implementations become very similar, and in the last patch I add ILK/SNB support to ivybridge_irq_handler and kill ironlake_irq_handler. This is mostly a coding-style/OCD series, but considering how complex our irq handlers are becoming, I think it's an advantage to be able to kill one irq handler. ALso, this series removes more lines than it adds :) Patches tested on SNB (which uses the ILK irq handler) and HSW (IVB irq handler). Paulo Zanoni (6): drm/i915: extract ilk_display_irq_handler drm/i915: extract ivb_display_irq_handler drm/i915: don't read or write GEN6_PMIIR on Gen 5 drm/i915: reorganize ironlake_irq_handler drm/i915: POSTING_READ(DEIER) on ivybridge_irq_handler drm/i915: add ILK/SNB support to ivybridge_irq_handler drivers/gpu/drm/i915/i915_irq.c | 273 ++++++++++++++++++---------------------- 1 file changed, 120 insertions(+), 153 deletions(-) -- 1.8.1.2
