[PATCH]: e533338c70 jtag/drivers/cmsis_dap.c: Fix Length of SWO Baudrate Command

[gerrit]

This is an automated email from Gerrit.

"Name of user not set <n...@nckraus.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7370

-- gerrit

commit e533338c70ec76e79482475c4edddd8566732b9e
Author: Nick Kraus <n...@nckraus.com>
Date:   Mon Nov 21 09:55:11 2022 -0500

    jtag/drivers/cmsis_dap.c: Fix Length of SWO Baudrate Command
    
    The command should now send the full 5 byte command length, which
    includes the command tag (0x19) and the 4-byte baudrate word, instead
    of only the last 3 bytes of the baudrate.
    
    Signed-off-by: Nick Kraus <n...@nckraus.com>
    Change-Id: Idd6e084efd7492489aa900cdbf08f540944041cb

diff --git a/src/jtag/drivers/cmsis_dap.c b/src/jtag/drivers/cmsis_dap.c
index 2f5f9c907b..44a7dd8cb6 100644
--- a/src/jtag/drivers/cmsis_dap.c
+++ b/src/jtag/drivers/cmsis_dap.c
@@ -660,7 +660,7 @@ static int cmsis_dap_cmd_dap_swo_baudrate(
        command[0] = CMD_DAP_SWO_BAUDRATE;
        h_u32_to_le(&command[1], in_baudrate);
 
-       int retval = cmsis_dap_xfer(cmsis_dap_handle, 4);
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 5);
        uint32_t rvbr = le_to_h_u32(&cmsis_dap_handle->response[1]);
        if (retval != ERROR_OK || rvbr == 0) {
                LOG_ERROR("CMSIS-DAP: command CMD_SWO_Baudrate(%u) -> %u 
failed.", in_baudrate, rvbr);

--