
17 May
2004
17 May
'04
6:11 p.m.
In message 200405171119.42555.pruhland@microwavedata.com you wrote:
- proper tx interrupt handling
- packets must still be freed/releases manually on error even with autorelease enabled).
- Tx int timeout is 100ms according to the manual.
- reverts CTL_DEFAULT back to register default. AUTORELEASE is not the default...or this with default at init
- removes 'getenv' and 'setenv' calls from 'smc_get_ethaddr'
- don't think the ethernet driver should need to know about the environment, the data need is already passed to function via board info.
- don't think ethernet driver should change environment, changing info is fine.
Please split this into separate patches. I may apply the first 2 parts, but will probably reject the third part. The current behaviour (as far as I can see) is exactly as documented. I don't see anything wrong there. YMMV.
Best regards,
Wolfgang Denk
--
Software Engineering: Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-4596-87 Fax: (+49)-8142-4596-88 Email: wd@denx.de
If you hear an onion ring, answer it.