From: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Date: Wed, 30 Oct 2019 03:14:46 +0000 (-0500)
Subject: jtag: jtag_vpi: Add missing 'default' to switch statement
X-Git-Tag: v0.11.0-rc1~574
X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=commitdiff_plain;h=4f779a88db4c03c3cf280275c383fd3f24f9fea3;ds=sidebyside

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.me@gmail.com>
Reviewed-on: http://openocd.zylin.com/5340
Tested-by: jenkins
Reviewed-by: Jan Matyas <matyas@codasip.com>
Reviewed-by: Franck Jullien <franck.jullien@gmail.com>
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
---

diff --git a/src/jtag/drivers/jtag_vpi.c b/src/jtag/drivers/jtag_vpi.c
index 1033cedb28..3e39420fb3 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;
 		}
 	}