Xiang Wang has posted comments on this change. ( https://review.coreboot.org/29023 )
Change subject: riscv: add support smp_pause / smp_resume ......................................................................
Patch Set 8:
(3 comments)
https://review.coreboot.org/#/c/29023/7/src/arch/riscv/include/mcall.h File src/arch/riscv/include/mcall.h:
https://review.coreboot.org/#/c/29023/7/src/arch/riscv/include/mcall.h@72 PS7, Line 72: /*
Can you clarify this comment?
Done
https://review.coreboot.org/#/c/29023/7/src/arch/riscv/smp.c File src/arch/riscv/smp.c:
https://review.coreboot.org/#/c/29023/7/src/arch/riscv/smp.c@22 PS7, Line 22: void smp_pause(int working_hartid)
Could you add some comments here, especially that this function never returns.
Done
https://review.coreboot.org/#/c/29023/7/src/arch/riscv/smp.c@24 PS7, Line 24: if (read_csr(mhartid) != working_hartid) {
Can we move this check outside of this function to make the flow clearer? […]
I want to pass the parameters to choose the cpu core to work.