Commit 85b0ee18 authored by Joe Perches's avatar Joe Perches Committed by Linus Torvalds

checkpatch: see if modified files are marked obsolete in MAINTAINERS

Use get_maintainer to check the status of individual files.  If
"obsolete", suggest leaving the files alone.

Link: http://lkml.kernel.org/r/7ceaa510dc9d2df05ec4b456baed7bb1415550b3.1471889575.git.joe@perches.comSigned-off-by: default avatarJoe Perches <joe@perches.com>
Cc: SF Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 2d13e6ca
...@@ -704,6 +704,16 @@ sub seed_camelcase_file { ...@@ -704,6 +704,16 @@ sub seed_camelcase_file {
} }
} }
sub is_maintained_obsolete {
my ($filename) = @_;
return 0 if (!(-e "$root/scripts/get_maintainer.pl"));
my $status = `perl $root/scripts/get_maintainer.pl --status --nom --nol --nogit --nogit-fallback $filename 2>&1`;
return $status =~ /obsolete/i;
}
my $camelcase_seeded = 0; my $camelcase_seeded = 0;
sub seed_camelcase_includes { sub seed_camelcase_includes {
return if ($camelcase_seeded); return if ($camelcase_seeded);
...@@ -2289,6 +2299,10 @@ sub process { ...@@ -2289,6 +2299,10 @@ sub process {
} }
if ($found_file) { if ($found_file) {
if (is_maintained_obsolete($realfile)) {
WARN("OBSOLETE",
"$realfile is marked as 'obsolete' in the MAINTAINERS hierarchy. No unnecessary modifications please.\n");
}
if ($realfile =~ m@^(?:drivers/net/|net/|drivers/staging/)@) { if ($realfile =~ m@^(?:drivers/net/|net/|drivers/staging/)@) {
$check = 1; $check = 1;
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment