
13 Dec
2010
13 Dec
'10
6:38 a.m.
Dear Macpaul Lin,
In message AANLkTikJM4hAw8yAWeUPiumiiN3vr9qgY-TT8KyYuSVZ@mail.gmail.com you wrote:
Please take a look at the similar driver "ftmac100.c", Some coding patterns in "ftgmac100.c" are followed as "ftmac100.c".
Oh. Thanks for pointing out. So this other driver needs to be fixed, too.
So I think you've permit that kind of coding style and related issues, this code of ftgmac100 shouldn't be a problem. ;p
The fact that some bad code slips through a review does not mean that it's blessed.
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
"The number of Unix installations has grown to 10, with more
expected." - The Unix Programmer's Manual, 2nd Edition, June, 1972