util/lint/checkpatch.pl: Update lines related to CONST_STRUCT
Update to v5.18-2 version. Signed-off-by: Elyes Haouas <ehaouas@noos.fr> Change-Id: I0fe2ec6a74a4b8c70452fbf05d534a37e1ea2c26 Reviewed-on: https://review.coreboot.org/c/coreboot/+/63582 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin L Roth <martinroth@google.com>
This commit is contained in:
parent
f0e150c35a
commit
43529c8b20
|
@ -867,9 +867,11 @@ sub read_words {
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
my $const_structs = "";
|
my $const_structs;
|
||||||
read_words(\$const_structs, $conststructsfile)
|
if (show_type("CONST_STRUCT")) {
|
||||||
or warn "No structs that should be const will be found - file '$conststructsfile': $!\n";
|
read_words(\$const_structs, $conststructsfile)
|
||||||
|
or warn "No structs that should be const will be found - file '$conststructsfile': $!\n";
|
||||||
|
}
|
||||||
|
|
||||||
my $typeOtherTypedefs = "";
|
my $typeOtherTypedefs = "";
|
||||||
if (length($typedefsfile)) {
|
if (length($typedefsfile)) {
|
||||||
|
@ -6603,7 +6605,8 @@ sub process {
|
||||||
|
|
||||||
# check for various structs that are normally const (ops, kgdb, device_tree)
|
# check for various structs that are normally const (ops, kgdb, device_tree)
|
||||||
# and avoid what seem like struct definitions 'struct foo {'
|
# and avoid what seem like struct definitions 'struct foo {'
|
||||||
if ($line !~ /\bconst\b/ &&
|
if (defined($const_structs) &&
|
||||||
|
$line !~ /\bconst\b/ &&
|
||||||
$line =~ /\bstruct\s+($const_structs)\b(?!\s*\{)/) {
|
$line =~ /\bstruct\s+($const_structs)\b(?!\s*\{)/) {
|
||||||
WARN("CONST_STRUCT",
|
WARN("CONST_STRUCT",
|
||||||
"struct $1 should normally be const\n" . $herecurr);
|
"struct $1 should normally be const\n" . $herecurr);
|
||||||
|
|
Loading…
Reference in New Issue