flash/stmqspi: fix build error with -Werror=maybe-uninitialized
authorTarek BOCHKATI <tarek.bouchkati@st.com>
Mon, 11 Jan 2021 18:21:06 +0000 (19:21 +0100)
committerAntonio Borneo <borneo.antonio@gmail.com>
Mon, 18 Jan 2021 15:31:51 +0000 (15:31 +0000)
using gcc 9.3 on ubuntu focal fossa with -Werror=maybe-uninitialized
we get this error:
  /src/flash/nor/stmqspi.c: In function ‘read_flash_id’:
  /src/flash/nor/stmqspi.c:1948:6: error: ‘retval’ may be used uninitialized

Change-Id: Ifd8ae60df847fc61e22ca100c008e3914c9af79b
Signed-off-by: Tarek BOCHKATI <tarek.bouchkati@st.com>
Reviewed-on: http://openocd.zylin.com/6012
Tested-by: jenkins
Reviewed-by: Tarek BOCHKATI <tarek.bouchkati@gmail.com>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
src/flash/nor/stmqspi.c

index f54e4975dd95ce2050896db7001730e4e3bfed22..a013336a0f8b1bdbea1800ddaab05d829aa79dd9 100644 (file)
@@ -1945,7 +1945,7 @@ static int read_flash_id(struct flash_bank *bank, uint32_t *id1, uint32_t *id2)
        uint32_t io_base = stmqspi_info->io_base;
        uint8_t byte;
        unsigned int type, count, len1, len2;
-       int retval;
+       int retval = ERROR_OK;
 
        /* invalidate both ids */
        *id1 = 0;