flash/nor/stm32f2x: fix clang static analyzer warning
authorTomas Vanek <vanekt@fbl.cz>
Tue, 10 Dec 2019 18:49:01 +0000 (19:49 +0100)
committerTomas Vanek <vanekt@fbl.cz>
Tue, 17 Dec 2019 15:35:24 +0000 (15:35 +0000)
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 <vanekt@fbl.cz>
Reviewed-on: http://openocd.zylin.com/5356
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
src/flash/nor/stm32f2x.c

index b49e76e3218eec8413e9362aa87494ce9a921a86..ea35fd05d691f196dce48c37ed740763e91b2c6e 100644 (file)
@@ -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