19 Dec
2013
19 Dec
'13
4:16 a.m.
On Tue, Dec 17, 2013 at 11:24:38AM +0800, Xiubo Li wrote:
This adds Freescale SAI ASoC Audio support. This implementation is only compatible with device tree definition.
This is mostly good, there are two small issues below but I've applied it since they don't really affect the driver as a whole. Please send followup patches to fix the issues below.
+static int fsl_sai_dai_probe(struct snd_soc_dai *cpu_dai) {
- struct fsl_sai *sai = dev_get_drvdata(cpu_dai->dev);
- cpu_dai->playback_dma_data = &sai->dma_params_tx;
- cpu_dai->capture_dma_data = &sai->dma_params_rx;
You should use snd_soc_dai_init_dma_data() for this.
Yes, I will use it.
+static int fsl_sai_dai_remove(struct snd_soc_dai *cpu_dai) {
- cpu_dai->playback_dma_data = NULL;
- cpu_dai->capture_dma_data = NULL;
- snd_soc_dai_set_drvdata(cpu_dai, NULL);
- return 0;
+}
This is all unneeded and this function should be removed entirely.
I will remove this and other unneeded functions like this.
Best Regards, Xiubo