This is an automated email from Gerrit.

Anonymous Coward (mr.nuke...@gmail.com) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/5340

-- gerrit

commit b324b0743e46fb64b417128a5805c37183672bfa
Author: Alexandru Gagniuc <mr.nuke...@gmail.com>
Date:   Tue Oct 29 22:14:46 2019 -0500

    jtag: jtag_vpi: Add missing 'default' to switch statement
    
    If a new JTAG command is added, then GCC will complain that
    enumeration value not handled in switch. Make this consistent with
    other drivers, and add a 'default' case.
    
    Change-Id: I66d6d0db3fcae93ea246f2d4882ffff5dec14693
    Signed-off-by: Alexandru Gagniuc <mr.nuke...@gmail.com>

diff --git a/src/jtag/drivers/jtag_vpi.c b/src/jtag/drivers/jtag_vpi.c
index 1033ced..3e39420 100644
--- a/src/jtag/drivers/jtag_vpi.c
+++ b/src/jtag/drivers/jtag_vpi.c
@@ -384,6 +384,11 @@ static int jtag_vpi_execute_queue(void)
                case JTAG_SCAN:
                        retval = jtag_vpi_scan(cmd->cmd.scan);
                        break;
+               default:
+                       LOG_ERROR("BUG: unknown JTAG command type 0x%X",
+                                 cmd->type);
+                       retval = ERROR_FAIL;
+                       break;
                }
        }
 

-- 


_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to