Use if and else instead of if(A) and if (!A). Signed-off-by: Wan Jiabing <wanjiabing@xxxxxxxx> --- drivers/mmc/host/atmel-mci.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/mmc/host/atmel-mci.c b/drivers/mmc/host/atmel-mci.c index 807177c953f3..98893ccad4bd 100644 --- a/drivers/mmc/host/atmel-mci.c +++ b/drivers/mmc/host/atmel-mci.c @@ -1125,8 +1125,7 @@ atmci_prepare_data_dma(struct atmel_mci *host, struct mmc_data *data) chan = host->dma.chan; if (chan) host->data_chan = chan; - - if (!chan) + else return -ENODEV; if (data->flags & MMC_DATA_READ) { -- 2.35.1