staging: mt29f_spinand: Remove unneeded else following return
authorJanani Ravichandran <janani.rvchndrn@gmail.com>
Thu, 18 Feb 2016 22:28:07 +0000 (17:28 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 20 Feb 2016 23:24:51 +0000 (15:24 -0800)
Remove unnecessary else when there is a return statement in the
corresponding if block. Coccinelle patch used:
@rule1@
expression e1;
@@

        if (e1) { ... return ...; }
-       else{
         ...
-     }

@rule2@
expression e2;
statement s1;
@@

if(e2) { ... return ...; }
-       else
s1

Signed-off-by: Janani Ravichandran <janani.rvchndrn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/mt29f_spinand/mt29f_spinand.c

index 3d9a4268ffa9889baf9485ca9fef362032a5ff5e..75fe61c54ee5e9951e08f237b5faf0d718949752 100644 (file)
@@ -175,7 +175,7 @@ static int wait_till_ready(struct spi_device *spi_nand)
                retval = spinand_read_status(spi_nand, &stat);
                if (retval < 0)
                        return -1;
-               else if (!(stat & 0x1))
+               if (!(stat & 0x1))
                        break;
 
                cond_resched();