2014-10-30 15:43 GMT-02:00 <[email protected]>: > From: Ville Syrjälä <[email protected]> > > 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 <[email protected]> > > Signed-off-by: Ville Syrjälä <[email protected]> > --- > 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 > [email protected] > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Paulo Zanoni _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
