From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6337F3B47E5 for ; Wed, 13 May 2026 19:18:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=216.40.44.16 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778699909; cv=none; b=fxPqAgNYk9kl3u3tbRtucDQR3KCV6Hxd6EEYj2oFopxoUOF6U+bQu3l/tSdFBSXrBBQBpv6mcF4jYBLX1hcHQK2Y3f07935LQPWVfowHNuDbHu8I+vcB5m/Uur4e0fKd9yCT+qza8+p6flx9nhEJvxEgIOUfzzRhzfALcTM30Ys= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778699909; c=relaxed/simple; bh=5N0vQuoYenMyUaYGb9Jk32Xg1uaeCabCLZuiARm6oCA=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=LkTF8GRWRQyBE7fwgqhhtCkVWsU9d+TQHj21lL3yc5s2rqEEMEGJ9EzW+24uwC9Z4mAEuXLm1JFUpIXgR08smPakPIFRFpjrZnqQkEbpKrUWQ4Dz4YSbX03qPFhoslG6aQ5W+L4XIbitLEW6sy2whrPtuKKqHOjcm/Hp10Wa6VI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=perches.com; spf=pass smtp.mailfrom=perches.com; arc=none smtp.client-ip=216.40.44.16 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=perches.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=perches.com Received: from omf15.hostedemail.com (lb01a-stub [10.200.18.249]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 1E0FA160590; Wed, 13 May 2026 19:18:27 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: joe@perches.com) by omf15.hostedemail.com (Postfix) with ESMTPA id D50BE1C; Wed, 13 May 2026 19:18:24 +0000 (UTC) Message-ID: Subject: Re: [PATCH] checkpatch: add .dtso to DT compatible string vendor check From: Joe Perches To: Vitor Soares , Andy Whitcroft Cc: Vitor Soares , linux-kernel@vger.kernel.org, Dwaipayan Ray , Lukas Bulwahn Date: Wed, 13 May 2026 12:18:23 -0700 In-Reply-To: <20260513171514.889168-2-ivitro@gmail.com> References: <20260513171514.889168-2-ivitro@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.58.3 (3.58.3-1.fc43) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Rspamd-Server: rspamout05 X-Rspamd-Queue-Id: D50BE1C X-Stat-Signature: c7jnyhgy5d9644ewu4dax8u3a5d4sd9e X-Session-Marker: 6A6F6540706572636865732E636F6D X-Session-ID: U2FsdGVkX19Ov9RvuWl0kTNqkmKr01lTp32yepXBWZs= X-HE-Tag: 1778699904-620189 X-HE-Meta: U2FsdGVkX1/6aBUHgd0gCQlIeaQSI1O1Z+jg3rsn4PeL8vBsnVRL9XILll/wyyRMPKGnZY9YWU6x2rsgGAJSPbaK/piQSMMosJu+9LrRdqrfKO+5aflgz+y4ad7weGcrnspzNvo50KwuWKGxadfE5DGDydZHoIklNRNtAL+7hPQypLvoEKBdoIp3nRjViQG6752WJ5hKGI4h3MozAlnVA2LJnEJeL4J8PVSH5zONyLyXbJsrfge6yNbVlOEHXU02AcOGQe3DmfetMVMb+Q1NvVfhKD8ZnZB6v0Fiuw/MFSjiGGarZ8B+eUt+0Lt9nmXUDA9o/5JOmhS8ahBRzv42Bb8+CE87St5N2zZF6rML/L89mqokO41Ozvsx8AEpJt9L 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. >=20 > Assisted-by: Claude:claude-sonnet-4.6 > Signed-off-by: Vitor Soares <[vitor.soares@toradex.com](mailto:vitor.soar= es@toradex.com)> [] > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl [] > @@ -3774,7 +3774,7 @@ sub process { > =20 > # check for DT compatible documentation > if (defined $root && > - (($realfile =3D~ /\.dtsi?$/ && $line =3D~ /^\+\s*compatible\s*=3D\s*\= "/) || > + (($realfile =3D~ /\.(dts|dtsi|dtso)$/ && $line =3D~ /^\+\s*compatible= \s*=3D\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 =3D~ /\.dtsi?$/= && $line =3D~ /^\+\s*compatible\s*=3D\s*\"/) || scripts/checkpatch.pl:3807: if ($realfile =3D~ /\.(dts|dtsi|dts= o)$/ && scripts/checkpatch.pl:3824: if ($realfile =3D~ = /\.(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|dt= si|dts)$/);