[ALSA] soc - ASoC Sharp corgi machine
This patch updates the Sharp corgi machine driver to the new API in ASoC
0.13.
Changes:-
o Manually configure DAI hardware format.
o Removed config_sysclk() function. No longer needed as clocking is now
configured manually.
Signed-off-by: Liam Girdwood <lg@opensource.wolfsonmicro.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@suse.cz>
diff --git a/sound/soc/pxa/corgi.c b/sound/soc/pxa/corgi.c
index 2b1c6e9..5ee51a9 100644
--- a/sound/soc/pxa/corgi.c
+++ b/sound/soc/pxa/corgi.c
@@ -37,6 +37,7 @@
#include "../codecs/wm8731.h"
#include "pxa2xx-pcm.h"
+#include "pxa2xx-i2s.h"
#define CORGI_HP 0
#define CORGI_MIC 1
@@ -119,8 +120,59 @@
return 0;
}
+static int corgi_hw_params(struct snd_pcm_substream *substream,
+ struct snd_pcm_hw_params *params)
+{
+ struct snd_soc_pcm_runtime *rtd = substream->private_data;
+ struct snd_soc_codec_dai *codec_dai = rtd->dai->codec_dai;
+ struct snd_soc_cpu_dai *cpu_dai = rtd->dai->cpu_dai;
+ unsigned int clk = 0;
+ int ret = 0;
+
+ switch (params_rate(params)) {
+ case 8000:
+ case 16000:
+ case 48000:
+ case 96000:
+ clk = 12288000;
+ break;
+ case 11025:
+ case 22050:
+ case 44100:
+ clk = 11289600;
+ break;
+ }
+
+ /* set codec DAI configuration */
+ ret = codec_dai->dai_ops.set_fmt(codec_dai, SND_SOC_DAIFMT_I2S |
+ SND_SOC_DAIFMT_NB_NF | SND_SOC_DAIFMT_CBS_CFS);
+ if (ret < 0)
+ return ret;
+
+ /* set cpu DAI configuration */
+ ret = cpu_dai->dai_ops.set_fmt(cpu_dai, SND_SOC_DAIFMT_I2S |
+ SND_SOC_DAIFMT_NB_NF | SND_SOC_DAIFMT_CBS_CFS);
+ if (ret < 0)
+ return ret;
+
+ /* set the codec system clock for DAC and ADC */
+ ret = codec_dai->dai_ops.set_sysclk(codec_dai, WM8731_SYSCLK, clk,
+ SND_SOC_CLOCK_IN);
+ if (ret < 0)
+ return ret;
+
+ /* set the I2S system clock as input (unused) */
+ ret = cpu_dai->dai_ops.set_sysclk(cpu_dai, PXA2XX_I2S_SYSCLK, 0,
+ SND_SOC_CLOCK_IN);
+ if (ret < 0)
+ return ret;
+
+ return 0;
+}
+
static struct snd_soc_ops corgi_ops = {
.startup = corgi_startup,
+ .hw_params = corgi_hw_params,
.shutdown = corgi_shutdown,
};
@@ -264,35 +316,6 @@
return 0;
}
-static unsigned int corgi_config_sysclk(struct snd_soc_pcm_runtime *rtd,
- struct snd_soc_clock_info *info)
-{
- if (info->bclk_master & SND_SOC_DAIFMT_CBS_CFS) {
- /* pxa2xx is i2s master */
- switch (info->rate) {
- case 44100:
- case 88200:
- /* configure codec digital filters for 44.1, 88.2 */
- rtd->codec_dai->config_sysclk(rtd->codec_dai, info,
- 11289600);
- break;
- default:
- /* configure codec digital filters for all other rates */
- rtd->codec_dai->config_sysclk(rtd->codec_dai, info,
- CORGI_AUDIO_CLOCK);
- break;
- }
- /* config pxa i2s as master */
- return rtd->cpu_dai->config_sysclk(rtd->cpu_dai, info,
- CORGI_AUDIO_CLOCK);
- } else {
- /* codec is i2s master -
- * only configure codec DAI clock and filters */
- return rtd->codec_dai->config_sysclk(rtd->codec_dai, info,
- CORGI_AUDIO_CLOCK);
- }
-}
-
/* corgi digital audio interface glue - connects codec <--> CPU */
static struct snd_soc_dai_link corgi_dai = {
.name = "WM8731",
@@ -300,7 +323,7 @@
.cpu_dai = &pxa_i2s_dai,
.codec_dai = &wm8731_dai,
.init = corgi_wm8731_init,
- .config_sysclk = corgi_config_sysclk,
+ .ops = &corgi_ops,
};
/* corgi audio machine driver */
@@ -308,7 +331,6 @@
.name = "Corgi",
.dai_link = &corgi_dai,
.num_links = 1,
- .ops = &corgi_ops,
};
/* corgi audio private data */