20:59 <balrog> borneoa___: 4284 is ready for review

20:59 <balrog> is there any interest in revisiting https://review.openocd.org/c/openocd/+/945?

21:12 <borneoa___> balrog: thanks, I will check it shortly. Indeed 945 is an interesting step ahead

21:12 <balrog> I could rework 945 if you'd like

21:44 <RolfNoot> I found an issue with 7966, where cmsis-dap quirk workarounds are introduced

21:45 <RolfNoot> On my DAP-LINK adapter, before 7966, programming / debugging is working. After 7966 it stopped

21:49 <RolfNoot> Line 1228 of the new patch adds the check if quirk mode is enabled to assert the SRST line, while the cmsis_dap_handle->quirk_mode is disabled by default. In the previous code the asserting of the SRST line is always done.

21:50 <RolfNoot> I don't know if this is an issue but the behavior changed after this patch for the DAPLINK adapter

21:58 <PaulFertser> RolfNoot: but SRST isn't supposed to be asserted by default normally. Or else you wouldn't be able to attach to a running target without altering its state.

21:59 <balrog> borneoa___: reworked, but untested

22:01 <RolfNoot> I have the reset_config srst_only'

22:01 <RolfNoot> added for my board

22:02 <RolfNoot> I didn't look into this well but I see the behavior change with the patch.

22:13 <PaulFertser> RolfNoot: srst_only means OpenOCD should use physical SRST line to reset when you ask the target to be reset.

22:13 <PaulFertser> RolfNoot: probably you want srst_only connect_assert_srst ?

22:14 <RolfNoot> PaulFertser: I still need to look into this. I want a physical assert of the SRST line but also noted different behavior between different adapters

22:17 <RolfNoot> PaulFertser: Although with the prior version the quirk mode was enabled by default (no check): if (seq != LINE_RESET &&

22:18 <RolfNoot> PaulFertser: The patch added the check (while quirk mode is disabled now by default): if (cmsis_dap_handle->quirk_mode && seq != LINE_RESET &&

22:19 <RolfNoot> PaulFertser: So shouldn't the quirk mode be enabled by default to maintain the same behavior? Or is this unwanted?

22:23 <balrog> borneoa___: will need more rework. how do I get OpenOCD to build all drivers?

22:35 <RolfNoot> PaulFertser: Also, OpenOCD crashes on 'cmsis-dap quirk enable' when issued before init

22:50 <karlp> RolfNoot: regardless of what it was before, it should not reset on connect by default. that should be requested behavioru.