This is an automated email from Gerrit.

Salvador Arroyo ([email protected]) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/549

-- gerrit

commit eecb12bff9bebd83ac4a67d7e5d491aae12f8e2a
Author: Salvador <[email protected]>
Date:   Mon Apr 2 16:48:52 2012 +0200

    Minor bug fixes in Mips32 code
    
    Now the the "Fast" version for memory blank check in pic32mx.c can be 
called:
    default_flash_blank_check()  instead of the "fallback"  
default_flash_mem_blank_check().
    
    The command "verify_image", without working area, now don't show:
     checksum mismatch - attempting binary compare
    when there are no real errors in flash.
    
    Change-Id: I256e8ae949289634e1de5c1c2861e4c4c4b7fdce
    Signed-off-by: Salvador <[email protected]>

diff --git a/src/target/mips32.c b/src/target/mips32.c
index 5682e3f..ab39e6e 100644
--- a/src/target/mips32.c
+++ b/src/target/mips32.c
@@ -631,7 +631,7 @@ int mips32_checksum_memory(struct target *target, uint32_t 
address,
                destroy_reg_param(&reg_params[0]);
                destroy_reg_param(&reg_params[1]);
                target_free_working_area(target, crc_algorithm);
-               return 0;
+               return retval;
        }
 
        *checksum = buf_get_u32(reg_params[0].value, 0, 32);
@@ -688,14 +688,14 @@ int mips32_blank_check_memory(struct target *target,
 
        retval = target_run_algorithm(target, 0, NULL, 3, reg_params,
                        erase_check_algorithm->address,
-                       erase_check_algorithm->address + 
(sizeof(erase_check_code)-2),
+                       erase_check_algorithm->address + 
(sizeof(erase_check_code)-4),
                        10000, &mips32_info);
        if (retval != ERROR_OK) {
                destroy_reg_param(&reg_params[0]);
                destroy_reg_param(&reg_params[1]);
                destroy_reg_param(&reg_params[2]);
                target_free_working_area(target, erase_check_algorithm);
-               return 0;
+               return retval;
        }
 
        *blank = buf_get_u32(reg_params[2].value, 0, 32);

-- 

------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to