[PATCH v5 0/3] Add riscv semihosting support in u-boot

[Kautuk Consul]

Hi,

Can someone pick this patchset up ?

It has been reviewed and there has been no comment on this in recent days.

Thanks.

On Fri, Sep 23, 2022 at 12:33 PM Kautuk Consul <kconsul at ventanamicro.com>
wrote:

> Semihosting is a mechanism that enables code running on
> a target to communicate and use the Input/Output
> facilities on a host computer that is running a debugger.
> This patchset adds support for semihosting in u-boot
> for RISCV64 targets.
>
> CHANGES since v4:
> -       Check arch dependencies for SEMIHOSTING as well as SPL_SEMIHOSTING
>         config options as per Sean's comment.
> -       arch/riscv/lib/interrupts.c: Check for post and pre instructions
>         of the ebreak statement whether they are as per the RISCV
>         semihosting specification. Only then do a disable_semihosting
>         and epc += 4 and return.
>
> Compilation and test commands for SPL and S-mode configurations
> =================================================================
>
> U-Boot S-mode on QEMU virt
> ----------------------------
> // Compilation of S-mode u-boot
> ARCH=riscv
> CROSS_COMPILE=riscv64-unknown-linux-gnu-
> make qemu-riscv64_smode_defconfig
> make
> // Run riscv 64-bit u-boot with opensbi on qemu
> qemu-system-riscv64 -M virt -m 256M -display none -serial stdio -bios\
> opensbi/build/platform/generic/firmware/fw_jump.bin -kernel\
> u-boot/u-boot.bin
>
> U-Boot SPL on QEMU virt
> ------------------------
> // Compilation of u-boot-spl
> ARCH=riscv
> CROSS_COMPILE=riscv64-unknown-linux-gnu-
> make qemu-riscv64_spl_defconfig
> make OPENSBI=opensbi/build/platform/generic/firmware/fw_dynamic.bin
> // Run 64-bit u-boot-spl in qemu
> qemu-system-riscv64 -M virt -m 256M -display none -serial stdio -bios\
> u-boot/spl/u-boot-spl.bin -device\
> loader,file=u-boot/u-boot.itb,addr=0x80200000
>
> Kautuk Consul (3):
>   lib: Add common semihosting library
>   arch/riscv: add semihosting support for RISC-V
>   board: qemu-riscv: enable semihosting
>
>  arch/arm/Kconfig                     |  46 -------
>  arch/arm/lib/semihosting.c           | 181 +-------------------------
>  arch/riscv/include/asm/spl.h         |   1 +
>  arch/riscv/lib/Makefile              |   2 +
>  arch/riscv/lib/interrupts.c          |  25 ++++
>  arch/riscv/lib/semihosting.c         |  24 ++++
>  configs/qemu-riscv32_defconfig       |   4 +
>  configs/qemu-riscv32_smode_defconfig |   4 +
>  configs/qemu-riscv32_spl_defconfig   |   7 +
>  configs/qemu-riscv64_defconfig       |   4 +
>  configs/qemu-riscv64_smode_defconfig |   4 +
>  configs/qemu-riscv64_spl_defconfig   |   7 +
>  include/semihosting.h                |  11 ++
>  lib/Kconfig                          |  47 +++++++
>  lib/Makefile                         |   2 +
>  lib/semihosting.c                    | 186 +++++++++++++++++++++++++++
>  16 files changed, 329 insertions(+), 226 deletions(-)
>  create mode 100644 arch/riscv/lib/semihosting.c
>  create mode 100644 lib/semihosting.c
>
> --
> 2.34.1
>
>