History

checkpatch: warn on unnecessary __dev<foo> section markings

Get people to stop adding __devinit and __devexit section markings.

Joe Perches committed 13 years ago

Commit 56e77d71 parent dc139311 file · +6 −0Browse files

Showing the changes to scripts/checkpatch.pl (1 of 1 file). Show all

Unified · Split

scripts/checkpatch.pl+6−0
@@ -1931,6 +1931,12 @@ sub process { · 1930 unmodified lines
1931 "use the SSYNC() macro in asm/blackfin.h\n" . $herevet);1931 "use the SSYNC() macro in asm/blackfin.h\n" . $herevet);
1932 }1932 }
1933 1933
1934# check for old HOTPLUG __dev<foo> section markings
1935 if ($line =~ /\b(__dev(init|exit)(data|const|))\b/) {
1936 WARN("HOTPLUG_SECTION",
1937 "Using $1 is unnecessary\n" . $herecurr);
1938 }
1939
1934# Check for potential 'bare' types1940# Check for potential 'bare' types
1935 my ($stat, $cond, $line_nr_next, $remain_next, $off_next,1941 my ($stat, $cond, $line_nr_next, $remain_next, $off_next,
1936 $realline_next);1942 $realline_next);