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

[matti.j.aaltonen at nokia.com]

Hi.

Thanks for the comments.
________________________________________
From: ext Mark Brown [broonie at opensource.wolfsonmicro.com]
Sent: Thursday, July 22, 2010 11:48 AM
To: Aaltonen Matti.J (Nokia-MS/Tampere)
Cc: alsa-devel at alsa-project.org; lrg at slimlogic.co.uk
Subject: Re: [PATCH v2 0/1] *** SUBJECT HERE ***

>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

Thanks...

>> > 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.

Yes, he's being CCed...

>> >> +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.

I get this error message:

aplay  -f dat -Dhw:wl1273 /root/sumppi.wav 
[  297.179626] Can't set codec DAI configuration: -22
[  297.184539] asoc: BT/FM PCM startup failed

>> >> +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...

OK, sorry... The problem in a sense is that the codec supports three different modes: BT, FM RX and
FM TX. All digital audio goes over a single McBSP connection. The BT part handles the audio routing, 
which is completely external to the radio driver and this codec. And the user of the codec is 
in principle responsible for keeping the codec mode in sync with the route setting. And the 
soc_card driver needs to know the setting also to do its thing.

Thanks,
Matti