On Mon, Feb 25, 2013 at 12:06:47PM -0500, Egbert Eich wrote: > I've reworked my 'hotplug interrupt storm detection'-patches and > included most of Daniel's suggestions. > I've looked into adding EDID caching but since this requires some > larger scale changes and some changes outside of the Intel driver > it seemed to be a good idea to propose those changes at a later time. > > Egbert Eich (12): > DRM/i915: Remove valleyview_hpd_irq_setup. > DRM/I915: Add enum hpd_pin to intel_encoder. > DRM/i915: Convert HPD interrupts to make use of HPD pin assignment in > encoders. > DRM/i915: Remove i965_hpd_irq_setup. > DRM/i915: Get rid if the 'hotplug_supported_mask' in struct > drm_i915_private. > DRM/i915: Add HPD IRQ storm detection. > DRM/i915: (re)init HPD interrupt storm statistics. > DRM/i915: Treat hpd_irq_setup() for ironake and older generations the > same way. > DRM/i915: Disable HPD interrupt on pin when irq storm is detected. > DRM/i915: Add Reenable Timer to turn Hotplug Detection back on. > DRM/i915: Add bit field to record which pins have received HPD > events. > DRM/i915: Only reprobe display on encoder which has received an HPD > event. I'm happy with this series. It seems to do everything that we need to enable/disable hpd, and tidies up hotplug detection in the process. Acked-by: Chris Wilson <chris at chris-wilson.co.uk> Didn't spot anything wrong in the first pass, but I'd like to let the details soak in a bit before r-b. -Chris -- Chris Wilson, Intel Open Source Technology Centre