[alsa-devel] [PATCH v3] ASoC: Add support for cs42l73 codec

[Austin, Brian]

On Oct 6, 2011, at 1:57 PM, Mark Brown wrote:
> On Thu, Oct 06, 2011 at 11:19:05AM -0500, Brian Austin wrote:
> 
>> +	{"DAC1 Mixer", "Headphone Switch", "XSPIN"},
>> +	{"DAC1 Mixer", "LineOut Switch", "XSPIN"},
>> +	{"DAC2 Mixer", "Speakerphone Switch", "XSPIN"},
>> +	{"DAC2 Mixer", "Speakerphone LineOut Switch", "XSPIN"},
>> +	{"DAC2 Mixer", "EarSpeaker Switch", "XSPIN"},
> 
>> +	{"DAC1 Mixer", "Headphone Switch", "ASPIN"},
>> +	{"DAC1 Mixer", "LineOut Switch", "ASPIN"},
>> +	{"DAC2 Mixer", "Speakerphone Switch", "ASPIN"},
>> +	{"DAC2 Mixer", "Speakerphone LineOut Switch", "ASPIN"},
>> +	{"DAC2 Mixer", "EarSpeaker Switch", "ASPIN"},
> 
>> +	{"DAC1 Mixer", "Headphone Switch", "VSPIN"},
>> +	{"DAC1 Mixer", "LineOut Switch", "VSPIN"},
>> +	{"DAC2 Mixer", "Speakerphone Switch", "VSPIN"},
>> +	{"DAC2 Mixer", "Speakerphone LineOut Switch", "VSPIN"},
>> +	{"DAC2 Mixer", "EarSpeaker Switch", "VSPIN"},
> 
> This all looks totally wrong and I'm surprised the driver loads without
> errors.  There's two issues:
> 
> - All the controls here are for the input to the DAC mixers but their
>   names are for physical outputs.  I'd expect the switches to name
>   inputs (for example, "DAC1 Mixer VSPIN Switch").  What's actually
>   going on here?
> 
> - You've got three copies of each of the controls for each mixer - I'm
>   surprsied this loads without the core complaining that it can't
>   create controls with duplicate names.
> 

This works fine on my setup with a beagleboard. 

There are 3 routes for the HP/LO/SPK/SPKLO/EAR. What I am trying to accomplish is the following.
Allow a stream from any serial port to go to any output.

 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3917 bytes
Desc: not available
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20111006/6b6800cc/attachment.p7s