2014-10-30 15:43 GMT-02:00 <ville.syrjala@xxxxxxxxxxxxxxx>: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > The extra VLV_IIR writes at the end of vlv_display_irq_postinstall() > serve no purpose. Remove them. > > The VLV_IMR/IER/IIR setup at the start of the function also seems a bit > pointless since it doesn't unmask/enable anything. But leave it be for > now. Reviewed-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_irq.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c > index 6a00e6e..628a129 100644 > --- a/drivers/gpu/drm/i915/i915_irq.c > +++ b/drivers/gpu/drm/i915/i915_irq.c > @@ -3444,9 +3444,6 @@ static void vlv_display_irq_postinstall(struct drm_i915_private *dev_priv) > if (dev_priv->display_irqs_enabled) > valleyview_display_irqs_install(dev_priv); > spin_unlock_irq(&dev_priv->irq_lock); > - > - I915_WRITE(VLV_IIR, 0xffffffff); > - I915_WRITE(VLV_IIR, 0xffffffff); > } > > static int valleyview_irq_postinstall(struct drm_device *dev) > -- > 2.0.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Paulo Zanoni _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
