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.