Skip to content

ILK/IVB irq handler merge

Paulo Zanoni intel-gfx 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6drm/i915: extract ilk_display_irq_handlerdrivers/gpu/drm/i915/i915_irq.c+56 -480 replies
2/6drm/i915: extract ivb_display_irq_handlerdrivers/gpu/drm/i915/i915_irq.c+35 -280 replies
3/6drm/i915: don't read or write GEN6_PMIIR on Gen 5drivers/gpu/drm/i915/i915_irq.c+5 -30 replies
4/6drm/i915: reorganize ironlake_irq_handlerdrivers/gpu/drm/i915/i915_irq.c+22 -220 replies
5/6drm/i915: POSTING_READ(DEIER) on ivybridge_irq_handlerdrivers/gpu/drm/i915/i915_irq.c+1 -00 replies
6/6drm/i915: add ILK/SNB support to ivybridge_irq_handlerdrivers/gpu/drm/i915/i915_irq.c+31 -820 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/6] ILK/IVB irq handler merge

From: Paulo Zanoni <[email protected]>

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(-)
Signature · 1 line
1.8.1.2