From: Hou Zhiqiang <b48...@freescale.com>

It doesn't make sense to compare a 'u8' element with Zero.

Signed-off-by: Hou Zhiqiang <b48...@freescale.com>
---
 drivers/mtd/spi/sf_ops.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/drivers/mtd/spi/sf_ops.c b/drivers/mtd/spi/sf_ops.c
index a5487ad..ba5ff0c 100644
--- a/drivers/mtd/spi/sf_ops.c
+++ b/drivers/mtd/spi/sf_ops.c
@@ -162,10 +162,7 @@ static inline int spi_flash_sr_ready(struct spi_flash 
*flash)
        if (ret < 0)
                return ret;
 
-       if (sr < 0)
-               return sr;
-       else
-               return !(sr & STATUS_WIP);
+       return !(sr & STATUS_WIP);
 }
 
 static inline int spi_flash_fsr_ready(struct spi_flash *flash)
@@ -177,10 +174,7 @@ static inline int spi_flash_fsr_ready(struct spi_flash 
*flash)
        if (ret < 0)
                return ret;
 
-       if (fsr < 0)
-               return fsr;
-       else
-               return fsr & STATUS_PEC;
+       return fsr & STATUS_PEC;
 }
 
 static int spi_flash_ready(struct spi_flash *flash)
-- 
2.1.0.27.g96db324

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to