mirror of
https://source.denx.de/u-boot/u-boot.git
synced 2026-06-13 15:03:58 +03:00
mmc: gen_atmel_mci: Remove duplicate checks
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 <andrew.goodbody@linaro.org> Reviewed-by: Peng Fan <peng.fan@nxp.com>
This commit is contained in:
committed by
Eugen Hristev
parent
b824136ab4
commit
4fb189a58e
@@ -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;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user