Re: [PATCH v4 2/2] checkpatch: Add option to not force /* */ for SPDX
From: Simon Glass
Date: Thu Apr 16 2026 - 14:47:25 EST
On Thu, 16 Apr 2026 at 02:36, Petr Vorel <pvorel@xxxxxxx> wrote:
>
> Add option --spdx-cxx-comments to not force C comments (/* */) for SPDX.
>
> As documented in aa19a176df95d6, this is required for some old
> toolchains still have older assembler tools which cannot handle C++
> style comments. This allows avoid forcing this for for projects which
> vendored checkpatch.pl (e.g. LTP or u-boot).
>
> Signed-off-by: Petr Vorel <pvorel@xxxxxxx>
> ---
> Changes in v4 (all by Simon):
> * Rename option to more specific (--ignore-old-toolchain =>
> --spdx-cxx-comments)
> * Fix the help text (C++ => C comments)
>
> Link to v3:
> https://lore.kernel.org/lkml/20260408120603.54351-2-pvorel@xxxxxxx/
>
> scripts/checkpatch.pl | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
Reviewed-by: Simon Glass <sjg@xxxxxxxxxxxx>