[PATCH] checkpatch: Don't emit warnings for USB & PCI device DT compatible prefixes

[PATCH] checkpatch: Don't emit warnings for USB & PCI device DT compatible prefixes

[Chen-Yu Tsai]

The USB and PCI device bindings define some compatible patterns based
on device IDs that use the comma to separate vendor and product IDs.

These patterns include:

  - usb[0-9a-f]{1,4},[0-9a-f]{1,4}
  - pci[0-9a-f]{2,4},[0-9a-f]{1,4}
  - pciclass,[01][0-9a-f]{3}([0-9a-f]{2})?

These are not real vendor prefixes. Don't emit warnings for them.

Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
---
This is a simplified version of what Brian Norris previously posted [1],
but more comprehensive and more perl-y than what Yingying Tang posted
[2], which only covered the second pattern.

Hopefully everyone likes this version.

This is based on next-20260508.

[1] https://lore.kernel.org/all/20190223022440.146915-1-briannorris@chromium.org/
[2] https://lore.kernel.org/all/20251210073812.1380803-1-yingying.tang@oss.qualcomm.com/

 scripts/checkpatch.pl | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 3727156e4cca..ccedf152acfd 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3795,6 +3795,9 @@ sub process {
 
 				next if $compat !~ /^([a-zA-Z0-9\-]+)\,/;
 				my $vendor = $1;
+				next if $vendor eq "pciclass";
+				next if $vendor =~ /^pci[a-f0-9]{2,4}$/;
+				next if $vendor =~ /^usb(if)?[a-f0-9]{1,4}$/;
 				`grep -Eq "\\"\\^\Q$vendor\E,\\.\\*\\":" $vp_file`;
 				if ( $? >> 8 ) {
 					WARN("UNDOCUMENTED_DT_STRING",
-- 
2.54.0.563.g4f69b47b94-goog