[PATCH v1 1/1] checkpatch: Don't warn on "orhapned" DT schema file

[PATCH v1 1/1] checkpatch: Don't warn on "orhapned" DT schema file

[Andy Shevchenko]

Currently checkpatch warns is the DT schema file is absent in MAINTAINERS.
However the DT schema files are self-contained in this sense and
have embedded information about maintainers of it. This is a requirement.
Hence, avoid checkpatch warning about it.

Requested-by: Krzysztof Kozlowski <krzk@kernel.org>
Tested-by: Akhilesh Patil <akhilesh@ee.iitb.ac.in>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 scripts/checkpatch.pl | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 6729f18e5654..818b49d314ce 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3441,11 +3441,17 @@ sub process {
 		     ($line =~ /\{\s*([\w\/\.\-]*)\s*\=\>\s*([\w\/\.\-]*)\s*\}/ &&
 		      (defined($1) || defined($2))))) {
 			$is_patch = 1;
-			$reported_maintainer_file = 1;
-			WARN("FILE_PATH_CHANGES",
-			     "added, moved or deleted file(s), does MAINTAINERS need updating?\n" . $herecurr);
+			# DT bindings are incorporate maintainer information, no need to report
+			if ($realfile !~ m@^Documentation/devicetree/bindings/@)) {
+				$reported_maintainer_file = 1;
+				WARN("FILE_PATH_CHANGES",
+				     "added, moved or deleted file(s), does MAINTAINERS need updating?\n" . $herecurr);
+			}
 		}
 
+		    ($realfile =~ m@^Documentation/devicetree/bindings/.*\.txt$@)) {
+			if ($realfile =~ m@^include/asm/@) {
+
 # Check for adding new DT bindings not in schema format
 		if (!$in_commit_log &&
 		    ($line =~ /^new file mode\s*\d+\s*$/) &&
-- 
2.50.1

Re: [PATCH v1 1/1] checkpatch: Don't warn on "orhapned" DT schema file

[Andy Shevchenko]

On Mon, Oct 27, 2025 at 10:06:35AM +0100, Andy Shevchenko wrote:
> Currently checkpatch warns is the DT schema file is absent in MAINTAINERS.
> However the DT schema files are self-contained in this sense and
> have embedded information about maintainers of it. This is a requirement.
> Hence, avoid checkpatch warning about it.

...

> +		    ($realfile =~ m@^Documentation/devicetree/bindings/.*\.txt$@)) {
> +			if ($realfile =~ m@^include/asm/@) {

Crap. I forgot leftovers once again, v2 is coming soon...

-- 
With Best Regards,
Andy Shevchenko