Re: [PATCH] alsa: Emagic Audiowerk8 low-level ALSA driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At 13.10.2008 12:32
tiwai@xxxxxxx wrote:

Hi Takashi,

>[please stop top-posting]

Ok, sorry for that. 

>I don't care in which direction, whether to merge yours into aw2 or
>aw2 into yours.  I just want to have a reasonable merge of two
>works.
>
>Well, this is basically an option, and can be achieved later, too.
>So, could you post the patches to ML for further reviews?

Yes, I'll do that soon.

>But before submitting patches, I recommend you to run once
>scripts/checkpatch.pl (found in kernel tree) with your patches.
>You'll find lots of coding style issues even by such a simple tool.
>Then, check Documentation/CodingStyle and fix your codes again.
>
>And, please remove the debug trace stuff unless you really need it in
>future.  This decreases readability fairly much.

I guess you refer to the 0.1 sourceforge release -> I actually did that before submitting aw-alsa-patch-0.2.
Anyway, I'll post the patch again, this time in Mail body, following your recommendations.

Thank you for helping.

Regards,

Matthias
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux