The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [PATCH] checkpatch: add .dtso to DT compatible string vendor check
@ 2026-05-13 17:15 Vitor Soares
  2026-05-13 19:18 ` Joe Perches
  0 siblings, 1 reply; 2+ messages in thread
From: Vitor Soares @ 2026-05-13 17:15 UTC (permalink / raw)
  To: Andy Whitcroft, Joe Perches
  Cc: Vitor Soares, linux-kernel, Dwaipayan Ray, Lukas Bulwahn

From: Vitor Soares <vitor.soares@toradex.com>

Extend the DT compatible documentation check to match .dtso files so
that vendor-prefixed compatibles in overlay files trigger the
UNDOCUMENTED_DT_STRING warning when the vendor is absent from
vendor-prefixes.yaml.

Assisted-by: Claude:claude-sonnet-4.6
Signed-off-by: Vitor Soares <vitor.soares@toradex.com>
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 3727156e4cca..fac38fd84856 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3774,7 +3774,7 @@ sub process {
 
 # check for DT compatible documentation
 		if (defined $root &&
-			(($realfile =~ /\.dtsi?$/ && $line =~ /^\+\s*compatible\s*=\s*\"/) ||
+			(($realfile =~ /\.(dts|dtsi|dtso)$/ && $line =~ /^\+\s*compatible\s*=\s*\"/) ||
 			 ($realfile =~ /\.[ch]$/ && $line =~ /^\+.*\.compatible\s*=\s*\"/))) {
 
 			my @compats = $rawline =~ /\"([a-zA-Z0-9\-\,\.\+_]+)\"/g;
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] checkpatch: add .dtso to DT compatible string vendor check
  2026-05-13 17:15 [PATCH] checkpatch: add .dtso to DT compatible string vendor check Vitor Soares
@ 2026-05-13 19:18 ` Joe Perches
  0 siblings, 0 replies; 2+ messages in thread
From: Joe Perches @ 2026-05-13 19:18 UTC (permalink / raw)
  To: Vitor Soares, Andy Whitcroft
  Cc: Vitor Soares, linux-kernel, Dwaipayan Ray, Lukas Bulwahn

On Wed, 2026-05-13 at 18:15 +0100, Vitor Soares wrote:
> Extend the DT compatible documentation check to match .dtso files so
> that vendor-prefixed compatibles in overlay files trigger the
> UNDOCUMENTED_DT_STRING warning when the vendor is absent from
> vendor-prefixes.yaml.
> 
> Assisted-by: Claude:claude-sonnet-4.6
> Signed-off-by: Vitor Soares <[vitor.soares@toradex.com](mailto:vitor.soares@toradex.com)>
[]
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
[]
> @@ -3774,7 +3774,7 @@ sub process {
>  
>  # check for DT compatible documentation
>  		if (defined $root &&
> -			(($realfile =~ /\.dtsi?$/ && $line =~ /^\+\s*compatible\s*=\s*\"/) ||
> +			(($realfile =~ /\.(dts|dtsi|dtso)$/ && $line =~ /^\+\s*compatible\s*=\s*\"/) ||

Use (?:...) to avoid capture groups.

And presumably all checks for dts.? files and locations should be modified

$ git grep -n dts scripts/checkpatch.pl
scripts/checkpatch.pl:3777:                     (($realfile =~ /\.dtsi?$/ && $line =~ /^\+\s*compatible\s*=\s*\"/) ||
scripts/checkpatch.pl:3807:             if ($realfile =~ /\.(dts|dtsi|dtso)$/ &&
scripts/checkpatch.pl:3824:                             if ($realfile =~ /\.(c|rs|dts|dtsi)$/) {
scripts/checkpatch.pl:3886:             next if ($realfile !~ /\.(h|c|rs|s|S|sh|dtsi|dts)$/);
scripts/checkpatch.pl:3985:             next if ($realfile !~ /\.(h|c|pl|dtsi|dts)$/);

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-05-13 19:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-13 17:15 [PATCH] checkpatch: add .dtso to DT compatible string vendor check Vitor Soares
2026-05-13 19:18 ` Joe Perches

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox