On Mon, 15 Dec 2014, Daniel Vetter <daniel@xxxxxxxx> wrote: > 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. Perhaps we should use the --subjective option instead. Not that it would actually change anything, it's exactly the same as --strict. We'd still require the same things for i915, but we'd be openly subjective about it. ;) BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
