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
>


Reply via email to