Mirrors/opensbi
3
0
Fork 1
mirror of https://github.com/riscv-software-src/opensbi.git synced 2026-05-23 14:21:32 +01:00
Code Issues Packages Projects Releases Wiki Activity

lib: sbi: Fix hw a/d updating defaults

Browse Source 
The Svade dt-binding description states that Svadu should only
be enabled at boot time when only Svadu is present in the DT.
Ensure that's the case. Also, when only Svadu is supported,
disable FWFT.PTE_AD_HW_UPDATING, as we need both to support
toggling.

Signed-off-by: Andrew Jones <andrew.jones@oss.qualcomm.com>
Reviewed-by: Radim Krčmář <radim.krcmar@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20260401220845.190680-1-andrew.jones@oss.qualcomm.com
Signed-off-by: Anup Patel <anup@brainfault.org>
This commit is contained in:
Andrew Jones
2026-04-01 17:08:45 -05:00
committed by Anup Patel
parent 8536a351fd
commit 5f13f9e13f
2 changed files with 19 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/sbi/sbi_fwft.c
+9 -1
View File
@@ -160,8 +160,16 @@ static int fwft_get_double_trap(struct fwft_config *conf, unsigned long *value)
static int fwft_adue_supported(struct fwft_config *conf) static int fwft_adue_supported(struct fwft_config *conf)
{ {
/*
* FWFT.PTE_AD_HW_UPDATING is only supported when both Svade and Svadu
* are supported. We need both in order to support toggling and to
* ensure the reset value of zero is valid (it wouldn't be when only
* Svadu is supported).
*/
if (!sbi_hart_has_extension(sbi_scratch_thishart_ptr(), if (!sbi_hart_has_extension(sbi_scratch_thishart_ptr(),
SBI_HART_EXT_SVADU)) SBI_HART_EXT_SVADU) ||
!sbi_hart_has_extension(sbi_scratch_thishart_ptr(),
SBI_HART_EXT_SVADE))
return SBI_ENOTSUPP; return SBI_ENOTSUPP;
return SBI_OK; return SBI_OK;
lib/sbi/sbi_hart.c
+10 -8
View File
@@ -137,6 +137,9 @@ static void mstatus_init(struct sbi_scratch *scratch)
if (sbi_hart_priv_version(scratch) >= SBI_HART_PRIV_VER_1_12) { if (sbi_hart_priv_version(scratch) >= SBI_HART_PRIV_VER_1_12) {
menvcfg_val = csr_read64(CSR_MENVCFG); menvcfg_val = csr_read64(CSR_MENVCFG);
/* Disable HW A/D updating by default */
menvcfg_val &= ~ENVCFG_ADUE;
/* Disable double trap by default */ /* Disable double trap by default */
menvcfg_val &= ~ENVCFG_DTE; menvcfg_val &= ~ENVCFG_DTE;
@@ -158,18 +161,17 @@ static void mstatus_init(struct sbi_scratch *scratch)
#endif #endif
__set_menvcfg_ext(SBI_HART_EXT_SSTC, ENVCFG_STCE) __set_menvcfg_ext(SBI_HART_EXT_SSTC, ENVCFG_STCE)
__set_menvcfg_ext(SBI_HART_EXT_SMCDELEG, ENVCFG_CDE); __set_menvcfg_ext(SBI_HART_EXT_SMCDELEG, ENVCFG_CDE);
__set_menvcfg_ext(SBI_HART_EXT_SVADU, ENVCFG_ADUE);
#undef __set_menvcfg_ext
/* /*
* When both Svade and Svadu are present in DT, the default scheme for managing * Assume only Svadu is supported when it is the only extension
* the PTE A/D bits should use Svade. Check Svadu before Svade extension to ensure * present in the ISA string. Svade is assumed when neither are
* that the ADUE bit is cleared when the Svade support are specified. * present. When both are present we must default to Svade (see
* the zero reset value of FWFT.PTE_AD_HW_UPDATING).
*/ */
if (!sbi_hart_has_extension(scratch, SBI_HART_EXT_SVADE))
__set_menvcfg_ext(SBI_HART_EXT_SVADU, ENVCFG_ADUE);
if (sbi_hart_has_extension(scratch, SBI_HART_EXT_SVADE)) #undef __set_menvcfg_ext
menvcfg_val &= ~ENVCFG_ADUE;
csr_write64(CSR_MENVCFG, menvcfg_val); csr_write64(CSR_MENVCFG, menvcfg_val);