On Wed, May 28, 2008 at 10:28:59AM -0500, Geoffrey Wossum wrote:
Do I need to submit patches to fix the style issues?
Ideally, yes; there are some not entirely mechanical things in there as well so it'd be good to make sure that what goes in has been tested as-is.
Will this driver work with a current mainline kernel or does it require other things from the Atmel kernel?
at32-pcm.c and at32-ssc.c will compile against a vanilla 2.6.24.3 kernel. playpaq_wm8510.c will not, since it references the WM8510 code that is not in the mainline kernel. Not sure if it would really work. Even though AVR32 is
We should be able to submit the WM8510 driver so it shouldn't be a blocker.
in the mainline kernel, Atmel still has a fairly extensive patch against the kernel. Until you can use an unpatched and current kernel for the AVR32, this code probably isn't a candidate for inclusion the mainline.
OTOH, if it will build with current mainline then that reduces the diff that one needs to carry in order to use this support and means there's one less thing in that diff that needs to be merged into mainline. That seems like a net win to me.