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);

-- 

Reply via email to