diff options
Diffstat (limited to 'src/flash/nor')
-rw-r--r-- | src/flash/nor/at91sam3.c | 1 | ||||
-rw-r--r-- | src/flash/nor/at91sam7.c | 4 | ||||
-rw-r--r-- | src/flash/nor/avrf.c | 7 | ||||
-rw-r--r-- | src/flash/nor/cfi.c | 3 | ||||
-rw-r--r-- | src/flash/nor/ecos.c | 3 | ||||
-rw-r--r-- | src/flash/nor/em357.c | 18 | ||||
-rw-r--r-- | src/flash/nor/faux.c | 3 | ||||
-rw-r--r-- | src/flash/nor/fm3.c | 8 | ||||
-rw-r--r-- | src/flash/nor/kinetis.c | 4 | ||||
-rw-r--r-- | src/flash/nor/lpc2000.c | 3 | ||||
-rw-r--r-- | src/flash/nor/lpc288x.c | 3 | ||||
-rw-r--r-- | src/flash/nor/lpc2900.c | 8 | ||||
-rw-r--r-- | src/flash/nor/ocl.c | 3 | ||||
-rw-r--r-- | src/flash/nor/pic32mx.c | 9 | ||||
-rw-r--r-- | src/flash/nor/stellaris.c | 8 | ||||
-rw-r--r-- | src/flash/nor/stm32f1x.c | 19 | ||||
-rw-r--r-- | src/flash/nor/stm32f2x.c | 3 | ||||
-rw-r--r-- | src/flash/nor/stm32lx.c | 3 | ||||
-rw-r--r-- | src/flash/nor/stmsmi.c | 3 | ||||
-rw-r--r-- | src/flash/nor/str7x.c | 7 | ||||
-rw-r--r-- | src/flash/nor/str9x.c | 3 | ||||
-rw-r--r-- | src/flash/nor/str9xpec.c | 43 | ||||
-rw-r--r-- | src/flash/nor/tms470.c | 6 | ||||
-rw-r--r-- | src/flash/nor/virtual.c | 3 |
24 files changed, 70 insertions, 105 deletions
diff --git a/src/flash/nor/at91sam3.c b/src/flash/nor/at91sam3.c index a5fbd43a..dac901ec 100644 --- a/src/flash/nor/at91sam3.c +++ b/src/flash/nor/at91sam3.c @@ -3117,7 +3117,6 @@ COMMAND_HANDLER(sam3_handle_gpnvm_command) switch (CMD_ARGC) { default: - command_print(CMD_CTX,"Too many parameters\n"); return ERROR_COMMAND_SYNTAX_ERROR; break; case 0: diff --git a/src/flash/nor/at91sam7.c b/src/flash/nor/at91sam7.c index 74d9ba3d..2b822b7d 100644 --- a/src/flash/nor/at91sam7.c +++ b/src/flash/nor/at91sam7.c @@ -1203,8 +1203,7 @@ COMMAND_HANDLER(at91sam7_handle_gpnvm_command) if (CMD_ARGC != 2) { - command_print(CMD_CTX, "at91sam7 gpnvm <bit> <set | clear>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } bank = get_flash_bank_by_num_noprobe(0); @@ -1295,6 +1294,7 @@ static const struct command_registration at91sam7_command_handlers[] = { struct flash_driver at91sam7_flash = { .name = "at91sam7", + .usage = "gpnvm <bit> <set | clear>", .commands = at91sam7_command_handlers, .flash_bank_command = at91sam7_flash_bank_command, .erase = at91sam7_erase, diff --git a/src/flash/nor/avrf.c b/src/flash/nor/avrf.c index 5a2482e4..1ade4526 100644 --- a/src/flash/nor/avrf.c +++ b/src/flash/nor/avrf.c @@ -191,8 +191,7 @@ FLASH_BANK_COMMAND_HANDLER(avrf_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank avr configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } avrf_info = malloc(sizeof(struct avrf_flash_bank)); @@ -451,8 +450,7 @@ COMMAND_HANDLER(avrf_handle_mass_erase_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "avr mass_erase <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -482,6 +480,7 @@ COMMAND_HANDLER(avrf_handle_mass_erase_command) static const struct command_registration avrf_exec_command_handlers[] = { { .name = "mass_erase", + .usage = "<bank>", .handler = avrf_handle_mass_erase_command, .mode = COMMAND_EXEC, .help = "erase entire device", diff --git a/src/flash/nor/cfi.c b/src/flash/nor/cfi.c index a4092b62..98633795 100644 --- a/src/flash/nor/cfi.c +++ b/src/flash/nor/cfi.c @@ -818,8 +818,7 @@ FLASH_BANK_COMMAND_HANDLER(cfi_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank cfi configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } /* both widths must: diff --git a/src/flash/nor/ecos.c b/src/flash/nor/ecos.c index c850f4f9..cae89358 100644 --- a/src/flash/nor/ecos.c +++ b/src/flash/nor/ecos.c @@ -111,8 +111,7 @@ FLASH_BANK_COMMAND_HANDLER(ecosflash_flash_bank_command) if (CMD_ARGC < 7) { - LOG_WARNING("incomplete flash_bank ecosflash configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } info = malloc(sizeof(struct ecosflash_flash_bank)); diff --git a/src/flash/nor/em357.c b/src/flash/nor/em357.c index c400187d..c045d577 100644 --- a/src/flash/nor/em357.c +++ b/src/flash/nor/em357.c @@ -104,8 +104,7 @@ FLASH_BANK_COMMAND_HANDLER(em357_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank em357 configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } em357_info = malloc(sizeof(struct em357_flash_bank)); @@ -768,8 +767,7 @@ COMMAND_HANDLER(em357_handle_lock_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "em357 lock <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -813,8 +811,7 @@ COMMAND_HANDLER(em357_handle_unlock_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "em357 unlock <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -892,8 +889,7 @@ COMMAND_HANDLER(em357_handle_mass_erase_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "em357 mass_erase <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -923,23 +919,23 @@ COMMAND_HANDLER(em357_handle_mass_erase_command) static const struct command_registration em357_exec_command_handlers[] = { { .name = "lock", + .usage = "<bank>", .handler = em357_handle_lock_command, .mode = COMMAND_EXEC, - .usage = "bank_id", .help = "Lock entire flash device.", }, { .name = "unlock", + .usage = "<bank>", .handler = em357_handle_unlock_command, .mode = COMMAND_EXEC, - .usage = "bank_id", .help = "Unlock entire protected flash device.", }, { .name = "mass_erase", + .usage = "<bank>", .handler = em357_handle_mass_erase_command, .mode = COMMAND_EXEC, - .usage = "bank_id", .help = "Erase entire flash device.", }, COMMAND_REGISTRATION_DONE diff --git a/src/flash/nor/faux.c b/src/flash/nor/faux.c index 92851ed0..75ea9d60 100644 --- a/src/flash/nor/faux.c +++ b/src/flash/nor/faux.c @@ -44,8 +44,7 @@ FLASH_BANK_COMMAND_HANDLER(faux_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank faux configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } info = malloc(sizeof(struct faux_flash_bank)); diff --git a/src/flash/nor/fm3.c b/src/flash/nor/fm3.c index e852589b..78e71d29 100644 --- a/src/flash/nor/fm3.c +++ b/src/flash/nor/fm3.c @@ -69,8 +69,7 @@ FLASH_BANK_COMMAND_HANDLER(fm3_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank fm3 configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } fm3_info = malloc(sizeof(struct fm3_flash_bank)); @@ -847,8 +846,7 @@ COMMAND_HANDLER(fm3_handle_chip_erase_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "fm3 chip_erase <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -875,9 +873,9 @@ COMMAND_HANDLER(fm3_handle_chip_erase_command) static const struct command_registration fm3_exec_command_handlers[] = { { .name = "chip_erase", + .usage = "<bank>", .handler = fm3_handle_chip_erase_command, .mode = COMMAND_EXEC, - .usage = "bank_id", .help = "Erase entire Flash device.", }, COMMAND_REGISTRATION_DONE diff --git a/src/flash/nor/kinetis.c b/src/flash/nor/kinetis.c index 18975498..0cfbd7ad 100644 --- a/src/flash/nor/kinetis.c +++ b/src/flash/nor/kinetis.c @@ -70,9 +70,7 @@ FLASH_BANK_COMMAND_HANDLER(kinetis_flash_bank_command) struct kinetis_flash_bank *bank_info; if (CMD_ARGC < 6) { - LOG_ERROR("incomplete flash_bank kinetis configuration %d", - CMD_ARGC); - return ERROR_FLASH_OPERATION_FAILED; + return ERROR_COMMAND_SYNTAX_ERROR; } LOG_INFO("add flash_bank kinetis %s", bank->name); diff --git a/src/flash/nor/lpc2000.c b/src/flash/nor/lpc2000.c index b639d1ef..b4e17e7e 100644 --- a/src/flash/nor/lpc2000.c +++ b/src/flash/nor/lpc2000.c @@ -481,8 +481,7 @@ FLASH_BANK_COMMAND_HANDLER(lpc2000_flash_bank_command) if (CMD_ARGC < 8) { - LOG_WARNING("incomplete flash_bank lpc2000 configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } lpc2000_info = malloc(sizeof(struct lpc2000_flash_bank)); diff --git a/src/flash/nor/lpc288x.c b/src/flash/nor/lpc288x.c index 589527f7..d5fde948 100644 --- a/src/flash/nor/lpc288x.c +++ b/src/flash/nor/lpc288x.c @@ -184,8 +184,7 @@ FLASH_BANK_COMMAND_HANDLER(lpc288x_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank LPC288x configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } lpc288x_info = malloc(sizeof(struct lpc288x_flash_bank)); diff --git a/src/flash/nor/lpc2900.c b/src/flash/nor/lpc2900.c index 7bb9f141..4960e9c6 100644 --- a/src/flash/nor/lpc2900.c +++ b/src/flash/nor/lpc2900.c @@ -542,8 +542,7 @@ COMMAND_HANDLER(lpc2900_handle_signature_command) if( CMD_ARGC < 1 ) { - LOG_WARNING( "Too few arguments. Call: lpc2900 signature <bank#>" ); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -954,9 +953,9 @@ COMMAND_HANDLER(lpc2900_handle_secure_jtag_command) static const struct command_registration lpc2900_exec_command_handlers[] = { { .name = "signature", + .usage = "<bank>", .handler = lpc2900_handle_signature_command, .mode = COMMAND_EXEC, - .usage = "bank_id", .help = "Calculate and display signature of flash bank.", }, { @@ -1017,8 +1016,7 @@ FLASH_BANK_COMMAND_HANDLER(lpc2900_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank LPC2900 configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } lpc2900_info = malloc(sizeof(struct lpc2900_flash_bank)); diff --git a/src/flash/nor/ocl.c b/src/flash/nor/ocl.c index 6c609239..2dd4f521 100644 --- a/src/flash/nor/ocl.c +++ b/src/flash/nor/ocl.c @@ -51,8 +51,7 @@ FLASH_BANK_COMMAND_HANDLER(ocl_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank ocl configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } arm7_9 = target_to_arm7_9(bank->target); diff --git a/src/flash/nor/pic32mx.c b/src/flash/nor/pic32mx.c index 65d93ad4..957bfc2a 100644 --- a/src/flash/nor/pic32mx.c +++ b/src/flash/nor/pic32mx.c @@ -172,8 +172,7 @@ FLASH_BANK_COMMAND_HANDLER(pic32mx_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank pic32mx configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } pic32mx_info = malloc(sizeof(struct pic32mx_flash_bank)); @@ -737,8 +736,7 @@ COMMAND_HANDLER(pic32mx_handle_pgm_word_command) if (CMD_ARGC != 3) { - command_print(CMD_CTX, "pic32mx pgm_word <addr> <value> <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } COMMAND_PARSE_NUMBER(u32, CMD_ARGV[0], address); @@ -781,7 +779,7 @@ COMMAND_HANDLER(pic32mx_handle_unlock_command) if (CMD_ARGC < 1) { command_print(CMD_CTX, "pic32mx unlock <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -837,6 +835,7 @@ COMMAND_HANDLER(pic32mx_handle_unlock_command) static const struct command_registration pic32mx_exec_command_handlers[] = { { .name = "pgm_word", + .usage = "<addr> <value> <bank>", .handler = pic32mx_handle_pgm_word_command, .mode = COMMAND_EXEC, .help = "program a word", diff --git a/src/flash/nor/stellaris.c b/src/flash/nor/stellaris.c index 7351aed9..c855995f 100644 --- a/src/flash/nor/stellaris.c +++ b/src/flash/nor/stellaris.c @@ -438,8 +438,7 @@ FLASH_BANK_COMMAND_HANDLER(stellaris_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank stellaris configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } stellaris_info = calloc(sizeof(struct stellaris_flash_bank), 1); @@ -1351,8 +1350,7 @@ COMMAND_HANDLER(stellaris_handle_mass_erase_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "stellaris mass_erase <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1439,9 +1437,9 @@ done: static const struct command_registration stellaris_exec_command_handlers[] = { { .name = "mass_erase", + .usage = "<bank>", .handler = stellaris_handle_mass_erase_command, .mode = COMMAND_EXEC, - .usage = "bank_id", .help = "erase entire device", }, { diff --git a/src/flash/nor/stm32f1x.c b/src/flash/nor/stm32f1x.c index 8e0fda7e..5d9da903 100644 --- a/src/flash/nor/stm32f1x.c +++ b/src/flash/nor/stm32f1x.c @@ -130,8 +130,7 @@ FLASH_BANK_COMMAND_HANDLER(stm32x_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank stm32x configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } stm32x_info = malloc(sizeof(struct stm32x_flash_bank)); @@ -1329,8 +1328,7 @@ COMMAND_HANDLER(stm32x_handle_lock_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "stm32x lock <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1378,8 +1376,7 @@ COMMAND_HANDLER(stm32x_handle_unlock_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "stm32x unlock <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1426,8 +1423,7 @@ COMMAND_HANDLER(stm32x_handle_options_read_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "stm32x options_read <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1496,9 +1492,7 @@ COMMAND_HANDLER(stm32x_handle_options_write_command) if (CMD_ARGC < 4) { - command_print(CMD_CTX, "stm32x options_write <bank> <SWWDG | HWWDG> " - "<RSTSTNDBY | NORSTSTNDBY> <RSTSTOP | NORSTSTOP> <BOOT0 | BOOT1>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1631,8 +1625,7 @@ COMMAND_HANDLER(stm32x_handle_mass_erase_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "stm32x mass_erase <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; diff --git a/src/flash/nor/stm32f2x.c b/src/flash/nor/stm32f2x.c index 7c07c536..ec5527bc 100644 --- a/src/flash/nor/stm32f2x.c +++ b/src/flash/nor/stm32f2x.c @@ -165,8 +165,7 @@ FLASH_BANK_COMMAND_HANDLER(stm32x_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank stm32x configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } stm32x_info = malloc(sizeof(struct stm32x_flash_bank)); diff --git a/src/flash/nor/stm32lx.c b/src/flash/nor/stm32lx.c index 32b3315d..8a6ad7bc 100644 --- a/src/flash/nor/stm32lx.c +++ b/src/flash/nor/stm32lx.c @@ -130,8 +130,7 @@ FLASH_BANK_COMMAND_HANDLER(stm32lx_flash_bank_command) struct stm32lx_flash_bank *stm32lx_info; if (CMD_ARGC < 6) { - LOG_ERROR("incomplete flash_bank stm32lx configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } // Create the bank structure diff --git a/src/flash/nor/stmsmi.c b/src/flash/nor/stmsmi.c index bfec7894..df8dbe75 100644 --- a/src/flash/nor/stmsmi.c +++ b/src/flash/nor/stmsmi.c @@ -206,8 +206,7 @@ FLASH_BANK_COMMAND_HANDLER(stmsmi_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank stmsmi configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } stmsmi_info = malloc(sizeof(struct stmsmi_flash_bank)); diff --git a/src/flash/nor/str7x.c b/src/flash/nor/str7x.c index 0084c67f..91ef77b7 100644 --- a/src/flash/nor/str7x.c +++ b/src/flash/nor/str7x.c @@ -208,8 +208,7 @@ FLASH_BANK_COMMAND_HANDLER(str7x_flash_bank_command) if (CMD_ARGC < 7) { - LOG_WARNING("incomplete flash_bank str7x configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } str7x_info = malloc(sizeof(struct str7x_flash_bank)); @@ -786,8 +785,7 @@ COMMAND_HANDLER(str7x_handle_disable_jtag_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "str7x disable_jtag <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -849,6 +847,7 @@ COMMAND_HANDLER(str7x_handle_disable_jtag_command) static const struct command_registration str7x_exec_command_handlers[] = { { .name = "disable_jtag", + .usage = "<bank>", .handler = str7x_handle_disable_jtag_command, .mode = COMMAND_EXEC, .help = "disable jtag access", diff --git a/src/flash/nor/str9x.c b/src/flash/nor/str9x.c index 63cfd18f..5bea2068 100644 --- a/src/flash/nor/str9x.c +++ b/src/flash/nor/str9x.c @@ -158,8 +158,7 @@ FLASH_BANK_COMMAND_HANDLER(str9x_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank str9x configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } str9x_info = malloc(sizeof(struct str9x_flash_bank)); diff --git a/src/flash/nor/str9xpec.c b/src/flash/nor/str9xpec.c index c06e6d16..6fa66a09 100644 --- a/src/flash/nor/str9xpec.c +++ b/src/flash/nor/str9xpec.c @@ -292,8 +292,7 @@ FLASH_BANK_COMMAND_HANDLER(str9xpec_flash_bank_command) if (CMD_ARGC < 6) { - LOG_WARNING("incomplete flash_bank str9x configuration"); - return ERROR_FLASH_BANK_INVALID; + return ERROR_COMMAND_SYNTAX_ERROR; } str9xpec_info = malloc(sizeof(struct str9xpec_flash_controller)); @@ -825,8 +824,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_options_read_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "str9xpec options_read <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -939,8 +937,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_options_write_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "str9xpec options_write <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -966,8 +963,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_options_cmap_command) if (CMD_ARGC < 2) { - command_print(CMD_CTX, "str9xpec options_cmap <bank> <bank0 | bank1>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -995,8 +991,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_options_lvdthd_command) if (CMD_ARGC < 2) { - command_print(CMD_CTX, "str9xpec options_lvdthd <bank> <2.4v | 2.7v>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1024,8 +1019,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_options_lvdsel_command) if (CMD_ARGC < 2) { - command_print(CMD_CTX, "str9xpec options_lvdsel <bank> <vdd | vdd_vddq>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1053,8 +1047,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_options_lvdwarn_command) if (CMD_ARGC < 2) { - command_print(CMD_CTX, "str9xpec options_lvdwarn <bank> <vdd | vdd_vddq>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1082,8 +1075,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_lock_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "str9xpec lock <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1105,8 +1097,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_unlock_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "str9xpec unlock <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1135,8 +1126,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_enable_turbo_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "str9xpec enable_turbo <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1182,8 +1172,7 @@ COMMAND_HANDLER(str9xpec_handle_flash_disable_turbo_command) if (CMD_ARGC < 1) { - command_print(CMD_CTX, "str9xpec disable_turbo <bank>"); - return ERROR_OK; + return ERROR_COMMAND_SYNTAX_ERROR; } struct flash_bank *bank; @@ -1213,60 +1202,70 @@ COMMAND_HANDLER(str9xpec_handle_flash_disable_turbo_command) static const struct command_registration str9xpec_config_command_handlers[] = { { .name = "enable_turbo", + .usage = "<bank>", .handler = str9xpec_handle_flash_enable_turbo_command, .mode = COMMAND_EXEC, .help = "enable str9xpec turbo mode", }, { .name = "disable_turbo", + .usage = "<bank>", .handler = str9xpec_handle_flash_disable_turbo_command, .mode = COMMAND_EXEC, .help = "disable str9xpec turbo mode", }, { .name = "options_cmap", + .usage = "<bank> <bank0 | bank1>", .handler = str9xpec_handle_flash_options_cmap_command, .mode = COMMAND_EXEC, .help = "configure str9xpec boot sector", }, { .name = "options_lvdthd", + .usage = "<bank> <2.4v | 2.7v>", .handler = str9xpec_handle_flash_options_lvdthd_command, .mode = COMMAND_EXEC, .help = "configure str9xpec lvd threshold", }, { .name = "options_lvdsel", + .usage = "<bank> <vdd | vdd_vddq>", .handler = str9xpec_handle_flash_options_lvdsel_command, .mode = COMMAND_EXEC, .help = "configure str9xpec lvd selection", }, { .name = "options_lvdwarn", + .usage = "<bank> <vdd | vdd_vddq>", .handler = str9xpec_handle_flash_options_lvdwarn_command, .mode = COMMAND_EXEC, .help = "configure str9xpec lvd warning", }, { .name = "options_read", + .usage = "<bank>", .handler = str9xpec_handle_flash_options_read_command, .mode = COMMAND_EXEC, .help = "read str9xpec options", }, { .name = "options_write", + .usage = "<bank>", .handler = str9xpec_handle_flash_options_write_command, .mode = COMMAND_EXEC, .help = "write str9xpec options", }, { .name = "lock", + .usage = "<bank>", .handler = str9xpec_handle_flash_lock_command, .mode = COMMAND_EXEC, .help = "lock str9xpec device", }, { .name = "unlock", + .usage = "<bank>", .handler = str9xpec_handle_flash_unlock_command, .mode = COMMAND_EXEC, .help = "unlock str9xpec device", diff --git a/src/flash/nor/tms470.c b/src/flash/nor/tms470.c index bf708be2..b28aee43 100644 --- a/src/flash/nor/tms470.c +++ b/src/flash/nor/tms470.c @@ -313,7 +313,6 @@ COMMAND_HANDLER(tms470_handle_flash_keyset_command) { if (CMD_ARGC > 4) { - command_print(CMD_CTX, "tms470 flash_keyset <key0> <key1> <key2> <key3>"); return ERROR_COMMAND_SYNTAX_ERROR; } else if (CMD_ARGC == 4) @@ -377,7 +376,6 @@ COMMAND_HANDLER(tms470_handle_osc_megahertz_command) { if (CMD_ARGC > 1) { - command_print(CMD_CTX, "tms470 osc_megahertz <MHz>"); return ERROR_COMMAND_SYNTAX_ERROR; } else if (CMD_ARGC == 1) @@ -406,7 +404,6 @@ COMMAND_HANDLER(tms470_handle_plldis_command) { if (CMD_ARGC > 1) { - command_print(CMD_CTX, "tms470 plldis <0 | 1>"); return ERROR_COMMAND_SYNTAX_ERROR; } else if (CMD_ARGC == 1) @@ -840,18 +837,21 @@ static int tms470_erase_sector(struct flash_bank *bank, int sector) static const struct command_registration tms470_any_command_handlers[] = { { .name = "flash_keyset", + .usage = "<key0> <key1> <key2> <key3>", .handler = tms470_handle_flash_keyset_command, .mode = COMMAND_ANY, .help = "tms470 flash_keyset <key0> <key1> <key2> <key3>", }, { .name = "osc_megahertz", + .usage = "<MHz>", .handler = tms470_handle_osc_megahertz_command, .mode = COMMAND_ANY, .help = "tms470 osc_megahertz <MHz>", }, { .name = "plldis", + .usage = "<0 | 1>", .handler = tms470_handle_plldis_command, .mode = COMMAND_ANY, .help = "tms470 plldis <0/1>", diff --git a/src/flash/nor/virtual.c b/src/flash/nor/virtual.c index e91dfc78..f1e84c29 100644 --- a/src/flash/nor/virtual.c +++ b/src/flash/nor/virtual.c @@ -55,8 +55,7 @@ FLASH_BANK_COMMAND_HANDLER(virtual_flash_bank_command) { if (CMD_ARGC < 7) { - LOG_WARNING("incomplete flash_bank virtual configuration"); - return ERROR_FLASH_OPERATION_FAILED; + return ERROR_COMMAND_SYNTAX_ERROR; } /* get the master flash bank */ |