gustavonihei commented on code in PR #6162: URL: https://github.com/apache/incubator-nuttx/pull/6162#discussion_r859832219
########## arch/risc-v/src/common/riscv_pmp.c: ########## @@ -600,9 +600,14 @@ int riscv_config_pmp_region(uintptr_t region, uintptr_t attr, # error "XLEN of risc-v not supported" # endif - /* fence is needed when page-based virtual memory is implemented */ +#ifdef CONFIG_ARCH_HAVE_S_MODE Review Comment: > `CONFIG_ARCH_HAVE_S_MODE` was originally intended to indicate software capability, not chip capability. But maybe I could add ARCH_RV_ISA_S like the other chip capabilities are listed for other chips and do `CONFIG_ARCH_HAVE_S_MODE` via another config parameter which depends ARCH_RV_ISA_S. You don't have to wait for this I can do the refactoring. So, for this PR, I'll just change `CONFIG_ARCH_HAVE_S_MODE` to `CONFIG_ARCH_USE_S_MODE`, which is already used in other places and will probably also change in a future refactor. I think this little change makes things a little more correct. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
