aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2016-11-23 05:02:24 -0500
committerUlf Hansson <ulf.hansson@linaro.org>2016-11-29 03:05:27 -0500
commitd3c6aac3bdfe97b8b44db6a8aba59786cb9531dc (patch)
treebff68f24e27d72562f187077dd899524cfddde2f
parente173f8911f091fa50ccf8cc1fa316dd5569bc470 (diff)
mmc: delete is_first_req parameter from pre-request callback
The void (*pre_req) callback in the struct mmc_host_ops vtable is passing an argument "is_first_req" indicating whether this is the first request or not. None of the in-kernel users use this parameter: instead, since they all just do variants of dma_map* they use the DMA cookie to indicate whether a pre* callback has already been done for a request when they decide how to handle it. Delete the parameter from the callback and all users, as it is just pointless cruft. Signed-off-by: Linus Walleij <linus.walleij@linaro.org> Acked-by: Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
-rw-r--r--drivers/mmc/core/core.c11
-rw-r--r--drivers/mmc/host/dw_mmc.c3
-rw-r--r--drivers/mmc/host/jz4740_mmc.c3
-rw-r--r--drivers/mmc/host/mmci.c3
-rw-r--r--drivers/mmc/host/mtk-sd.c3
-rw-r--r--drivers/mmc/host/omap_hsmmc.c3
-rw-r--r--drivers/mmc/host/rtsx_pci_sdmmc.c3
-rw-r--r--drivers/mmc/host/sdhci.c3
-rw-r--r--include/linux/mmc/host.h3
9 files changed, 12 insertions, 23 deletions
diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index 060f76742550..f39397f7c8dc 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -611,18 +611,15 @@ EXPORT_SYMBOL(mmc_is_req_done);
611 * mmc_pre_req - Prepare for a new request 611 * mmc_pre_req - Prepare for a new request
612 * @host: MMC host to prepare command 612 * @host: MMC host to prepare command
613 * @mrq: MMC request to prepare for 613 * @mrq: MMC request to prepare for
614 * @is_first_req: true if there is no previous started request
615 * that may run in parellel to this call, otherwise false
616 * 614 *
617 * mmc_pre_req() is called in prior to mmc_start_req() to let 615 * mmc_pre_req() is called in prior to mmc_start_req() to let
618 * host prepare for the new request. Preparation of a request may be 616 * host prepare for the new request. Preparation of a request may be
619 * performed while another request is running on the host. 617 * performed while another request is running on the host.
620 */ 618 */
621static void mmc_pre_req(struct mmc_host *host, struct mmc_request *mrq, 619static void mmc_pre_req(struct mmc_host *host, struct mmc_request *mrq)
622 bool is_first_req)
623{ 620{
624 if (host->ops->pre_req) 621 if (host->ops->pre_req)
625 host->ops->pre_req(host, mrq, is_first_req); 622 host->ops->pre_req(host, mrq);
626} 623}
627 624
628/** 625/**
@@ -667,7 +664,7 @@ struct mmc_async_req *mmc_start_req(struct mmc_host *host,
667 664
668 /* Prepare a new request */ 665 /* Prepare a new request */
669 if (areq) 666 if (areq)
670 mmc_pre_req(host, areq->mrq, !host->areq); 667 mmc_pre_req(host, areq->mrq);
671 668
672 if (host->areq) { 669 if (host->areq) {
673 status = mmc_wait_for_data_req_done(host, host->areq->mrq, areq); 670 status = mmc_wait_for_data_req_done(host, host->areq->mrq, areq);
@@ -696,7 +693,7 @@ struct mmc_async_req *mmc_start_req(struct mmc_host *host,
696 693
697 /* prepare the request again */ 694 /* prepare the request again */
698 if (areq) 695 if (areq)
699 mmc_pre_req(host, areq->mrq, !host->areq); 696 mmc_pre_req(host, areq->mrq);
700 } 697 }
701 } 698 }
702 699
diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
index 881ca3e37da4..d400afc74719 100644
--- a/drivers/mmc/host/dw_mmc.c
+++ b/drivers/mmc/host/dw_mmc.c
@@ -886,8 +886,7 @@ static int dw_mci_pre_dma_transfer(struct dw_mci *host,
886} 886}
887 887
888static void dw_mci_pre_req(struct mmc_host *mmc, 888static void dw_mci_pre_req(struct mmc_host *mmc,
889 struct mmc_request *mrq, 889 struct mmc_request *mrq)
890 bool is_first_req)
891{ 890{
892 struct dw_mci_slot *slot = mmc_priv(mmc); 891 struct dw_mci_slot *slot = mmc_priv(mmc);
893 struct mmc_data *data = mrq->data; 892 struct mmc_data *data = mrq->data;
diff --git a/drivers/mmc/host/jz4740_mmc.c b/drivers/mmc/host/jz4740_mmc.c
index 684087db170b..819ad32964fc 100644
--- a/drivers/mmc/host/jz4740_mmc.c
+++ b/drivers/mmc/host/jz4740_mmc.c
@@ -320,8 +320,7 @@ dma_unmap:
320} 320}
321 321
322static void jz4740_mmc_pre_request(struct mmc_host *mmc, 322static void jz4740_mmc_pre_request(struct mmc_host *mmc,
323 struct mmc_request *mrq, 323 struct mmc_request *mrq)
324 bool is_first_req)
325{ 324{
326 struct jz4740_mmc_host *host = mmc_priv(mmc); 325 struct jz4740_mmc_host *host = mmc_priv(mmc);
327 struct mmc_data *data = mrq->data; 326 struct mmc_data *data = mrq->data;
diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c
index 6af3ee7b452a..01a804792f30 100644
--- a/drivers/mmc/host/mmci.c
+++ b/drivers/mmc/host/mmci.c
@@ -699,8 +699,7 @@ static void mmci_get_next_data(struct mmci_host *host, struct mmc_data *data)
699 next->dma_chan = NULL; 699 next->dma_chan = NULL;
700} 700}
701 701
702static void mmci_pre_request(struct mmc_host *mmc, struct mmc_request *mrq, 702static void mmci_pre_request(struct mmc_host *mmc, struct mmc_request *mrq)
703 bool is_first_req)
704{ 703{
705 struct mmci_host *host = mmc_priv(mmc); 704 struct mmci_host *host = mmc_priv(mmc);
706 struct mmc_data *data = mrq->data; 705 struct mmc_data *data = mrq->data;
diff --git a/drivers/mmc/host/mtk-sd.c b/drivers/mmc/host/mtk-sd.c
index 86af0b199a54..10ef2ae1d2f6 100644
--- a/drivers/mmc/host/mtk-sd.c
+++ b/drivers/mmc/host/mtk-sd.c
@@ -927,8 +927,7 @@ static void msdc_ops_request(struct mmc_host *mmc, struct mmc_request *mrq)
927 msdc_start_command(host, mrq, mrq->cmd); 927 msdc_start_command(host, mrq, mrq->cmd);
928} 928}
929 929
930static void msdc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, 930static void msdc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq)
931 bool is_first_req)
932{ 931{
933 struct msdc_host *host = mmc_priv(mmc); 932 struct msdc_host *host = mmc_priv(mmc);
934 struct mmc_data *data = mrq->data; 933 struct mmc_data *data = mrq->data;
diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
index 5f2f24a7360d..ad11c4cc12ed 100644
--- a/drivers/mmc/host/omap_hsmmc.c
+++ b/drivers/mmc/host/omap_hsmmc.c
@@ -1565,8 +1565,7 @@ static void omap_hsmmc_post_req(struct mmc_host *mmc, struct mmc_request *mrq,
1565 } 1565 }
1566} 1566}
1567 1567
1568static void omap_hsmmc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, 1568static void omap_hsmmc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq)
1569 bool is_first_req)
1570{ 1569{
1571 struct omap_hsmmc_host *host = mmc_priv(mmc); 1570 struct omap_hsmmc_host *host = mmc_priv(mmc);
1572 1571
diff --git a/drivers/mmc/host/rtsx_pci_sdmmc.c b/drivers/mmc/host/rtsx_pci_sdmmc.c
index 3ccaa1415f33..ecb99a8d2fa2 100644
--- a/drivers/mmc/host/rtsx_pci_sdmmc.c
+++ b/drivers/mmc/host/rtsx_pci_sdmmc.c
@@ -190,8 +190,7 @@ static int sd_pre_dma_transfer(struct realtek_pci_sdmmc *host,
190 return using_cookie; 190 return using_cookie;
191} 191}
192 192
193static void sdmmc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, 193static void sdmmc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq)
194 bool is_first_req)
195{ 194{
196 struct realtek_pci_sdmmc *host = mmc_priv(mmc); 195 struct realtek_pci_sdmmc *host = mmc_priv(mmc);
197 struct mmc_data *data = mrq->data; 196 struct mmc_data *data = mrq->data;
diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
index 62aedf191b0c..7f2526c9572f 100644
--- a/drivers/mmc/host/sdhci.c
+++ b/drivers/mmc/host/sdhci.c
@@ -2202,8 +2202,7 @@ static void sdhci_post_req(struct mmc_host *mmc, struct mmc_request *mrq,
2202 data->host_cookie = COOKIE_UNMAPPED; 2202 data->host_cookie = COOKIE_UNMAPPED;
2203} 2203}
2204 2204
2205static void sdhci_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, 2205static void sdhci_pre_req(struct mmc_host *mmc, struct mmc_request *mrq)
2206 bool is_first_req)
2207{ 2206{
2208 struct sdhci_host *host = mmc_priv(mmc); 2207 struct sdhci_host *host = mmc_priv(mmc);
2209 2208
diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
index 68639295148d..2a6418d0c343 100644
--- a/include/linux/mmc/host.h
+++ b/include/linux/mmc/host.h
@@ -93,8 +93,7 @@ struct mmc_host_ops {
93 */ 93 */
94 void (*post_req)(struct mmc_host *host, struct mmc_request *req, 94 void (*post_req)(struct mmc_host *host, struct mmc_request *req,
95 int err); 95 int err);
96 void (*pre_req)(struct mmc_host *host, struct mmc_request *req, 96 void (*pre_req)(struct mmc_host *host, struct mmc_request *req);
97 bool is_first_req);
98 void (*request)(struct mmc_host *host, struct mmc_request *req); 97 void (*request)(struct mmc_host *host, struct mmc_request *req);
99 98
100 /* 99 /*