[PATCH v5 3/3] board: qemu-riscv: enable semihosting

[Sean Anderson]

On 12/5/22 00:51, Kautuk Consul wrote:
> Hi,
> 
> On Sat, Dec 3, 2022 at 9:44 AM Bin Meng <bmeng.cn at gmail.com> wrote:
>>
>> On Fri, Sep 23, 2022 at 3:03 PM Kautuk Consul <kconsul at ventanamicro.com> wrote:
>> >
>> > To enable semihosting we also need to enable the following
>> > configs in defconfigs:
>> > CONFIG_SEMIHOSTING
>> > CONFIG_SPL_SEMIHOSTING
>> > CONFIG_SEMIHOSTING_SERIAL
>> > CONFIG_SERIAL_PROBE_ALL
>> > CONFIG_SPL_FS_EXT4
>> > CONFIG_SPL_FS_FAT
>>
>> Why should these _SPL_FS_xxx be required? If it's required by
>> SEMIHOSTING, could the dependency be fixed there?
> 
> The build dependencies require that these options be there.

What error do you get?

--Sean

>>
>> >
>> > Signed-off-by: Kautuk Consul <kconsul at ventanamicro.com>
>> > ---
>> >  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 +++++++
>> >  6 files changed, 30 insertions(+)
>> >
>>
>> Regards,
>> Bin