29 May
2007
29 May
'07
4:28 p.m.
At Tue, 29 May 2007 09:09:39 -0500, Ash Willis wrote:
Also I changed "if ((err = ...))" to two individual lines, which is a newer standard kernel coding style.
Yeah, I left that for now. I'll go through the tree and tidy up remaining occurences of this style when I have some time.
Note that I didn't rewrite the existing lines but just changes in the patch. It'll be appreciated if you send a patch to clean up these things in your driver.
It seems that when people add drivers to the tree, they generally copy already existing drivers. It doesn't make much sense to be copying code that doesn't fit CodingStyle.
Right. A general clean up work is appreciated.
thanks,
Takashi