[PATCH]: 04daa3397 flash/nor/numicro: remove useless architecture check

[gerrit]

This is an automated email from Gerrit.

"Tomas Vanek <van...@fbl.cz>" just uploaded a new patch set to Gerrit, which 
you can find at https://review.openocd.org/c/openocd/+/6755

-- gerrit

commit 04daa339738011b621fe93ae69674c4848749fb2
Author: Tomas Vanek <van...@fbl.cz>
Date:   Wed Nov 24 20:44:49 2021 +0100

    flash/nor/numicro: remove useless architecture check
    
    target_to_armv7m() just returns a type-cast of target->arch_info,
    so the test has no value.
    
    Following target_run_algorithm() checks magic number so
    we need not worry about execution on mismatched architecture.
    
    Change-Id: Ic9892a488a42af1d8e8731eddb39240deeb26020
    Signed-off-by: Tomas Vanek <van...@fbl.cz>

diff --git a/src/flash/nor/numicro.c b/src/flash/nor/numicro.c
index cb7c1df83..34875084c 100644
--- a/src/flash/nor/numicro.c
+++ b/src/flash/nor/numicro.c
@@ -1385,13 +1385,6 @@ static int numicro_writeblock(struct flash_bank *bank, 
const uint8_t *buffer,
        init_reg_param(&reg_params[1], "r1", 32, PARAM_OUT);    /* faddr */
        init_reg_param(&reg_params[2], "r2", 32, PARAM_OUT);    /* number of 
words to program */
 
-       struct armv7m_common *armv7m = target_to_armv7m(target);
-       if (!armv7m) {
-               /* something is very wrong if armv7m is NULL */
-               LOG_ERROR("unable to get armv7m target");
-               return retval;
-       }
-
        /* write code buffer and use Flash programming code within NuMicro     
*/
        /* Set breakpoint to 0 with time-out of 1000 ms                        
*/
        while (count > 0) {

--