This is an automated email from Gerrit.

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

-- gerrit

commit bc88144c532e75066efba78d3da83c7ae5a555bd
Author: Tomas Vanek <[email protected]>
Date:   Sun Aug 21 15:27:23 2016 +0200

    flash Kinetis: fix mdm check_security regression in JTAG mode
    
    Commit 6d5b4d709cb05e8daf0350ac507b928da95d0abb causes failed assert
    when JTAG transport is selected:
    
    src/target/arm_adi_v5.h:346: dap_queue_ap_read:
      Assertion `ap->dap->ops != ((void *)0)' failed.
    
    As check_security runs early in examine-start event, dap->ops must
    be checked.
    
    Change-Id: Ibd8312a3c668fbce834eed9790eabeed794117aa
    Signed-off-by: Tomas Vanek <[email protected]>

diff --git a/src/flash/nor/kinetis.c b/src/flash/nor/kinetis.c
index d02918b..0b738e2 100644
--- a/src/flash/nor/kinetis.c
+++ b/src/flash/nor/kinetis.c
@@ -609,6 +609,9 @@ COMMAND_HANDLER(kinetis_check_flash_security_status)
                return ERROR_OK;
        }
 
+       if (!dap->ops)
+               return ERROR_OK;        /* too early to check, in JTAG mode ops 
may not be initialised */
+
        uint32_t val;
        int retval;
 
@@ -623,7 +626,7 @@ COMMAND_HANDLER(kinetis_check_flash_security_status)
        }
 
        if (val == 0)
-               return ERROR_OK;
+               return ERROR_OK;        /* dap not yet initialised */
 
        bool found = false;
        for (size_t i = 0; i < ARRAY_SIZE(kinetis_known_mdm_ids); i++) {

-- 

------------------------------------------------------------------------------
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to