[PATCH 2/6] checkpatch.pl: Add a U-Boot option

Simon Glass sjg at chromium.org
Sat May 23 00:32:36 CEST 2020


Add an option to indicate that U-Boot-specific checks should be enabled.
Add a function to house the code that will be added.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

 .checkpatch.conf      |  3 +++
 scripts/checkpatch.pl | 12 ++++++++++++
 2 files changed, 15 insertions(+)

diff --git a/.checkpatch.conf b/.checkpatch.conf
index 95f19635d35..ed0c2150ba8 100644
--- a/.checkpatch.conf
+++ b/.checkpatch.conf
@@ -28,3 +28,6 @@
 
 # A bit shorter of a description is OK with us.
 --min-conf-desc-length=2
+
+# Extra checks for U-Boot
+--u-boot
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index cec09fbade8..93863c8f086 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -60,6 +60,7 @@ my $codespell = 0;
 my $codespellfile = "/usr/share/codespell/dictionary.txt";
 my $conststructsfile = "$D/const_structs.checkpatch";
 my $typedefsfile = "";
+my $u_boot = 0;
 my $color = "auto";
 my $allow_c99_comments = 1;
 
@@ -121,6 +122,7 @@ Options:
   --typedefsfile             Read additional types from this file
   --color[=WHEN]             Use colors 'always', 'never', or only when output
                              is a terminal ('auto'). Default is 'auto'.
+  --u-boot                   Run additional checks for U-Boot
   -h, --help, --version      display this help and exit
 
 When FILE is - read standard input.
@@ -225,6 +227,7 @@ GetOptions(
 	'codespell!'	=> \$codespell,
 	'codespellfile=s'	=> \$codespellfile,
 	'typedefsfile=s'	=> \$typedefsfile,
+	'u-boot'	=> \$u_boot,
 	'color=s'	=> \$color,
 	'no-color'	=> \$color,	#keep old behaviors of -nocolor
 	'nocolor'	=> \$color,	#keep old behaviors of -nocolor
@@ -2235,6 +2238,11 @@ sub pos_last_openparen {
 	return length(expand_tabs(substr($line, 0, $last_openparen))) + 1;
 }
 
+# Checks specific to U-Boot
+sub u_boot_line {
+	my ($realfile, $line,  $herecurr) = @_;
+}
+
 sub process {
 	my $filename = shift;
 
@@ -3102,6 +3110,10 @@ sub process {
 			     "adding a line without newline at end of file\n" . $herecurr);
 		}
 
+		if ($u_boot) {
+			u_boot_line($realfile, $line,  $herecurr);
+		}
+
 # check we are in a valid source file C or perl if not then ignore this hunk
 		next if ($realfile !~ /\.(h|c|pl|dtsi|dts)$/);
 
-- 
2.27.0.rc0.183.gde8f92d652-goog



More information about the U-Boot mailing list