Hi Thomas, I have made the suggested change and submitted it to the OpenOCD gerrit. Please let me know if there is anything additional to be done, and please note I have not tested the changes as I have not yet gone through the trouble of getting OpenOCD to build on Windows.
On Sun, Nov 20, 2022 at 2:06 PM Tomas Vanek <tom_...@users.sourceforge.net> wrote: > On 20/11/2022 19:47, Nick Kraus wrote: > > When trying to use a cmsis-dap debugger to capture SWO output, I think > that the command to change baudrate doesn't send enough bytes to the > debug probe. At line 663 of cmsis_dap.c (function > cmsis_dap_cmd_dap_swo_baudrate()) > it appears that 4 bytes are transferred for this command. I believe it > should be 5 total (the command tag 0x19, then the 4-byte baudrate word), as > described here > <https://arm-software.github.io/CMSIS_5/DAP/html/group__DAP__SWO__Baudrate.html> > . > > > Nick, > > your report looks correct. > Could you please submit a fix to https://review.openocd.org ? > For details see https://openocd.org/doc/doxygen/html/patchguide.html > > Thanks > Tomas >