Re: Ответ: [PATCH 01/24] sound/pci: coding style fixes: ac97

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

 



At Tue, 9 Sep 2008 19:15:36 +0400,
Alexander Beregalov wrote:
> 
> 2008/9/9, Takashi Iwai <tiwai@xxxxxxx>:
> >> - <asm/io.h> -> <linux/io.h>
> >> - remove trailing whitespaces
> >> - convert comments
> >>
> >> Only compile tested.
> >
> > Thanks for patches.  It takes some time to check all of them because
> > apparently you tested only compilation :)
> > I'll let you know when these are merged.
> Yes, sure.
> I did not just modify files and test compilation, I certainly read all
> that changes, but it would be better if someone inspects them too.

One thing I noticed that spaces around open parentheses aren't always
fixed, e.g. in sound/pci/cs46xx/*.  Could you fix them?

BTW, I'll likely postpone some (or most) of these patches until
shortly before the final push to the upstream, because these spacing
changes break all other upcoming patches.  That is, this can be even
worse than nothing in such a case.
So, applying this at the last would be a better option.


thanks,

Takashi
_______________________________________________
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