[U-Boot] DHCP regression on 2009-06
Ben Warren
biggerbadderben at gmail.com
Tue Jul 14 16:57:36 CEST 2009
On Tue, Jul 14, 2009 at 2:26 AM, Michael Zaidman
<michael.zaidman at gmail.com>wrote:
> On Mon, Jul 13, 2009 at 9:40 PM, Robin Getz <rgetz at blackfin.uclinux.org>
> wrote:
> >
> > On Mon 13 Jul 2009 11:11, Michael Zaidman pondered:
> > > > I did verify that reverting the line exposes the bug that Michael
> fixed, ...
> > >
> > > Ok, my target uses static IP configuration so I did not verified the
> DHCP
> > > behavior. Now I did it. I also reverted the line and did DHCP
> afterwards
> > > changed the subnet and did DHCP again. It works as expected.
> > >
> > > diff --git a/net/net.c b/net/net.c
> > > index 5637cf5..5e43dd1 100644
> > > --- a/net/net.c
> > > +++ b/net/net.c
> > > @@ -388,6 +388,7 @@ restart:
> > > #if defined(CONFIG_CMD_DHCP)
> > > case DHCP:
> > > BootpTry = 0;
> > > + NetOurIP = 0;
> > > DhcpRequest(); /* Basically same as
> BOOTP */
> > > break;
> > > #endif
> > >
> >
> > OK - I tested it - and it seems to work for me.
> >
> > Some of my debugging crap when I was tracking this down must have messed
> this up.
> >
> > Ack-by: Robin Getz <rgetz at blackfin.uclinux.org>
>
> If nobody has objections I can submit this fix.
>
> Michael.
>
Please do.
regards,
Ben
More information about the U-Boot
mailing list