This is an automated email from Gerrit. "Jan Matyas <mat...@codasip.com>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/6845
-- gerrit commit 6c394300bb627698a19a6aba94c9c68de183d743 Author: Jan Matyas <mat...@codasip.com> Date: Fri Feb 4 14:44:24 2022 +0100 jtag_vpi: Minor update to several log prints Several log prints in the "jtag_vpi" driver were adjusted to make them more clear to the user. No change in functionality - just the prints. Change-Id: Ibe386403a179adab5edb69c77fa408aef55701bd Signed-off-by: Jan Matyas <mat...@codasip.com> diff --git a/src/jtag/drivers/jtag_vpi.c b/src/jtag/drivers/jtag_vpi.c index 0fc688edf..64e5889e3 100644 --- a/src/jtag/drivers/jtag_vpi.c +++ b/src/jtag/drivers/jtag_vpi.c @@ -605,11 +605,11 @@ static int jtag_vpi_quit(void) COMMAND_HANDLER(jtag_vpi_set_port) { if (CMD_ARGC == 0) - LOG_WARNING("You need to set a port number"); + LOG_ERROR("Command \"jtag_vpi set_port\" expects 1 argument (TCP port number)"); else COMMAND_PARSE_NUMBER(int, CMD_ARGV[0], server_port); - LOG_INFO("Set server port to %u", server_port); + LOG_INFO("jtag_vpi: server port set to %u", server_port); return ERROR_OK; } @@ -619,12 +619,12 @@ COMMAND_HANDLER(jtag_vpi_set_address) free(server_address); if (CMD_ARGC == 0) { - LOG_WARNING("You need to set an address"); + LOG_ERROR("Command \"jtag_vpi set_address\" expects 1 argument (IP address)"); server_address = strdup(SERVER_ADDRESS); } else server_address = strdup(CMD_ARGV[0]); - LOG_INFO("Set server address to %s", server_address); + LOG_INFO("jtag_vpi: server address set to %s", server_address); return ERROR_OK; } @@ -632,7 +632,7 @@ COMMAND_HANDLER(jtag_vpi_set_address) COMMAND_HANDLER(jtag_vpi_stop_sim_on_exit_handler) { if (CMD_ARGC != 1) { - LOG_ERROR("jtag_vpi_stop_sim_on_exit expects 1 argument (on|off)"); + LOG_ERROR("Command \"jtag_vpi stop_sim_on_exit\" expects 1 argument (on|off)"); return ERROR_COMMAND_SYNTAX_ERROR; } else { COMMAND_PARSE_ON_OFF(CMD_ARGV[0], stop_sim_on_exit); --