On Mon, Jul 04, 2016 at 11:53:01AM +0300, Mika Kuoppala wrote: > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c > > index 3da36db9c830..1da821479161 100644 > > --- a/drivers/gpu/drm/i915/i915_debugfs.c > > +++ b/drivers/gpu/drm/i915/i915_debugfs.c > > @@ -4615,17 +4615,10 @@ i915_ring_test_irq_set(void *data, u64 val) > > { > > struct drm_device *dev = data; > > struct drm_i915_private *dev_priv = dev->dev_private; > > - int ret; > > > > DRM_DEBUG_DRIVER("Masking interrupts on rings 0x%08llx\n", val); > > Moving this dbg message past masking would make more sense? Yes, it does. Done and pushed. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx