From d335a178d8cfdf2ed13af70fe385f7e38162e171 Mon Sep 17 00:00:00 2001 From: Xiang W Date: Mon, 8 Nov 2021 17:43:03 +0800 Subject: [PATCH] lib: sbi: clear pmpcfg.A before setting in pmp_set() We should clear A bits in prot variable before enabling A_NA4 or A_NAPOT. Signed-off-by: Xiang W Reviewed-by: Anup Patel --- lib/sbi/riscv_asm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/sbi/riscv_asm.c b/lib/sbi/riscv_asm.c index 25b40cf..1642ac6 100644 --- a/lib/sbi/riscv_asm.c +++ b/lib/sbi/riscv_asm.c @@ -268,6 +268,7 @@ int pmp_set(unsigned int n, unsigned long prot, unsigned long addr, return SBI_ENOTSUPP; /* encode PMP config */ + prot &= ~PMP_A; prot |= (log2len == PMP_SHIFT) ? PMP_A_NA4 : PMP_A_NAPOT; cfgmask = ~(0xffUL << pmpcfg_shift); pmpcfg = (csr_read_num(pmpcfg_csr) & cfgmask); -- 2.7.4