On 22/05/14 20:53, Mark Brown wrote:
On Thu, May 22, 2014 at 01:55:08PM +0200, Sylwester Nawrocki wrote:
From: Chen Zhen zhen1.chen@samsung.com
This machine driver primary defines the audio path, the supported data formats and sample frequency.
I guess it can't yet, it would have been good to explain that in the changelog for the bindings (and include the binding in this patch...).
Thanks for the review, I'm still rather inexperienced in the ASoC stuff, I've obviously overlooked there is actually no routing handling. I'd rather add those bits. As for the DT binding documentation patch, I thought we're supposed to keep documentation in separate patches, so processes of separating it from the kernel is easier ?
- ret = snd_soc_dai_set_fmt(codec_dai, SND_SOC_DAIFMT_I2S
| SND_SOC_DAIFMT_NB_NF| SND_SOC_DAIFMT_CBM_CFM);- if (ret < 0)
return ret;- ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_I2S
| SND_SOC_DAIFMT_NB_NF| SND_SOC_DAIFMT_CBM_CFM);- if (ret < 0)
return ret;These are constant, set these in the dai_link.
- ret = snd_soc_dai_set_sysclk(codec_dai, 3, MAX98090_MCLK_FREQ,
SND_SOC_CLOCK_IN);- if (ret < 0) {
dev_err(codec_dai->dev,"Unable to switch to FLL1: %d\n", ret);return ret;- }
So's this - also what is 3? Set it on init.
OK, will move it. I'm not sure why 3, it seems to be ignored by the codec anyway. Chen, can you confirm the second argument is actually a don't care value ?
- ret = snd_soc_dai_set_sysclk(cpu_dai, SAMSUNG_I2S_OPCLK,
0, MOD_OPCLK_PCLK);Similarly for the rest of the function.
- return snd_soc_register_card(card);
devm_snd_soc_register_card().
Thanks for the suggestions, I'll amend that.
-- Regards, Sylwester