summaryrefslogtreecommitdiffstats
path: root/sound/soc/fsl
diff options
context:
space:
mode:
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>2019-06-27 21:47:22 -0400
committerMark Brown <broonie@kernel.org>2019-06-28 10:11:38 -0400
commit8bebc822a50a9b5ae3dd88ebec7a3f4872da219c (patch)
tree3d397f21578fc8e0f2810ab9732fb701bc375396 /sound/soc/fsl
parent9998d3e2add865723a4dffabaa0a8c12c7ee589b (diff)
ASoC: fsl: imx-es8328: consider CPU-Platform possibility
commit 577cf50d4dc8 ("ASoC: fsl: imx-es8328: don't select unnecessary Platform") Current ALSA SoC avoid to add duplicate component to rtd, and this driver was selecting CPU component as Platform component. Thus, above patch removed Platform settings from this driver, because it assumed these are same component. But, some CPU driver is using generic DMAEngine, in such case, both CPU component and Platform component will have same of_node/name. In other words, there are some components which are different but have same of_node/name. In such case, Card driver definitely need to select Platform even though it is same as CPU. It is depends on CPU driver, but is difficult to know it from Card driver. This patch reverts above patch. Fixes: commit 577cf50d4dc8 ("ASoC: fsl: imx-es8328: don't select unnecessary Platform") Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/fsl')
-rw-r--r--sound/soc/fsl/imx-es8328.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/sound/soc/fsl/imx-es8328.c b/sound/soc/fsl/imx-es8328.c
index 089ee140c718..15a27a2cd0ca 100644
--- a/sound/soc/fsl/imx-es8328.c
+++ b/sound/soc/fsl/imx-es8328.c
@@ -148,7 +148,7 @@ static int imx_es8328_probe(struct platform_device *pdev)
148 goto fail; 148 goto fail;
149 } 149 }
150 150
151 comp = devm_kzalloc(dev, 2 * sizeof(*comp), GFP_KERNEL); 151 comp = devm_kzalloc(dev, 3 * sizeof(*comp), GFP_KERNEL);
152 if (!comp) { 152 if (!comp) {
153 ret = -ENOMEM; 153 ret = -ENOMEM;
154 goto fail; 154 goto fail;
@@ -160,15 +160,18 @@ static int imx_es8328_probe(struct platform_device *pdev)
160 160
161 data->dai.cpus = &comp[0]; 161 data->dai.cpus = &comp[0];
162 data->dai.codecs = &comp[1]; 162 data->dai.codecs = &comp[1];
163 data->dai.platforms = &comp[2];
163 164
164 data->dai.num_cpus = 1; 165 data->dai.num_cpus = 1;
165 data->dai.num_codecs = 1; 166 data->dai.num_codecs = 1;
167 data->dai.num_platforms = 1;
166 168
167 data->dai.name = "hifi"; 169 data->dai.name = "hifi";
168 data->dai.stream_name = "hifi"; 170 data->dai.stream_name = "hifi";
169 data->dai.codecs->dai_name = "es8328-hifi-analog"; 171 data->dai.codecs->dai_name = "es8328-hifi-analog";
170 data->dai.codecs->of_node = codec_np; 172 data->dai.codecs->of_node = codec_np;
171 data->dai.cpus->of_node = ssi_np; 173 data->dai.cpus->of_node = ssi_np;
174 data->dai.platforms->of_node = ssi_np;
172 data->dai.init = &imx_es8328_dai_init; 175 data->dai.init = &imx_es8328_dai_init;
173 data->dai.dai_fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF | 176 data->dai.dai_fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF |
174 SND_SOC_DAIFMT_CBM_CFM; 177 SND_SOC_DAIFMT_CBM_CFM;