
9 Jan
2011
9 Jan
'11
10:17 p.m.
Dear Macpaul Lin,
In message 1292921986-23419-1-git-send-email-macpaul@andestech.com you wrote:
Add Faraday's ftgmac100 (gigabit ethernet) MAC controller's driver.
Signed-off-by: Macpaul Lin macpaul@andestech.com
Changes for v2:
- Coding Style cleanup
- Makefile Order sorted
- Volatile statement eliminated
- Error handling added
- Autonegotiation timeout value rechecked
- Link speed detection code is reduced
- Logic of autonegotiation and Link speed update reorganized
- Redundent registers offset table cleared
- The explaination of sub configuration of this driver has moved into README
Changes for v3:
- Coding Style clean up about
- 2 continue blank line
- incorrect multi-line comment style
- useless else statement and return value statement
- make multi-line statement into single line which doesn't exceed 80 chars
- brance with multi-line statement (comments included before one line code)
README | 12 + drivers/net/Makefile | 1 + drivers/net/ftgmac100.c | 570 +++++++++++++++++++++++++++++++++++++++++++++++ drivers/net/ftgmac100.h | 255 +++++++++++++++++++++ include/netdev.h | 1 + 5 files changed, 839 insertions(+), 0 deletions(-) create mode 100644 drivers/net/ftgmac100.c create mode 100644 drivers/net/ftgmac100.h
Applied, thanks.
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
Madness takes its toll. Please have exact change.