|
@@ -3348,18 +3348,20 @@ sub process {
|
|
|
if ($line =~ /^.\s*sema_init.+,\W?0\W?\)/) {
|
|
|
WARN("CONSIDER_COMPLETION",
|
|
|
"consider using a completion\n" . $herecurr);
|
|
|
-
|
|
|
}
|
|
|
+
|
|
|
# recommend kstrto* over simple_strto* and strict_strto*
|
|
|
if ($line =~ /\b((simple|strict)_(strto(l|ll|ul|ull)))\s*\(/) {
|
|
|
WARN("CONSIDER_KSTRTO",
|
|
|
"$1 is obsolete, use k$3 instead\n" . $herecurr);
|
|
|
}
|
|
|
+
|
|
|
# check for __initcall(), use device_initcall() explicitly please
|
|
|
if ($line =~ /^.\s*__initcall\s*\(/) {
|
|
|
WARN("USE_DEVICE_INITCALL",
|
|
|
"please use device_initcall() instead of __initcall()\n" . $herecurr);
|
|
|
}
|
|
|
+
|
|
|
# check for various ops structs, ensure they are const.
|
|
|
my $struct_ops = qr{acpi_dock_ops|
|
|
|
address_space_operations|
|