
2 May
2006
2 May
'06
3:17 p.m.
In message 20060502115845.GA20961@tuxedo.skovlyporten.dk you wrote:
makes no sense and always returns true. IMHO we should remove it completely and always run the PhyOps.Init function. Checking on the EMAC_ISR register is also wrong since it indicates link changes and is AFAIK phy dependent and needs to have interrupts enabled.
On most at91rm9200 based systems I have worked on, I do not have link yet when PhyOps.Init is called. So I have added a small loop in my phy code waiting for link before returning (max 5 seconds).
Maybe you can submit a patch?
Thanks in advance.
Best regards,
Wolfgang Denk
--
Software Engineering: Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
In the realm of scientific observation, luck is granted only to those
who are prepared. - Louis Pasteur