[PATCH] src/target/semihosting_common.c: add stdfd fd control command

2022-10-17 Thread Kautuk Consul
for this new command. Signed-off-by: Kautuk Consul --- doc/openocd.texi| 7 src/target/semihosting_common.c | 58 + 2 files changed, 65 insertions(+) diff --git a/doc/openocd.texi b/doc/openocd.texi index 2a8626b80..b0bd976b0 100644 --- a/

Re: [PATCH] src/target/semihosting_common.c: add stdfd fd control command

2022-11-21 Thread Kautuk Consul
Hi, Can anyone review my patch ? This patch can improve performance of semihosting with respect to key-strokes. Thanks. On Tue, Oct 18, 2022 at 10:10 AM Kautuk Consul wrote: > The semihosting->stdin_fd is initially in blocking mode. > Due to this there is a possibility that t

Re: [PATCH] src/target/semihosting_common.c: add stdfd fd control command

2022-11-23 Thread Kautuk Consul
rtser wrote: > Hi, > > On Tue, Nov 22, 2022 at 12:18:30PM +0530, Kautuk Consul wrote: > > Can anyone review my patch ? > > This patch can improve performance of semihosting with respect to > key-strokes. > > Thank you for the patch. Have you considered sending

Re: [PATCH] src/target/semihosting_common.c: add stdfd fd control command

2022-11-23 Thread Kautuk Consul
PM +0530, Kautuk Consul wrote: > > I don't have access to gerrit and I am not a regular contributor. > > It's open for public, and anybody can send and review patches there. > > > I am doing board bring-up for our RISCV board and I noticed this issue > with our > &

Re: [PATCH] src/target/semihosting_common.c: add stdfd fd control command

2022-11-24 Thread Kautuk Consul
resolve this compilation problem before anyone reviews the patch ? If yes, then can you help me resolve this issue as I can't figure out what went wrong in the compilation from the console logs. Thanks! On Wed, Nov 23, 2022 at 5:07 PM Kautuk Consul wrote: > Ok, then let me try to find

Re: [PATCH] src/target/semihosting_common.c: add stdfd fd control command

2022-11-24 Thread Kautuk Consul
: > Hi, > > On Thu, Nov 24, 2022 at 10:10:59PM +0530, Kautuk Consul wrote: > > So I managed to create a gerrit account and followed the instructions in > > HACKING. > > Thanks, you did everything right. > > > But, I got a build failure for mingw64 as given >

Re: [PATCH] src/target/semihosting_common.c: add stdfd fd control command

2022-11-24 Thread Kautuk Consul
Ok anyway I resolved the issue myself. It is now verified. Thanks for your help! :) On Fri, Nov 25, 2022 at 11:57 AM Kautuk Consul wrote: > Ok, so I have taken in all the comments I could find. > But now the build is showing that mingw64 compiled fine but the build got > aborted: