This is an automated email from Gerrit.

"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7413

-- gerrit

commit 68c85ffd379131e1c93e09d9b208e5127da2d7f8
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Mon Dec 19 17:20:06 2022 +0100

    target: arc: fix error handling in command 'arc set-reg-exists'
    
    The command is specified through COMMAND_HANDLER. It should not
    return JIM_OK / JIM_ERR.
    
    Change-Id: I56666414d49b0298ecc23ec7ef30c77e1e27afa8
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/target/arc_cmd.c b/src/target/arc_cmd.c
index e7c54446e2..fb1817de6e 100644
--- a/src/target/arc_cmd.c
+++ b/src/target/arc_cmd.c
@@ -763,7 +763,7 @@ COMMAND_HANDLER(arc_set_reg_exists)
        struct target * const target = get_current_target(CMD_CTX);
        if (!target) {
                command_print(CMD, "Unable to get current target.");
-               return JIM_ERR;
+               return ERROR_FAIL;
        }
 
        if (!CMD_ARGC) {
@@ -783,7 +783,7 @@ COMMAND_HANDLER(arc_set_reg_exists)
                r->exist = true;
        }
 
-       return JIM_OK;
+       return ERROR_OK;
 }
 
 /* arc reg-field  ($reg_name) ($reg_field)

-- 

Reply via email to