[PATCH v4 3/8] firmware: fw_base: Simplified setup trap handler
Xiang W
wxjstz at 126.com
Sun Feb 11 08:11:27 PST 2024
The same detection was done twice when setting mtvec and trap_exit.
Merging can reduce code size.
Signed-off-by: Xiang W <wxjstz at 126.com>
Reviewed-by: Anup Patel <anup at brainfault.org>
---
firmware/fw_base.S | 16 ++++------------
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/firmware/fw_base.S b/firmware/fw_base.S
index d418041..f0dd0da 100644
--- a/firmware/fw_base.S
+++ b/firmware/fw_base.S
@@ -362,22 +362,14 @@ _start_warm:
srli a5, a5, ('H' - 'A')
andi a5, a5, 0x1
beq a5, zero, _skip_trap_handler_rv32_hyp
- lla a4, _trap_handler_rv32_hyp
-_skip_trap_handler_rv32_hyp:
-#endif
- csrw CSR_MTVEC, a4
-
-#if __riscv_xlen == 32
/* Override trap exit for H-extension */
- csrr a5, CSR_MISA
- srli a5, a5, ('H' - 'A')
- andi a5, a5, 0x1
- beq a5, zero, _skip_trap_exit_rv32_hyp
- lla a4, _trap_exit_rv32_hyp
csrr a5, CSR_MSCRATCH
+ lla a4, _trap_exit_rv32_hyp
REG_S a4, SBI_SCRATCH_TRAP_EXIT_OFFSET(a5)
-_skip_trap_exit_rv32_hyp:
+ lla a4, _trap_handler_rv32_hyp
+_skip_trap_handler_rv32_hyp:
#endif
+ csrw CSR_MTVEC, a4
/* Initialize SBI runtime */
csrr a0, CSR_MSCRATCH
--
2.43.0
More information about the opensbi
mailing list