]> git.sur5r.net Git - openocd/commitdiff
stm32f2x: Fix left shift of negative value
authorMarc Schink <openocd-dev@marcschink.de>
Thu, 24 Aug 2017 15:42:28 +0000 (17:42 +0200)
committerSpencer Oliver <spen@spen-soft.co.uk>
Tue, 3 Oct 2017 10:19:45 +0000 (11:19 +0100)
Use unsigned constant for left shift operation in order to avoid the
following error with GCC >= 6.0:

../src/flash/nor/stm32f2x.c: In function ‘stm32x_handle_unlock_command’:
../src/flash/nor/stm32f2x.c:1324:67: error: left shift of negative value [-Werror=shift-negative-value]
   stm32x_info->option_bytes.optcr2_pcrop = OPTCR2_PCROP_RDP | (~1 << bank->num_sectors);

Change-Id: I0ac082bd0dbb8dc2f61ffff8fdf486ab7962d2e0
Signed-off-by: Marc Schink <openocd-dev@marcschink.de>
Reviewed-on: http://openocd.zylin.com/4207
Tested-by: jenkins
Reviewed-by: Andreas Bolsch <hyphen0break@gmail.com>
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-by: Anton Fosselius <anton.fosselius@gmail.com>
Reviewed-by: Esben Haabendal <esbenhaabendal@gmail.com>
src/flash/nor/stm32f2x.c

index 0e4abb533c6ff672f50a20d752e64be57c957120..65cb212b6d9c8dbbe66bf2e6ba21892868068b1a 100644 (file)
@@ -1321,7 +1321,7 @@ COMMAND_HANDLER(stm32x_handle_unlock_command)
         * this will also force a device unlock if set */
        stm32x_info->option_bytes.RDP = 0xAA;
        if (stm32x_info->has_optcr2_pcrop) {
-               stm32x_info->option_bytes.optcr2_pcrop = OPTCR2_PCROP_RDP | (~1 << bank->num_sectors);
+               stm32x_info->option_bytes.optcr2_pcrop = OPTCR2_PCROP_RDP | (~1U << bank->num_sectors);
        }
 
        if (stm32x_write_options(bank) != ERROR_OK) {