In message <[EMAIL PROTECTED]> you wrote: > Robin Getz wrote: > > On Fri 11 Jul 2008 18:05, Ben Warren pondered: > > > >> Hi Robin, > >> > >> Robin Getz wrote: > >> > >>> On Fri 11 Jul 2008 14:21, Ben Warren pondered: > >>> > >>>> Robin Getz wrote:
Hm... this looks as if there was some longer discussion, but I cannot find any traces of this on the mailing list. Would be interesting to know what you discussed. At least if it should result in any changes to the existing code. > > Before - the U-Boot had set source IP number to the IP number it had been > > offered (my first patch leaves it that way). With the call to > > BootpCopyNetParams() removed, it does a broadcast (Source IP is 0.0.0.0 - > > just like the RFC says it should do). ??? Setting the source IP addr to 0 does not mean we're broadcasting? Could you please fill us in on what you discussed? > > I have no idea if some other broken DHCP server historically needed that, > > or > > what was going on - so that is why I only fixed the operation on the wire - > > not changed it from it's existing state (which AFAICT - _is_ wrong). It seems Robin is claiming this - what exactly is wrong? > > I'm more than happy to send the patch that remove the call (and makes > > things > > more correct, fixes the bug, and makes things smaller) - but I'm also > > hesitant since I don't want to break it for anyone else :) Before anybody removes anything I want to know exactly what's going on here, please. > > The code does do that. The call to BootpCopyNetParams() sets up the > > network, > > and allows the ARP code in ./net/net.c to respond (before it should) this > > is > > what causes the problem I was/am having. Please explain... > I'm a bit of an idealist, so I say let's do it right (remove the call to > BootpCopyNetParams()). Unless somebody with some historical perspective > weighs in, we'll pull it in as soon as the next merge window opens and > see what happens. Please explain... 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: [EMAIL PROTECTED] A conservative is a man with two perfectly good legs who has never learned to walk. - Franklin D. Roosevelt ------------------------------------------------------------------------- Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW! Studies have shown that voting for your favorite open source project, along with a healthy diet, reduces your potential for chronic lameness and boredom. Vote Now at http://www.sourceforge.net/community/cca08 _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users