This is an automated email from Gerrit.

"zapb <d...@zapb.de>" just uploaded a new patch set to Gerrit, which you can 
find at https://review.openocd.org/c/openocd/+/7992

-- gerrit

commit cf19d580f47ec74d1b54ec1651ddbf122aa830c0
Author: Marc Schink <d...@zapb.de>
Date:   Thu Nov 9 10:13:10 2023 +0100

    jtag/drivers/rshim: Remove redundant error message
    
    The correct syntax is already suggested due to the return
    value used.
    
    Change-Id: I0f4a7f93fdf056e7517c754d6d4ecd7928f1d226
    Signed-off-by: Marc Schink <d...@zapb.de>

diff --git a/src/jtag/drivers/rshim.c b/src/jtag/drivers/rshim.c
index 6170e86bdb..21fc7fd378 100644
--- a/src/jtag/drivers/rshim.c
+++ b/src/jtag/drivers/rshim.c
@@ -434,10 +434,8 @@ static void rshim_disconnect(struct adiv5_dap *dap)
 
 COMMAND_HANDLER(rshim_dap_device_command)
 {
-       if (CMD_ARGC != 1) {
-               command_print(CMD, "Too many arguments");
+       if (CMD_ARGC != 1)
                return ERROR_COMMAND_SYNTAX_ERROR;
-       }
 
        free(rshim_dev_path);
        rshim_dev_path = strdup(CMD_ARGV[0]);

-- 

Reply via email to