jtag/drivers/cmsis_dap: improve USB packets filling
[openocd.git] / src / jtag / drivers / cmsis_dap.c
index b5ceb6cefbc7f5fe68d3f6cf6904e663b07f9d19..10e6638627def00d9f9050fe9a8ca56c9bd30bf1 100644 (file)
@@ -1,4 +1,9 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+
 /***************************************************************************
+ *   Copyright (C) 2021 by Adrian Negreanu                                 *
+ *   groleo@gmail.com                                                      *
+ *                                                                         *
  *   Copyright (C) 2018 by MickaĆ«l Thomas                                  *
  *   mickael9@gmail.com                                                    *
  *                                                                         *
  *                                                                         *
  *   Copyright (C) 2013 by Spencer Oliver                                  *
  *   spen@spen-soft.co.uk                                                  *
- *                                                                         *
- *   This program is free software; you can redistribute it and/or modify  *
- *   it under the terms of the GNU General Public License as published by  *
- *   the Free Software Foundation; either version 2 of the License, or     *
- *   (at your option) any later version.                                   *
- *                                                                         *
- *   This program is distributed in the hope that it will be useful,       *
- *   but WITHOUT ANY WARRANTY; without even the implied warranty of        *
- *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the         *
- *   GNU General Public License for more details.                          *
- *                                                                         *
- *   You should have received a copy of the GNU General Public License     *
- *   along with this program.  If not, see <http://www.gnu.org/licenses/>. *
  ***************************************************************************/
 
 #ifdef HAVE_CONFIG_H
 #endif
 
 #include <transport/transport.h>
+#include "helper/replacements.h"
+#include <jtag/adapter.h>
 #include <jtag/swd.h>
 #include <jtag/interface.h>
 #include <jtag/commands.h>
 #include <jtag/tcl.h>
+#include <target/cortex_m.h>
 
 #include "cmsis_dap.h"
+#include "libusb_helper.h"
 
 static const struct cmsis_dap_backend *const cmsis_dap_backends[] = {
 #if BUILD_CMSIS_DAP_USB == 1
@@ -70,12 +66,9 @@ static const struct cmsis_dap_backend *const cmsis_dap_backends[] = {
 /* vid = pid = 0 marks the end of the list */
 static uint16_t cmsis_dap_vid[MAX_USB_IDS + 1] = { 0 };
 static uint16_t cmsis_dap_pid[MAX_USB_IDS + 1] = { 0 };
-static char *cmsis_dap_serial;
 static int cmsis_dap_backend = -1;
 static bool swd_mode;
 
-#define USB_TIMEOUT       1000
-
 /* CMSIS-DAP General Commands */
 #define CMD_DAP_INFO              0x00
 #define CMD_DAP_LED               0x01
@@ -95,9 +88,18 @@ static bool swd_mode;
 #define INFO_ID_CAPS              0xf0      /* byte */
 #define INFO_ID_PKT_CNT           0xfe      /* byte */
 #define INFO_ID_PKT_SZ            0xff      /* short */
-
-#define INFO_CAPS_SWD             0x01
-#define INFO_CAPS_JTAG            0x02
+#define INFO_ID_SWO_BUF_SZ        0xfd      /* word */
+
+#define INFO_CAPS_SWD                 BIT(0)
+#define INFO_CAPS_JTAG                BIT(1)
+#define INFO_CAPS_SWO_UART            BIT(2)
+#define INFO_CAPS_SWO_MANCHESTER      BIT(3)
+#define INFO_CAPS_ATOMIC_CMDS         BIT(4)
+#define INFO_CAPS_TEST_DOMAIN_TIMER   BIT(5)
+#define INFO_CAPS_SWO_STREAMING_TRACE BIT(6)
+#define INFO_CAPS_UART_PORT           BIT(7)
+#define INFO_CAPS_USB_COM_PORT        BIT(8)
+#define INFO_CAPS__NUM_CAPS               9
 
 /* CMD_LED */
 #define LED_ID_CONNECT            0x00
@@ -136,6 +138,7 @@ static bool swd_mode;
 
 /* CMSIS-DAP SWD Commands */
 #define CMD_DAP_SWD_CONFIGURE     0x13
+#define CMD_DAP_SWD_SEQUENCE      0x1D
 
 /* CMSIS-DAP JTAG Commands */
 #define CMD_DAP_JTAG_SEQ          0x14
@@ -161,46 +164,66 @@ static bool swd_mode;
 #define DAP_OK                    0
 #define DAP_ERROR                 0xFF
 
+/* CMSIS-DAP SWO Commands */
+#define CMD_DAP_SWO_TRANSPORT     0x17
+#define CMD_DAP_SWO_MODE          0x18
+#define CMD_DAP_SWO_BAUDRATE      0x19
+#define CMD_DAP_SWO_CONTROL       0x1A
+#define CMD_DAP_SWO_STATUS        0x1B
+#define CMD_DAP_SWO_DATA          0x1C
+#define CMD_DAP_SWO_EX_STATUS     0x1E
+
+/* SWO transport mode for reading trace data */
+#define DAP_SWO_TRANSPORT_NONE    0
+#define DAP_SWO_TRANSPORT_DATA    1
+#define DAP_SWO_TRANSPORT_WINUSB  2
+
+/* SWO trace capture mode */
+#define DAP_SWO_MODE_OFF          0
+#define DAP_SWO_MODE_UART         1
+#define DAP_SWO_MODE_MANCHESTER   2
+
+/* SWO trace data capture */
+#define DAP_SWO_CONTROL_STOP      0
+#define DAP_SWO_CONTROL_START     1
+
+/* SWO trace status */
+#define DAP_SWO_STATUS_CAPTURE_INACTIVE      0
+#define DAP_SWO_STATUS_CAPTURE_ACTIVE        1
+#define DAP_SWO_STATUS_CAPTURE_MASK          BIT(0)
+#define DAP_SWO_STATUS_STREAM_ERROR_MASK     BIT(6)
+#define DAP_SWO_STATUS_BUFFER_OVERRUN_MASK   BIT(7)
+
 /* CMSIS-DAP Vendor Commands
  * None as yet... */
 
-static const char * const info_caps_str[] = {
-       "SWD  Supported",
-       "JTAG Supported"
-};
-
-struct pending_transfer_result {
-       uint8_t cmd;
-       uint32_t data;
-       void *buffer;
-};
-
-struct pending_request_block {
-       struct pending_transfer_result *transfers;
-       int transfer_count;
+static const char * const info_caps_str[INFO_CAPS__NUM_CAPS] = {
+       "SWD supported",
+       "JTAG supported",
+       "SWO-UART supported",
+       "SWO-MANCHESTER supported",
+       "Atomic commands supported",
+       "Test domain timer supported",
+       "SWO streaming trace supported",
+       "UART communication port supported",
+       "UART via USB COM port supported",
 };
 
 struct pending_scan_result {
        /** Offset in bytes in the CMD_DAP_JTAG_SEQ response buffer. */
-       unsigned first;
+       unsigned int first;
        /** Number of bits to read. */
-       unsigned length;
+       unsigned int length;
        /** Location to store the result */
        uint8_t *buffer;
        /** Offset in the destination buffer */
-       unsigned buffer_offset;
+       unsigned int buffer_offset;
 };
 
-/* Up to MIN(packet_count, MAX_PENDING_REQUESTS) requests may be issued
- * until the first response arrives */
-#define MAX_PENDING_REQUESTS 3
-
-/* Pending requests are organized as a FIFO - circular buffer */
 /* Each block in FIFO can contain up to pending_queue_len transfers */
-static int pending_queue_len;
-static struct pending_request_block pending_fifo[MAX_PENDING_REQUESTS];
-static int pending_fifo_put_idx, pending_fifo_get_idx;
-static int pending_fifo_block_count;
+static unsigned int pending_queue_len;
+static unsigned int tfer_max_command_size;
+static unsigned int tfer_max_response_size;
 
 /* pointers to buffers that will receive jtag scan results on the next flush */
 #define MAX_PENDING_SCAN_RESULTS 256
@@ -208,7 +231,7 @@ static int pending_scan_result_count;
 static struct pending_scan_result pending_scan_results[MAX_PENDING_SCAN_RESULTS];
 
 /* queued JTAG sequences that will be executed on the next flush */
-#define QUEUED_SEQ_BUF_LEN (cmsis_dap_handle->packet_size - 3)
+#define QUEUED_SEQ_BUF_LEN (cmsis_dap_handle->packet_usable_size - 3)
 static int queued_seq_count;
 static int queued_seq_buf_end;
 static int queued_seq_tdo_ptr;
@@ -228,7 +251,7 @@ static int cmsis_dap_open(void)
        const struct cmsis_dap_backend *backend = NULL;
 
        struct cmsis_dap *dap = calloc(1, sizeof(struct cmsis_dap));
-       if (dap == NULL) {
+       if (!dap) {
                LOG_ERROR("unable to allocate memory");
                return ERROR_FAIL;
        }
@@ -236,20 +259,20 @@ static int cmsis_dap_open(void)
        if (cmsis_dap_backend >= 0) {
                /* Use forced backend */
                backend = cmsis_dap_backends[cmsis_dap_backend];
-               if (backend->open(dap, cmsis_dap_vid, cmsis_dap_pid, cmsis_dap_serial) != ERROR_OK)
+               if (backend->open(dap, cmsis_dap_vid, cmsis_dap_pid, adapter_get_required_serial()) != ERROR_OK)
                        backend = NULL;
        } else {
                /* Try all backends */
                for (unsigned int i = 0; i < ARRAY_SIZE(cmsis_dap_backends); i++) {
                        backend = cmsis_dap_backends[i];
-                       if (backend->open(dap, cmsis_dap_vid, cmsis_dap_pid, cmsis_dap_serial) == ERROR_OK)
+                       if (backend->open(dap, cmsis_dap_vid, cmsis_dap_pid, adapter_get_required_serial()) == ERROR_OK)
                                break;
                        else
                                backend = NULL;
                }
        }
 
-       if (backend == NULL) {
+       if (!backend) {
                LOG_ERROR("unable to find a matching CMSIS-DAP device");
                free(dap);
                return ERROR_FAIL;
@@ -269,42 +292,56 @@ static void cmsis_dap_close(struct cmsis_dap *dap)
                dap->backend = NULL;
        }
 
-       free(cmsis_dap_handle->packet_buffer);
+       free(dap->packet_buffer);
+
+       for (unsigned int i = 0; i < MAX_PENDING_REQUESTS; i++) {
+               free(dap->pending_fifo[i].transfers);
+               dap->pending_fifo[i].transfers = NULL;
+       }
+
        free(cmsis_dap_handle);
        cmsis_dap_handle = NULL;
-       free(cmsis_dap_serial);
-       cmsis_dap_serial = NULL;
+}
 
-       for (int i = 0; i < MAX_PENDING_REQUESTS; i++) {
-               free(pending_fifo[i].transfers);
-               pending_fifo[i].transfers = NULL;
+static void cmsis_dap_flush_read(struct cmsis_dap *dap)
+{
+       unsigned int i;
+       /* Some CMSIS-DAP adapters keep buffered packets over
+        * USB close/open so we need to flush up to 64 old packets
+        * to be sure all buffers are empty */
+       for (i = 0; i < 64; i++) {
+               int retval = dap->backend->read(dap, 10);
+               if (retval == ERROR_TIMEOUT_REACHED)
+                       break;
        }
+       if (i)
+               LOG_DEBUG("Flushed %u packets", i);
 }
 
 /* Send a message and receive the reply */
 static int cmsis_dap_xfer(struct cmsis_dap *dap, int txlen)
 {
-       if (pending_fifo_block_count) {
-               LOG_ERROR("pending %d blocks, flushing", pending_fifo_block_count);
-               while (pending_fifo_block_count) {
+       if (dap->pending_fifo_block_count) {
+               LOG_ERROR("pending %u blocks, flushing", dap->pending_fifo_block_count);
+               while (dap->pending_fifo_block_count) {
                        dap->backend->read(dap, 10);
-                       pending_fifo_block_count--;
+                       dap->pending_fifo_block_count--;
                }
-               pending_fifo_put_idx = 0;
-               pending_fifo_get_idx = 0;
+               dap->pending_fifo_put_idx = 0;
+               dap->pending_fifo_get_idx = 0;
        }
 
-       uint8_t current_cmd = cmsis_dap_handle->command[0];
-       int retval = dap->backend->write(dap, txlen, USB_TIMEOUT);
+       uint8_t current_cmd = dap->command[0];
+       int retval = dap->backend->write(dap, txlen, LIBUSB_TIMEOUT_MS);
        if (retval < 0)
                return retval;
 
        /* get reply */
-       retval = dap->backend->read(dap, USB_TIMEOUT);
+       retval = dap->backend->read(dap, LIBUSB_TIMEOUT_MS);
        if (retval < 0)
                return retval;
 
-       uint8_t *resp = cmsis_dap_handle->response;
+       uint8_t *resp = dap->response;
        if (resp[0] == DAP_ERROR) {
                LOG_ERROR("CMSIS-DAP command 0x%" PRIx8 " not implemented", current_cmd);
                return ERROR_NOT_IMPLEMENTED;
@@ -313,15 +350,16 @@ static int cmsis_dap_xfer(struct cmsis_dap *dap, int txlen)
        if (resp[0] != current_cmd) {
                LOG_ERROR("CMSIS-DAP command mismatch. Sent 0x%" PRIx8
                         " received 0x%" PRIx8, current_cmd, resp[0]);
+
+               cmsis_dap_flush_read(dap);
                return ERROR_FAIL;
        }
 
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_SWJ_Pins(uint8_t pins, uint8_t mask, uint32_t delay, uint8_t *input)
+static int cmsis_dap_cmd_dap_swj_pins(uint8_t pins, uint8_t mask, uint32_t delay, uint8_t *input)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_SWJ_PINS;
@@ -329,8 +367,7 @@ static int cmsis_dap_cmd_DAP_SWJ_Pins(uint8_t pins, uint8_t mask, uint32_t delay
        command[2] = mask;
        h_u32_to_le(&command[3], delay);
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 7);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 7);
        if (retval != ERROR_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_DAP_SWJ_PINS failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -342,9 +379,8 @@ static int cmsis_dap_cmd_DAP_SWJ_Pins(uint8_t pins, uint8_t mask, uint32_t delay
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_SWJ_Clock(uint32_t swj_clock)
+static int cmsis_dap_cmd_dap_swj_clock(uint32_t swj_clock)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        /* set clock in Hz */
@@ -353,8 +389,7 @@ static int cmsis_dap_cmd_DAP_SWJ_Clock(uint32_t swj_clock)
        command[0] = CMD_DAP_SWJ_CLOCK;
        h_u32_to_le(&command[1], swj_clock);
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 5);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 5);
        if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_DAP_SWJ_CLOCK failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -364,14 +399,13 @@ static int cmsis_dap_cmd_DAP_SWJ_Clock(uint32_t swj_clock)
 }
 
 /* clock a sequence of bits out on TMS, to change JTAG states */
-static int cmsis_dap_cmd_DAP_SWJ_Sequence(uint8_t s_len, const uint8_t *sequence)
+static int cmsis_dap_cmd_dap_swj_sequence(uint8_t s_len, const uint8_t *sequence)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
 #ifdef CMSIS_DAP_JTAG_DEBUG
        LOG_DEBUG("cmsis-dap TMS sequence: len=%d", s_len);
-       for (int i = 0; i < DIV_ROUND_UP(s_len, 8); ++i)
+       for (unsigned int i = 0; i < DIV_ROUND_UP(s_len, 8); ++i)
                printf("%02X ", sequence[i]);
 
        printf("\n");
@@ -381,23 +415,21 @@ static int cmsis_dap_cmd_DAP_SWJ_Sequence(uint8_t s_len, const uint8_t *sequence
        command[1] = s_len;
        bit_copy(&command[2], 0, sequence, 0, s_len);
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 2 + DIV_ROUND_UP(s_len, 8));
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 2 + DIV_ROUND_UP(s_len, 8));
        if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK)
                return ERROR_FAIL;
 
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_Info(uint8_t info, uint8_t **data)
+static int cmsis_dap_cmd_dap_info(uint8_t info, uint8_t **data)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_INFO;
        command[1] = info;
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
        if (retval != ERROR_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_INFO failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -408,17 +440,15 @@ static int cmsis_dap_cmd_DAP_Info(uint8_t info, uint8_t **data)
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_LED(uint8_t led, uint8_t state)
+static int cmsis_dap_cmd_dap_led(uint8_t led, uint8_t state)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_LED;
        command[1] = led;
        command[2] = state;
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 3);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 3);
        if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_LED failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -427,16 +457,14 @@ static int cmsis_dap_cmd_DAP_LED(uint8_t led, uint8_t state)
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_Connect(uint8_t mode)
+static int cmsis_dap_cmd_dap_connect(uint8_t mode)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_CONNECT;
        command[1] = mode;
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
        if (retval != ERROR_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_CONNECT failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -450,15 +478,13 @@ static int cmsis_dap_cmd_DAP_Connect(uint8_t mode)
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_Disconnect(void)
+static int cmsis_dap_cmd_dap_disconnect(void)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_DISCONNECT;
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 1);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 1);
        if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_DISCONNECT failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -467,9 +493,8 @@ static int cmsis_dap_cmd_DAP_Disconnect(void)
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_TFER_Configure(uint8_t idle, uint16_t retry_count, uint16_t match_retry)
+static int cmsis_dap_cmd_dap_tfer_configure(uint8_t idle, uint16_t retry_count, uint16_t match_retry)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_TFER_CONFIGURE;
@@ -477,8 +502,7 @@ static int cmsis_dap_cmd_DAP_TFER_Configure(uint8_t idle, uint16_t retry_count,
        h_u16_to_le(&command[2], retry_count);
        h_u16_to_le(&command[4], match_retry);
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 6);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 6);
        if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_TFER_Configure failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -487,16 +511,14 @@ static int cmsis_dap_cmd_DAP_TFER_Configure(uint8_t idle, uint16_t retry_count,
        return ERROR_OK;
 }
 
-static int cmsis_dap_cmd_DAP_SWD_Configure(uint8_t cfg)
+static int cmsis_dap_cmd_dap_swd_configure(uint8_t cfg)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_SWD_CONFIGURE;
        command[1] = cfg;
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
        if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_SWD_Configure failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -506,16 +528,14 @@ static int cmsis_dap_cmd_DAP_SWD_Configure(uint8_t cfg)
 }
 
 #if 0
-static int cmsis_dap_cmd_DAP_Delay(uint16_t delay_us)
+static int cmsis_dap_cmd_dap_delay(uint16_t delay_us)
 {
-       int retval;
        uint8_t *command = cmsis_dap_handle->command;
 
        command[0] = CMD_DAP_DELAY;
        h_u16_to_le(&command[1], delay_us);
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, 3);
-
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 3);
        if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
                LOG_ERROR("CMSIS-DAP command CMD_Delay failed.");
                return ERROR_JTAG_DEVICE_ERROR;
@@ -525,12 +545,219 @@ static int cmsis_dap_cmd_DAP_Delay(uint16_t delay_us)
 }
 #endif
 
-static void cmsis_dap_swd_write_from_queue(struct cmsis_dap *dap)
+static int cmsis_dap_metacmd_targetsel(uint32_t instance_id)
 {
        uint8_t *command = cmsis_dap_handle->command;
-       struct pending_request_block *block = &pending_fifo[pending_fifo_put_idx];
+       const uint32_t SEQ_RD = 0x80, SEQ_WR = 0x00;
+
+       /* SWD multi-drop requires a transfer ala CMD_DAP_TFER,
+       but with no expectation of an SWD ACK response.  In
+       CMSIS-DAP v1.20 and v2.00, CMD_DAP_SWD_SEQUENCE was
+       added to allow this special sequence to be generated.
+       The purpose of this operation is to select the target
+       corresponding to the instance_id that is written */
+
+       LOG_DEBUG_IO("DP write reg TARGETSEL %" PRIx32, instance_id);
+
+       size_t idx = 0;
+       command[idx++] = CMD_DAP_SWD_SEQUENCE;
+       command[idx++] = 3;     /* sequence count */
+
+       /* sequence 0: packet request for TARGETSEL */
+       command[idx++] = SEQ_WR | 8;
+       command[idx++] = SWD_CMD_START | swd_cmd(false, false, DP_TARGETSEL) | SWD_CMD_STOP | SWD_CMD_PARK;
+
+       /* sequence 1: read Trn ACK Trn, no expectation for target to ACK  */
+       command[idx++] = SEQ_RD | 5;
+
+       /* sequence 2: WDATA plus parity */
+       command[idx++] = SEQ_WR | (32 + 1);
+       h_u32_to_le(command + idx, instance_id);
+       idx += 4;
+       command[idx++] = parity_u32(instance_id);
+
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, idx);
+       if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
+               LOG_ERROR("CMSIS-DAP command SWD_Sequence failed.");
+               return ERROR_JTAG_DEVICE_ERROR;
+       }
+
+       return ERROR_OK;
+}
+
+/**
+ * Sets the SWO transport mode.
+ * @param[in] transport     The transport mode. Can be None, SWO_Data or
+ *                          WinUSB (requires CMSIS-DAP v2).
+ */
+static int cmsis_dap_cmd_dap_swo_transport(uint8_t transport)
+{
+       uint8_t *command = cmsis_dap_handle->command;
+
+       command[0] = CMD_DAP_SWO_TRANSPORT;
+       command[1] = transport;
+
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
+       if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
+               LOG_ERROR("CMSIS-DAP: command CMD_SWO_Transport(%d) failed.", transport);
+               return ERROR_JTAG_DEVICE_ERROR;
+       }
+
+       return ERROR_OK;
+}
+
+/**
+ * Sets the SWO trace capture mode.
+ * @param[in] mode          Trace capture mode. Can be UART or MANCHESTER.
+ */
+static int cmsis_dap_cmd_dap_swo_mode(uint8_t mode)
+{
+       uint8_t *command = cmsis_dap_handle->command;
+
+       command[0] = CMD_DAP_SWO_MODE;
+       command[1] = mode;
+
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
+       if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
+               LOG_ERROR("CMSIS-DAP: command CMD_SWO_Mode(%d) failed.", mode);
+               return ERROR_JTAG_DEVICE_ERROR;
+       }
+
+       return ERROR_OK;
+}
+
+/**
+ * Sets the baudrate for capturing SWO trace data.
+ * Can be called iteratively to determine supported baudrates.
+ * @param[in]  in_baudrate  Requested baudrate.
+ * @param[out] dev_baudrate Actual baudrate or 0 (baudrate not configured).
+ *                          When requested baudrate is not achievable the
+ *                          closest configured baudrate can be returned or
+ *                          0 which indicates that baudrate was not configured.
+ */
+static int cmsis_dap_cmd_dap_swo_baudrate(
+                                       uint32_t in_baudrate,
+                                       uint32_t *dev_baudrate)
+{
+       uint8_t *command = cmsis_dap_handle->command;
+
+       command[0] = CMD_DAP_SWO_BAUDRATE;
+       h_u32_to_le(&command[1], in_baudrate);
+
+       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);
+               if (dev_baudrate)
+                       *dev_baudrate = 0;
+               return ERROR_JTAG_DEVICE_ERROR;
+       }
+
+       if (dev_baudrate)
+               *dev_baudrate = rvbr;
+
+       return ERROR_OK;
+}
+
+/**
+ * Controls the SWO trace data capture.
+ * @param[in] control       Start or stop a trace. Starting capture automatically
+ *                          flushes any existing trace data in buffers which has
+ *                          not yet been read.
+ */
+static int cmsis_dap_cmd_dap_swo_control(uint8_t control)
+{
+       uint8_t *command = cmsis_dap_handle->command;
+
+       command[0] = CMD_DAP_SWO_CONTROL;
+       command[1] = control;
+
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 2);
+       if (retval != ERROR_OK || cmsis_dap_handle->response[1] != DAP_OK) {
+               LOG_ERROR("CMSIS-DAP: command CMD_SWO_Control(%d) failed.", control);
+               return ERROR_JTAG_DEVICE_ERROR;
+       }
+
+       return ERROR_OK;
+}
+
+/**
+ * Reads the SWO trace status.
+ * @param[out] trace_status The trace's status.
+ *                          Bit0: Trace Capture (1 - active, 0 - inactive).
+ *                          Bit6: Trace Stream Error.
+ *                          Bit7: Trace Buffer Overrun.
+ * @param[out] trace_count  Number of bytes in Trace Buffer (not yet read).
+ */
+static int cmsis_dap_cmd_dap_swo_status(
+                                       uint8_t *trace_status,
+                                       size_t *trace_count)
+{
+       uint8_t *command = cmsis_dap_handle->command;
+
+       command[0] = CMD_DAP_SWO_STATUS;
+
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 1);
+       if (retval != ERROR_OK) {
+               LOG_ERROR("CMSIS-DAP: command CMD_SWO_Status failed.");
+               return ERROR_JTAG_DEVICE_ERROR;
+       }
+
+       if (trace_status)
+               *trace_status = cmsis_dap_handle->response[1];
+       if (trace_count)
+               *trace_count = le_to_h_u32(&cmsis_dap_handle->response[2]);
+
+       return ERROR_OK;
+}
+
+/**
+ * Reads the captured SWO trace data from Trace Buffer.
+ * @param[in]  max_trace_count Maximum number of Trace Data bytes to read.
+ * @param[out] trace_status The trace's status.
+ * @param[out] trace_count  Number of Trace Data bytes read.
+ * @param[out] data         Trace Data bytes read.
+ */
+static int cmsis_dap_cmd_dap_swo_data(
+                                       size_t max_trace_count,
+                                       uint8_t *trace_status,
+                                       size_t *trace_count,
+                                       uint8_t *data)
+{
+       uint8_t *command = cmsis_dap_handle->command;
+
+       command[0] = CMD_DAP_SWO_DATA;
+       h_u16_to_le(&command[1], max_trace_count);
+
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, 3);
+       if (retval != ERROR_OK) {
+               LOG_ERROR("CMSIS-DAP: command CMD_SWO_Data failed.");
+               return ERROR_JTAG_DEVICE_ERROR;
+       }
+
+       *trace_status = cmsis_dap_handle->response[1];
+       *trace_count = le_to_h_u16(&cmsis_dap_handle->response[2]);
+
+       if (*trace_count > 0)
+               memcpy(data, &cmsis_dap_handle->response[4], *trace_count);
+
+       return ERROR_OK;
+}
 
-       LOG_DEBUG_IO("Executing %d queued transactions from FIFO index %d", block->transfer_count, pending_fifo_put_idx);
+
+static void cmsis_dap_swd_write_from_queue(struct cmsis_dap *dap)
+{
+       uint8_t *command = dap->command;
+       struct pending_request_block *block = &dap->pending_fifo[dap->pending_fifo_put_idx];
+
+       assert(dap->write_count + dap->read_count == block->transfer_count);
+
+       /* Reset packet size check counters for the next packet */
+       dap->write_count = 0;
+       dap->read_count = 0;
+
+       LOG_DEBUG_IO("Executing %d queued transactions from FIFO index %u",
+                         block->transfer_count, dap->pending_fifo_put_idx);
 
        if (queued_retval != ERROR_OK) {
                LOG_DEBUG("Skipping due to previous errors: %d", queued_retval);
@@ -545,14 +772,14 @@ static void cmsis_dap_swd_write_from_queue(struct cmsis_dap *dap)
        command[2] = block->transfer_count;
        size_t idx = 3;
 
-       for (int i = 0; i < block->transfer_count; i++) {
+       for (unsigned int i = 0; i < block->transfer_count; i++) {
                struct pending_transfer_result *transfer = &(block->transfers[i]);
                uint8_t cmd = transfer->cmd;
                uint32_t data = transfer->data;
 
-               LOG_DEBUG_IO("%s %s reg %x %"PRIx32,
-                               cmd & SWD_CMD_APnDP ? "AP" : "DP",
-                               cmd & SWD_CMD_RnW ? "read" : "write",
+               LOG_DEBUG_IO("%s %s reg %x %" PRIx32,
+                               cmd & SWD_CMD_APNDP ? "AP" : "DP",
+                               cmd & SWD_CMD_RNW ? "read" : "write",
                          (cmd & SWD_CMD_A32) >> 1, data);
 
                /* When proper WAIT handling is implemented in the
@@ -562,12 +789,12 @@ static void cmsis_dap_swd_write_from_queue(struct cmsis_dap *dap)
                 * able to automatically retry anything (because ARM
                 * has forgotten to implement sticky error flags
                 * clearing). See also comments regarding
-                * cmsis_dap_cmd_DAP_TFER_Configure() and
-                * cmsis_dap_cmd_DAP_SWD_Configure() in
+                * cmsis_dap_cmd_dap_tfer_configure() and
+                * cmsis_dap_cmd_dap_swd_configure() in
                 * cmsis_dap_init().
                 */
-               if (!(cmd & SWD_CMD_RnW) &&
-                   !(cmd & SWD_CMD_APnDP) &&
+               if (!(cmd & SWD_CMD_RNW) &&
+                   !(cmd & SWD_CMD_APNDP) &&
                    (cmd & SWD_CMD_A32) >> 1 == DP_CTRL_STAT &&
                    (data & CORUNDETECT)) {
                        LOG_DEBUG("refusing to enable sticky overrun detection");
@@ -575,14 +802,13 @@ static void cmsis_dap_swd_write_from_queue(struct cmsis_dap *dap)
                }
 
                command[idx++] = (cmd >> 1) & 0x0f;
-               if (!(cmd & SWD_CMD_RnW)) {
+               if (!(cmd & SWD_CMD_RNW)) {
                        h_u32_to_le(&command[idx], data);
                        idx += 4;
                }
        }
 
-       int retval = dap->backend->write(dap, idx, USB_TIMEOUT);
-
+       int retval = dap->backend->write(dap, idx, LIBUSB_TIMEOUT_MS);
        if (retval < 0) {
                queued_retval = retval;
                goto skip;
@@ -590,10 +816,10 @@ static void cmsis_dap_swd_write_from_queue(struct cmsis_dap *dap)
                queued_retval = ERROR_OK;
        }
 
-       pending_fifo_put_idx = (pending_fifo_put_idx + 1) % dap->packet_count;
-       pending_fifo_block_count++;
-       if (pending_fifo_block_count > dap->packet_count)
-               LOG_ERROR("too much pending writes %d", pending_fifo_block_count);
+       dap->pending_fifo_put_idx = (dap->pending_fifo_put_idx + 1) % dap->packet_count;
+       dap->pending_fifo_block_count++;
+       if (dap->pending_fifo_block_count > dap->packet_count)
+               LOG_ERROR("too much pending writes %u", dap->pending_fifo_block_count);
 
        return;
 
@@ -603,14 +829,14 @@ skip:
 
 static void cmsis_dap_swd_read_process(struct cmsis_dap *dap, int timeout_ms)
 {
-       struct pending_request_block *block = &pending_fifo[pending_fifo_get_idx];
+       struct pending_request_block *block = &dap->pending_fifo[dap->pending_fifo_get_idx];
 
-       if (pending_fifo_block_count == 0)
+       if (dap->pending_fifo_block_count == 0)
                LOG_ERROR("no pending write");
 
        /* get reply */
        int retval = dap->backend->read(dap, timeout_ms);
-       if (retval == ERROR_TIMEOUT_REACHED && timeout_ms < USB_TIMEOUT)
+       if (retval == ERROR_TIMEOUT_REACHED && timeout_ms < LIBUSB_TIMEOUT_MS)
                return;
 
        if (retval <= 0) {
@@ -621,13 +847,13 @@ static void cmsis_dap_swd_read_process(struct cmsis_dap *dap, int timeout_ms)
 
        uint8_t *resp = dap->response;
        if (resp[0] != CMD_DAP_TFER) {
-               LOG_ERROR("CMSIS-DAP command mismatch. Expected 0x%" PRIx8
-                        " received 0x%" PRIx8, CMD_DAP_TFER, resp[0]);
+               LOG_ERROR("CMSIS-DAP command mismatch. Expected 0x%x received 0x%" PRIx8,
+                       CMD_DAP_TFER, resp[0]);
                queued_retval = ERROR_FAIL;
                goto skip;
        }
 
-       uint8_t transfer_count = resp[1];
+       unsigned int transfer_count = resp[1];
        uint8_t ack = resp[2] & 0x07;
        if (resp[2] & 0x08) {
                LOG_DEBUG("CMSIS-DAP Protocol Error @ %d (wrong parity)", transfer_count);
@@ -646,21 +872,21 @@ static void cmsis_dap_swd_read_process(struct cmsis_dap *dap, int timeout_ms)
                LOG_ERROR("CMSIS-DAP transfer count mismatch: expected %d, got %d",
                          block->transfer_count, transfer_count);
 
-       LOG_DEBUG_IO("Received results of %d queued transactions FIFO index %d",
-                transfer_count, pending_fifo_get_idx);
-       size_t idx = 3;
-       for (int i = 0; i < transfer_count; i++) {
+       LOG_DEBUG_IO("Received results of %d queued transactions FIFO index %u timeout %i",
+                transfer_count, dap->pending_fifo_get_idx, timeout_ms);
+       unsigned int idx = 3;
+       for (unsigned int i = 0; i < transfer_count; i++) {
                struct pending_transfer_result *transfer = &(block->transfers[i]);
-               if (transfer->cmd & SWD_CMD_RnW) {
+               if (transfer->cmd & SWD_CMD_RNW) {
                        static uint32_t last_read;
                        uint32_t data = le_to_h_u32(&resp[idx]);
                        uint32_t tmp = data;
                        idx += 4;
 
-                       LOG_DEBUG_IO("Read result: %"PRIx32, data);
+                       LOG_DEBUG_IO("Read result: %" PRIx32, data);
 
                        /* Imitate posted AP reads */
-                       if ((transfer->cmd & SWD_CMD_APnDP) ||
+                       if ((transfer->cmd & SWD_CMD_APNDP) ||
                            ((transfer->cmd & SWD_CMD_A32) >> 1 == DP_RDBUFF)) {
                                tmp = last_read;
                                last_read = data;
@@ -673,22 +899,22 @@ static void cmsis_dap_swd_read_process(struct cmsis_dap *dap, int timeout_ms)
 
 skip:
        block->transfer_count = 0;
-       pending_fifo_get_idx = (pending_fifo_get_idx + 1) % dap->packet_count;
-       pending_fifo_block_count--;
+       dap->pending_fifo_get_idx = (dap->pending_fifo_get_idx + 1) % dap->packet_count;
+       dap->pending_fifo_block_count--;
 }
 
 static int cmsis_dap_swd_run_queue(void)
 {
-       if (pending_fifo_block_count)
+       if (cmsis_dap_handle->pending_fifo_block_count)
                cmsis_dap_swd_read_process(cmsis_dap_handle, 0);
 
        cmsis_dap_swd_write_from_queue(cmsis_dap_handle);
 
-       while (pending_fifo_block_count)
-               cmsis_dap_swd_read_process(cmsis_dap_handle, USB_TIMEOUT);
+       while (cmsis_dap_handle->pending_fifo_block_count)
+               cmsis_dap_swd_read_process(cmsis_dap_handle, LIBUSB_TIMEOUT_MS);
 
-       pending_fifo_put_idx = 0;
-       pending_fifo_get_idx = 0;
+       cmsis_dap_handle->pending_fifo_put_idx = 0;
+       cmsis_dap_handle->pending_fifo_get_idx = 0;
 
        int retval = queued_retval;
        queued_retval = ERROR_OK;
@@ -696,42 +922,87 @@ static int cmsis_dap_swd_run_queue(void)
        return retval;
 }
 
+static unsigned int cmsis_dap_tfer_cmd_size(unsigned int write_count,
+                                                                                       unsigned int read_count)
+{
+       unsigned int size = 3;                  /* header */
+       size += write_count * (1 + 4);  /* DAP register + data */
+       size += read_count;                             /* DAP register */
+       return size;
+}
+
+static unsigned int cmsis_dap_tfer_resp_size(unsigned int write_count,
+                                                                                        unsigned int read_count)
+{
+       unsigned int size = 3;                  /* header */
+       size += read_count * 4;                 /* data */
+       return size;
+}
+
 static void cmsis_dap_swd_queue_cmd(uint8_t cmd, uint32_t *dst, uint32_t data)
 {
-       if (pending_fifo[pending_fifo_put_idx].transfer_count == pending_queue_len) {
-               if (pending_fifo_block_count)
+       /* Compute sizes of the DAP Transfer command and the expected response
+        * for all queued and this operation */
+       bool targetsel_cmd = swd_cmd(false, false, DP_TARGETSEL) == cmd;
+
+       unsigned int write_count = cmsis_dap_handle->write_count;
+       unsigned int read_count = cmsis_dap_handle->read_count;
+       if (cmd & SWD_CMD_RNW)
+               read_count++;
+       else
+               write_count++;
+
+       unsigned int cmd_size = cmsis_dap_tfer_cmd_size(write_count, read_count);
+       unsigned int resp_size = cmsis_dap_tfer_resp_size(write_count, read_count);
+
+       /* Does the DAP Transfer command and the expected response fit into one packet?
+        * Run the queue also before a targetsel - it cannot be queued */
+       if (cmd_size > tfer_max_command_size
+                       || resp_size > tfer_max_response_size
+                       || targetsel_cmd) {
+               if (cmsis_dap_handle->pending_fifo_block_count)
                        cmsis_dap_swd_read_process(cmsis_dap_handle, 0);
 
                /* Not enough room in the queue. Run the queue. */
                cmsis_dap_swd_write_from_queue(cmsis_dap_handle);
 
-               if (pending_fifo_block_count >= cmsis_dap_handle->packet_count)
-                       cmsis_dap_swd_read_process(cmsis_dap_handle, USB_TIMEOUT);
+               if (cmsis_dap_handle->pending_fifo_block_count >= cmsis_dap_handle->packet_count)
+                       cmsis_dap_swd_read_process(cmsis_dap_handle, LIBUSB_TIMEOUT_MS);
        }
 
+       assert(cmsis_dap_handle->pending_fifo[cmsis_dap_handle->pending_fifo_put_idx].transfer_count < pending_queue_len);
+
        if (queued_retval != ERROR_OK)
                return;
 
-       struct pending_request_block *block = &pending_fifo[pending_fifo_put_idx];
+       if (targetsel_cmd) {
+               cmsis_dap_metacmd_targetsel(data);
+               return;
+       }
+
+       struct pending_request_block *block = &cmsis_dap_handle->pending_fifo[cmsis_dap_handle->pending_fifo_put_idx];
        struct pending_transfer_result *transfer = &(block->transfers[block->transfer_count]);
        transfer->data = data;
        transfer->cmd = cmd;
-       if (cmd & SWD_CMD_RnW) {
+       if (cmd & SWD_CMD_RNW) {
                /* Queue a read transaction */
                transfer->buffer = dst;
+               cmsis_dap_handle->read_count++;
+       } else {
+               cmsis_dap_handle->write_count++;
        }
        block->transfer_count++;
 }
 
 static void cmsis_dap_swd_write_reg(uint8_t cmd, uint32_t value, uint32_t ap_delay_clk)
 {
-       assert(!(cmd & SWD_CMD_RnW));
+       assert(!(cmd & SWD_CMD_RNW));
        cmsis_dap_swd_queue_cmd(cmd, NULL, value);
 }
 
 static void cmsis_dap_swd_read_reg(uint8_t cmd, uint32_t *value, uint32_t ap_delay_clk)
 {
-       assert(cmd & SWD_CMD_RnW);
+       assert(cmd & SWD_CMD_RNW);
        cmsis_dap_swd_queue_cmd(cmd, value, 0);
 }
 
@@ -739,7 +1010,7 @@ static int cmsis_dap_get_serial_info(void)
 {
        uint8_t *data;
 
-       int retval = cmsis_dap_cmd_DAP_Info(INFO_ID_SERNUM, &data);
+       int retval = cmsis_dap_cmd_dap_info(INFO_ID_SERNUM, &data);
        if (retval != ERROR_OK)
                return retval;
 
@@ -754,7 +1025,7 @@ static int cmsis_dap_get_version_info(void)
        uint8_t *data;
 
        /* INFO_ID_FW_VER - string */
-       int retval = cmsis_dap_cmd_DAP_Info(INFO_ID_FW_VER, &data);
+       int retval = cmsis_dap_cmd_dap_info(INFO_ID_FW_VER, &data);
        if (retval != ERROR_OK)
                return retval;
 
@@ -769,29 +1040,50 @@ static int cmsis_dap_get_caps_info(void)
        uint8_t *data;
 
        /* INFO_ID_CAPS - byte */
-       int retval = cmsis_dap_cmd_DAP_Info(INFO_ID_CAPS, &data);
+       int retval = cmsis_dap_cmd_dap_info(INFO_ID_CAPS, &data);
        if (retval != ERROR_OK)
                return retval;
 
-       if (data[0] == 1) {
-               uint8_t caps = data[1];
+       if (data[0] == 1 || data[0] == 2) {
+               uint16_t caps = data[1];
+               if (data[0] == 2)
+                       caps |= (uint16_t)data[2] << 8;
 
                cmsis_dap_handle->caps = caps;
 
-               if (caps & INFO_CAPS_SWD)
-                       LOG_INFO("CMSIS-DAP: %s", info_caps_str[0]);
-               if (caps & INFO_CAPS_JTAG)
-                       LOG_INFO("CMSIS-DAP: %s", info_caps_str[1]);
+               for (unsigned int i = 0; i < INFO_CAPS__NUM_CAPS; ++i) {
+                       if (caps & BIT(i))
+                               LOG_INFO("CMSIS-DAP: %s", info_caps_str[i]);
+               }
        }
 
        return ERROR_OK;
 }
 
+static int cmsis_dap_get_swo_buf_sz(uint32_t *swo_buf_sz)
+{
+       uint8_t *data;
+
+       /* INFO_ID_SWO_BUF_SZ - word */
+       int retval = cmsis_dap_cmd_dap_info(INFO_ID_SWO_BUF_SZ, &data);
+       if (retval != ERROR_OK)
+               return retval;
+
+       if (data[0] != 4)
+               return ERROR_FAIL;
+
+       *swo_buf_sz = le_to_h_u32(&data[1]);
+
+       LOG_INFO("CMSIS-DAP: SWO Trace Buffer Size = %u bytes", *swo_buf_sz);
+
+       return ERROR_OK;
+}
+
 static int cmsis_dap_get_status(void)
 {
        uint8_t d;
 
-       int retval = cmsis_dap_cmd_DAP_SWJ_Pins(0, 0, 0, &d);
+       int retval = cmsis_dap_cmd_dap_swj_pins(0, 0, 0, &d);
 
        if (retval == ERROR_OK) {
                LOG_INFO("SWCLK/TCK = %d SWDIO/TMS = %d TDI = %d TDO = %d nTRST = %d nRESET = %d",
@@ -818,18 +1110,18 @@ static int cmsis_dap_swd_switch_seq(enum swd_special_seq seq)
                 * Reconnecting would break connecting under reset. */
 
                /* First disconnect before connecting, Atmel EDBG needs it for SAMD/R/L/C */
-               cmsis_dap_cmd_DAP_Disconnect();
+               cmsis_dap_cmd_dap_disconnect();
 
                /* When we are reconnecting, DAP_Connect needs to be rerun, at
                 * least on Keil ULINK-ME */
-               retval = cmsis_dap_cmd_DAP_Connect(CONNECT_SWD);
+               retval = cmsis_dap_cmd_dap_connect(CONNECT_SWD);
                if (retval != ERROR_OK)
                        return retval;
        }
 
        switch (seq) {
        case LINE_RESET:
-               LOG_DEBUG("SWD line reset");
+               LOG_DEBUG_IO("SWD line reset");
                s = swd_seq_line_reset;
                s_len = swd_seq_line_reset_len;
                break;
@@ -838,35 +1130,53 @@ static int cmsis_dap_swd_switch_seq(enum swd_special_seq seq)
                s = swd_seq_jtag_to_swd;
                s_len = swd_seq_jtag_to_swd_len;
                break;
+       case JTAG_TO_DORMANT:
+               LOG_DEBUG("JTAG-to-DORMANT");
+               s = swd_seq_jtag_to_dormant;
+               s_len = swd_seq_jtag_to_dormant_len;
+               break;
        case SWD_TO_JTAG:
                LOG_DEBUG("SWD-to-JTAG");
                s = swd_seq_swd_to_jtag;
                s_len = swd_seq_swd_to_jtag_len;
                break;
+       case SWD_TO_DORMANT:
+               LOG_DEBUG("SWD-to-DORMANT");
+               s = swd_seq_swd_to_dormant;
+               s_len = swd_seq_swd_to_dormant_len;
+               break;
+       case DORMANT_TO_SWD:
+               LOG_DEBUG("DORMANT-to-SWD");
+               s = swd_seq_dormant_to_swd;
+               s_len = swd_seq_dormant_to_swd_len;
+               break;
+       case DORMANT_TO_JTAG:
+               LOG_DEBUG("DORMANT-to-JTAG");
+               s = swd_seq_dormant_to_jtag;
+               s_len = swd_seq_dormant_to_jtag_len;
+               break;
        default:
                LOG_ERROR("Sequence %d not supported", seq);
                return ERROR_FAIL;
        }
 
-       retval = cmsis_dap_cmd_DAP_SWJ_Sequence(s_len, s);
+       retval = cmsis_dap_cmd_dap_swj_sequence(s_len, s);
        if (retval != ERROR_OK)
                return retval;
 
        /* Atmel EDBG needs renew clock setting after SWJ_Sequence
         * otherwise default frequency is used */
-       return cmsis_dap_cmd_DAP_SWJ_Clock(jtag_get_speed_khz());
+       return cmsis_dap_cmd_dap_swj_clock(adapter_get_speed_khz());
 }
 
 static int cmsis_dap_swd_open(void)
 {
-       int retval;
-
        if (!(cmsis_dap_handle->caps & INFO_CAPS_SWD)) {
                LOG_ERROR("CMSIS-DAP: SWD not supported");
                return ERROR_JTAG_DEVICE_ERROR;
        }
 
-       retval = cmsis_dap_cmd_DAP_Connect(CONNECT_SWD);
+       int retval = cmsis_dap_cmd_dap_connect(CONNECT_SWD);
        if (retval != ERROR_OK)
                return retval;
 
@@ -878,13 +1188,14 @@ static int cmsis_dap_swd_open(void)
 
 static int cmsis_dap_init(void)
 {
-       int retval;
        uint8_t *data;
 
-       retval = cmsis_dap_open();
+       int retval = cmsis_dap_open();
        if (retval != ERROR_OK)
                return retval;
 
+       cmsis_dap_flush_read(cmsis_dap_handle);
+
        retval = cmsis_dap_get_caps_info();
        if (retval != ERROR_OK)
                return retval;
@@ -908,7 +1219,7 @@ static int cmsis_dap_init(void)
                        return ERROR_JTAG_DEVICE_ERROR;
                }
 
-               retval = cmsis_dap_cmd_DAP_Connect(CONNECT_JTAG);
+               retval = cmsis_dap_cmd_dap_connect(CONNECT_JTAG);
                if (retval != ERROR_OK)
                        return retval;
 
@@ -918,22 +1229,15 @@ static int cmsis_dap_init(void)
        /* Be conservative and suppress submitting multiple HID requests
         * until we get packet count info from the adaptor */
        cmsis_dap_handle->packet_count = 1;
-       pending_queue_len = 12;
 
        /* INFO_ID_PKT_SZ - short */
-       retval = cmsis_dap_cmd_DAP_Info(INFO_ID_PKT_SZ, &data);
+       retval = cmsis_dap_cmd_dap_info(INFO_ID_PKT_SZ, &data);
        if (retval != ERROR_OK)
                goto init_err;
 
        if (data[0] == 2) {  /* short */
                uint16_t pkt_sz = data[1] + (data[2] << 8);
                if (pkt_sz != cmsis_dap_handle->packet_size) {
-
-                       /* 4 bytes of command header + 5 bytes per register
-                        * write. For bulk read sequences just 4 bytes are
-                        * needed per transfer, so this is suboptimal. */
-                       pending_queue_len = (pkt_sz - 4) / 5;
-
                        free(cmsis_dap_handle->packet_buffer);
                        retval = cmsis_dap_handle->backend->packet_buffer_alloc(cmsis_dap_handle, pkt_sz);
                        if (retval != ERROR_OK)
@@ -943,23 +1247,34 @@ static int cmsis_dap_init(void)
                }
        }
 
+       /* Maximal number of transfers which fit to one packet:
+        * Limited by response size: 3 bytes of response header + 4 per read
+        * Plus writes to full command size: 3 bytes cmd header + 1 per read + 5 per write */
+       tfer_max_command_size = cmsis_dap_handle->packet_usable_size;
+       tfer_max_response_size = cmsis_dap_handle->packet_usable_size;
+       unsigned int max_reads = tfer_max_response_size / 4;
+       pending_queue_len = max_reads + (tfer_max_command_size - max_reads) / 5;
+       cmsis_dap_handle->write_count = 0;
+       cmsis_dap_handle->read_count = 0;
+
        /* INFO_ID_PKT_CNT - byte */
-       retval = cmsis_dap_cmd_DAP_Info(INFO_ID_PKT_CNT, &data);
+       retval = cmsis_dap_cmd_dap_info(INFO_ID_PKT_CNT, &data);
        if (retval != ERROR_OK)
                goto init_err;
 
        if (data[0] == 1) { /* byte */
-               int pkt_cnt = data[1];
+               unsigned int pkt_cnt = data[1];
                if (pkt_cnt > 1)
                        cmsis_dap_handle->packet_count = MIN(MAX_PENDING_REQUESTS, pkt_cnt);
 
-               LOG_DEBUG("CMSIS-DAP: Packet Count = %d", pkt_cnt);
+               LOG_DEBUG("CMSIS-DAP: Packet Count = %u", pkt_cnt);
        }
 
-       LOG_DEBUG("Allocating FIFO for %d pending packets", cmsis_dap_handle->packet_count);
-       for (int i = 0; i < cmsis_dap_handle->packet_count; i++) {
-               pending_fifo[i].transfers = malloc(pending_queue_len * sizeof(struct pending_transfer_result));
-               if (!pending_fifo[i].transfers) {
+       LOG_DEBUG("Allocating FIFO for %u pending packets", cmsis_dap_handle->packet_count);
+       for (unsigned int i = 0; i < cmsis_dap_handle->packet_count; i++) {
+               cmsis_dap_handle->pending_fifo[i].transfers = malloc(pending_queue_len
+                                                                        * sizeof(struct pending_transfer_result));
+               if (!cmsis_dap_handle->pending_fifo[i].transfers) {
                        LOG_ERROR("Unable to allocate memory for CMSIS-DAP queue");
                        retval = ERROR_FAIL;
                        goto init_err;
@@ -972,36 +1287,36 @@ static int cmsis_dap_init(void)
 
        /* Now try to connect to the target
         * TODO: This is all SWD only @ present */
-       retval = cmsis_dap_cmd_DAP_SWJ_Clock(jtag_get_speed_khz());
+       retval = cmsis_dap_cmd_dap_swj_clock(adapter_get_speed_khz());
        if (retval != ERROR_OK)
                goto init_err;
 
        /* Ask CMSIS-DAP to automatically retry on receiving WAIT for
         * up to 64 times. This must be changed to 0 if sticky
         * overrun detection is enabled. */
-       retval = cmsis_dap_cmd_DAP_TFER_Configure(0, 64, 0);
+       retval = cmsis_dap_cmd_dap_tfer_configure(0, 64, 0);
        if (retval != ERROR_OK)
                goto init_err;
 
        if (swd_mode) {
                /* Data Phase (bit 2) must be set to 1 if sticky overrun
                 * detection is enabled */
-               retval = cmsis_dap_cmd_DAP_SWD_Configure(0);    /* 1 TRN, no Data Phase */
+               retval = cmsis_dap_cmd_dap_swd_configure(0);    /* 1 TRN, no Data Phase */
                if (retval != ERROR_OK)
                        goto init_err;
        }
        /* Both LEDs on */
        /* Intentionally not checked for error, debugging will work
         * without LEDs */
-       (void)cmsis_dap_cmd_DAP_LED(LED_ID_CONNECT, LED_ON);
-       (void)cmsis_dap_cmd_DAP_LED(LED_ID_RUN, LED_ON);
+       (void)cmsis_dap_cmd_dap_led(LED_ID_CONNECT, LED_ON);
+       (void)cmsis_dap_cmd_dap_led(LED_ID_RUN, LED_ON);
 
        /* support connecting with srst asserted */
        enum reset_types jtag_reset_config = jtag_get_reset_config();
 
        if (jtag_reset_config & RESET_CNCT_UNDER_SRST) {
                if (jtag_reset_config & RESET_SRST_NO_GATING) {
-                       retval = cmsis_dap_cmd_DAP_SWJ_Pins(0, SWJ_PIN_SRST, 0, NULL);
+                       retval = cmsis_dap_cmd_dap_swj_pins(0, SWJ_PIN_SRST, 0, NULL);
                        if (retval != ERROR_OK)
                                goto init_err;
                        LOG_INFO("Connecting under reset");
@@ -1023,10 +1338,11 @@ static int cmsis_dap_swd_init(void)
 
 static int cmsis_dap_quit(void)
 {
-       cmsis_dap_cmd_DAP_Disconnect();
+       cmsis_dap_cmd_dap_disconnect();
+
        /* Both LEDs off */
-       cmsis_dap_cmd_DAP_LED(LED_ID_RUN, LED_OFF);
-       cmsis_dap_cmd_DAP_LED(LED_ID_CONNECT, LED_OFF);
+       cmsis_dap_cmd_dap_led(LED_ID_RUN, LED_OFF);
+       cmsis_dap_cmd_dap_led(LED_ID_CONNECT, LED_OFF);
 
        cmsis_dap_close(cmsis_dap_handle);
 
@@ -1044,7 +1360,7 @@ static int cmsis_dap_reset(int trst, int srst)
        if (!trst)
                output_pins |= SWJ_PIN_TRST;
 
-       int retval = cmsis_dap_cmd_DAP_SWJ_Pins(output_pins,
+       int retval = cmsis_dap_cmd_dap_swj_pins(output_pins,
                        SWJ_PIN_TRST | SWJ_PIN_SRST, 0, NULL);
        if (retval != ERROR_OK)
                LOG_ERROR("CMSIS-DAP: Interface reset failed");
@@ -1054,7 +1370,7 @@ static int cmsis_dap_reset(int trst, int srst)
 static void cmsis_dap_execute_sleep(struct jtag_command *cmd)
 {
 #if 0
-       int retval = cmsis_dap_cmd_DAP_Delay(cmd->cmd.sleep->us);
+       int retval = cmsis_dap_cmd_dap_delay(cmd->cmd.sleep->us);
        if (retval != ERROR_OK)
 #endif
                jtag_sleep(cmd->cmd.sleep->us);
@@ -1065,10 +1381,11 @@ static int cmsis_dap_execute_tlr_reset(struct jtag_command *cmd)
 {
        LOG_INFO("cmsis-dap JTAG TLR_RESET");
        uint8_t seq = 0xff;
-       int ret = cmsis_dap_cmd_DAP_SWJ_Sequence(8, &seq);
-       if (ret == ERROR_OK)
+
+       int retval = cmsis_dap_cmd_dap_swj_sequence(8, &seq);
+       if (retval == ERROR_OK)
                tap_set_state(TAP_RESET);
-       return ret;
+       return retval;
 }
 
 /* Set new end state */
@@ -1083,7 +1400,7 @@ static void cmsis_dap_end_state(tap_state_t state)
 }
 
 #ifdef SPRINT_BINARY
-static void sprint_binary(char *s, const uint8_t *buf, int offset, int len)
+static void sprint_binary(char *s, const uint8_t *buf, unsigned int offset, unsigned int len)
 {
        if (!len)
                return;
@@ -1094,7 +1411,7 @@ static void sprint_binary(char *s, const uint8_t *buf, int offset, int len)
        buf = { 0xc0 0x18 } offset=3 len=10 should result in: 11000 11000
                i=3 there means i/8 = 0 so c = 0xFF, and
        */
-       for (int i = offset; i < offset + len; ++i) {
+       for (unsigned int i = offset; i < offset + len; ++i) {
                uint8_t c = buf[i / 8], mask = 1 << (i % 8);
                if ((i != offset) && !(i % 8))
                        putchar(' ');
@@ -1112,18 +1429,18 @@ static void debug_parse_cmsis_buf(const uint8_t *cmd, int cmdlen)
        for (int i = 0; i < cmdlen; ++i)
                printf(" %02x", cmd[i]);
        printf("\n");
-       switch (cmd[1]) {
+       switch (cmd[0]) {
                case CMD_DAP_JTAG_SEQ: {
-                       printf("cmsis-dap jtag sequence command %02x (n=%d)\n", cmd[1], cmd[2]);
+                       printf("cmsis-dap jtag sequence command %02x (n=%d)\n", cmd[0], cmd[1]);
                        /*
-                        * #2 = number of sequences
-                        * #3 = sequence info 1
-                        * #4...4+n_bytes-1 = sequence 1
+                        * #1 = number of sequences
+                        * #2 = sequence info 1
+                        * #3...4+n_bytes-1 = sequence 1
                         * #4+n_bytes = sequence info 2
                         * #5+n_bytes = sequence 2 (single bit)
                         */
-                       int pos = 3;
-                       for (int seq = 0; seq < cmd[2]; ++seq) {
+                       int pos = 2;
+                       for (int seq = 0; seq < cmd[1]; ++seq) {
                                uint8_t info = cmd[pos++];
                                int len = info & DAP_JTAG_SEQ_TCK;
                                if (len == 0)
@@ -1208,10 +1525,11 @@ static void cmsis_dap_flush(void)
  * sequence=NULL means clock out zeros on TDI
  * tdo_buffer=NULL means don't capture TDO
  */
-static void cmsis_dap_add_jtag_sequence(int s_len, const uint8_t *sequence, int s_offset,
-                                       bool tms, uint8_t *tdo_buffer, int tdo_buffer_offset)
+static void cmsis_dap_add_jtag_sequence(unsigned int s_len, const uint8_t *sequence,
+                                       unsigned int s_offset, bool tms,
+                                       uint8_t *tdo_buffer, unsigned int tdo_buffer_offset)
 {
-       LOG_DEBUG_IO("[at %d] %d bits, tms %s, seq offset %d, tdo buf %p, tdo offset %d",
+       LOG_DEBUG_IO("[at %d] %u bits, tms %s, seq offset %u, tdo buf %p, tdo offset %u",
                queued_seq_buf_end,
                s_len, tms ? "HIGH" : "LOW", s_offset, tdo_buffer, tdo_buffer_offset);
 
@@ -1220,25 +1538,25 @@ static void cmsis_dap_add_jtag_sequence(int s_len, const uint8_t *sequence, int
 
        if (s_len > 64) {
                LOG_DEBUG_IO("START JTAG SEQ SPLIT");
-               for (int offset = 0; offset < s_len; offset += 64) {
-                       int len = s_len - offset;
+               for (unsigned int offset = 0; offset < s_len; offset += 64) {
+                       unsigned int len = s_len - offset;
                        if (len > 64)
                                len = 64;
-                       LOG_DEBUG_IO("Splitting long jtag sequence: %d-bit chunk starting at offset %d", len, offset);
+                       LOG_DEBUG_IO("Splitting long jtag sequence: %u-bit chunk starting at offset %u", len, offset);
                        cmsis_dap_add_jtag_sequence(
                                len,
                                sequence,
                                s_offset + offset,
                                tms,
                                tdo_buffer,
-                               tdo_buffer == NULL ? 0 : (tdo_buffer_offset + offset)
+                               !tdo_buffer ? 0 : (tdo_buffer_offset + offset)
                                );
                }
                LOG_DEBUG_IO("END JTAG SEQ SPLIT");
                return;
        }
 
-       int cmd_len = 1 + DIV_ROUND_UP(s_len, 8);
+       unsigned int cmd_len = 1 + DIV_ROUND_UP(s_len, 8);
        if (queued_seq_count >= 255 || queued_seq_buf_end + cmd_len > QUEUED_SEQ_BUF_LEN)
                /* empty out the buffer */
                cmsis_dap_flush();
@@ -1248,17 +1566,17 @@ static void cmsis_dap_add_jtag_sequence(int s_len, const uint8_t *sequence, int
        /* control byte */
        queued_seq_buf[queued_seq_buf_end] =
                (tms ? DAP_JTAG_SEQ_TMS : 0) |
-               (tdo_buffer != NULL ? DAP_JTAG_SEQ_TDO : 0) |
+               (tdo_buffer ? DAP_JTAG_SEQ_TDO : 0) |
                (s_len == 64 ? 0 : s_len);
 
-       if (sequence != NULL)
+       if (sequence)
                bit_copy(&queued_seq_buf[queued_seq_buf_end + 1], 0, sequence, s_offset, s_len);
        else
                memset(&queued_seq_buf[queued_seq_buf_end + 1], 0, DIV_ROUND_UP(s_len, 8));
 
        queued_seq_buf_end += cmd_len;
 
-       if (tdo_buffer != NULL) {
+       if (tdo_buffer) {
                struct pending_scan_result *scan = &pending_scan_results[pending_scan_result_count++];
                scan->first = queued_seq_tdo_ptr;
                queued_seq_tdo_ptr += DIV_ROUND_UP(s_len, 8);
@@ -1286,11 +1604,8 @@ static void cmsis_dap_add_tms_sequence(const uint8_t *sequence, int s_len)
 /* Move to the end state by queuing a sequence to clock into TMS */
 static void cmsis_dap_state_move(void)
 {
-       uint8_t tms_scan;
-       uint8_t tms_scan_bits;
-
-       tms_scan = tap_get_tms_path(tap_get_state(), tap_get_end_state());
-       tms_scan_bits = tap_get_tms_path_len(tap_get_state(), tap_get_end_state());
+       uint8_t tms_scan = tap_get_tms_path(tap_get_state(), tap_get_end_state());
+       uint8_t tms_scan_bits = tap_get_tms_path_len(tap_get_state(), tap_get_end_state());
 
        LOG_DEBUG_IO("state move from %s to %s: %d clocks, %02X on tms",
                tap_state_name(tap_get_state()), tap_state_name(tap_get_end_state()),
@@ -1405,11 +1720,10 @@ static void cmsis_dap_execute_scan(struct jtag_command *cmd)
 
 static void cmsis_dap_pathmove(int num_states, tap_state_t *path)
 {
-       int i;
        uint8_t tms0 = 0x00;
        uint8_t tms1 = 0xff;
 
-       for (i = 0; i < num_states; i++) {
+       for (int i = 0; i < num_states; i++) {
                if (path[i] == tap_state_transition(tap_get_state(), false))
                        cmsis_dap_add_tms_sequence(&tms0, 1);
                else if (path[i] == tap_state_transition(tap_get_state(), true))
@@ -1437,12 +1751,10 @@ static void cmsis_dap_execute_pathmove(struct jtag_command *cmd)
 
 static void cmsis_dap_stableclocks(int num_cycles)
 {
-       int i;
-
        uint8_t tms = tap_get_state() == TAP_RESET;
        /* TODO: Perform optimizations? */
        /* Execute num_cycles. */
-       for (i = 0; i < num_cycles; i++)
+       for (int i = 0; i < num_cycles; i++)
                cmsis_dap_add_tms_sequence(&tms, 1);
 }
 
@@ -1482,7 +1794,7 @@ static void cmsis_dap_execute_stableclocks(struct jtag_command *cmd)
 static void cmsis_dap_execute_tms(struct jtag_command *cmd)
 {
        LOG_DEBUG_IO("TMS: %d bits", cmd->cmd.tms->num_bits);
-       cmsis_dap_cmd_DAP_SWJ_Sequence(cmd->cmd.tms->num_bits, cmd->cmd.tms->bits);
+       cmsis_dap_cmd_dap_swj_sequence(cmd->cmd.tms->num_bits, cmd->cmd.tms->bits);
 }
 
 /* TODO: Is there need to call cmsis_dap_flush() for the JTAG_PATHMOVE,
@@ -1523,7 +1835,7 @@ static int cmsis_dap_execute_queue(void)
 {
        struct jtag_command *cmd = jtag_command_queue;
 
-       while (cmd != NULL) {
+       while (cmd) {
                cmsis_dap_execute_command(cmd);
                cmd = cmd->next;
        }
@@ -1540,7 +1852,7 @@ static int cmsis_dap_speed(int speed)
                return ERROR_JTAG_NOT_IMPLEMENTED;
        }
 
-       return cmsis_dap_cmd_DAP_SWJ_Clock(speed);
+       return cmsis_dap_cmd_dap_swj_clock(speed);
 }
 
 static int cmsis_dap_speed_div(int speed, int *khz)
@@ -1555,6 +1867,157 @@ static int cmsis_dap_khz(int khz, int *jtag_speed)
        return ERROR_OK;
 }
 
+static bool calculate_swo_prescaler(unsigned int traceclkin_freq,
+               uint32_t trace_freq, uint16_t *prescaler)
+{
+       unsigned int presc = (traceclkin_freq + trace_freq / 2) / trace_freq;
+       if (presc == 0 || presc > TPIU_ACPR_MAX_SWOSCALER + 1)
+               return false;
+
+       /* Probe's UART speed must be within 3% of the TPIU's SWO baud rate. */
+       unsigned int max_deviation = (traceclkin_freq * 3) / 100;
+       if (presc * trace_freq < traceclkin_freq - max_deviation ||
+           presc * trace_freq > traceclkin_freq + max_deviation)
+               return false;
+
+       *prescaler = presc;
+
+       return true;
+}
+
+/**
+ * @see adapter_driver::config_trace
+ */
+static int cmsis_dap_config_trace(
+                               bool trace_enabled,
+                               enum tpiu_pin_protocol pin_protocol,
+                               uint32_t port_size,
+                               unsigned int *swo_freq,
+                               unsigned int traceclkin_hz,
+                               uint16_t *swo_prescaler)
+{
+       int retval;
+
+       if (!trace_enabled) {
+               if (cmsis_dap_handle->trace_enabled) {
+                       retval = cmsis_dap_cmd_dap_swo_control(DAP_SWO_CONTROL_STOP);
+                       if (retval != ERROR_OK) {
+                               LOG_ERROR("Failed to disable the SWO-trace.");
+                               return retval;
+                       }
+               }
+               cmsis_dap_handle->trace_enabled = false;
+               LOG_INFO("SWO-trace disabled.");
+               return ERROR_OK;
+       }
+
+       if (!(cmsis_dap_handle->caps & INFO_CAPS_SWO_UART) &&
+           !(cmsis_dap_handle->caps & INFO_CAPS_SWO_MANCHESTER)) {
+               LOG_ERROR("SWO-trace is not supported by the device.");
+               return ERROR_FAIL;
+       }
+
+       uint8_t swo_mode;
+       if (pin_protocol == TPIU_PIN_PROTOCOL_ASYNC_UART &&
+          (cmsis_dap_handle->caps & INFO_CAPS_SWO_UART)) {
+               swo_mode = DAP_SWO_MODE_UART;
+       } else if (pin_protocol == TPIU_PIN_PROTOCOL_ASYNC_MANCHESTER &&
+                 (cmsis_dap_handle->caps & INFO_CAPS_SWO_MANCHESTER)) {
+               swo_mode = DAP_SWO_MODE_MANCHESTER;
+       } else {
+               LOG_ERROR("Selected pin protocol is not supported.");
+               return ERROR_FAIL;
+       }
+
+       if (*swo_freq == 0) {
+               LOG_INFO("SWO-trace frequency autodetection not implemented.");
+               return ERROR_FAIL;
+       }
+
+       retval = cmsis_dap_cmd_dap_swo_control(DAP_SWO_CONTROL_STOP);
+       if (retval != ERROR_OK)
+               return retval;
+
+       cmsis_dap_handle->trace_enabled = false;
+
+       retval = cmsis_dap_get_swo_buf_sz(&cmsis_dap_handle->swo_buf_sz);
+       if (retval != ERROR_OK)
+               return retval;
+
+       retval = cmsis_dap_cmd_dap_swo_transport(DAP_SWO_TRANSPORT_DATA);
+       if (retval != ERROR_OK)
+               return retval;
+
+       retval = cmsis_dap_cmd_dap_swo_mode(swo_mode);
+       if (retval != ERROR_OK)
+               return retval;
+
+       retval = cmsis_dap_cmd_dap_swo_baudrate(*swo_freq, swo_freq);
+       if (retval != ERROR_OK)
+               return retval;
+
+       if (!calculate_swo_prescaler(traceclkin_hz, *swo_freq,
+                       swo_prescaler)) {
+               LOG_ERROR("SWO frequency is not suitable. Please choose a "
+                       "different frequency or use auto-detection.");
+               return ERROR_FAIL;
+       }
+
+       LOG_INFO("SWO frequency: %u Hz.", *swo_freq);
+       LOG_INFO("SWO prescaler: %u.", *swo_prescaler);
+
+       retval = cmsis_dap_cmd_dap_swo_control(DAP_SWO_CONTROL_START);
+       if (retval != ERROR_OK)
+               return retval;
+
+       cmsis_dap_handle->trace_enabled = true;
+
+       return ERROR_OK;
+}
+
+/**
+ * @see adapter_driver::poll_trace
+ */
+static int cmsis_dap_poll_trace(uint8_t *buf, size_t *size)
+{
+       uint8_t trace_status;
+       size_t trace_count;
+
+       if (!cmsis_dap_handle->trace_enabled) {
+               *size = 0;
+               return ERROR_OK;
+       }
+
+       int retval = cmsis_dap_cmd_dap_swo_status(&trace_status, &trace_count);
+       if (retval != ERROR_OK)
+               return retval;
+       if ((trace_status & DAP_SWO_STATUS_CAPTURE_MASK) != DAP_SWO_STATUS_CAPTURE_ACTIVE)
+               return ERROR_FAIL;
+
+       *size = trace_count < *size ? trace_count : *size;
+       size_t read_so_far = 0;
+       do {
+               size_t rb = 0;
+               uint32_t packet_size = cmsis_dap_handle->packet_size - 4 /*data-reply*/;
+               uint32_t remaining = *size - read_so_far;
+               if (remaining < packet_size)
+                       packet_size = remaining;
+               retval = cmsis_dap_cmd_dap_swo_data(
+                                               packet_size,
+                                               &trace_status,
+                                               &rb,
+                                               &buf[read_so_far]);
+               if (retval != ERROR_OK)
+                       return retval;
+               if ((trace_status & DAP_SWO_STATUS_CAPTURE_MASK) != DAP_SWO_STATUS_CAPTURE_ACTIVE)
+                       return ERROR_FAIL;
+
+               read_so_far += rb;
+       } while (read_so_far < *size);
+
+       return ERROR_OK;
+}
+
 COMMAND_HANDLER(cmsis_dap_handle_info_command)
 {
        if (cmsis_dap_get_version_info() == ERROR_OK)
@@ -1565,14 +2028,12 @@ COMMAND_HANDLER(cmsis_dap_handle_info_command)
 
 COMMAND_HANDLER(cmsis_dap_handle_cmd_command)
 {
-       int retval;
-       unsigned i;
        uint8_t *command = cmsis_dap_handle->command;
 
-       for (i = 0; i < CMD_ARGC; i++)
-               command[i] = strtoul(CMD_ARGV[i], NULL, 16);
+       for (unsigned i = 0; i < CMD_ARGC; i++)
+               COMMAND_PARSE_NUMBER(u8, CMD_ARGV[i], command[i]);
 
-       retval = cmsis_dap_xfer(cmsis_dap_handle, CMD_ARGC);
+       int retval = cmsis_dap_xfer(cmsis_dap_handle, CMD_ARGC);
 
        if (retval != ERROR_OK) {
                LOG_ERROR("CMSIS-DAP command failed.");
@@ -1616,16 +2077,6 @@ COMMAND_HANDLER(cmsis_dap_handle_vid_pid_command)
        return ERROR_OK;
 }
 
-COMMAND_HANDLER(cmsis_dap_handle_serial_command)
-{
-       if (CMD_ARGC == 1)
-               cmsis_dap_serial = strdup(CMD_ARGV[0]);
-       else
-               LOG_ERROR("expected exactly one argument to cmsis_dap_serial <serial-number>");
-
-       return ERROR_OK;
-}
-
 COMMAND_HANDLER(cmsis_dap_handle_backend_command)
 {
        if (CMD_ARGC == 1) {
@@ -1680,14 +2131,7 @@ static const struct command_registration cmsis_dap_command_handlers[] = {
                .handler = &cmsis_dap_handle_vid_pid_command,
                .mode = COMMAND_CONFIG,
                .help = "the vendor ID and product ID of the CMSIS-DAP device",
-               .usage = "(vid pid)* ",
-       },
-       {
-               .name = "cmsis_dap_serial",
-               .handler = &cmsis_dap_handle_serial_command,
-               .mode = COMMAND_CONFIG,
-               .help = "set the serial number of the adapter",
-               .usage = "serial_string",
+               .usage = "(vid pid)*",
        },
        {
                .name = "cmsis_dap_backend",
@@ -1734,6 +2178,8 @@ struct adapter_driver cmsis_dap_adapter_driver = {
        .speed = cmsis_dap_speed,
        .khz = cmsis_dap_khz,
        .speed_div = cmsis_dap_speed_div,
+       .config_trace = cmsis_dap_config_trace,
+       .poll_trace = cmsis_dap_poll_trace,
 
        .jtag_ops = &cmsis_dap_interface,
        .swd_ops = &cmsis_dap_swd_driver,

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)