From 16b6b5e7a86353dbc0c4823fe3d772c0faca7c1c Mon Sep 17 00:00:00 2001 From: Mathias K Date: Fri, 16 Dec 2011 07:48:39 +0100 Subject: Change return value on error. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On wrong parameters a error is signalized to the calling function. Change-Id: I484443fdb39938e20382edc9246d5ec546a5c960 Signed-off-by: Mathias K Signed-off-by: Øyvind Harboe Reviewed-on: http://openocd.zylin.com/282 Tested-by: jenkins Reviewed-by: Øyvind Harboe --- src/jtag/drivers/buspirate.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) (limited to 'src/jtag/drivers/buspirate.c') diff --git a/src/jtag/drivers/buspirate.c b/src/jtag/drivers/buspirate.c index 41e3e194..f3edc8a5 100644 --- a/src/jtag/drivers/buspirate.c +++ b/src/jtag/drivers/buspirate.c @@ -280,8 +280,7 @@ COMMAND_HANDLER(buspirate_handle_adc_command) COMMAND_HANDLER(buspirate_handle_vreg_command) { if (CMD_ARGC < 1) { - LOG_ERROR("usage: buspirate_vreg <1|0>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } if (atoi(CMD_ARGV[0]) == 1) @@ -298,8 +297,7 @@ COMMAND_HANDLER(buspirate_handle_vreg_command) COMMAND_HANDLER(buspirate_handle_pullup_command) { if (CMD_ARGC < 1) { - LOG_ERROR("usage: buspirate_pullup <1|0>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } if (atoi(CMD_ARGV[0]) == 1) @@ -316,8 +314,7 @@ COMMAND_HANDLER(buspirate_handle_pullup_command) COMMAND_HANDLER(buspirate_handle_led_command) { if (CMD_ARGC < 1) { - LOG_ERROR("usage: buspirate_led <1|0>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } if (atoi(CMD_ARGV[0]) == 1) { @@ -339,8 +336,7 @@ COMMAND_HANDLER(buspirate_handle_led_command) COMMAND_HANDLER(buspirate_handle_mode_command) { if (CMD_ARGC < 1) { - LOG_ERROR("usage: buspirate_mode "); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } if (CMD_ARGV[0][0] == 'n') @@ -357,8 +353,7 @@ COMMAND_HANDLER(buspirate_handle_mode_command) COMMAND_HANDLER(buspirate_handle_speed_command) { if (CMD_ARGC < 1) { - LOG_ERROR("usage: buspirate_speed "); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } if (CMD_ARGV[0][0] == 'n') @@ -375,8 +370,7 @@ COMMAND_HANDLER(buspirate_handle_speed_command) COMMAND_HANDLER(buspirate_handle_port_command) { if (CMD_ARGC < 1) { - LOG_ERROR("usage: buspirate_port /dev/ttyUSB0"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } if (buspirate_port == NULL) @@ -395,36 +389,42 @@ static const struct command_registration buspirate_command_handlers[] = { }, { .name = "buspirate_vreg", + .usage = "<1|0>", .handler = &buspirate_handle_vreg_command, .mode = COMMAND_CONFIG, .help = "changes the state of voltage regulators", }, { .name = "buspirate_pullup", + .usage = "<1|0>", .handler = &buspirate_handle_pullup_command, .mode = COMMAND_CONFIG, .help = "changes the state of pullup", }, { .name = "buspirate_led", + .usage = "<1|0>", .handler = &buspirate_handle_led_command, .mode = COMMAND_EXEC, .help = "changes the state of led", }, { .name = "buspirate_speed", + .usage = "", .handler = &buspirate_handle_speed_command, .mode = COMMAND_CONFIG, .help = "speed of the interface", }, { .name = "buspirate_mode", + .usage = "", .handler = &buspirate_handle_mode_command, .mode = COMMAND_CONFIG, .help = "pin mode of the interface", }, { .name = "buspirate_port", + .usage = "/dev/ttyUSB0", .handler = &buspirate_handle_port_command, .mode = COMMAND_CONFIG, .help = "name of the serial port to open", -- cgit v1.2.3-18-g5258