From 22911a3aedfa01c7a5643de9c21fbb94f6219c38 Mon Sep 17 00:00:00 2001 From: =?utf8?q?=C3=98yvind=20Harboe?= Date: Mon, 20 Sep 2010 09:22:46 +0200 Subject: [PATCH 1/1] flash: fix error handling MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit sensible error must be reported at failure site Signed-off-by: Øyvind Harboe --- src/flash/nand/tcl.c | 32 +++----------------------------- src/flash/nor/tcl.c | 13 ++----------- 2 files changed, 5 insertions(+), 40 deletions(-) diff --git a/src/flash/nand/tcl.c b/src/flash/nand/tcl.c index 7b848880db..57bbe00e5c 100644 --- a/src/flash/nand/tcl.c +++ b/src/flash/nand/tcl.c @@ -150,16 +150,8 @@ COMMAND_HANDLER(handle_nand_probe_command) command_print(CMD_CTX, "NAND flash device '%s (%s)' found", p->device->name, p->manufacturer->name); } - else if (retval == ERROR_NAND_OPERATION_FAILED) - { - command_print(CMD_CTX, "probing failed for NAND flash device"); - } - else - { - command_print(CMD_CTX, "unknown error when probing NAND flash device"); - } - return ERROR_OK; + return retval; } COMMAND_HANDLER(handle_nand_erase_command) @@ -206,16 +198,8 @@ COMMAND_HANDLER(handle_nand_erase_command) offset, offset + length, CMD_ARGV[0], p->device->name); } - else if (retval == ERROR_NAND_OPERATION_FAILED) - { - command_print(CMD_CTX, "erase failed"); - } - else - { - command_print(CMD_CTX, "unknown error when erasing NAND flash device"); - } - return ERROR_OK; + return retval; } COMMAND_HANDLER(handle_nand_check_bad_blocks_command) @@ -261,18 +245,8 @@ COMMAND_HANDLER(handle_nand_check_bad_blocks_command) command_print(CMD_CTX, "checked NAND flash device for bad blocks, " "use \"nand info\" command to list blocks"); } - else if (retval == ERROR_NAND_OPERATION_FAILED) - { - command_print(CMD_CTX, "error when checking for bad blocks on " - "NAND flash device"); - } - else - { - command_print(CMD_CTX, "unknown error when checking for bad " - "blocks on NAND flash device"); - } - return ERROR_OK; + return retval; } COMMAND_HANDLER(handle_nand_write_command) diff --git a/src/flash/nor/tcl.c b/src/flash/nor/tcl.c index 3dc6cff3fd..8604b4b33a 100644 --- a/src/flash/nor/tcl.c +++ b/src/flash/nor/tcl.c @@ -132,23 +132,14 @@ COMMAND_HANDLER(handle_flash_probe_command) { command_print(CMD_CTX, "flash '%s' found at 0x%8.8" PRIx32, p->driver->name, p->base); } - else if (retval == ERROR_FLASH_BANK_INVALID) - { - command_print(CMD_CTX, "probing failed for flash bank '#%s' at 0x%8.8" PRIx32, - CMD_ARGV[0], p->base); - } - else - { - command_print(CMD_CTX, "unknown error when probing flash bank '#%s' at 0x%8.8" PRIx32, - CMD_ARGV[0], p->base); - } } else { command_print(CMD_CTX, "flash bank '#%s' is out of bounds", CMD_ARGV[0]); + retval = ERROR_FAIL; } - return ERROR_OK; + return retval; } COMMAND_HANDLER(handle_flash_erase_check_command) -- 2.30.2