Hi Connie, On 6/17/25 5:45 PM, Cornelia Huck wrote: > On Tue, Jun 17 2025, Cornelia Huck <coh...@redhat.com> wrote: > >> diff --git a/scripts/arm-gen-cpu-sysregs-header.awk >> b/scripts/arm-gen-cpu-sysregs-header.awk >> new file mode 100755 >> index 000000000000..f92bbbafa727 >> --- /dev/null >> +++ b/scripts/arm-gen-cpu-sysregs-header.awk >> @@ -0,0 +1,37 @@ >> +#!/bin/awk -f >> +# SPDX-License-Identifier: GPL-2.0-or-later >> +# arm-gen-cpu-sysregs-header.awk: arm64 sysreg header include generator >> +# >> +# Usage: awk -f arm-gen-cpu-sysregs-header.awk >> $LINUX_PATH/arch/arm64/tools/sysreg >> + >> +BEGIN { >> + print "/* SPDX-License-Identifier: GPL-2.0-or-later */" > FWIW, checkpatch.pl chokes on this: > > ^* matches null string many times in regex; marked by <-- HERE in m/^* <-- > HERE /"$/ at scripts/checkpatch.pl line 1389. > ERROR: Saw unacceptable licenses '*/"', valid choices for QEMU are: > GPL-2.0-or-later > GPL-2.0-only > LGPL-2.1-only > LGPL-2.1-or-later > BSD-2-Clause > BSD-3-Clause > MIT > total: 1 errors, 0 warnings, 76 lines checked
so don't we need to improve checkpatch.pl to silence this error? Cheers Eric > > >> + print "/* GENERATED FILE, DO NOT EDIT */" >> + print "/* use arm-gen-cpu-sysregs-header.awk to regenerate */" >> +} END { >> + print "" >> +} >> +