Compare commits
3 Commits
9238b877ef
...
master
Author | SHA1 | Date | |
---|---|---|---|
aeb2707d80 | |||
cb937262aa | |||
3a58e3dd1c |
@@ -3021,35 +3021,6 @@ sub process {
|
||||
}
|
||||
}
|
||||
|
||||
# check for using SPDX license tag at beginning of files
|
||||
if ($realline == $checklicenseline) {
|
||||
if ($rawline =~ /^[ \+]\s*\#\!\s*\//) {
|
||||
$checklicenseline = 2;
|
||||
} elsif ($rawline =~ /^\+/) {
|
||||
my $comment = "";
|
||||
if ($realfile =~ /\.(h|s|S)$/) {
|
||||
$comment = '/*';
|
||||
} elsif ($realfile =~ /\.(c|dts|dtsi)$/) {
|
||||
$comment = '//';
|
||||
} elsif (($checklicenseline == 2) || $realfile =~ /\.(sh|pl|py|awk|tc)$/) {
|
||||
$comment = '#';
|
||||
} elsif ($realfile =~ /\.rst$/) {
|
||||
$comment = '..';
|
||||
}
|
||||
|
||||
if ($comment !~ /^$/ &&
|
||||
$rawline !~ /^\+\Q$comment\E SPDX-License-Identifier: /) {
|
||||
WARN("SPDX_LICENSE_TAG",
|
||||
"Missing or malformed SPDX-License-Identifier tag in line $checklicenseline\n" . $herecurr);
|
||||
} elsif ($rawline =~ /(SPDX-License-Identifier: .*)/) {
|
||||
my $spdx_license = $1;
|
||||
if (!is_SPDX_License_valid($spdx_license)) {
|
||||
WARN("SPDX_LICENSE_TAG",
|
||||
"'$spdx_license' is not supported in LICENSES/...\n" . $herecurr);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
# check we are in a valid source file if not then ignore this hunk
|
||||
next if ($realfile !~ /\.(h|c|s|S|sh|dtsi|dts)$/);
|
||||
@@ -3849,15 +3820,6 @@ sub process {
|
||||
$fixed[$fixlinenr] =~ s/(^.$Type\s*$Ident(?:\s+$Modifier)*)\s*=\s*$zero_initializer\s*;/$1;/;
|
||||
}
|
||||
}
|
||||
# check for static initialisers.
|
||||
if ($line =~ /^\+.*\bstatic\s.*=\s*($zero_initializer)\s*;/) {
|
||||
if (ERROR("INITIALISED_STATIC",
|
||||
"do not initialise statics to $1\n" .
|
||||
$herecurr) &&
|
||||
$fix) {
|
||||
$fixed[$fixlinenr] =~ s/(\bstatic\s.*?)\s*=\s*$zero_initializer\s*;/$1;/;
|
||||
}
|
||||
}
|
||||
|
||||
# check for misordered declarations of char/short/int/long with signed/unsigned
|
||||
while ($sline =~ m{(\b$TypeMisordered\b)}g) {
|
||||
@@ -5367,10 +5329,10 @@ sub process {
|
||||
|
||||
# no volatiles please
|
||||
my $asm_volatile = qr{\b(__asm__|asm)\s+(__volatile__|volatile)\b};
|
||||
if ($line =~ /\bvolatile\b/ && $line !~ /$asm_volatile/) {
|
||||
WARN("VOLATILE",
|
||||
"Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst\n" . $herecurr);
|
||||
}
|
||||
#if ($line =~ /\bvolatile\b/ && $line !~ /$asm_volatile/) {
|
||||
# WARN("VOLATILE",
|
||||
# "Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst\n" . $herecurr);
|
||||
#}
|
||||
|
||||
# Check for user-visible strings broken across lines, which breaks the ability
|
||||
# to grep for the string. Make exceptions when the previous string ends in a
|
||||
|
1
const_structs.checkpatch
Normal file
1
const_structs.checkpatch
Normal file
@@ -0,0 +1 @@
|
||||
__unused__dummy_struct__
|
1344
spelling.txt
Normal file
1344
spelling.txt
Normal file
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user