From patchwork Sun Jan 27 11:35:39 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: checkpatch.pl: Fix warnings on code comments Date: Sun, 27 Jan 2013 01:35:39 -0000 From: Jeff Kirsher X-Patchwork-Id: 215986 Message-Id: <1359286539-18395-1-git-send-email-jeffrey.t.kirsher@intel.com> To: apw@canonical.com Cc: Jeff Kirsher , linux-kernel@vger.kernel.org, davem@davemloft.net, netdev@vger.kernel.org The following commit: commit 058806007450489bb8f457b275e5cb5c946320c1 Author: Joe Perches Date: Thu Oct 4 17:13:35 2012 -0700 checkpatch: check networking specific block comment style Produces warnings on code comments which follow the Linux coding style guide. While the desired code comment style for networking my differ from the rest of the kernel, both styles should be permitted. This patch reverts a portion of the commit to allow multi-line code comments to use either style. Signed-off-by: Jeff Kirsher Tested-by: Jeff Pieper --- scripts/checkpatch.pl | 7 ------- 1 file changed, 7 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 4d2c7df..d3ffec5 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -1878,13 +1878,6 @@ sub process { } if ($realfile =~ m@^(drivers/net/|net/)@ && - $rawline =~ /^\+[ \t]*\/\*[ \t]*$/ && - $prevrawline =~ /^\+[ \t]*$/) { - WARN("NETWORKING_BLOCK_COMMENT_STYLE", - "networking block comments don't use an empty /* line, use /* Comment...\n" . $hereprev); - } - - if ($realfile =~ m@^(drivers/net/|net/)@ && $rawline !~ m@^\+[ \t]*\*/[ \t]*$@ && #trailing */ $rawline !~ m@^\+.*/\*.*\*/[ \t]*$@ && #inline /*...*/ $rawline !~ m@^\+.*\*{2,}/[ \t]*$@ && #trailing **/