openocd: fix SPDX tag format for files .c
[fw/openocd] / src / jtag / drivers / jlink.c
index 3dcadc81361d718d65bd34677e8a63dfbdbf7bb1..0a96ac255dc03830b77f5ffd062e0afe6c843eb5 100644 (file)
@@ -1,3 +1,5 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+
 /***************************************************************************
  *   Copyright (C) 2007 by Juergen Stuber <juergen@jstuber.net>            *
  *   based on Dominic Rath's and Benedikt Sauter's usbprog.c               *
  *                                                                         *
  *   Copyright (C) 2015 by Paul Fertser                                    *
  *   fercerpav@gmail.com                                                   *
- *                                                                         *
- *   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
@@ -38,7 +27,8 @@
 #include <jtag/interface.h>
 #include <jtag/swd.h>
 #include <jtag/commands.h>
-#include <jtag/drivers/jtag_usb_common.h>
+#include <jtag/adapter.h>
+#include <helper/replacements.h>
 #include <target/cortex_m.h>
 
 #include <libjaylink/libjaylink.h>
@@ -113,8 +103,6 @@ static int jlink_flush(void);
  * @param in A pointer to store TDO data to, if NULL the data will be discarded.
  * @param in_offset A bit offset for TDO data.
  * @param length Amount of bits to transfer out and in.
- *
- * @retval This function doesn't return any value.
  */
 static void jlink_clock_data(const uint8_t *out, unsigned out_offset,
                             const uint8_t *tms_out, unsigned tms_offset,
@@ -254,7 +242,7 @@ static void jlink_execute_scan(struct jtag_command *cmd)
 
 static void jlink_execute_sleep(struct jtag_command *cmd)
 {
-       LOG_DEBUG_IO("sleep %" PRIi32 "", cmd->cmd.sleep->us);
+       LOG_DEBUG_IO("sleep %" PRIu32 "", cmd->cmd.sleep->us);
        jlink_flush();
        jtag_sleep(cmd->cmd.sleep->us);
 }
@@ -281,7 +269,7 @@ static int jlink_execute_command(struct jtag_command *cmd)
                        jlink_execute_sleep(cmd);
                        break;
                default:
-                       LOG_ERROR("BUG: Unknown JTAG command type encountered.");
+                       LOG_ERROR("BUG: Unknown JTAG command type encountered");
                        return ERROR_JTAG_QUEUE_FAILED;
        }
 
@@ -293,7 +281,7 @@ static int jlink_execute_queue(void)
        int ret;
        struct jtag_command *cmd = jtag_command_queue;
 
-       while (cmd != NULL) {
+       while (cmd) {
                ret = jlink_execute_command(cmd);
 
                if (ret != ERROR_OK)
@@ -315,7 +303,7 @@ static int jlink_speed(int speed)
                ret = jaylink_get_speeds(devh, &tmp);
 
                if (ret != JAYLINK_OK) {
-                       LOG_ERROR("jaylink_get_speeds() failed: %s.",
+                       LOG_ERROR("jaylink_get_speeds() failed: %s",
                                jaylink_strerror(ret));
                        return ERROR_JTAG_DEVICE_ERROR;
                }
@@ -328,13 +316,13 @@ static int jlink_speed(int speed)
 
        if (!speed) {
                if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_ADAPTIVE_CLOCKING)) {
-                       LOG_ERROR("Adaptive clocking is not supported by the device.");
+                       LOG_ERROR("Adaptive clocking is not supported by the device");
                        return ERROR_JTAG_NOT_IMPLEMENTED;
                }
 
                speed = JAYLINK_SPEED_ADAPTIVE_CLOCKING;
        } else if (speed > max_speed) {
-               LOG_INFO("Reduced speed from %d kHz to %d kHz (maximum).", speed,
+               LOG_INFO("Reduced speed from %d kHz to %d kHz (maximum)", speed,
                        max_speed);
                speed = max_speed;
        }
@@ -342,7 +330,7 @@ static int jlink_speed(int speed)
        ret = jaylink_set_speed(devh, speed);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_set_speed() failed: %s.",
+               LOG_ERROR("jaylink_set_speed() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_JTAG_DEVICE_ERROR;
        }
@@ -371,7 +359,7 @@ static bool read_device_config(struct device_config *cfg)
        ret = jaylink_read_raw_config(devh, (uint8_t *)cfg);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_read_raw_config() failed: %s.",
+               LOG_ERROR("jaylink_read_raw_config() failed: %s",
                        jaylink_strerror(ret));
                return false;
        }
@@ -392,7 +380,7 @@ static int select_interface(void)
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_SELECT_TIF)) {
                if (iface != JAYLINK_TIF_JTAG) {
-                       LOG_ERROR("Device supports JTAG transport only.");
+                       LOG_ERROR("Device supports JTAG transport only");
                        return ERROR_JTAG_INIT_FAILED;
                }
 
@@ -402,20 +390,20 @@ static int select_interface(void)
        ret = jaylink_get_available_interfaces(devh, &interfaces);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_get_available_interfaces() failed: %s.",
+               LOG_ERROR("jaylink_get_available_interfaces() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_JTAG_INIT_FAILED;
        }
 
        if (!(interfaces & (1 << iface))) {
-               LOG_ERROR("Selected transport is not supported by the device.");
+               LOG_ERROR("Selected transport is not supported by the device");
                return ERROR_JTAG_INIT_FAILED;
        }
 
        ret = jaylink_select_interface(devh, iface, NULL);
 
        if (ret < 0) {
-               LOG_ERROR("jaylink_select_interface() failed: %s.",
+               LOG_ERROR("jaylink_select_interface() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_JTAG_INIT_FAILED;
        }
@@ -436,7 +424,7 @@ static int jlink_register(void)
        ret = jaylink_register(devh, &conn, connlist, &count);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_register() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_register() failed: %s", jaylink_strerror(ret));
                return ERROR_FAIL;
        }
 
@@ -451,7 +439,7 @@ static int jlink_register(void)
 
        if (!handle_found) {
                LOG_ERROR("Registration failed: maximum number of connections on the "
-                       "device reached.");
+                       "device reached");
                return ERROR_FAIL;
        }
 
@@ -474,13 +462,13 @@ static bool adjust_swd_buffer_size(void)
        ret = jaylink_get_free_memory(devh, &tmp);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_get_free_memory() failed: %s.",
+               LOG_ERROR("jaylink_get_free_memory() failed: %s",
                        jaylink_strerror(ret));
                return false;
        }
 
        if (tmp < 143) {
-               LOG_ERROR("Not enough free device internal memory: %u bytes.", tmp);
+               LOG_ERROR("Not enough free device internal memory: %" PRIu32 " bytes", tmp);
                return false;
        }
 
@@ -488,7 +476,7 @@ static bool adjust_swd_buffer_size(void)
 
        if (tmp != swd_buffer_size) {
                swd_buffer_size = tmp;
-               LOG_DEBUG("Adjusted SWD transaction buffer size to %u bytes.",
+               LOG_DEBUG("Adjusted SWD transaction buffer size to %u bytes",
                        swd_buffer_size);
        }
 
@@ -541,100 +529,59 @@ static bool jlink_usb_location_equal(struct jaylink_device *dev)
        if (retval == JAYLINK_ERR_NOT_SUPPORTED) {
                return false;
        } else if (retval != JAYLINK_OK) {
-               LOG_WARNING("jaylink_device_get_usb_bus_ports() failed: %s.",
+               LOG_WARNING("jaylink_device_get_usb_bus_ports() failed: %s",
                        jaylink_strerror(retval));
                return false;
        }
 
-       equal = jtag_usb_location_equal(bus, ports,     num_ports);
+       equal = adapter_usb_location_equal(bus, ports, num_ports);
        free(ports);
 
        return equal;
 }
 
 
-static int jlink_init(void)
+static int jlink_open_device(uint32_t ifaces, bool *found_device)
 {
-       int ret;
-       struct jaylink_device **devs;
-       unsigned int i;
-       bool found_device;
-       uint32_t tmp;
-       char *firmware_version;
-       struct jaylink_hardware_version hwver;
-       struct jaylink_hardware_status hwstatus;
-       enum jaylink_usb_address address;
-       size_t length;
-       size_t num_devices;
-       uint32_t host_interfaces;
-
-       LOG_DEBUG("Using libjaylink %s (compiled with %s).",
-               jaylink_version_package_get_string(), JAYLINK_VERSION_PACKAGE_STRING);
-
-       if (!jaylink_library_has_cap(JAYLINK_CAP_HIF_USB) && use_usb_address) {
-               LOG_ERROR("J-Link driver does not support USB devices.");
-               return ERROR_JTAG_INIT_FAILED;
-       }
-
-       ret = jaylink_init(&jayctx);
-
-       if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_init() failed: %s.", jaylink_strerror(ret));
-               return ERROR_JTAG_INIT_FAILED;
-       }
-
-       ret = jaylink_log_set_callback(jayctx, &jaylink_log_handler, NULL);
-
-       if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_log_set_callback() failed: %s.",
-                       jaylink_strerror(ret));
-               jaylink_exit(jayctx);
-               return ERROR_JTAG_INIT_FAILED;
-       }
-
-       host_interfaces = JAYLINK_HIF_USB;
-
-       if (use_serial_number)
-               host_interfaces |= JAYLINK_HIF_TCP;
-
-       ret = jaylink_discovery_scan(jayctx, host_interfaces);
-
+       int ret = jaylink_discovery_scan(jayctx, ifaces);
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_discovery_scan() failed: %s.",
-                       jaylink_strerror(ret));
+               LOG_ERROR("jaylink_discovery_scan() failed: %s", jaylink_strerror(ret));
                jaylink_exit(jayctx);
                return ERROR_JTAG_INIT_FAILED;
        }
 
+       size_t num_devices;
+       struct jaylink_device **devs;
        ret = jaylink_get_devices(jayctx, &devs, &num_devices);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_get_devices() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_get_devices() failed: %s", jaylink_strerror(ret));
                jaylink_exit(jayctx);
                return ERROR_JTAG_INIT_FAILED;
        }
 
-       use_usb_location = (jtag_usb_get_location() != NULL);
+       use_usb_location = !!adapter_usb_get_location();
 
        if (!use_serial_number && !use_usb_address && !use_usb_location && num_devices > 1) {
-               LOG_ERROR("Multiple devices found, specify the desired device.");
+               LOG_ERROR("Multiple devices found, specify the desired device");
                jaylink_free_devices(devs, true);
                jaylink_exit(jayctx);
                return ERROR_JTAG_INIT_FAILED;
        }
 
-       found_device = false;
+       *found_device = false;
 
-       for (i = 0; devs[i]; i++) {
+       for (size_t i = 0; devs[i]; i++) {
                struct jaylink_device *dev = devs[i];
 
                if (use_serial_number) {
+                       uint32_t tmp;
                        ret = jaylink_device_get_serial_number(dev, &tmp);
 
                        if (ret == JAYLINK_ERR_NOT_AVAILABLE) {
                                continue;
                        } else if (ret != JAYLINK_OK) {
-                               LOG_WARNING("jaylink_device_get_serial_number() failed: %s.",
+                               LOG_WARNING("jaylink_device_get_serial_number() failed: %s",
                                        jaylink_strerror(ret));
                                continue;
                        }
@@ -644,12 +591,13 @@ static int jlink_init(void)
                }
 
                if (use_usb_address) {
+                       enum jaylink_usb_address address;
                        ret = jaylink_device_get_usb_address(dev, &address);
 
                        if (ret == JAYLINK_ERR_NOT_SUPPORTED) {
                                continue;
                        } else if (ret != JAYLINK_OK) {
-                               LOG_WARNING("jaylink_device_get_usb_address() failed: %s.",
+                               LOG_WARNING("jaylink_device_get_usb_address() failed: %s",
                                        jaylink_strerror(ret));
                                continue;
                        }
@@ -664,17 +612,80 @@ static int jlink_init(void)
                ret = jaylink_open(dev, &devh);
 
                if (ret == JAYLINK_OK) {
-                       found_device = true;
+                       *found_device = true;
                        break;
                }
 
-               LOG_ERROR("Failed to open device: %s.", jaylink_strerror(ret));
+               LOG_ERROR("Failed to open device: %s", jaylink_strerror(ret));
        }
 
        jaylink_free_devices(devs, true);
+       return ERROR_OK;
+}
+
+
+static int jlink_init(void)
+{
+       int ret;
+       char *firmware_version;
+       struct jaylink_hardware_version hwver;
+       struct jaylink_hardware_status hwstatus;
+       size_t length;
+
+       LOG_DEBUG("Using libjaylink %s (compiled with %s)",
+               jaylink_version_package_get_string(), JAYLINK_VERSION_PACKAGE_STRING);
+
+       if (!jaylink_library_has_cap(JAYLINK_CAP_HIF_USB) && use_usb_address) {
+               LOG_ERROR("J-Link driver does not support USB devices");
+               return ERROR_JTAG_INIT_FAILED;
+       }
+
+       ret = jaylink_init(&jayctx);
+
+       if (ret != JAYLINK_OK) {
+               LOG_ERROR("jaylink_init() failed: %s", jaylink_strerror(ret));
+               return ERROR_JTAG_INIT_FAILED;
+       }
+
+       ret = jaylink_log_set_callback(jayctx, &jaylink_log_handler, NULL);
+
+       if (ret != JAYLINK_OK) {
+               LOG_ERROR("jaylink_log_set_callback() failed: %s",
+                       jaylink_strerror(ret));
+               jaylink_exit(jayctx);
+               return ERROR_JTAG_INIT_FAILED;
+       }
+
+       const char *serial = adapter_get_required_serial();
+       if (serial) {
+               ret = jaylink_parse_serial_number(serial, &serial_number);
+               if (ret == JAYLINK_ERR) {
+                       LOG_ERROR("Invalid serial number: %s", serial);
+                       jaylink_exit(jayctx);
+                       return ERROR_JTAG_INIT_FAILED;
+               }
+               if (ret != JAYLINK_OK) {
+                       LOG_ERROR("jaylink_parse_serial_number() failed: %s", jaylink_strerror(ret));
+                       jaylink_exit(jayctx);
+                       return ERROR_JTAG_INIT_FAILED;
+               }
+               use_serial_number = true;
+               use_usb_address = false;
+       }
+
+       bool found_device;
+       ret = jlink_open_device(JAYLINK_HIF_USB, &found_device);
+       if (ret != ERROR_OK)
+               return ret;
+
+       if (!found_device && use_serial_number) {
+               ret = jlink_open_device(JAYLINK_HIF_TCP, &found_device);
+               if (ret != ERROR_OK)
+                       return ret;
+       }
 
        if (!found_device) {
-               LOG_ERROR("No J-Link device found.");
+               LOG_ERROR("No J-Link device found");
                jaylink_exit(jayctx);
                return ERROR_JTAG_INIT_FAILED;
        }
@@ -687,7 +698,7 @@ static int jlink_init(void)
        ret = jaylink_get_firmware_version(devh, &firmware_version, &length);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_get_firmware_version() failed: %s.",
+               LOG_ERROR("jaylink_get_firmware_version() failed: %s",
                        jaylink_strerror(ret));
                jaylink_close(devh);
                jaylink_exit(jayctx);
@@ -696,14 +707,14 @@ static int jlink_init(void)
                LOG_INFO("%s", firmware_version);
                free(firmware_version);
        } else {
-               LOG_WARNING("Device responds empty firmware version string.");
+               LOG_WARNING("Device responds empty firmware version string");
        }
 
        memset(caps, 0, JAYLINK_DEV_EXT_CAPS_SIZE);
        ret = jaylink_get_caps(devh, caps);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_get_caps() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_get_caps() failed: %s", jaylink_strerror(ret));
                jaylink_close(devh);
                jaylink_exit(jayctx);
                return ERROR_JTAG_INIT_FAILED;
@@ -713,7 +724,7 @@ static int jlink_init(void)
                ret = jaylink_get_extended_caps(devh, caps);
 
                if (ret != JAYLINK_OK) {
-                       LOG_ERROR("jaylink_get_extended_caps() failed:  %s.",
+                       LOG_ERROR("jaylink_get_extended_caps() failed:  %s",
                                jaylink_strerror(ret));
                        jaylink_close(devh);
                        jaylink_exit(jayctx);
@@ -727,7 +738,7 @@ static int jlink_init(void)
                ret = jaylink_get_hardware_version(devh, &hwver);
 
                if (ret != JAYLINK_OK) {
-                       LOG_ERROR("Failed to retrieve hardware version: %s.",
+                       LOG_ERROR("Failed to retrieve hardware version: %s",
                                jaylink_strerror(ret));
                        jaylink_close(devh);
                        jaylink_exit(jayctx);
@@ -756,7 +767,7 @@ static int jlink_init(void)
 
        if (jaylink_has_cap(caps, JAYLINK_DEV_CAP_READ_CONFIG)) {
                if (!read_device_config(&config)) {
-                       LOG_ERROR("Failed to read device configuration data.");
+                       LOG_ERROR("Failed to read device configuration data");
                        jaylink_close(devh);
                        jaylink_exit(jayctx);
                        return ERROR_JTAG_INIT_FAILED;
@@ -768,7 +779,7 @@ static int jlink_init(void)
        ret = jaylink_get_hardware_status(devh, &hwstatus);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_get_hardware_status() failed: %s.",
+               LOG_ERROR("jaylink_get_hardware_status() failed: %s",
                        jaylink_strerror(ret));
                jaylink_close(devh);
                jaylink_exit(jayctx);
@@ -804,7 +815,7 @@ static int jlink_init(void)
        jtag_sleep(3000);
        jlink_tap_init();
 
-       jlink_speed(jtag_get_speed_khz());
+       jlink_speed(adapter_get_speed_khz());
 
        if (iface == JAYLINK_TIF_JTAG) {
                /*
@@ -830,14 +841,14 @@ static int jlink_quit(void)
                ret = jaylink_swo_stop(devh);
 
                if (ret != JAYLINK_OK)
-                       LOG_ERROR("jaylink_swo_stop() failed: %s.", jaylink_strerror(ret));
+                       LOG_ERROR("jaylink_swo_stop() failed: %s", jaylink_strerror(ret));
        }
 
        if (jaylink_has_cap(caps, JAYLINK_DEV_CAP_REGISTER)) {
                ret = jaylink_unregister(devh, &conn, connlist, &count);
 
                if (ret != JAYLINK_OK)
-                       LOG_ERROR("jaylink_unregister() failed: %s.",
+                       LOG_ERROR("jaylink_unregister() failed: %s",
                                jaylink_strerror(ret));
        }
 
@@ -885,7 +896,7 @@ static void jlink_path_move(int num_states, tap_state_t *path)
                else if (path[i] == tap_state_transition(tap_get_state(), true))
                        jlink_clock_data(NULL, 0, &tms, 0, NULL, 0, 1);
                else {
-                       LOG_ERROR("BUG: %s -> %s isn't a valid TAP transition.",
+                       LOG_ERROR("BUG: %s -> %s isn't a valid TAP transition",
                                tap_state_name(tap_get_state()), tap_state_name(path[i]));
                        exit(-1);
                }
@@ -928,7 +939,7 @@ static void jlink_runtest(int num_cycles)
 
 static void jlink_reset(int trst, int srst)
 {
-       LOG_DEBUG("TRST: %i, SRST: %i.", trst, srst);
+       LOG_DEBUG("TRST: %i, SRST: %i", trst, srst);
 
        /* Signals are active low. */
        if (srst == 0)
@@ -956,54 +967,27 @@ COMMAND_HANDLER(jlink_usb_command)
        int tmp;
 
        if (CMD_ARGC != 1) {
-               command_print(CMD, "Need exactly one argument for jlink usb.");
+               command_print(CMD, "Need exactly one argument for jlink usb");
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
 
        if (sscanf(CMD_ARGV[0], "%i", &tmp) != 1) {
-               command_print(CMD, "Invalid USB address: %s.", CMD_ARGV[0]);
+               command_print(CMD, "Invalid USB address: %s", CMD_ARGV[0]);
                return ERROR_FAIL;
        }
 
        if (tmp < JAYLINK_USB_ADDRESS_0 || tmp > JAYLINK_USB_ADDRESS_3) {
-               command_print(CMD, "Invalid USB address: %s.", CMD_ARGV[0]);
+               command_print(CMD, "Invalid USB address: %s", CMD_ARGV[0]);
                return ERROR_FAIL;
        }
 
        usb_address = tmp;
 
-       use_serial_number = false;
        use_usb_address = true;
 
        return ERROR_OK;
 }
 
-COMMAND_HANDLER(jlink_serial_command)
-{
-       int ret;
-
-       if (CMD_ARGC != 1) {
-               command_print(CMD, "Need exactly one argument for jlink serial.");
-               return ERROR_COMMAND_SYNTAX_ERROR;
-       }
-
-       ret = jaylink_parse_serial_number(CMD_ARGV[0], &serial_number);
-
-       if (ret == JAYLINK_ERR) {
-               command_print(CMD, "Invalid serial number: %s.", CMD_ARGV[0]);
-               return ERROR_FAIL;
-       } else if (ret != JAYLINK_OK) {
-               command_print(CMD, "jaylink_parse_serial_number() failed: %s.",
-                       jaylink_strerror(ret));
-               return ERROR_FAIL;
-       }
-
-       use_serial_number = true;
-       use_usb_address = false;
-
-       return ERROR_OK;
-}
-
 COMMAND_HANDLER(jlink_handle_hwstatus_command)
 {
        int ret;
@@ -1012,7 +996,7 @@ COMMAND_HANDLER(jlink_handle_hwstatus_command)
        ret = jaylink_get_hardware_status(devh, &status);
 
        if (ret != JAYLINK_OK) {
-               command_print(CMD, "jaylink_get_hardware_status() failed: %s.",
+               command_print(CMD, "jaylink_get_hardware_status() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_FAIL;
        }
@@ -1025,7 +1009,7 @@ COMMAND_HANDLER(jlink_handle_hwstatus_command)
                status.tres, status.trst);
 
        if (status.target_voltage < 1500)
-               command_print(CMD, "Target voltage too low. Check target power.");
+               command_print(CMD, "Target voltage too low. Check target power");
 
        return ERROR_OK;
 }
@@ -1037,19 +1021,19 @@ COMMAND_HANDLER(jlink_handle_free_memory_command)
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_GET_FREE_MEMORY)) {
                command_print(CMD, "Retrieval of free memory is not supported by "
-                       "the device.");
+                       "the device");
                return ERROR_OK;
        }
 
        ret = jaylink_get_free_memory(devh, &tmp);
 
        if (ret != JAYLINK_OK) {
-               command_print(CMD, "jaylink_get_free_memory() failed: %s.",
+               command_print(CMD, "jaylink_get_free_memory() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_FAIL;
        }
 
-       command_print(CMD, "Device has %u bytes of free memory.", tmp);
+       command_print(CMD, "Device has %" PRIu32 " bytes of free memory", tmp);
 
        return ERROR_OK;
 }
@@ -1074,7 +1058,7 @@ COMMAND_HANDLER(jlink_handle_jlink_jtag_command)
                command_print(CMD, "JTAG command version: %i", version);
        } else if (CMD_ARGC == 1) {
                if (sscanf(CMD_ARGV[0], "%i", &tmp) != 1) {
-                       command_print(CMD, "Invalid argument: %s.", CMD_ARGV[0]);
+                       command_print(CMD, "Invalid argument: %s", CMD_ARGV[0]);
                        return ERROR_COMMAND_SYNTAX_ERROR;
                }
 
@@ -1086,11 +1070,11 @@ COMMAND_HANDLER(jlink_handle_jlink_jtag_command)
                                jtag_command_version = JAYLINK_JTAG_VERSION_3;
                                break;
                        default:
-                               command_print(CMD, "Invalid argument: %s.", CMD_ARGV[0]);
+                               command_print(CMD, "Invalid argument: %s", CMD_ARGV[0]);
                                return ERROR_COMMAND_SYNTAX_ERROR;
                }
        } else {
-               command_print(CMD, "Need exactly one argument for jlink jtag.");
+               command_print(CMD, "Need exactly one argument for jlink jtag");
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
 
@@ -1103,14 +1087,13 @@ COMMAND_HANDLER(jlink_handle_target_power_command)
        int enable;
 
        if (CMD_ARGC != 1) {
-               command_print(CMD, "Need exactly one argument for jlink "
-                       "targetpower.");
+               command_print(CMD, "Need exactly one argument for jlink targetpower");
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_SET_TARGET_POWER)) {
                command_print(CMD, "Target power supply is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
@@ -1119,14 +1102,14 @@ COMMAND_HANDLER(jlink_handle_target_power_command)
        } else if (!strcmp(CMD_ARGV[0], "off")) {
                enable = false;
        } else {
-               command_print(CMD, "Invalid argument: %s.", CMD_ARGV[0]);
+               command_print(CMD, "Invalid argument: %s", CMD_ARGV[0]);
                return ERROR_FAIL;
        }
 
        ret = jaylink_set_target_power(devh, enable);
 
        if (ret != JAYLINK_OK) {
-               command_print(CMD, "jaylink_set_target_power() failed: %s.",
+               command_print(CMD, "jaylink_set_target_power() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_FAIL;
        }
@@ -1233,7 +1216,7 @@ static int poll_trace(uint8_t *buf, size_t *size)
        ret = jaylink_swo_read(devh, buf, &length);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_swo_read() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_swo_read() failed: %s", jaylink_strerror(ret));
                return ERROR_FAIL;
        }
 
@@ -1253,7 +1236,7 @@ static uint32_t calculate_trace_buffer_size(void)
        ret = jaylink_get_free_memory(devh, &tmp);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_get_free_memory() failed: %s.",
+               LOG_ERROR("jaylink_get_free_memory() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_FAIL;
        }
@@ -1269,17 +1252,14 @@ static uint32_t calculate_trace_buffer_size(void)
 static bool calculate_swo_prescaler(unsigned int traceclkin_freq,
                uint32_t trace_freq, uint16_t *prescaler)
 {
-       unsigned int presc;
-       double deviation;
-
-       presc = ((1.0 - SWO_MAX_FREQ_DEV) * traceclkin_freq) / trace_freq + 1;
-
-       if (presc > TPIU_ACPR_MAX_SWOSCALER)
+       unsigned int presc = (traceclkin_freq + trace_freq / 2) / trace_freq;
+       if (presc == 0 || presc > TPIU_ACPR_MAX_SWOSCALER + 1)
                return false;
 
-       deviation = fabs(1.0 - ((double)trace_freq * presc / traceclkin_freq));
-
-       if (deviation > SWO_MAX_FREQ_DEV)
+       /* 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;
@@ -1288,7 +1268,7 @@ static bool calculate_swo_prescaler(unsigned int traceclkin_freq,
 }
 
 static bool detect_swo_freq_and_prescaler(struct jaylink_swo_speed speed,
-               unsigned int traceclkin_freq, uint32_t *trace_freq,
+               unsigned int traceclkin_freq, unsigned int *trace_freq,
                uint16_t *prescaler)
 {
        uint32_t divider;
@@ -1299,7 +1279,7 @@ static bool detect_swo_freq_and_prescaler(struct jaylink_swo_speed speed,
                *trace_freq = speed.freq / divider;
                presc = ((1.0 - SWO_MAX_FREQ_DEV) * traceclkin_freq) / *trace_freq + 1;
 
-               if (presc > TPIU_ACPR_MAX_SWOSCALER)
+               if (presc > TPIU_ACPR_MAX_SWOSCALER + 1)
                        break;
 
                deviation = fabs(1.0 - ((double)*trace_freq * presc / traceclkin_freq));
@@ -1324,22 +1304,20 @@ static int config_trace(bool enabled, enum tpiu_pin_protocol pin_protocol,
        uint32_t min_freq;
        uint32_t max_freq;
 
+       trace_enabled = enabled;
+
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_SWO)) {
-               LOG_ERROR("Trace capturing is not supported by the device.");
-               return ERROR_FAIL;
-       }
+               if (!enabled)
+                       return ERROR_OK;
 
-       if (pin_protocol != TPIU_PIN_PROTOCOL_ASYNC_UART) {
-               LOG_ERROR("Selected pin protocol is not supported.");
+               LOG_ERROR("Trace capturing is not supported by the device");
                return ERROR_FAIL;
        }
 
-       trace_enabled = enabled;
-
        ret = jaylink_swo_stop(devh);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_swo_stop() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_swo_stop() failed: %s", jaylink_strerror(ret));
                return ERROR_FAIL;
        }
 
@@ -1354,17 +1332,22 @@ static int config_trace(bool enabled, enum tpiu_pin_protocol pin_protocol,
                return ERROR_OK;
        }
 
+       if (pin_protocol != TPIU_PIN_PROTOCOL_ASYNC_UART) {
+               LOG_ERROR("Selected pin protocol is not supported");
+               return ERROR_FAIL;
+       }
+
        buffer_size = calculate_trace_buffer_size();
 
        if (!buffer_size) {
-               LOG_ERROR("Not enough free device memory to start trace capturing.");
+               LOG_ERROR("Not enough free device memory to start trace capturing");
                return ERROR_FAIL;
        }
 
        ret = jaylink_swo_get_speeds(devh, JAYLINK_SWO_MODE_UART, &speed);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_swo_get_speeds() failed: %s.",
+               LOG_ERROR("jaylink_swo_get_speeds() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_FAIL;
        }
@@ -1375,49 +1358,49 @@ static int config_trace(bool enabled, enum tpiu_pin_protocol pin_protocol,
                max_freq = speed.freq / speed.min_div;
 
                if (*trace_freq > max_freq) {
-                       LOG_INFO("Given SWO frequency too high, using %u Hz instead.",
+                       LOG_INFO("Given SWO frequency too high, using %" PRIu32 " Hz instead",
                                max_freq);
                        *trace_freq = max_freq;
                } else if (*trace_freq < min_freq) {
-                       LOG_INFO("Given SWO frequency too low, using %u Hz instead.",
+                       LOG_INFO("Given SWO frequency too low, using %" PRIu32 " Hz instead",
                                min_freq);
                        *trace_freq = min_freq;
                } else if (*trace_freq != speed.freq / divider) {
                        *trace_freq = speed.freq / divider;
 
                        LOG_INFO("Given SWO frequency is not supported by the device, "
-                               "using %u Hz instead.", *trace_freq);
+                               "using %u Hz instead", *trace_freq);
                }
 
                if (!calculate_swo_prescaler(traceclkin_freq, *trace_freq,
                                prescaler)) {
                        LOG_ERROR("SWO frequency is not suitable. Please choose a "
-                               "different frequency or use auto-detection.");
+                               "different frequency or use auto-detection");
                        return ERROR_FAIL;
                }
        } else {
-               LOG_INFO("Trying to auto-detect SWO frequency.");
+               LOG_INFO("Trying to auto-detect SWO frequency");
 
                if (!detect_swo_freq_and_prescaler(speed, traceclkin_freq, trace_freq,
                                prescaler)) {
                        LOG_ERROR("Maximum permitted frequency deviation of %.02f %% "
-                               "could not be achieved.", SWO_MAX_FREQ_DEV);
-                       LOG_ERROR("Auto-detection of SWO frequency failed.");
+                               "could not be achieved", SWO_MAX_FREQ_DEV);
+                       LOG_ERROR("Auto-detection of SWO frequency failed");
                        return ERROR_FAIL;
                }
 
-               LOG_INFO("Using SWO frequency of %u Hz.", *trace_freq);
+               LOG_INFO("Using SWO frequency of %u Hz", *trace_freq);
        }
 
        ret = jaylink_swo_start(devh, JAYLINK_SWO_MODE_UART, *trace_freq,
                buffer_size);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_start_swo() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_start_swo() failed: %s", jaylink_strerror(ret));
                return ERROR_FAIL;
        }
 
-       LOG_DEBUG("Using %u bytes device memory for trace capturing.",
+       LOG_DEBUG("Using %" PRIu32 " bytes device memory for trace capturing",
                buffer_size);
 
        /*
@@ -1436,7 +1419,7 @@ COMMAND_HANDLER(jlink_handle_config_usb_address_command)
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_READ_CONFIG)) {
                command_print(CMD, "Reading configuration is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
@@ -1444,19 +1427,18 @@ COMMAND_HANDLER(jlink_handle_config_usb_address_command)
                show_config_usb_address(CMD);
        } else if (CMD_ARGC == 1) {
                if (sscanf(CMD_ARGV[0], "%" SCNd8, &tmp) != 1) {
-                       command_print(CMD, "Invalid USB address: %s.", CMD_ARGV[0]);
+                       command_print(CMD, "Invalid USB address: %s", CMD_ARGV[0]);
                        return ERROR_FAIL;
                }
 
                if (tmp > JAYLINK_USB_ADDRESS_3) {
-                       command_print(CMD, "Invalid USB address: %u.", tmp);
+                       command_print(CMD, "Invalid USB address: %u", tmp);
                        return ERROR_FAIL;
                }
 
                tmp_config.usb_address = tmp;
        } else {
-               command_print(CMD, "Need exactly one argument for jlink config "
-                       "usb.");
+               command_print(CMD, "Need exactly one argument for jlink config usb");
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
 
@@ -1469,13 +1451,13 @@ COMMAND_HANDLER(jlink_handle_config_target_power_command)
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_READ_CONFIG)) {
                command_print(CMD, "Reading configuration is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_SET_TARGET_POWER)) {
                command_print(CMD, "Target power supply is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
@@ -1487,14 +1469,14 @@ COMMAND_HANDLER(jlink_handle_config_target_power_command)
                } else if (!strcmp(CMD_ARGV[0], "off")) {
                        enable = false;
                } else {
-                       command_print(CMD, "Invalid argument: %s.", CMD_ARGV[0]);
+                       command_print(CMD, "Invalid argument: %s", CMD_ARGV[0]);
                        return ERROR_FAIL;
                }
 
                tmp_config.target_power = enable;
        } else {
                command_print(CMD, "Need exactly one argument for jlink config "
-                       "targetpower.");
+                       "targetpower");
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
 
@@ -1510,13 +1492,13 @@ COMMAND_HANDLER(jlink_handle_config_mac_address_command)
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_READ_CONFIG)) {
                command_print(CMD, "Reading configuration is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_ETHERNET)) {
                command_print(CMD, "Ethernet connectivity is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
@@ -1525,9 +1507,9 @@ COMMAND_HANDLER(jlink_handle_config_mac_address_command)
        } else if (CMD_ARGC == 1) {
                str = CMD_ARGV[0];
 
-               if ((strlen(str) != 17) || (str[2] != ':' || str[5] != ':' || \
+               if ((strlen(str) != 17) || (str[2] != ':' || str[5] != ':' ||
                                str[8] != ':' || str[11] != ':' || str[14] != ':')) {
-                       command_print(CMD, "Invalid MAC address format.");
+                       command_print(CMD, "Invalid MAC address format");
                        return ERROR_COMMAND_SYNTAX_ERROR;
                }
 
@@ -1537,19 +1519,18 @@ COMMAND_HANDLER(jlink_handle_config_mac_address_command)
                }
 
                if (!(addr[0] | addr[1] | addr[2] | addr[3] | addr[4] | addr[5])) {
-                       command_print(CMD, "Invalid MAC address: zero address.");
+                       command_print(CMD, "Invalid MAC address: zero address");
                        return ERROR_COMMAND_SYNTAX_ERROR;
                }
 
                if (!(0x01 & addr[0])) {
-                       command_print(CMD, "Invalid MAC address: multicast address.");
+                       command_print(CMD, "Invalid MAC address: multicast address");
                        return ERROR_COMMAND_SYNTAX_ERROR;
                }
 
                memcpy(tmp_config.mac_address, addr, sizeof(addr));
        } else {
-               command_print(CMD, "Need exactly one argument for jlink config "
-                       " mac.");
+               command_print(CMD, "Need exactly one argument for jlink config mac");
                return ERROR_COMMAND_SYNTAX_ERROR;
        }
 
@@ -1598,13 +1579,13 @@ COMMAND_HANDLER(jlink_handle_config_ip_address_command)
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_READ_CONFIG)) {
                command_print(CMD, "Reading configuration is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_ETHERNET)) {
                command_print(CMD, "Ethernet connectivity is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
@@ -1655,44 +1636,44 @@ COMMAND_HANDLER(jlink_handle_config_write_command)
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_READ_CONFIG)) {
                command_print(CMD, "Reading configuration is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_WRITE_CONFIG)) {
                command_print(CMD, "Writing configuration is not supported by the "
-                       "device.");
+                       "device");
                return ERROR_OK;
        }
 
        if (!memcmp(&config, &tmp_config, sizeof(struct device_config))) {
                command_print(CMD, "Operation not performed due to no changes in "
-                       "the configuration.");
+                       "the configuration");
                return ERROR_OK;
        }
 
        ret = jaylink_write_raw_config(devh, (const uint8_t *)&tmp_config);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_write_raw_config() failed: %s.",
+               LOG_ERROR("jaylink_write_raw_config() failed: %s",
                        jaylink_strerror(ret));
                return ERROR_FAIL;
        }
 
        if (!read_device_config(&config)) {
-               LOG_ERROR("Failed to read device configuration for verification.");
+               LOG_ERROR("Failed to read device configuration for verification");
                return ERROR_FAIL;
        }
 
        if (memcmp(&config, &tmp_config, sizeof(struct device_config))) {
                LOG_ERROR("Verification of device configuration failed. Please check "
-                       "your device.");
+                       "your device");
                return ERROR_FAIL;
        }
 
        memcpy(&tmp_config, &config, sizeof(struct device_config));
        command_print(CMD, "The new device configuration applies after power "
-               "cycling the J-Link device.");
+               "cycling the J-Link device");
 
        return ERROR_OK;
 }
@@ -1700,7 +1681,7 @@ COMMAND_HANDLER(jlink_handle_config_write_command)
 COMMAND_HANDLER(jlink_handle_config_command)
 {
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_READ_CONFIG)) {
-               command_print(CMD, "Device doesn't support reading configuration.");
+               command_print(CMD, "Device doesn't support reading configuration");
                return ERROR_OK;
        }
 
@@ -1723,7 +1704,7 @@ COMMAND_HANDLER(jlink_handle_emucom_write_command)
                return ERROR_COMMAND_SYNTAX_ERROR;
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_EMUCOM)) {
-               LOG_ERROR("Device does not support EMUCOM.");
+               LOG_ERROR("Device does not support EMUCOM");
                return ERROR_FAIL;
        }
 
@@ -1732,21 +1713,21 @@ COMMAND_HANDLER(jlink_handle_emucom_write_command)
        tmp = strlen(CMD_ARGV[1]);
 
        if (tmp % 2 != 0) {
-               LOG_ERROR("Data must be encoded as hexadecimal pairs.");
+               LOG_ERROR("Data must be encoded as hexadecimal pairs");
                return ERROR_COMMAND_ARGUMENT_INVALID;
        }
 
        buf = malloc(tmp / 2);
 
        if (!buf) {
-               LOG_ERROR("Failed to allocate buffer.");
+               LOG_ERROR("Failed to allocate buffer");
                return ERROR_FAIL;
        }
 
        dummy = unhexify(buf, CMD_ARGV[1], tmp / 2);
 
        if (dummy != (tmp / 2)) {
-               LOG_ERROR("Data must be encoded as hexadecimal pairs.");
+               LOG_ERROR("Data must be encoded as hexadecimal pairs");
                free(buf);
                return ERROR_COMMAND_ARGUMENT_INVALID;
        }
@@ -1757,15 +1738,15 @@ COMMAND_HANDLER(jlink_handle_emucom_write_command)
        free(buf);
 
        if (ret == JAYLINK_ERR_DEV_NOT_SUPPORTED) {
-               LOG_ERROR("Channel not supported by the device.");
+               LOG_ERROR("Channel not supported by the device");
                return ERROR_FAIL;
        } else if (ret != JAYLINK_OK) {
-               LOG_ERROR("Failed to write to channel: %s.", jaylink_strerror(ret));
+               LOG_ERROR("Failed to write to channel: %s", jaylink_strerror(ret));
                return ERROR_FAIL;
        }
 
        if (length != (tmp / 2))
-               LOG_WARNING("Only %" PRIu32 " bytes written to the channel.", length);
+               LOG_WARNING("Only %" PRIu32 " bytes written to the channel", length);
 
        return ERROR_OK;
 }
@@ -1782,7 +1763,7 @@ COMMAND_HANDLER(jlink_handle_emucom_read_command)
                return ERROR_COMMAND_SYNTAX_ERROR;
 
        if (!jaylink_has_cap(caps, JAYLINK_DEV_CAP_EMUCOM)) {
-               LOG_ERROR("Device does not support EMUCOM.");
+               LOG_ERROR("Device does not support EMUCOM");
                return ERROR_FAIL;
        }
 
@@ -1792,23 +1773,23 @@ COMMAND_HANDLER(jlink_handle_emucom_read_command)
        buf = malloc(length * 3 + 1);
 
        if (!buf) {
-               LOG_ERROR("Failed to allocate buffer.");
+               LOG_ERROR("Failed to allocate buffer");
                return ERROR_FAIL;
        }
 
        ret = jaylink_emucom_read(devh, channel, buf, &length);
 
        if (ret == JAYLINK_ERR_DEV_NOT_SUPPORTED) {
-               LOG_ERROR("Channel is not supported by the device.");
+               LOG_ERROR("Channel is not supported by the device");
                free(buf);
                return ERROR_FAIL;
        } else if (ret == JAYLINK_ERR_DEV_NOT_AVAILABLE) {
                LOG_ERROR("Channel is not available for the requested amount of data. "
-                       "%" PRIu32 " bytes are avilable.", length);
+                       "%" PRIu32 " bytes are available", length);
                free(buf);
                return ERROR_FAIL;
        } else if (ret != JAYLINK_OK) {
-               LOG_ERROR("Failed to read from channel: %s.", jaylink_strerror(ret));
+               LOG_ERROR("Failed to read from channel: %s", jaylink_strerror(ret));
                free(buf);
                return ERROR_FAIL;
        }
@@ -1816,7 +1797,7 @@ COMMAND_HANDLER(jlink_handle_emucom_read_command)
        tmp = hexify((char *)buf + length, buf, length, 2 * length + 1);
 
        if (tmp != 2 * length) {
-               LOG_ERROR("Failed to convert data into hexadecimal string.");
+               LOG_ERROR("Failed to convert data into hexadecimal string");
                free(buf);
                return ERROR_FAIL;
        }
@@ -1928,13 +1909,6 @@ static const struct command_registration jlink_subcommand_handlers[] = {
                .help = "set the USB address of the device that should be used",
                .usage = "<0-3>"
        },
-       {
-               .name = "serial",
-               .handler = &jlink_serial_command,
-               .mode = COMMAND_CONFIG,
-               .help = "set the serial number of the device that should be used",
-               .usage = "<serial number>"
-       },
        {
                .name = "config",
                .handler = &jlink_handle_config_command,
@@ -1974,13 +1948,13 @@ static int jlink_swd_init(void)
 
 static void jlink_swd_write_reg(uint8_t cmd, uint32_t value, uint32_t ap_delay_clk)
 {
-       assert(!(cmd & SWD_CMD_RnW));
+       assert(!(cmd & SWD_CMD_RNW));
        jlink_swd_queue_cmd(cmd, NULL, value, ap_delay_clk);
 }
 
 static void jlink_swd_read_reg(uint8_t cmd, uint32_t *value, uint32_t ap_delay_clk)
 {
-       assert(cmd & SWD_CMD_RnW);
+       assert(cmd & SWD_CMD_RNW);
        jlink_swd_queue_cmd(cmd, value, 0, ap_delay_clk);
 }
 
@@ -2074,7 +2048,7 @@ static int jlink_flush(void)
                tap_length, jtag_command_version);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_jtag_io() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_jtag_io() failed: %s", jaylink_strerror(ret));
                jlink_tap_init();
                return ERROR_JTAG_QUEUE_FAILED;
        }
@@ -2085,7 +2059,7 @@ static int jlink_flush(void)
                buf_set_buf(tdo_buffer, p->first, p->buffer,
                            p->buffer_offset, p->length);
 
-               LOG_DEBUG_IO("Pending scan result, length = %d.", p->length);
+               LOG_DEBUG_IO("Pending scan result, length = %d", p->length);
        }
 
        jlink_tap_init();
@@ -2144,13 +2118,33 @@ static int jlink_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);
+                       LOG_ERROR("Sequence %d not supported", seq);
                        return ERROR_FAIL;
        }
 
@@ -2164,10 +2158,10 @@ static int jlink_swd_run_queue(void)
        int i;
        int ret;
 
-       LOG_DEBUG("Executing %d queued transactions.", pending_scan_results_length);
+       LOG_DEBUG("Executing %d queued transactions", pending_scan_results_length);
 
        if (queued_retval != ERROR_OK) {
-               LOG_DEBUG("Skipping due to previous errors: %d.", queued_retval);
+               LOG_DEBUG("Skipping due to previous errors: %d", queued_retval);
                goto skip;
        }
 
@@ -2180,7 +2174,7 @@ static int jlink_swd_run_queue(void)
        ret = jaylink_swd_io(devh, tms_buffer, tdi_buffer, tdo_buffer, tap_length);
 
        if (ret != JAYLINK_OK) {
-               LOG_ERROR("jaylink_swd_io() failed: %s.", jaylink_strerror(ret));
+               LOG_ERROR("jaylink_swd_io() failed: %s", jaylink_strerror(ret));
                goto skip;
        }
 
@@ -2197,7 +2191,7 @@ static int jlink_swd_run_queue(void)
                        int parity = buf_get_u32(tdo_buffer, 3 + 32 + pending_scan_results_buffer[i].first, 1);
 
                        if (parity != parity_u32(data)) {
-                               LOG_ERROR("SWD: Read data parity mismatch.");
+                               LOG_ERROR("SWD: Read data parity mismatch");
                                queued_retval = ERROR_FAIL;
                                goto skip;
                        }
@@ -2233,7 +2227,7 @@ static void jlink_swd_queue_cmd(uint8_t cmd, uint32_t *dst, uint32_t data, uint3
 
        pending_scan_results_buffer[pending_scan_results_length].first = tap_length;
 
-       if (cmd & SWD_CMD_RnW) {
+       if (cmd & SWD_CMD_RNW) {
                /* Queue a read transaction. */
                pending_scan_results_buffer[pending_scan_results_length].length = 32;
                pending_scan_results_buffer[pending_scan_results_length].buffer = dst;
@@ -2253,7 +2247,7 @@ static void jlink_swd_queue_cmd(uint8_t cmd, uint32_t *dst, uint32_t data, uint3
        pending_scan_results_length++;
 
        /* Insert idle cycles after AP accesses to avoid WAIT. */
-       if (cmd & SWD_CMD_APnDP)
+       if (cmd & SWD_CMD_APNDP)
                jlink_queue_data_out(NULL, ap_delay_clk);
 }