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

2022-12-07 Thread Kautuk Consul
Hi Leo, Thanks! I have sent a v6 of this patchset wherein I have rebased the patchset on the latest master. I have removed the last patch with default config options for qemu-riscv64 targets and have replaced it with a patch with Sean's suggestion for adding to the dependencies. Thanks again. On

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

2022-12-06 Thread Leo Liang
Hi Kautuk, On Tue, Dec 06, 2022 at 05:02:49PM +0530, Kautuk Consul wrote: > Hi Leo, > > On Tue, Dec 6, 2022 at 4:29 PM Leo Liang wrote: > > > > Hi Kautuk, > > > > We have tested your patchset with QEMU 7.1.0. > > It generally looks fine, but CI error seems to persist. > > https://source.denx.de/

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

2022-12-06 Thread Sean Anderson
On 12/6/22 00:42, Kautuk Consul wrote: > Hi, > > On Mon, Dec 5, 2022 at 8:46 PM Sean Anderson wrote: >> >> On 12/5/22 00:51, Kautuk Consul wrote: >> > Hi, >> > >> > On Sat, Dec 3, 2022 at 9:44 AM Bin Meng wrote: >> >> >> >> On Fri, Sep 23, 2022 at 3:03 PM Kautuk Consul >> >> wrote: >> >> > >>

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

2022-12-06 Thread Kautuk Consul
Hi Leo, On Tue, Dec 6, 2022 at 4:29 PM Leo Liang wrote: > > Hi Kautuk, > > We have tested your patchset with QEMU 7.1.0. > It generally looks fine, but CI error seems to persist. > https://source.denx.de/u-boot/custodians/u-boot-riscv/-/pipelines/14314 > > The error comes from CI testcase timed-o

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

2022-12-06 Thread Leo Liang
Hi Kautuk, We have tested your patchset with QEMU 7.1.0. It generally looks fine, but CI error seems to persist. https://source.denx.de/u-boot/custodians/u-boot-riscv/-/pipelines/14314 The error comes from CI testcase timed-out. The reason for the time-out is not yet confirmed, but we suspect it

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

2022-12-05 Thread Kautuk Consul
Hi, On Mon, Dec 5, 2022 at 8:46 PM Sean Anderson wrote: > > On 12/5/22 00:51, Kautuk Consul wrote: > > Hi, > > > > On Sat, Dec 3, 2022 at 9:44 AM Bin Meng wrote: > >> > >> On Fri, Sep 23, 2022 at 3:03 PM Kautuk Consul > >> wrote: > >> > > >> > To enable semihosting we also need to enable the f

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

2022-12-05 Thread Sean Anderson
On 12/5/22 00:51, Kautuk Consul wrote: > Hi, > > On Sat, Dec 3, 2022 at 9:44 AM Bin Meng wrote: >> >> On Fri, Sep 23, 2022 at 3:03 PM Kautuk Consul >> wrote: >> > >> > To enable semihosting we also need to enable the following >> > configs in defconfigs: >> > CONFIG_SEMIHOSTING >> > CONFIG_SPL_

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

2022-12-05 Thread Bin Meng
On Mon, Dec 5, 2022 at 1:51 PM Kautuk Consul wrote: > > Hi, > > On Sat, Dec 3, 2022 at 9:44 AM Bin Meng wrote: > > > > On Fri, Sep 23, 2022 at 3:03 PM Kautuk Consul > > wrote: > > > > > > To enable semihosting we also need to enable the following > > > configs in defconfigs: > > > CONFIG_SEMIHO

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

2022-12-04 Thread Kautuk Consul
Hi, On Sat, Dec 3, 2022 at 9:44 AM Bin Meng wrote: > > On Fri, Sep 23, 2022 at 3:03 PM Kautuk Consul > wrote: > > > > To enable semihosting we also need to enable the following > > configs in defconfigs: > > CONFIG_SEMIHOSTING > > CONFIG_SPL_SEMIHOSTING > > CONFIG_SEMIHOSTING_SERIAL > > CONFIG_

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

2022-12-02 Thread Bin Meng
On Fri, Sep 23, 2022 at 3:03 PM Kautuk Consul 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 thes

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

2022-10-11 Thread Rick Chen
dt > ; Rasmus Villemoes ; Eugen > Hristev ; Stefan Roese ; Loic > Poulain ; Peng Fan ; Michal Simek > > Cc: u-boot@lists.denx.de; Kautuk Consul > Subject: [PATCH v5 3/3] board: qemu-riscv: enable semihosting > > To enable semihosting we also need to enable the followin

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

2022-10-05 Thread Kautuk Consul
Hi Leo, I took a look at the raw log. It seems that the test-case is outputting too many characters onto the u-boot prompt due to which the log size has gotten exceeded. Can you somehow increase the log size and then try ? Thanks. On Wed, Oct 5, 2022 at 11:47 AM Leo Liang wrote: > > Hi Kautuk,

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

2022-10-04 Thread Leo Liang
Hi Kautuk, On Fri, Sep 23, 2022 at 12:33:20PM +0530, Kautuk Consul 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_F

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

2022-09-23 Thread Kautuk Consul
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 Signed-off-by: Kautuk Consul --- configs/qemu-riscv32_defconfig | 4 confi