diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/jtag/drivers/stlink_usb.c | 12 | ||||
-rw-r--r-- | src/jtag/hla/hla_interface.c | 20 |
2 files changed, 18 insertions, 14 deletions
diff --git a/src/jtag/drivers/stlink_usb.c b/src/jtag/drivers/stlink_usb.c index 31f08cbb..e94e6c8a 100644 --- a/src/jtag/drivers/stlink_usb.c +++ b/src/jtag/drivers/stlink_usb.c @@ -855,9 +855,11 @@ static void stlink_usb_trace_read(void *handle) res = stlink_usb_read_trace(handle, buf, size); if (res == ERROR_OK) { - /* Log retrieved trace output */ - if (fwrite(buf, 1, size, h->trace.output_f) > 0) - fflush(h->trace.output_f); + if (h->trace.output_f) { + /* Log retrieved trace output */ + if (fwrite(buf, 1, size, h->trace.output_f) > 0) + fflush(h->trace.output_f); + } } } } @@ -1113,7 +1115,7 @@ static int stlink_usb_run(void *handle) res = stlink_usb_write_debug_reg(handle, DCB_DHCSR, DBGKEY|C_DEBUGEN); /* Try to start tracing, if requested */ - if (res == ERROR_OK && h->trace.output_f) { + if (res == ERROR_OK && h->trace.source_hz) { if (stlink_usb_trace_enable(handle) == ERROR_OK) LOG_DEBUG("Tracing: enabled\n"); else @@ -1732,7 +1734,7 @@ static int stlink_usb_open(struct hl_interface_param_s *param, void **fd) /* set the used jtag api, this will default to the newest supported version */ h->jtag_api = api; - if (h->jtag_api >= 2 && param->trace_f && param->trace_source_hz > 0) { + if (h->jtag_api >= 2 && param->trace_source_hz > 0) { uint32_t prescale; prescale = param->trace_source_hz > STLINK_TRACE_MAX_HZ ? diff --git a/src/jtag/hla/hla_interface.c b/src/jtag/hla/hla_interface.c index 02d0f2fb..ae33f54e 100644 --- a/src/jtag/hla/hla_interface.c +++ b/src/jtag/hla/hla_interface.c @@ -119,6 +119,7 @@ static int hl_interface_quit(void) fclose(hl_if.param.trace_f); hl_if.param.trace_f = NULL; } + hl_if.param.trace_source_hz = 0; return ERROR_OK; } @@ -230,22 +231,23 @@ COMMAND_HANDLER(stlink_interface_handle_api_command) COMMAND_HANDLER(interface_handle_trace_command) { - FILE *f; + FILE *f = NULL; unsigned source_hz; - if (CMD_ARGC != 2) + if ((CMD_ARGC < 1) || (CMD_ARGC > 2)) return ERROR_COMMAND_SYNTAX_ERROR; - f = fopen(CMD_ARGV[0], "a"); - if (!f) - return ERROR_COMMAND_SYNTAX_ERROR; - - COMMAND_PARSE_NUMBER(uint, CMD_ARGV[1], source_hz); + COMMAND_PARSE_NUMBER(uint, CMD_ARGV[0], source_hz); if (source_hz == 0) { - fclose(f); return ERROR_COMMAND_SYNTAX_ERROR; } + if (CMD_ARGC == 2) { + f = fopen(CMD_ARGV[0], "a"); + if (!f) + return ERROR_COMMAND_SYNTAX_ERROR; + } + hl_if.param.trace_f = f; hl_if.param.trace_source_hz = source_hz; @@ -293,7 +295,7 @@ static const struct command_registration hl_interface_command_handlers[] = { .handler = &interface_handle_trace_command, .mode = COMMAND_CONFIG, .help = "configure trace reception", - .usage = "destination_path source_lock_hz", + .usage = "source_lock_hz [destination_path]", }, COMMAND_REGISTRATION_DONE }; |