diff options
author | Joe Perches <joe@perches.com> | 2016-10-11 16:51:44 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-10-11 18:06:30 -0400 |
commit | 85b0ee18bbf82cb3e1880c718749149c6ed61058 (patch) | |
tree | 4b889c89cfa46e1ea1d145601b5e677cc40c1b17 /scripts | |
parent | 2d13e6ca429c0a6fbc82750acbece829facceec5 (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-x | scripts/checkpatch.pl | 14 |
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 | ||
707 | sub 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 | |||
707 | my $camelcase_seeded = 0; | 717 | my $camelcase_seeded = 0; |
708 | sub seed_camelcase_includes { | 718 | sub 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 { |