[Intel-gfx] [PATCH v3 0/8] Add enlightenments for vGPU
Daniel Vetter
daniel at ffwll.ch
Mon Dec 15 00:18:49 PST 2014
On Thu, Dec 11, 2014 at 05:03:46PM +0000, Tvrtko Ursulin wrote:
>
> Hi,
>
> I'll try to do the detailed review of your series in the following few days.
> I might ask some questions on the design also to help me understand the
> bigger picture.
>
> First thing, I see that patches are checkpatch.pl clean, apart when run in
> strict mode. I think Daniel prefers "--strict" nowadays, at least I needed
> to fix up those in my patches so you should probably do the same.
Well I mostly run --strict since that checks for the alignment of
continuation lines for function parameters. Imo not following the i915
coding style for that looks a bit too jarring ;-)
Still checkpatch is just a tool and occasionally seriously lacks taste, so
please use --strict wisely instead of strictly.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
More information about the Intel-gfx
mailing list