arm64/cache: Fix cache_type_cwg() for register generation
authorMark Brown <broonie@kernel.org>
Thu, 18 Aug 2022 21:36:13 +0000 (22:36 +0100)
committerWill Deacon <will@kernel.org>
Tue, 23 Aug 2022 10:26:01 +0000 (11:26 +0100)
commit53d2d84a1f6d68e036adab71df8e0f37cd2724f6
treeb05ac940956fbe1673ce34f5c55bc75d9ff67e39
parenta10edea4efbba4e8756f493781e953dd3592f24a
arm64/cache: Fix cache_type_cwg() for register generation

Ard noticed that since we converted CTR_EL0 to automatic generation we have
been seeing errors on some systems handling the value of cache_type_cwg()
such as

   CPU features: No Cache Writeback Granule information, assuming 128

This is because the manual definition of CTR_EL0_CWG_MASK was done without
a shift while our convention is to define the mask after shifting. This
means that the user in cache_type_cwg() was broken as it was written for
the manually written shift then mask. Fix this by converting to use
SYS_FIELD_GET().

The only other field where the _MASK for this register is used is IminLine
which is at offset 0 so unaffected.

Fixes: 9a3634d02301 ("arm64/sysreg: Convert CTR_EL0 to automatic generation")
Reported-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20220818213613.733091-4-broonie@kernel.org
Signed-off-by: Will Deacon <will@kernel.org>
arch/arm64/include/asm/cache.h