Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32394 )
Change subject: riscv: add support to boot linux kernel binary using opensbi ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/#/c/32394/4/payloads/Kconfig File payloads/Kconfig:
https://review.coreboot.org/#/c/32394/4/payloads/Kconfig@31 PS4, Line 31: config PAYLOAD_RISCV_LINUX_BINARY as far as I understand the OpenSBI payload/bootloader can be anything, not only a linux binary.
https://review.coreboot.org/#/c/32394/4/src/arch/riscv/Makefile.inc File src/arch/riscv/Makefile.inc:
https://review.coreboot.org/#/c/32394/4/src/arch/riscv/Makefile.inc@193 PS4, Line 193: FW_PAYLOAD_PATH=$(abspath $(CONFIG_PAYLOAD_FILE)) Why do you use FW_PAYLOAD? Using FW_JUMP or FW_DYNAMIC seem to match what is done on aarch64 with BL31, which is supported in coreboot and works well, as we can handle the payload in coreboot and don't need to rebuild OpenSBI for every change.