Hi Roman, can you please submit your patch in OpenOCD's gerrit? The instructions are both in the file HACKING and online in http://openocd.org/doc-release/doxygen/patchguide.html
In meantime, I have checked your patch against ARM code CMSIS/DAP/Firmware/Source/DAP.c with both current version https://github.com/ARM-software/CMSIS_5.git and legacy one https://github.com/ARM-software/CMSIS.git The patch is in line with ARM code. Actually I don't know if ulink2 FW implements exactly the same ARM code or has some major differences. I will test it (hopefully next week) when I will have access to a ulink2 with cmsis-dap FW. On Fri, Dec 27, 2019 at 9:16 AM Roman Elshin via OpenOCD-devel <[email protected]> wrote: > > It is with keil ulink2, attached patch make it functional but RUN and > CONNECT leds works similary (i don't know how to make it properly). I'm not sure I understand what you say. What do you mean for "...leds works similary"? If you mean you have to call cmsis_dap_cmd_DAP_LED() two times, that's correct, accordingly to ARM code. Thanks, Antonio _______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
