aboutsummaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorJoe Perches <joe@perches.com>2016-10-11 16:51:44 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2016-10-11 18:06:30 -0400
commit85b0ee18bbf82cb3e1880c718749149c6ed61058 (patch)
tree4b889c89cfa46e1ea1d145601b5e677cc40c1b17 /scripts
parent2d13e6ca429c0a6fbc82750acbece829facceec5 (diff)
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.com Signed-off-by: Joe Perches <joe@perches.com> Cc: SF Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/checkpatch.pl14
1 files changed, 14 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 206a6b346a8d..34b445574011 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -704,6 +704,16 @@ sub seed_camelcase_file {
704 } 704 }
705} 705}
706 706
707sub is_maintained_obsolete {
708 my ($filename) = @_;
709
710 return 0 if (!(-e "$root/scripts/get_maintainer.pl"));
711
712 my $status = `perl $root/scripts/get_maintainer.pl --status --nom --nol --nogit --nogit-fallback $filename 2>&1`;
713
714 return $status =~ /obsolete/i;
715}
716
707my $camelcase_seeded = 0; 717my $camelcase_seeded = 0;
708sub seed_camelcase_includes { 718sub seed_camelcase_includes {
709 return if ($camelcase_seeded); 719 return if ($camelcase_seeded);
@@ -2289,6 +2299,10 @@ sub process {
2289 } 2299 }
2290 2300
2291 if ($found_file) { 2301 if ($found_file) {
2302 if (is_maintained_obsolete($realfile)) {
2303 WARN("OBSOLETE",
2304 "$realfile is marked as 'obsolete' in the MAINTAINERS hierarchy. No unnecessary modifications please.\n");
2305 }
2292 if ($realfile =~ m@^(?:drivers/net/|net/|drivers/staging/)@) { 2306 if ($realfile =~ m@^(?:drivers/net/|net/|drivers/staging/)@) {
2293 $check = 1; 2307 $check = 1;
2294 } else { 2308 } else {