
1 Feb
2009
1 Feb
'09
4:12 p.m.
Dear Jean-Christophe PLAGNIOL-VILLARD,
In message 20090201130727.GF9067@game.jcrosoft.org you wrote:
-static int iomux_getc(void) +static int iomux_getc (void)
...
-static int iomux_tstc(int file) +static int iomux_tstc (int file)
NAK.
This Linux kernel coding style does NOT insert spaces between function names; your patch makes the style worse, not better.
It's the coding style use in this file
We have a mix of styles here, so if we change it, then please to the "right" way.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
Given a choice between two theories, take the one which is funnier.