[alsa-devel] [PATCH v2] soc/lapis: add machine driver

[Tomoya MORINAGA]

2011/12/6 Tomoya MORINAGA <tomoya.rohm at gmail.com>:
>>> >> +     ret = snd_soc_dai_set_fmt(codec_dai, SND_SOC_DAIFMT_I2S |
>>> >> +                             SND_SOC_DAIFMT_NB_NF | SND_SOC_DAIFMT_CBS_CFS);
>>> >> +     if (ret < 0)
>>> >> +             return ret;
>>
>>> > Use the dai_fmt field in the dai_link to set this.
>>
>>> Sorry, I can't understand your saying.
>>> Let me know in more detail.
>>
>> I'm not sure what more to say...  Have you looked at that field and how
>> it is both implemented and used in other drivers, or at the commit logs
>> for relevant changes?  What do you find unclear?
>
> You said "Use the dai_fmt field in the dai_link to set this."
> However, both dai_fmt and dai_link are already implemented like below.
> static struct snd_soc_dai_link ioh_i2s_dai = {
> ...
> };
>
> static struct snd_soc_card ioh_i2s_card = {
> ...
>        .dai_link       = &ioh_i2s_dai,
> ...
> };
> So, I can't understand your saying.

Could you give me your answer ?

thanks,

tomoya