Message ID | 1439868955-43291-1-git-send-email-B48286@freescale.com |
---|---|
State | Superseded |
Delegated to: | Jagannadha Sutradharudu Teki |
Headers | show |
On 18 August 2015 at 09:05, Zhiqiang Hou <B48286@freescale.com> wrote: > From: Hou Zhiqiang <B48286@freescale.com> > > It doesn't make sense to compare a 'u8' element with Zero. > > Signed-off-by: Hou Zhiqiang <B48286@freescale.com> Reviewed-by: Jagan Teki <jteki@openedev.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 thanks!
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)