mmc: mxcmmc: check the return value of mxcmci_finish_data
authorShawn Lin <shawn.lin@rock-chips.com>
Thu, 6 Jul 2017 08:09:12 +0000 (16:09 +0800)
committerUlf Hansson <ulf.hansson@linaro.org>
Wed, 30 Aug 2017 12:01:28 +0000 (14:01 +0200)
We got a compile warning for mxcmmc,

drivers/mmc/host/mxcmmc.c: In function 'mxcmci_data_done':
drivers/mmc/host/mxcmmc.c:661:6: warning: variable 'data_error' set but
not used [-Wunused-but-set-variable]

The easiest method is to remove the data_error. But looking into
the code closely, I think we should check the return value of
mxcmci_finish_data as if it got data->error(the same as data_error),
we shouldn't try to read the response.

Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/mxcmmc.c

index f3c28326c4db45543755382de3e8bf63658eaf59..f2b6639634d264c77301d0f1cbc8a0bf9e0b8000 100644 (file)
@@ -681,6 +681,9 @@ static void mxcmci_data_done(struct mxcmci_host *host, unsigned int stat)
 
        spin_unlock_irqrestore(&host->lock, flags);
 
+       if (data_error)
+               return;
+
        mxcmci_read_response(host, stat);
        host->cmd = NULL;