scripts: checkpatch: enable language-independent checks for Rust
authorMiguel Ojeda <ojeda@kernel.org>
Sun, 22 May 2022 15:22:58 +0000 (17:22 +0200)
committerMiguel Ojeda <ojeda@kernel.org>
Wed, 28 Sep 2022 07:01:15 +0000 (09:01 +0200)
Include Rust in the "source code files" category, so that
the language-independent tests are checked for Rust too,
and teach `checkpatch` about the comment style for Rust files.

This enables the malformed SPDX check, the misplaced SPDX license
tag check, the long line checks, the lines without a newline check
and the embedded filename check.

Reviewed-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Co-developed-by: Alex Gaynor <alex.gaynor@gmail.com>
Signed-off-by: Alex Gaynor <alex.gaynor@gmail.com>
Co-developed-by: Wedson Almeida Filho <wedsonaf@google.com>
Signed-off-by: Wedson Almeida Filho <wedsonaf@google.com>
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
scripts/checkpatch.pl

index 74a7693..b5ed31d 100755 (executable)
@@ -3616,7 +3616,7 @@ sub process {
                                my $comment = "";
                                if ($realfile =~ /\.(h|s|S)$/) {
                                        $comment = '/*';
-                               } elsif ($realfile =~ /\.(c|dts|dtsi)$/) {
+                               } elsif ($realfile =~ /\.(c|rs|dts|dtsi)$/) {
                                        $comment = '//';
                                } elsif (($checklicenseline == 2) || $realfile =~ /\.(sh|pl|py|awk|tc|yaml)$/) {
                                        $comment = '#';
@@ -3664,7 +3664,7 @@ sub process {
                }
 
 # check we are in a valid source file if not then ignore this hunk
-               next if ($realfile !~ /\.(h|c|s|S|sh|dtsi|dts)$/);
+               next if ($realfile !~ /\.(h|c|rs|s|S|sh|dtsi|dts)$/);
 
 # check for using SPDX-License-Identifier on the wrong line number
                if ($realline != $checklicenseline &&