diff options
| author | Bill Fischofer <bill.fischofer@linaro.org> | 2017-05-04 14:33:14 -0500 |
|---|---|---|
| committer | Maxim Uvarov <maxim.uvarov@linaro.org> | 2017-05-22 21:52:04 +0300 |
| commit | f4386378e466a519d8f97923ba43ea22dec1e933 (patch) | |
| tree | 99c56e59f6c6bf5aedafaa821bf0cd609339685a /scripts | |
| parent | 3ca26420f0e06fdb1b2cb9192461dbc32a421de5 (diff) | |
scripts: checkpatch: update to allow additional exceptions
Update checkpatch.pl to avoid issuing warnings for use of externs,
volatile, or camelCase.
Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org>
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/checkpatch.pl | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 16316b928..1c27ac60c 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -4273,7 +4273,7 @@ sub process { $camelcase_file_seeded = 1; } } - if (!defined $camelcase{$word}) { + if (!defined $camelcase{$word} && 0) { $camelcase{$word} = 1; CHK("CAMELCASE", "Avoid CamelCase: <$word>\n" . $herecurr); @@ -4620,7 +4620,7 @@ sub process { # no volatiles please my $asm_volatile = qr{\b(__asm__|asm)\s+(__volatile__|volatile)\b}; - if ($line =~ /\bvolatile\b/ && $line !~ /$asm_volatile/) { + if ($line =~ /\bvolatile\b/ && 0 && $line !~ /$asm_volatile/) { WARN("VOLATILE", "Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt\n" . $herecurr); } @@ -5134,7 +5134,7 @@ sub process { if (defined $cond) { substr($s, 0, length($cond), ''); } - if ($s =~ /^\s*;/ && + if ($s =~ /^\s*;/ && 0 && $function_name ne 'uninitialized_var') { WARN("AVOID_EXTERNS", |