Remove duplicate checks on status from mci_data_read and mci_data_write
which are guaranteed to be true as exiting the above do..while loop
above requires that to be so.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody <[email protected]>
---
 drivers/mmc/gen_atmel_mci.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/drivers/mmc/gen_atmel_mci.c b/drivers/mmc/gen_atmel_mci.c
index 6a531fa0961..62aa6b3cb0b 100644
--- a/drivers/mmc/gen_atmel_mci.c
+++ b/drivers/mmc/gen_atmel_mci.c
@@ -206,10 +206,9 @@ static u32 mci_data_read(atmel_mci_t *mci, u32* data, u32 
error_flags)
                        goto io_fail;
        } while (!(status & MMCI_BIT(RXRDY)));
 
-       if (status & MMCI_BIT(RXRDY)) {
-               *data = readl(&mci->rdr);
-               status = 0;
-       }
+       *data = readl(&mci->rdr);
+       status = 0;
+
 io_fail:
        return status;
 }
@@ -225,10 +224,9 @@ static u32 mci_data_write(atmel_mci_t *mci, u32* data, u32 
error_flags)
                        goto io_fail;
        } while (!(status & MMCI_BIT(TXRDY)));
 
-       if (status & MMCI_BIT(TXRDY)) {
-               writel(*data, &mci->tdr);
-               status = 0;
-       }
+       writel(*data, &mci->tdr);
+       status = 0;
+
 io_fail:
        return status;
 }

-- 
2.39.5

Reply via email to