diff options
| author | Linus Walleij <linus.walleij@linaro.org> | 2016-11-23 05:02:24 -0500 |
|---|---|---|
| committer | Ulf Hansson <ulf.hansson@linaro.org> | 2016-11-29 03:05:27 -0500 |
| commit | d3c6aac3bdfe97b8b44db6a8aba59786cb9531dc (patch) | |
| tree | bff68f24e27d72562f187077dd899524cfddde2f | |
| parent | e173f8911f091fa50ccf8cc1fa316dd5569bc470 (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.c | 11 | ||||
| -rw-r--r-- | drivers/mmc/host/dw_mmc.c | 3 | ||||
| -rw-r--r-- | drivers/mmc/host/jz4740_mmc.c | 3 | ||||
| -rw-r--r-- | drivers/mmc/host/mmci.c | 3 | ||||
| -rw-r--r-- | drivers/mmc/host/mtk-sd.c | 3 | ||||
| -rw-r--r-- | drivers/mmc/host/omap_hsmmc.c | 3 | ||||
| -rw-r--r-- | drivers/mmc/host/rtsx_pci_sdmmc.c | 3 | ||||
| -rw-r--r-- | drivers/mmc/host/sdhci.c | 3 | ||||
| -rw-r--r-- | include/linux/mmc/host.h | 3 |
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 | */ |
| 621 | static void mmc_pre_req(struct mmc_host *host, struct mmc_request *mrq, | 619 | static 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 | ||
| 888 | static void dw_mci_pre_req(struct mmc_host *mmc, | 888 | static 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 | ||
| 322 | static void jz4740_mmc_pre_request(struct mmc_host *mmc, | 322 | static 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 | ||
| 702 | static void mmci_pre_request(struct mmc_host *mmc, struct mmc_request *mrq, | 702 | static 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 | ||
| 930 | static void msdc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, | 930 | static 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 | ||
| 1568 | static void omap_hsmmc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, | 1568 | static 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 | ||
| 193 | static void sdmmc_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, | 193 | static 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 | ||
| 2205 | static void sdhci_pre_req(struct mmc_host *mmc, struct mmc_request *mrq, | 2205 | static 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 | /* |
