This is an automated email from Gerrit.

Spencer Oliver (s...@spen-soft.co.uk) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/391

-- gerrit

commit c02c62428f4d91a342638e66140e83980fa04d33
Author: Spencer Oliver <s...@spen-soft.co.uk>
Date:   Thu Jan 19 15:08:50 2012 +0000

    flash: print bank usage on failure
    
    This makes use of the newly introduced usage field in the flash bank
    structure.
    
    Also remove the assertion if usage field is null and
    lets print a DEBUG_LOG message instead.
    
    Change-Id: I384bf0e2c444fcc99deef73aec9ef01149a91c76
    Signed-off-by: Spencer Oliver <s...@spen-soft.co.uk>

diff --git a/src/flash/nand/tcl.c b/src/flash/nand/tcl.c
index 29d4e1e..6389377 100644
--- a/src/flash/nand/tcl.c
+++ b/src/flash/nand/tcl.c
@@ -581,7 +581,6 @@ static COMMAND_HELPER(create_nand_device, const char 
*bank_name,
        retval = CALL_COMMAND_HANDLER(controller->nand_device_command, c);
        if (ERROR_OK != retval)
        {
-               assert(controller->usage != NULL);
                LOG_ERROR("'%s' driver rejected nand flash. Usage: %s",
                        controller->name,
                        controller->usage);
@@ -589,6 +588,9 @@ static COMMAND_HELPER(create_nand_device, const char 
*bank_name,
                return retval;
        }
 
+       if (controller->usage == NULL)
+               LOG_DEBUG("'%s' driver usage field missing", controller->name);
+
        nand_device_add(c);
 
        return ERROR_OK;
diff --git a/src/flash/nor/tcl.c b/src/flash/nor/tcl.c
index 4ba5f47..735caa8 100644
--- a/src/flash/nor/tcl.c
+++ b/src/flash/nor/tcl.c
@@ -834,12 +834,15 @@ COMMAND_HANDLER(handle_flash_bank_command)
        retval = CALL_COMMAND_HANDLER(driver->flash_bank_command, c);
        if (ERROR_OK != retval)
        {
-               LOG_ERROR("'%s' driver rejected flash bank at 0x%8.8" PRIx32,
-                               driver_name, c->base);
+               LOG_ERROR("'%s' driver rejected flash bank at 0x%8.8" PRIx32 
"Usage %s",
+                               driver_name, c->base, driver->usage);
                free(c);
                return retval;
        }
 
+       if (driver->usage == NULL)
+               LOG_DEBUG("'%s' driver usage field missing", driver_name);
+
        flash_bank_add(c);
 
        return ERROR_OK;

-- 

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to