[U-Boot-Users] quit unclear code in cpu/arm920t/at91rm9200/ether.c Line:220
Wolfgang Denk
wd at denx.de
Tue May 2 15:17:57 CEST 2006
In message <20060502115845.GA20961 at 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 at denx.de
In the realm of scientific observation, luck is granted only to those
who are prepared. - Louis Pasteur
More information about the U-Boot
mailing list