[alsa-devel] [PATCH v2 0/1] *** SUBJECT HERE ***

[Mark Brown]

On Thu, Jul 22, 2010 at 12:38:53PM +0300, Matti J. Aaltonen wrote:

> > My major comment about this driver is that it'd probably make sense to
> > redo it on top of Liam's multi-component branch, though it shouldn't be
> > a pressing concern for merge.

> I didn't do this yet. But could you give me the git URL and then I'll resend?

git://git.kernel.org/pub/scm/linux/kernel/git/lrg/sound-2.6.git

> > Wouldn't it be nicer to do this stuff with the ALSA constraint APIs
> > rather than futzing with the constant data?

> I principle I of course agree. But it seems - also discussed with the local
> ALSA specialist - that using static constraints does not work here.
> HALF_DUPLEX is not one of the SNDRV_PCM_HW_PARAM_'s... or something like that.
> As a ALSA non-specialist I'd be willing to this leave as it is...

Could you please get whoever the expert is to get involved in this
thread?  If this is not currently possible it should probably be.

> >> +static int wl1273_set_dai_fmt(struct snd_soc_dai *codec_dai,
> >> +                           unsigned int fmt)
> >> +{
> >> +     return 0;
> >> +}

> >Remove unused functions.

> I tried leaving this out but then the codec stopped working. I guess I should
> mention that my test environment is 2.6.32. I'm only able to test that
> the codec compiles under 2.6.35. Anyway I left the function in.

Could you please be a little more specific - what do you mean when you
say "the CODEC stopped working?  The set_dai_fmt() function has always
been optional.

> >> +enum wl1273_mode wl1273_get_codec_mode(struct snd_soc_codec *codec)
> >> +{
> >> +     struct wl1273_priv *wl1273 = snd_soc_codec_get_drvdata(codec);
> >> +     return wl1273->mode;
> >> +}
> >> +EXPORT_SYMBOL_GPL(wl1273_get_codec_mode);

> > Why is this being exported?

> This is because the soc_card driver needs to know the codec mode and accessing
> the internals of the codec struct is ugly.

So why do you believe that the soc_card driver needs to know the CODEC
mode?  This was the gist of my original question...