On Mon, Oct 29, 2012 at 7:17 PM, Mark Brown broonie@opensource.wolfsonmicro.com wrote:
On Mon, Oct 29, 2012 at 06:49:18PM +0100, Paolo Doz wrote:
Please don't top post.
@Lars-Peter: thanks for the code review @Mark: You're right. The original source code (that I found on the web) was for kernel 2.6.35, and then I adapted it to work with kernel 3.2 series. Can you suggest me a "state of the art" codec driver where can I have a look at best practice?
Anything that's been getting work recently... wm8731 is probably similar to what you have there.
In the end the codec should work with alsa infrastructure available in kernel 3.2 so I can test it's correctness directly on my hw.
That's a *very* old kernel now, any mainline submissions should be against a current development kernel.
Hi, I'm rewriting the ak5702 codec driver following you suggestions.
From you space should be possible to select input source. The codec
has 2 ADC with 2 differents inputs and another one shared by both (input5)
Inside the codec the input selection is made in the following way
8 bit register (ie ADC A) bit 0 -> select left1 or left2 bit 1 -> select right1 or right2
bit2/3/4 other settings
bit5 -> if 1 select left5 otherwise select (left1 or left2) depending on bit0 bit6 -> if 1 select right5 otherwise select (right1 or rigth2) depending on bit1
Is there any SOC_ENUM that can be used to represent this situation? I mean something like "Left1, Left2, Left5" ?
At the moment I have one SOC_ENUM to select "Left1, Left2" and another for "Left1/2, Left5"
Thanks, Paolo