On Fri, 29 Aug 2025 10:51:40 +0100, Fuad Tabba wrote:

The sysreg definitions file has a few errors. In Enum Security,
NSACR_RFR is set to 0b0001 instead of the spec value of 0b0010.
Additionaly, the sign of the Enums DoubleLock and EIESB are wrong
(should be Signed and Unsigned, respectively).

Moreover, the file has some redundant definitions, e.g.,
RCWSMASK_EL1. While these redundant definitions are not wrong per se,
they add unnecessary code into the generated header file and could be a
source of future bugs.

[...]

Applied to arm64 (for-next/sysregs), thanks!

[1/3] arm64: sysreg: Fix and tidy up sysreg field definitions
      https://git.kernel.org/arm64/c/2c2529e47062
[2/3] arm64: sysreg: Correct sign definitions for EIESB and DoubleLock
      https://git.kernel.org/arm64/c/f4d4ebc84995
[3/3] arm64: sysreg: Add validation checks to sysreg header generation script
      https://git.kernel.org/arm64/c/382cbbe7fb2a

Cheers,
-- 
Will

https://fixes.arm64.dev
https://next.arm64.dev
https://will.arm64.dev