Xiang Wang 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.
Yes! You are right.
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 BL […]
This is simple.
It can be made like the ARM BL31, but it does not separate the firmware and the kernel.
I wrote a loader as a coreboot payload, loading opensbi and the kernel from the sd card. But still in debugging