Revert "riscv: qemu: clear kernel-start/-end in device tree as workaround for BBL"

The commit was added as a workaround required in QEMU when using BBL as
the supervisor binary interface (SBI) for Linux. We are now using
OpenSBI to provide the SBI, the workaround is therefore not required
anymore and can be removed.

This reverts commit 897206c5cc5c6ac0dc2ab851044e42baada3785b.

Signed-off-by: Lukas Auer <lukas.auer@aisec.fraunhofer.de>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

diff --git a/board/emulation/qemu-riscv/Kconfig b/board/emulation/qemu-riscv/Kconfig
index 7ce12018e7af4b9ed86edd46a2ca619b383e5d7f..ad99b08b4478c9265ed2e92ae9bd284dcdbe0855 100644 (file)
--- a/board/emulation/qemu-riscv/Kconfig
+++ b/board/emulation/qemu-riscv/Kconfig
@@ -43,7 +43,6 @@ config BOARD_SPECIFIC_OPTIONS # dummy
        imply CMD_EXT4
        imply CMD_FAT
        imply BOARD_LATE_INIT
-       imply OF_BOARD_SETUP
        imply SIFIVE_SERIAL
        imply SMP
        imply PCI

diff --git a/board/emulation/qemu-riscv/qemu-riscv.c b/board/emulation/qemu-riscv/qemu-riscv.c
index cbce5ffe6e336fd52c9f86955aa81fae51337966..c3f96988b1d49c63ec012da5c09772db2304dfd7 100644 (file)
--- a/board/emulation/qemu-riscv/qemu-riscv.c
+++ b/board/emulation/qemu-riscv/qemu-riscv.c
@@ -52,45 +52,6 @@ int board_late_init(void)
        return 0;
 }
 
-/*
- * QEMU specifies the location of Linux (supplied with the -kernel argument)
- * in the device tree using the riscv,kernel-start and riscv,kernel-end
- * properties. We currently rely on the SBI implementation of BBL to run
- * Linux and therefore embed Linux as payload in BBL. This causes an issue,
- * because BBL detects the kernel properties in the device tree and ignores
- * the Linux payload as a result. To work around this issue, we clear the
- * kernel properties before booting Linux.
- *
- * This workaround can be removed, once we do not require BBL for its SBI
- * implementation anymore.
- */
-int ft_board_setup(void *blob, bd_t *bd)
-{
-       int chosen_offset, ret;
-
-       chosen_offset = fdt_path_offset(blob, "/chosen");
-       if (chosen_offset < 0)
-               return 0;
-
-#ifdef CONFIG_ARCH_RV64I
-       ret = fdt_setprop_u64(blob, chosen_offset, "riscv,kernel-start", 0);
-#else
-       ret = fdt_setprop_u32(blob, chosen_offset, "riscv,kernel-start", 0);
-#endif
-       if (ret)
-               return ret;
-
-#ifdef CONFIG_ARCH_RV64I
-       ret = fdt_setprop_u64(blob, chosen_offset, "riscv,kernel-end", 0);
-#else
-       ret = fdt_setprop_u32(blob, chosen_offset, "riscv,kernel-end", 0);
-#endif
-       if (ret)
-               return ret;
-
-       return 0;
-}
-
 #ifdef CONFIG_SPL
 u32 spl_boot_device(void)
 {