On Thu, Apr 18, 2013 at 11:25:44AM +0200, Arnd Bergmann wrote: > +static void mmci_dma_setup(struct device *dev, struct mmci_host *host) > +{ > + const char *rxname, *txname; > + > + host->dma_rx_channel = dma_request_slave_channel(dev, "rx"); > + host->dma_tx_channel = dma_request_slave_channel(dev, "tx"); > + > + if (!host->dma_rx_channel && !host->dma_tx_channel) { > + if (host->plat && host->plat->dma_filter) > + mmci_dma_plat_setup(host); > + else > + dev_info(mmc_dev(host->mmc), "no DMA platform data\n"); > + return; You may like to fix the sillies in this patch... but yes, the above would be better. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html