mmc: mmci: Enforce DMA configuration through DT
authorUlf Hansson <ulf.hansson@linaro.org>
Tue, 20 May 2014 04:45:54 +0000 (06:45 +0200)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 2 Jun 2014 09:20:25 +0000 (11:20 +0200)
Remove the option to provide DMA configuration as platform data,
enforce it through DT.

Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>
Cc: Roland Stigge <stigge@antcom.de>
Acked-by: Arnd Bergmann <arnd@arndb.de>
arch/arm/mach-lpc32xx/phy3250.c
drivers/mmc/host/mmci.c
include/linux/amba/mmci.h

index 34932e0e31fad10686ad3077b68a64003dd6e9a9..7858d5b6f6ce3b0a5de55c1464e98786b0a1c1f8 100644 (file)
@@ -202,9 +202,6 @@ static struct mmci_platform_data lpc32xx_mmci_data = {
        .ocr_mask       = MMC_VDD_30_31 | MMC_VDD_31_32 |
                          MMC_VDD_32_33 | MMC_VDD_33_34,
        .ios_handler    = mmc_handle_ios,
-       .dma_filter     = NULL,
-       /* No DMA for now since AMBA PL080 dmaengine driver only does scatter
-        * gather, and the MMCI driver doesn't do it this way */
 };
 
 static struct lpc32xx_slc_platform_data lpc32xx_slc_data = {
index 758efea184c963146cec06b3d408da56fdd0802a..a084edd37af53f379a429d40ea0f9ab6851b80f5 100644 (file)
@@ -366,7 +366,6 @@ static void mmci_init_sg(struct mmci_host *host, struct mmc_data *data)
 #ifdef CONFIG_DMA_ENGINE
 static void mmci_dma_setup(struct mmci_host *host)
 {
-       struct mmci_platform_data *plat = host->plat;
        const char *rxname, *txname;
        dma_cap_mask_t mask;
 
@@ -380,25 +379,6 @@ static void mmci_dma_setup(struct mmci_host *host)
        dma_cap_zero(mask);
        dma_cap_set(DMA_SLAVE, mask);
 
-       if (plat && plat->dma_filter) {
-               if (!host->dma_rx_channel && plat->dma_rx_param) {
-                       host->dma_rx_channel = dma_request_channel(mask,
-                                                          plat->dma_filter,
-                                                          plat->dma_rx_param);
-                       /* E.g if no DMA hardware is present */
-                       if (!host->dma_rx_channel)
-                               dev_err(mmc_dev(host->mmc), "no RX DMA channel\n");
-               }
-
-               if (!host->dma_tx_channel && plat->dma_tx_param) {
-                       host->dma_tx_channel = dma_request_channel(mask,
-                                                          plat->dma_filter,
-                                                          plat->dma_tx_param);
-                       if (!host->dma_tx_channel)
-                               dev_warn(mmc_dev(host->mmc), "no TX DMA channel\n");
-               }
-       }
-
        /*
         * If only an RX channel is specified, the driver will
         * attempt to use it bidirectionally, however if it is
@@ -446,11 +426,9 @@ static void mmci_dma_setup(struct mmci_host *host)
  */
 static inline void mmci_dma_release(struct mmci_host *host)
 {
-       struct mmci_platform_data *plat = host->plat;
-
        if (host->dma_rx_channel)
                dma_release_channel(host->dma_rx_channel);
-       if (host->dma_tx_channel && plat->dma_tx_param)
+       if (host->dma_tx_channel)
                dma_release_channel(host->dma_tx_channel);
        host->dma_rx_channel = host->dma_tx_channel = NULL;
 }
index 3f95d32d527796aff0144c34dd4e32b7fce42a3d..8c98113069ce9264a2bdf2fe7ae10198ceed0b9d 100644 (file)
@@ -6,9 +6,6 @@
 
 #include <linux/mmc/host.h>
 
-/* Just some dummy forwarding */
-struct dma_chan;
-
 /**
  * struct mmci_platform_data - platform configuration for the MMCI
  * (also known as PL180) block.
@@ -26,17 +23,6 @@ struct dma_chan;
  * @gpio_wp: read this GPIO pin to see if the card is write protected
  * @gpio_cd: read this GPIO pin to detect card insertion
  * @cd_invert: true if the gpio_cd pin value is active low
- * @dma_filter: function used to select an appropriate RX and TX
- * DMA channel to be used for DMA, if and only if you're deploying the
- * generic DMA engine
- * @dma_rx_param: parameter passed to the DMA allocation
- * filter in order to select an appropriate RX channel. If
- * there is a bidirectional RX+TX channel, then just specify
- * this and leave dma_tx_param set to NULL
- * @dma_tx_param: parameter passed to the DMA allocation
- * filter in order to select an appropriate TX channel. If this
- * is NULL the driver will attempt to use the RX channel as a
- * bidirectional channel
  */
 struct mmci_platform_data {
        unsigned int ocr_mask;
@@ -45,9 +31,6 @@ struct mmci_platform_data {
        int     gpio_wp;
        int     gpio_cd;
        bool    cd_invert;
-       bool (*dma_filter)(struct dma_chan *chan, void *filter_param);
-       void *dma_rx_param;
-       void *dma_tx_param;
 };
 
 #endif