Jan Matyas

Results 79 comments of Jan Matyas

@TommyMurphyTM1234 Thank you for your reply. > If anything I was suggesting that the master branch should be eliminated (or, as you suggest, marked in some way as "archived"). Sounds...

Closing this stale pull request as there has not been any development for the past 2 months.

@kr-sc I haven't yet tried to reproduce it locally. What values of `dmstatus` are you getting, please? Are they unexpected? Also your added LOG_INFO print in the patch is put...

@aap-sc - Thank you for correcting me, I believe you're right. I will double-check it and update the issue.

> In the other functions it looks to us that assert(!ac_busy) would be more appropriate. Or, possibly `LOG_ERROR("BUG: Unexpected abstractcs.busy ...");` could be used if asserts seem too harsh. Still,...

@en-sc: > Currently, in general, working with abstract commands is something along the lines of ... > > However, [the loop in `wait_for_idle()` is time-bound](https://github.com/riscv-collab/riscv-openocd/pull/1023/files#diff-701864e2be3210275085c84b8f6fa7f9eb6171a2a18ca71638be42a155c08c63R915), so ... > > ......

@MrAlexei Thank you for the contribution and I am sorry for a not very timely review. I will be able to take a look at the changes tomorrow morning.

@TommyMurphyTM1234 or @MrAlexei, please, don't forget to mark the above threads as resolved when done so. Thank you.

@aap-sc @MrAlexei Please, what is the status on the two last currently open threads ([1](https://github.com/riscv-collab/riscv-openocd/pull/1029#discussion_r1579574008), [2](https://github.com/riscv-collab/riscv-openocd/pull/1029#discussion_r1574955154)) in this PR? Thank you.