Robin, Please keep in mind that the exact subject text becomes the changelog title:
" - save the server's mac address..." looks pretty dumb. I took the liberty of cleaning it up. Robin Getz wrote: > On Tue 21 Jul 2009 02:37, Ben Warren pondered: > >> Can you please re-submit using git tools? >> > > From: Robin Getz <rg...@blackfin.uclinux.org> > > Linux's netconsole works much better when you can pass it the MAC address of > the server. (otherwise it just uses broadcast, which everyone else on my > network complains about :) > > This sets the env var "serveraddr" (to match ethaddr), so that you can pass > it to linux with whatever bootargs you want to.... > > addnetconsole=set bootargs $(bootargs) > netconso...@$(ipaddr)/eth0,@$(serverip)/$(serveraddr) > > Signed-of-by: Robin Getz <rg...@blackfin.uclinux.org> > > ----- > > Please create the patch using git tools, 'git format-patch' in particular. It takes care of generating nice lines with three hyphens. This one that you've created messes things up. > diff --git a/README b/README > index 4c74cb7..9071472 100644 > --- a/README > +++ b/README > @@ -1184,6 +1184,11 @@ The following options need to be configured: > Defines a default value for the IP address of a TFTP > server to contact when using the "tftboot" command. > > + CONFIG_KEEP_SERVERADDR > + > + Keeps the server's MAC address, in the env 'serveraddr' > + for passing to bootargs (like Linux's netconsole option) > + > - Multicast TFTP Mode: > CONFIG_MCAST_TFTP > > diff --git a/net/net.c b/net/net.c > index 7ce947d..641c37c 100644 > --- a/net/net.c > +++ b/net/net.c > @@ -1287,6 +1287,15 @@ NetReceive(volatile uchar * inpkt, int len) > /* are we waiting for a reply */ > if (!NetArpWaitPacketIP || !NetArpWaitPacketMAC) > break; > + > +#ifdef CONFIG_KEEP_SERVERADDR > + if (NetServerIP == NetArpWaitPacketIP) { > + char buf[20]; > + sprintf(buf, "%pM", arp->ar_data); > + setenv("serveraddr", buf); > + } > +#endif > + > #ifdef ET_DEBUG > printf("Got ARP REPLY, set server/gtwy eth addr > (%pM)\n", > arp->ar_data); > Anti-climatically... applied to net repo. thanks, Ben _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot