mmc: core: move some code in mmc_start_areq()
authorLinus Walleij <linus.walleij@linaro.org>
Tue, 28 Mar 2017 08:40:29 +0000 (10:40 +0200)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 24 Apr 2017 19:42:06 +0000 (21:42 +0200)
"previous" is a better name for the variable storing the previous
asynchronous request, better than the opaque name "data" atleast.
We see that we assign the return status to the returned variable
on all code paths, so we might as well just do that immediately
after calling mmc_finalize_areq().

Reviewed-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/core/core.c

index ffc263283f54f3cc665f3158ceb45272e08e9cb9..4c3e23f310f8c1616f50a55f0b1f82f1a8bed93b 100644 (file)
@@ -704,7 +704,7 @@ struct mmc_async_req *mmc_start_areq(struct mmc_host *host,
 {
        enum mmc_blk_status status;
        int start_err = 0;
-       struct mmc_async_req *data = host->areq;
+       struct mmc_async_req *previous = host->areq;
 
        /* Prepare a new request */
        if (areq)
@@ -712,13 +712,12 @@ struct mmc_async_req *mmc_start_areq(struct mmc_host *host,
 
        /* Finalize previous request */
        status = mmc_finalize_areq(host);
+       if (ret_stat)
+               *ret_stat = status;
 
        /* The previous request is still going on... */
-       if (status == MMC_BLK_NEW_REQUEST) {
-               if (ret_stat)
-                       *ret_stat = status;
+       if (status == MMC_BLK_NEW_REQUEST)
                return NULL;
-       }
 
        /* Fine so far, start the new request! */
        if (status == MMC_BLK_SUCCESS && areq)
@@ -737,9 +736,7 @@ struct mmc_async_req *mmc_start_areq(struct mmc_host *host,
        else
                host->areq = areq;
 
-       if (ret_stat)
-               *ret_stat = status;
-       return data;
+       return previous;
 }
 EXPORT_SYMBOL(mmc_start_areq);