On 11/14/07, Detlev Zundel <dzu@denx.de> wrote:

Hi Upakul,

> Hi all, I observed that when the Ethernet initialization fails, it is not
> properly halting the operation and exiting. On walking through the Networking
> files, I saw that the eth_init() in eth.c either returns a 0 or 1. Now, in the
> NetLoop() in net.c file, from where the eth_init() gets called, checks the
> condition if (eth_init(bd) < 0) { eth_halt(); return(-1); } which is thus never
> true. Thus the network operation, never exits gracefully, if Ethernet init
> fails. Any of you, have any clues about this??

I think your analysis is right and thus the code in NetLoop should be
fixed.  Feel free to send a patch and I am pretty sure Ben Warren will
pick it up into the u-boot-net repo.

Cheers
  Detlev

--
Two  monks went  fishing in an  electron river.  The first monk drew out his
network, and out flopped a hacker.  The second monk cried, "The poor hacker!
How can  it live outside of the  network?"   The first monk said,  "When you
have learned to live outside the network, then you will know."
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu@denx.de