On Tue, May 13 2025, Daniel P. Berrangé <berra...@redhat.com> wrote:
> On Tue, May 06, 2025 at 10:52:32AM +0200, Cornelia Huck wrote: >> From: Eric Auger <eric.au...@redhat.com> >> >> Introduce scripts that automate the generation of system register >> definitions from a given linux source tree arch/arm64/tools/sysreg. >> >> Invocation of >> ./update-aarch64-sysreg-code.sh $PATH_TO_LINUX_SOURCE_TREE >> in scripts directory generates target/arm/cpu-sysregs.h.inc >> containing defines for all system registers. >> >> [CH: update to handle current kernel sysregs structure, and to emit >> the re-worked register structures; cpu properties will be added >> later] >> Reviewed-by: Sebastian Ott <seb...@redhat.com> >> Signed-off-by: Eric Auger <eric.au...@redhat.com> >> Signed-off-by: Cornelia Huck <coh...@redhat.com> >> --- >> scripts/gen-cpu-sysregs-header.awk | 35 +++++++++++++++++++++++++++ >> scripts/update-aarch64-sysreg-code.sh | 25 +++++++++++++++++++ >> 2 files changed, 60 insertions(+) >> create mode 100755 scripts/gen-cpu-sysregs-header.awk >> create mode 100755 scripts/update-aarch64-sysreg-code.sh >> >> diff --git a/scripts/gen-cpu-sysregs-header.awk >> b/scripts/gen-cpu-sysregs-header.awk >> new file mode 100755 >> index 000000000000..b6b207e3c0fd >> --- /dev/null >> +++ b/scripts/gen-cpu-sysregs-header.awk >> @@ -0,0 +1,35 @@ >> +#!/bin/awk -f >> +# SPDX-License-Identifier: GPL-2.0 > > Invalid license name here - hopefully checkpatch.pl is warning > about this. It might not check .awk scripts? > > Same issue as in the other sysreg script in the host CPU model > series. > > >> diff --git a/scripts/update-aarch64-sysreg-code.sh >> b/scripts/update-aarch64-sysreg-code.sh >> new file mode 100755 >> index 000000000000..721f41a9a516 >> --- /dev/null >> +++ b/scripts/update-aarch64-sysreg-code.sh >> @@ -0,0 +1,25 @@ >> +#!/bin/sh -e >> +# >> +# Update target/arm/cpu-sysregs.h >> +# from a linux source tree (arch/arm64/tools/sysreg) >> +# >> +# Copyright Red Hat, Inc. 2024 >> +# >> +# Authors: >> +# Eric Auger <eric.au...@redhat.com> > > Although its fairly short, this still deserves SPDX-License-Identifier > to be set. @Eric: do you agree on "GPL-2.0-or-later"?
