Commit d1d84b5f authored by Miguel Ojeda's avatar Miguel Ojeda
Browse files

scripts: checkpatch: enable language-independent checks for Rust 



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: default avatarKees Cook <keescook@chromium.org>
Reviewed-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Co-developed-by: default avatarAlex Gaynor <alex.gaynor@gmail.com>
Signed-off-by: default avatarAlex Gaynor <alex.gaynor@gmail.com>
Co-developed-by: default avatarWedson Almeida Filho <wedsonaf@google.com>
Signed-off-by: default avatarWedson Almeida Filho <wedsonaf@google.com>
Signed-off-by: default avatarMiguel Ojeda <ojeda@kernel.org>
parent de48fa1a

scripts/checkpatch.pl

+2 −2
Original line number Diff line number Diff line
@@ -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 &&