ASoC: Revert "samsung: Remove unneeded initialization of chan_name"
authorSylwester Nawrocki <s.nawrocki@samsung.com>
Tue, 17 Jan 2017 13:16:42 +0000 (14:16 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 23 Jan 2017 18:16:33 +0000 (18:16 +0000)
This reverts commit cdaf9af1eaeb539e32bfd6da6310b41ad6c3ba23
which breaks I2S support on the non-DT Samsung SoC platforms,
since the default "tx", "rx" DMA channel names for playback
and capture streams or custom channel names in struct
snd_dmaengine_pcm_config are supported in the ASoC dmaengine
module only for devicetree booting case.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/samsung/dmaengine.c
sound/soc/samsung/i2s.c
sound/soc/samsung/s3c2412-i2s.c
sound/soc/samsung/s3c24xx-i2s.c

index cda656e4afc6985e0fe6ffb78f3abe486d7374e2..9104c98deeb75ab8ac3113e5be85ff4958fa2bdf 100644 (file)
@@ -37,8 +37,12 @@ int samsung_asoc_dma_platform_register(struct device *dev, dma_filter_fn filter,
        pcm_conf->prepare_slave_config = snd_dmaengine_pcm_prepare_slave_config;
        pcm_conf->compat_filter_fn = filter;
 
-       pcm_conf->chan_names[SNDRV_PCM_STREAM_PLAYBACK] = tx;
-       pcm_conf->chan_names[SNDRV_PCM_STREAM_CAPTURE] = rx;
+       if (dev->of_node) {
+               pcm_conf->chan_names[SNDRV_PCM_STREAM_PLAYBACK] = tx;
+               pcm_conf->chan_names[SNDRV_PCM_STREAM_CAPTURE] = rx;
+       } else {
+               flags |= SND_DMAENGINE_PCM_FLAG_CUSTOM_CHANNEL_NAME;
+       }
 
        return devm_snd_dmaengine_pcm_register(dev, pcm_conf, flags);
 }
index e00974bc561670ff30413a85e5fe2644148617e3..85324e61cbd56da5c07a55ae02151516ce4a8f00 100644 (file)
@@ -1305,6 +1305,8 @@ static int samsung_i2s_probe(struct platform_device *pdev)
        }
        pri_dai->dma_playback.addr = regs_base + I2STXD;
        pri_dai->dma_capture.addr = regs_base + I2SRXD;
+       pri_dai->dma_playback.chan_name = "tx";
+       pri_dai->dma_capture.chan_name = "rx";
        pri_dai->dma_playback.addr_width = 4;
        pri_dai->dma_capture.addr_width = 4;
        pri_dai->quirks = quirks;
@@ -1329,6 +1331,7 @@ static int samsung_i2s_probe(struct platform_device *pdev)
                sec_dai->lock = &pri_dai->spinlock;
                sec_dai->variant_regs = pri_dai->variant_regs;
                sec_dai->dma_playback.addr = regs_base + I2STXDS;
+               sec_dai->dma_playback.chan_name = "tx-sec";
 
                if (!np) {
                        sec_dai->dma_playback.filter_data = i2s_pdata->dma_play_sec;
index 6d0b8897fa6cdf64e77aec7c951234a2e285d89d..0a4718207e6ec41ae9f1535c5757066c73710f6a 100644 (file)
 #include <linux/platform_data/asoc-s3c.h>
 
 static struct snd_dmaengine_dai_dma_data s3c2412_i2s_pcm_stereo_out = {
+       .chan_name      = "tx",
        .addr_width     = 4,
 };
 
 static struct snd_dmaengine_dai_dma_data s3c2412_i2s_pcm_stereo_in = {
+       .chan_name      = "rx",
        .addr_width     = 4,
 };
 
index 07f5091b33e898bf7e6af19b02802db9645f2390..91e6871e5413a64898e33339d41903d15cba7f41 100644 (file)
 #include "s3c24xx-i2s.h"
 
 static struct snd_dmaengine_dai_dma_data s3c24xx_i2s_pcm_stereo_out = {
+       .chan_name      = "tx",
        .addr_width     = 2,
 };
 
 static struct snd_dmaengine_dai_dma_data s3c24xx_i2s_pcm_stereo_in = {
+       .chan_name      = "rx",
        .addr_width     = 2,
 };