[U-Boot] [PATCH] bootp: Fix bug in auto_load function
Wolfgang Denk
wd at denx.de
Wed Aug 31 22:01:52 CEST 2011
Dear Michal Simek,
In message <1314786967-23261-1-git-send-email-monstr at monstr.eu> you wrote:
> Patch: "Put common autoload code into auto_load() function"
> (sha1: 093498669e77597635a24f326f11efeab213d394) is not simple code
> cleanup but code change which introduce new bug.
>
> If autoload variable is not setup it worked as autoload=yes.
Yes, and this is intentionally, because it's what poople expect when
they run the "dhcpboot" command.
> Currently if autoload is not setup dhcp sends request in
> forever loop.
That's not correct. Only until a DHCP server provides the needed
configurationinformation so TFTP gets started.
> There are two options how to fix it:
> 1. Move TftpStart() which is in this patch
> 2. Change functionality if autoload is not setup, set NetSate and ends.
That would change behavious in a way that is not wanted.
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 at denx.de
Status quo. Latin for "the mess we're in."
More information about the U-Boot
mailing list