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/+/9035

-- gerrit

commit c3cddf1bc443b7c4cee4eaa376f50b9dbc773859
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Fri Jul 25 20:07:27 2025 +0200

    flash: nor: jtagspi: invert the operands in check
    
    Check the variable against the constant.
    No functional change.
    
    Change-Id: Ie3138e516f4b81bf544780981863e856f8ffc528
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/flash/nor/jtagspi.c b/src/flash/nor/jtagspi.c
index 4b975390be..e1bb17ab1f 100644
--- a/src/flash/nor/jtagspi.c
+++ b/src/flash/nor/jtagspi.c
@@ -230,7 +230,7 @@ COMMAND_HANDLER(jtagspi_handle_set)
        */
        retval = CALL_COMMAND_HANDLER(flash_command_get_bank_probe_optional, 0,
                &bank, false);
-       if (ERROR_OK != retval)
+       if (retval != ERROR_OK)
                return retval;
        info = bank->driver_priv;
 
@@ -420,7 +420,7 @@ COMMAND_HANDLER(jtagspi_handle_always_4byte)
                return ERROR_COMMAND_SYNTAX_ERROR;
 
        retval = CALL_COMMAND_HANDLER(flash_command_get_bank, 0, &bank);
-       if (ERROR_OK != retval)
+       if (retval != ERROR_OK)
                return retval;
 
        jtagspi_info = bank->driver_priv;

-- 

Reply via email to