Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63439 )
Change subject: util/lint/checkpatch.pl: Update lines related to max_line_length ......................................................................
util/lint/checkpatch.pl: Update lines related to max_line_length
Signed-off-by: Elyes Haouas ehaouas@noos.fr Change-Id: Ib9927bfa98e20d4b621bf7abecec234b4754ee9c --- M util/lint/checkpatch.pl 1 file changed, 8 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/63439/1
diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl index 6a9c345..4151e82 100755 --- a/util/lint/checkpatch.pl +++ b/util/lint/checkpatch.pl @@ -56,7 +56,7 @@ my @exclude = (); #coreboot my $help = 0; my $configuration_file = ".checkpatch.conf"; -my $max_line_length = 80; +my $max_line_length = 100; my $ignore_perl_version = 0; my $minimum_perl_version = 5.10.0; my $min_conf_desc_length = 4; @@ -111,7 +111,9 @@ --ignore TYPE(,TYPE2...) ignore various comma separated message types --exclude DIR(,DIR22...) exclude directories --show-types show the specific message type in the output - --max-line-length=n set the maximum line length, if exceeded, warn + --max-line-length=n set the maximum line length, (default $max_line_length) + if exceeded, warn on patches + requires --strict for use with --file --min-conf-desc-length=n set the min description length, if shorter, warn --tab-size=n set the number of spaces for tab (default $tabsize) --root=PATH PATH to the kernel tree root @@ -3196,8 +3198,10 @@
if ($msg_type ne "" && (show_type("LONG_LINE") || show_type($msg_type))) { - WARN($msg_type, - "line over $max_line_length characters\n" . $herecurr); + my $msg_level = &WARN; + $msg_level = &CHK if ($file); + &{$msg_level}($msg_type, + "line length of $length exceeds $max_line_length columns\n" . $herecurr); } }