From 99939c3c75f3bef44d4cd176e90a6c5fe8b833da Mon Sep 17 00:00:00 2001 From: David Brownell Date: Wed, 3 Mar 2010 12:59:53 -0800 Subject: NOR: stellaris message tweaks Give a more accurate failure message when trying to unprotect; don't complain about pages being write protected, just say that unprotect is not supported by the hardware ... referencing the new "recover" command, which is the way to achieve that. Likewise, when trying to protect, talk about "pages" (matching hardware doc) not "sectors" (an concept that's alien to these chips). Also make the helptext for the "recover" command mention that it also erases the device. Signed-off-by: David Brownell --- src/flash/nor/stellaris.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/flash/nor/stellaris.c b/src/flash/nor/stellaris.c index c9c800ea..0b7c45a3 100644 --- a/src/flash/nor/stellaris.c +++ b/src/flash/nor/stellaris.c @@ -693,8 +693,8 @@ static int stellaris_protect(struct flash_bank *bank, int set, int first, int la if (!set) { - LOG_ERROR("Can't unprotect write-protected pages."); - /* except by the "recover locked device" procedure ... */ + LOG_ERROR("Hardware doesn't suppport page-level unprotect. " + "Try the 'recover' command."); return ERROR_INVALID_ARGUMENTS; } @@ -706,7 +706,7 @@ static int stellaris_protect(struct flash_bank *bank, int set, int first, int la || (last < first) || !(last & 1) || (last >= 2 * stellaris_info->num_lockbits)) { - LOG_ERROR("Can't protect unaligned or out-of-range sectors."); + LOG_ERROR("Can't protect unaligned or out-of-range pages."); return ERROR_FLASH_SECTOR_INVALID; } @@ -1240,7 +1240,7 @@ static const struct command_registration stellaris_exec_command_handlers[] = { .handler = stellaris_handle_recover_command, .mode = COMMAND_EXEC, .usage = "bank_id", - .help = "recover locked device", + .help = "recover (and erase) locked device", }, COMMAND_REGISTRATION_DONE }; -- cgit v1.2.3