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/5356

-- gerrit

commit 90fedf81d51fe28517fb3012e34e661e3e992743
Author: Tomas Vanek <[email protected]>
Date:   Tue Dec 10 19:49:01 2019 +0100

    flash/nor/stm32f2x: fix clang static analyzer warning
    
    Using a signed variable as a parameter of FLASH_SNB() macro
    generated "warning: The result of the left shift is undefined
    because the left operand is negative"
    
    Change-Id: I8b3fe840f9308962460906097df6ddd848c07b25
    Signed-off-by: Tomas Vanek <[email protected]>

diff --git a/src/flash/nor/stm32f2x.c b/src/flash/nor/stm32f2x.c
index b49e76e..ea35fd0 100644
--- a/src/flash/nor/stm32f2x.c
+++ b/src/flash/nor/stm32f2x.c
@@ -634,7 +634,7 @@ static int stm32x_erase(struct flash_bank *bank, int first, 
int last)
         */
 
        for (i = first; i <= last; i++) {
-               int snb;
+               unsigned int snb;
                if (stm32x_info->has_large_mem && i >= 12)
                        snb = (i - 12) | 0x10;
                else

-- 


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

Reply via email to