ASoC: Intel: Remove vague commit about slave DMA config from firmware loader
authorJarkko Nikula <jarkko.nikula@linux.intel.com>
Wed, 18 Mar 2015 08:07:20 +0000 (10:07 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 18 Mar 2015 11:43:26 +0000 (11:43 +0000)
Intel MID DMA driver is going to be removed, commit should be a few lines
down near to dmaengine_slave_config() call in order to not confuse and at
quick look Synopsys DesignWare does seem to use some of the slave config
structure fields (see drivers/dma/dw/core.c: dwc_config()).

Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/sst-firmware.c

index 38881f1fb990b203ba6de78a5e70914bec9c9ecb..b5659ecb80de6f9fd8f7cfab454b7a04276efc39 100644 (file)
@@ -221,8 +221,6 @@ int sst_dsp_dma_get_channel(struct sst_dsp *dsp, int chan_id)
        dma_cap_mask_t mask;
        int ret;
 
-       /* The Intel MID DMA engine driver needs the slave config set but
-        * Synopsis DMA engine driver safely ignores the slave config */
        dma_cap_zero(mask);
        dma_cap_set(DMA_SLAVE, mask);
        dma_cap_set(DMA_MEMCPY, mask);