On Fri, Aug 04, 2017 at 07:00:33PM +0100, Chris Wilson wrote:
> Quoting Michal Wajdeczko (2017-08-04 17:27:02)
> > To allow future code reuse. While here, fix comment style.
>
> Might as well fix the alignment as well then...
>
> > + /* Handle flush interrupt in bottom half */
> > + queue_work(dev_priv->guc.log.runtime.flush_wq,
> > + &dev_priv->guc.log.runtime.flush_work);
> > +
> > + dev_priv->guc.log.flush_interrupt_count++;
> > + } else {
> > + /*
> > + * Not clearing of unhandled event bits won't result in
> > + * re-triggering of the interrupt.
> > + */
> > + }
> > +}
>
> Argh, and those are spaces not tabs! Back to the checkpatch dungeon you
> go, until you are sorry!
Hmm, that's weird. On my side there were all tabs.
$ scripts/checkpatch.pl ../RFC/0005-drm-i915-guc-Move-Guc-notification-handling-to-separ.patch
total: 0 errors, 0 warnings, 89 lines checked
../RFC/0005-drm-i915-guc-Move-Guc-notification-handling-to-separ.patch has no obvious style problems and is ready for submission.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
