This is an automated email from Gerrit. "Daniel Anselmi <danse...@gmx.ch>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7236
-- gerrit commit 0ab07976d2079e273a7abbc08aafbbaa3e5a0807 Author: Daniel Anselmi <danse...@gmx.ch> Date: Sun Oct 2 02:16:28 2022 +0200 don't return ERROR_OK in error cases Change-Id: I7e046df85838692c9044fe9c9d67e8b2c821eb0f Signed-off-by: Daniel Anselmi <danse...@gmx.ch> diff --git a/src/pld/virtex2.c b/src/pld/virtex2.c index b4861e9cb8..98fd58adbb 100644 --- a/src/pld/virtex2.c +++ b/src/pld/virtex2.c @@ -175,7 +175,7 @@ COMMAND_HANDLER(virtex2_handle_read_stat_command) device = get_pld_device_by_num(dev_id); if (!device) { command_print(CMD, "pld device '#%s' is out of bounds", CMD_ARGV[0]); - return ERROR_OK; + return ERROR_FAIL; } virtex2_read_stat(device, &status); @@ -197,7 +197,7 @@ PLD_DEVICE_COMMAND_HANDLER(virtex2_pld_device_command) tap = jtag_tap_by_string(CMD_ARGV[1]); if (!tap) { command_print(CMD, "Tap: %s does not exist", CMD_ARGV[1]); - return ERROR_OK; + return ERROR_FAIL; } virtex2_info = malloc(sizeof(struct virtex2_pld_device)); --