This is an automated email from Gerrit.
"Marcus Nilsson (1002225)" just uploaded a new patch set to Gerrit, which you
can find at https://review.openocd.org/c/openocd/+/8233
-- gerrit
commit 83bbea287ae99257a0eac12dceeefd71b11a6139
Author: Marcus Nilsson <brainb...@gmail.com>
Date: Mon May 6 17:07:07 2024 +0200
drivers/cmsis_dap: Allow `quirk` command to be used before init
Commit ba16fd ("drivers/cmsis_dap: use quirk workarounds optionally")
added the `quirk` command to be able to enable or disable quirks when
using cmsis-dap. This only worked after the cmsis_dap_handle struct was
initialized however, resulting in a segmentation fault if enabled before
the init call.
Since the quirks can be needed during initialization of the SWD
interface, move the flag out of the cmsis_dap_handle struct.
Change-Id: I8ba9de84b9f985dd041257dc37048da5e57de181
Signed-off-by: Marcus Nilsson <brainb...@gmail.com>
diff --git a/src/jtag/drivers/cmsis_dap.c b/src/jtag/drivers/cmsis_dap.c
index d7367d8137..0e92a0149d 100644
--- a/src/jtag/drivers/cmsis_dap.c
+++ b/src/jtag/drivers/cmsis_dap.c
@@ -68,6 +68,7 @@ static uint16_t cmsis_dap_vid[MAX_USB_IDS + 1] = { 0 };
static uint16_t cmsis_dap_pid[MAX_USB_IDS + 1] = { 0 };
static int cmsis_dap_backend = -1;
static bool swd_mode;
+static bool quirk_mode; /* enable expensive workarounds */
/* CMSIS-DAP General Commands */
#define CMD_DAP_INFO 0x00
@@ -861,7 +862,7 @@ static void cmsis_dap_swd_write_from_queue(struct cmsis_dap
*dap)
goto skip;
}
- unsigned int packet_count = dap->quirk_mode ? 1 : dap->packet_count;
+ unsigned int packet_count = quirk_mode ? 1 : dap->packet_count;
dap->pending_fifo_put_idx = (dap->pending_fifo_put_idx + 1) %
packet_count;
dap->pending_fifo_block_count++;
if (dap->pending_fifo_block_count > packet_count)
@@ -985,7 +986,7 @@ static void cmsis_dap_swd_read_process(struct cmsis_dap
*dap, enum cmsis_dap_blo
skip:
block->transfer_count = 0;
- if (!dap->quirk_mode && dap->packet_count > 1)
+ if (!quirk_mode && dap->packet_count > 1)
dap->pending_fifo_get_idx = (dap->pending_fifo_get_idx + 1) %
dap->packet_count;
dap->pending_fifo_block_count--;
}
@@ -1081,7 +1082,7 @@ static void cmsis_dap_swd_queue_cmd(uint8_t cmd, uint32_t
*dst, uint32_t data)
/* Not enough room in the queue. Run the queue. */
cmsis_dap_swd_write_from_queue(cmsis_dap_handle);
- unsigned int packet_count = cmsis_dap_handle->quirk_mode ? 1 :
cmsis_dap_handle->packet_count;
+ unsigned int packet_count = quirk_mode ? 1 :
cmsis_dap_handle->packet_count;
if (cmsis_dap_handle->pending_fifo_block_count >= packet_count)
cmsis_dap_swd_read_process(cmsis_dap_handle,
CMSIS_DAP_BLOCKING);
}
@@ -1225,7 +1226,7 @@ static int cmsis_dap_swd_switch_seq(enum swd_special_seq
seq)
if (swd_mode)
queued_retval = cmsis_dap_swd_run_queue();
- if (cmsis_dap_handle->quirk_mode && seq != LINE_RESET &&
+ if (quirk_mode && seq != LINE_RESET &&
(output_pins & (SWJ_PIN_SRST | SWJ_PIN_TRST))
== (SWJ_PIN_SRST | SWJ_PIN_TRST)) {
/* Following workaround deasserts reset on most adapters.
@@ -2229,10 +2230,10 @@ COMMAND_HANDLER(cmsis_dap_handle_quirk_command)
return ERROR_COMMAND_SYNTAX_ERROR;
if (CMD_ARGC == 1)
- COMMAND_PARSE_ENABLE(CMD_ARGV[0], cmsis_dap_handle->quirk_mode);
+ COMMAND_PARSE_ENABLE(CMD_ARGV[0], quirk_mode);
command_print(CMD, "CMSIS-DAP quirk workarounds %s",
- cmsis_dap_handle->quirk_mode ? "enabled" :
"disabled");
+ quirk_mode ? "enabled" : "disabled");
return ERROR_OK;
}
diff --git a/src/jtag/drivers/cmsis_dap.h b/src/jtag/drivers/cmsis_dap.h
index e47697d1f9..817636f346 100644
--- a/src/jtag/drivers/cmsis_dap.h
+++ b/src/jtag/drivers/cmsis_dap.h
@@ -52,7 +52,6 @@ struct cmsis_dap {
unsigned int pending_fifo_block_count;
uint16_t caps;
- bool quirk_mode; /* enable expensive workarounds */
uint32_t swo_buf_sz;
bool trace_enabled;
--
