On Mon, 2017-12-18 at 07:30 -0800, Joe Perches wrote: > On Mon, 2017-12-18 at 14:05 +0100, Knut Omang wrote: > > > Here is a list of the checkpatch messages for drivers/infiniband > > > sorted by type. > > > > > > Many of these might be corrected by using > > > > > > $ ./scripts/checkpatch.pl -f --fix-inplace --types=<TYPE> \ > > > $(git ls-files drivers/infiniband/) > > > > Yes, and I already did that work piece by piece for individual types, > > just to test the runchecks tool, and want to post that set once the > > runchecks script and Makefile changes itself are in, > > I think those are independent of any runcheck tool changes and > could be posted now. In general, don't keep patches in a local > tree waiting on some other unrelated patch. It becomes related in that the runchecks.cfg file is updated in all the patches to keep 'make C=2' run with 0 errors while enabling more checks. I think they serve well as examples of how a workflow with runchecks could be. > Just fyi: > > There is a script that helps automate checkpatch "by-type" conversions > with compilation, .o difference checking, and git commit editing. > > https://lkml.org/lkml/2014/7/11/794 oh - good to know - seems it would have been a good help during my little exercise.. Thanks, Knut -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html