On Sat, 2012-12-01 at 13:53 +0100, Daniel Vetter wrote: > This is for legacy legacy stuff, and checking with the leftover > pipe from the previous loop is propably not what we want. Since > pipe == 2 after the loop ... This doesn't seem to match the change, blc_event is simply never used here. > > Cc: Jesse Barnes <jbarnes at virtuousgeek.org> > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > --- > drivers/gpu/drm/i915/i915_irq.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c > index ff78818..2028137 100644 > --- a/drivers/gpu/drm/i915/i915_irq.c > +++ b/drivers/gpu/drm/i915/i915_irq.c > @@ -533,7 +533,6 @@ static irqreturn_t valleyview_irq_handler(int irq, void *arg) > unsigned long irqflags; > int pipe; > u32 pipe_stats[I915_MAX_PIPES]; > - bool blc_event; > > atomic_inc(&dev_priv->irq_received); > > @@ -590,9 +589,6 @@ static irqreturn_t valleyview_irq_handler(int irq, void *arg) > I915_READ(PORT_HOTPLUG_STAT); > } > > - if (pipe_stats[pipe] & PIPE_LEGACY_BLC_EVENT_STATUS) > - blc_event = true; > - > if (pm_iir & GEN6_PM_DEFERRED_EVENTS) > gen6_queue_rps_work(dev_priv, pm_iir); >
