Re: [PATCH V3 18/19] mmc: mmci: Enforce DMA configuration through DT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 10 April 2014 14:21, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote:
> Remove the option to provide DMA configuration as platform data,
> enforce it through DT.
>
> Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>

This patch breaks compiling of arch/arm/mach-lpc32xx/phy3250.c. Found
by Fengguang's "kbuild test robot" while testing in Linux next.

The .dma_filter platform callback were assigned to NULL, thus the
callback can't just be removed - stupid me.

I drop this patch from my next branch for now - and will send a v4.

Kind regards
Ulf Hansson

> ---
>  drivers/mmc/host/mmci.c   |   24 +-----------------------
>  include/linux/amba/mmci.h |   17 -----------------
>  2 files changed, 1 insertion(+), 40 deletions(-)
>
> diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c
> index 9a87bad..04cff5e 100644
> --- a/drivers/mmc/host/mmci.c
> +++ b/drivers/mmc/host/mmci.c
> @@ -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;
>  }
> diff --git a/include/linux/amba/mmci.h b/include/linux/amba/mmci.h
> index b992fc9..bf339ff 100644
> --- a/include/linux/amba/mmci.h
> +++ b/include/linux/amba/mmci.h
> @@ -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.
> @@ -29,17 +26,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 f_max;
> @@ -49,9 +35,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
> --
> 1.7.9.5
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux