flash/nor/jtagspi: 'retval' may be used uninitialized
authorAlamy Liu <alamy.liu@gmail.com>
Thu, 6 Aug 2015 21:45:19 +0000 (14:45 -0700)
committerAndreas Fritiofson <andreas.fritiofson@gmail.com>
Wed, 19 Aug 2015 20:35:10 +0000 (21:35 +0100)
Problem
As my compiler has "warnings being treated as errors" on, it shows the
error message:
  error: 'retval' may be used uninitialized in this function

Investigation
Nothing wrong with the logic, 'retval' would have a value before returning.
Just wanna get rid of the compiling "warning as error" message.

Solution
Provide a reasonable default value

Change-Id: I712c15f82819c6c48bee9dceca8de4b18aeb29b0
Signed-off-by: Alamy Liu <alamy.liu@gmail.com>
Reviewed-on: http://openocd.zylin.com/2905
Tested-by: jenkins
Reviewed-by: Robert Jordens <jordens@gmail.com>
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
src/flash/nor/jtagspi.c

index 6ca5c3c25e2bf747ef7ece6a938a42fcbe3ee0e2..1e623a71b65303aebc85caa3cee7ff354ecaca0a 100644 (file)
@@ -272,7 +272,7 @@ static int jtagspi_erase(struct flash_bank *bank, int first, int last)
 {
        int sector;
        struct jtagspi_flash_bank *info = bank->driver_priv;
-       int retval;
+       int retval = ERROR_OK;
 
        LOG_DEBUG("erase from sector %d to sector %d", first, last);