08:04 <gamiee> hi borneoa__ . Thank you for feedback in https://review.openocd.org/c/openocd/+/8529/1/tcl/target/bl702.cfg#b57

08:04 <gamiee> I wasn't aware of "catch" clause

08:05 <gamiee> I re-implemented it with catch clause, and I needed to modify some stuff (like reseting in -pre event).

08:05 <gamiee> It works, but it anyway shows error :/ But it works.

08:12 <gamiee> It works even without catch, just of course TCL execution is break, but that's okay.

08:18 <gamiee> https://paste.debian.net/1333571/ the error is still shown, but it works.

08:56 <borneoa___> gamiee: what is printed out by echo $result ? I use the catch command when I know the command will return error, like on this case, but I often ignore the result

08:58 <borneoa___> gamiee: the TCL command to write in memory should not output errors using LOG_ERROR(); it should return the error through the normal TCL output so catch can take it and either analyze or drop it

08:59 <borneoa___> gamiee: but unfortunately LOG_xxx() is widely abused in current code!

10:44 <gamiee> borneoa___ : the $result is empty, that's the empty line between "Error" and "Info" messages.

10:44 <gamiee> ./configure CFLAGS="-ggdb3 -O0" CXXFLAGS="-ggdb3 -O0" LDFLAGS="-ggdb3"

10:44 <gamiee> those are my build flags, but I don't think those changes anything.

10:46 <gamiee> both Warn and Error logs came from riscv-013.c

11:07 <borneoa___> gamiee: it's not a problem of compile options. I'm checking for a workaround...

12:03 <gamiee> borneoa___ : Thanks! Eventually, easy workaround would be to print before that this error is normal, and just leave it as is...

12:03 <gamiee> But well, having proper solution would be probably the best

12:42 <borneoa___> gamiee: proper solution will take too much

12:42 <borneoa___> gamiee: before the command that give error, add:

12:43 <borneoa___> gamiee: set lvl [lindex [debug_level] 1]

12:44 <borneoa___> gamiee: debug_level -1

12:44 <borneoa___> gamiee: then the command that give error, and then

12:44 <borneoa___> gamiee: debug_level $lvl

12:45 <borneoa___> gamiee: this will silent every LOG_xx() caused by your command

12:46 <borneoa___> gamiee: I haven't check if an attached GDB will still receive the errors...

13:31 <gamiee> borneoa___ : i am gonna try

13:32 <gamiee> It works perfectly! Error is not shown in both GDB and OpenOCD.

13:33 <gamiee> I will clean it up a bit and send it to gerrit. Thanks for help :)

13:33 <gamiee> Even it uses this workaround, it's much more readable than assembly and still works well.

15:10 <borneoa___> gamiee: cool!

15:59 <gamiee> borneoa___ : Patch is up. (Sorry, messed up twice some stuff, this patch is final).

15:59 <gamiee> Also, I hope it will be possible to get my BL602 flash bank driver ( https://review.openocd.org/c/openocd/+/8527/3 ) to upstream too.

15:59 <gamiee> I tried to make it as best as I could, so it would be suitable for upstream, and I wouldn't like to see it rot on fork :( I plan to maintain it in future (it's definitely not "upstream & abandon"), I already working on BL602 support for it :)

15:59 <gamiee> s/rot on/rot in/

17:30 <borneoa___> gamiee: I have added a minor comment to your patch

17:31 <borneoa___> gamiee: for the flash driver, I only do some minor check, as I'm not familiar enough with the flash subsystem to run a full review

19:34 <gamiee> borneoa___ I sent fixed patch, sorry, I probably introduced the space when commenting the code and forgot to remove space.

19:34 <gamiee> About the flash driver, thx for minor check. I will try to take look who does upstreaming of those and contact them about request for review.