Hi Connie, On 5/14/25 4:54 PM, Cornelia Huck wrote: > 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"?
Yes I agree Eric >
