]> git.kernelconcepts.de Git - karo-tx-uboot.git/blobdiff - net/bootp.c
dm: Adjust lists_bind_fdt() to return the bound device
[karo-tx-uboot.git] / net / bootp.c
index 77057c6c0a7b2c51e29dc4ed0482a6f8d1cd5992..d4c86cf1794e7d78feafc837195f41a596a7dd6d 100644 (file)
@@ -8,17 +8,6 @@
  *     Copyright 2000-2004 Wolfgang Denk, wd@denx.de
  */
 
-#if 0
-#define DEBUG          1       /* general debug */
-#define DEBUG_BOOTP_EXT 1      /* Debug received vendor fields */
-#endif
-
-#ifdef DEBUG_BOOTP_EXT
-#define debug_ext(fmt,args...) printf (fmt ,##args)
-#else
-#define debug_ext(fmt,args...)
-#endif
-
 #include <common.h>
 #include <command.h>
 #include <net.h>
 #ifdef CONFIG_STATUS_LED
 #include <status_led.h>
 #endif
+#ifdef CONFIG_BOOTP_RANDOM_DELAY
+#include "net_rand.h"
+#endif
 
-#define BOOTP_VENDOR_MAGIC     0x63825363      /* RFC1048 Magic Cookie         */
-
-#if defined(CONFIG_CMD_NET)
+#define BOOTP_VENDOR_MAGIC     0x63825363      /* RFC1048 Magic Cookie */
 
-#define TIMEOUT                5000UL  /* Milliseconds before trying BOOTP again */
+/*
+ * The timeout for the initial BOOTP/DHCP request used to be described by a
+ * counter of fixed-length timeout periods. TIMEOUT_COUNT represents
+ * that counter
+ *
+ * Now that the timeout periods are variable (exponential backoff and retry)
+ * we convert the timeout count to the absolute time it would have take to
+ * execute that many retries, and keep sending retry packets until that time
+ * is reached.
+ */
 #ifndef CONFIG_NET_RETRY_COUNT
-# define TIMEOUT_COUNT 5               /* # of timeouts before giving up  */
+# define TIMEOUT_COUNT 5               /* # of timeouts before giving up */
 #else
 # define TIMEOUT_COUNT (CONFIG_NET_RETRY_COUNT)
 #endif
+#define TIMEOUT_MS     ((3 + (TIMEOUT_COUNT * 5)) * 1000)
 
-#define PORT_BOOTPS    67              /* BOOTP server UDP port                */
-#define PORT_BOOTPC    68              /* BOOTP client UDP port                */
+#define PORT_BOOTPS    67              /* BOOTP server UDP port */
+#define PORT_BOOTPC    68              /* BOOTP client UDP port */
 
-#ifndef CONFIG_DHCP_MIN_EXT_LEN                /* minimal length of extension list     */
+#ifndef CONFIG_DHCP_MIN_EXT_LEN                /* minimal length of extension list */
 #define CONFIG_DHCP_MIN_EXT_LEN 64
 #endif
 
-ulong          BootpID;
-int            BootpTry;
-#ifdef CONFIG_BOOTP_RANDOM_DELAY
-ulong          seed1, seed2;
+#ifndef CONFIG_BOOTP_ID_CACHE_SIZE
+#define CONFIG_BOOTP_ID_CACHE_SIZE 4
 #endif
 
+ulong          bootp_ids[CONFIG_BOOTP_ID_CACHE_SIZE];
+unsigned int   bootp_num_ids;
+int            BootpTry;
+ulong          bootp_start;
+ulong          bootp_timeout;
+
 #if defined(CONFIG_CMD_DHCP)
-dhcp_state_t dhcp_state = INIT;
-unsigned long dhcp_leasetime = 0;
-IPaddr_t NetDHCPServerIP = 0;
-static void DhcpHandler(uchar * pkt, unsigned dest, unsigned src, unsigned len);
+static dhcp_state_t dhcp_state = INIT;
+static unsigned long dhcp_leasetime;
+static IPaddr_t NetDHCPServerIP;
+static void DhcpHandler(uchar *pkt, unsigned dest, IPaddr_t sip, unsigned src,
+                       unsigned len);
 
 /* For Debug */
 #if 0
@@ -75,39 +80,55 @@ static char *dhcpmsg2str(int type)
        }
 }
 #endif
-
-#if defined(CONFIG_BOOTP_VENDOREX)
-extern u8 *dhcp_vendorex_prep (u8 *e); /*rtn new e after add own opts. */
-extern u8 *dhcp_vendorex_proc (u8 *e); /*rtn next e if mine,else NULL  */
 #endif
 
-#endif
+static void bootp_add_id(ulong id)
+{
+       if (bootp_num_ids >= ARRAY_SIZE(bootp_ids)) {
+               size_t size = sizeof(bootp_ids) - sizeof(id);
+
+               memmove(bootp_ids, &bootp_ids[1], size);
+               bootp_ids[bootp_num_ids - 1] = id;
+       } else {
+               bootp_ids[bootp_num_ids] = id;
+               bootp_num_ids++;
+       }
+}
+
+static bool bootp_match_id(ulong id)
+{
+       unsigned int i;
+
+       for (i = 0; i < bootp_num_ids; i++)
+               if (bootp_ids[i] == id)
+                       return true;
+
+       return false;
+}
 
 static int BootpCheckPkt(uchar *pkt, unsigned dest, unsigned src, unsigned len)
 {
-       Bootp_t *bp = (Bootp_t *) pkt;
+       struct Bootp_t *bp = (struct Bootp_t *) pkt;
        int retval = 0;
 
        if (dest != PORT_BOOTPC || src != PORT_BOOTPS)
                retval = -1;
-       else if (len < sizeof (Bootp_t) - OPT_SIZE)
+       else if (len < sizeof(struct Bootp_t) - OPT_FIELD_SIZE)
                retval = -2;
        else if (bp->bp_op != OP_BOOTREQUEST &&
-           bp->bp_op != OP_BOOTREPLY &&
-           bp->bp_op != DHCP_OFFER &&
-           bp->bp_op != DHCP_ACK &&
-           bp->bp_op != DHCP_NAK ) {
+                       bp->bp_op != OP_BOOTREPLY &&
+                       bp->bp_op != DHCP_OFFER &&
+                       bp->bp_op != DHCP_ACK &&
+                       bp->bp_op != DHCP_NAK)
                retval = -3;
-       }
        else if (bp->bp_htype != HWT_ETHER)
                retval = -4;
        else if (bp->bp_hlen != HWL_ETHER)
                retval = -5;
-       else if (NetReadLong((ulong*)&bp->bp_id) != BootpID) {
+       else if (!bootp_match_id(NetReadLong((ulong *)&bp->bp_id)))
                retval = -6;
-       }
 
-       debug ("Filtering pkt = %d\n", retval);
+       debug("Filtering pkt = %d\n", retval);
 
        return retval;
 }
@@ -115,117 +136,115 @@ static int BootpCheckPkt(uchar *pkt, unsigned dest, unsigned src, unsigned len)
 /*
  * Copy parameters of interest from BOOTP_REPLY/DHCP_OFFER packet
  */
-static void BootpCopyNetParams(Bootp_t *bp)
+static void BootpCopyNetParams(struct Bootp_t *bp)
 {
+#if !defined(CONFIG_BOOTP_SERVERIP)
        IPaddr_t tmp_ip;
 
-       NetCopyIP(&NetOurIP, &bp->bp_yiaddr);
-#if !defined(CONFIG_BOOTP_SERVERIP)
        NetCopyIP(&tmp_ip, &bp->bp_siaddr);
        if (tmp_ip != 0)
                NetCopyIP(&NetServerIP, &bp->bp_siaddr);
-       memcpy (NetServerEther, ((Ethernet_t *)NetRxPkt)->et_src, 6);
+       memcpy(NetServerEther, ((struct ethernet_hdr *)NetRxPacket)->et_src, 6);
 #endif
+       NetCopyIP(&NetOurIP, &bp->bp_yiaddr);
        if (strlen(bp->bp_file) > 0)
-               copy_filename (BootFile, bp->bp_file, sizeof(BootFile));
+               copy_filename(BootFile, bp->bp_file, sizeof(BootFile));
 
-       debug ("Bootfile: %s\n", BootFile);
+       debug("Bootfile: %s\n", BootFile);
 
        /* Propagate to environment:
         * don't delete exising entry when BOOTP / DHCP reply does
         * not contain a new value
         */
-       if (*BootFile) {
-               setenv ("bootfile", BootFile);
-       }
+       if (*BootFile)
+               setenv("bootfile", BootFile);
 }
 
-static int truncate_sz (const char *name, int maxlen, int curlen)
+static int truncate_sz(const char *name, int maxlen, int curlen)
 {
        if (curlen >= maxlen) {
-               printf("*** WARNING: %s is too long (%d - max: %d) - truncated\n",
-                       name, curlen, maxlen);
+               printf("*** WARNING: %s is too long (%d - max: %d)"
+                       " - truncated\n", name, curlen, maxlen);
                curlen = maxlen - 1;
        }
-       return (curlen);
+       return curlen;
 }
 
 #if !defined(CONFIG_CMD_DHCP)
 
-static void BootpVendorFieldProcess (u8 * ext)
+static void BootpVendorFieldProcess(u8 *ext)
 {
        int size = *(ext + 1);
 
-       debug_ext ("[BOOTP] Processing extension %d... (%d bytes)\n", *ext,
-                  *(ext + 1));
+       debug("[BOOTP] Processing extension %d... (%d bytes)\n", *ext,
+               *(ext + 1));
 
        NetBootFileSize = 0;
 
        switch (*ext) {
                /* Fixed length fields */
-       case 1:                 /* Subnet mask                                  */
+       case 1:                 /* Subnet mask */
                if (NetOurSubnetMask == 0)
-                       NetCopyIP (&NetOurSubnetMask, (IPaddr_t *) (ext + 2));
+                       NetCopyIP(&NetOurSubnetMask, (IPaddr_t *) (ext + 2));
                break;
-       case 2:                 /* Time offset - Not yet supported              */
+       case 2:                 /* Time offset - Not yet supported */
                break;
                /* Variable length fields */
-       case 3:                 /* Gateways list                                */
-               if (NetOurGatewayIP == 0) {
-                       NetCopyIP (&NetOurGatewayIP, (IPaddr_t *) (ext + 2));
-               }
+       case 3:                 /* Gateways list */
+               if (NetOurGatewayIP == 0)
+                       NetCopyIP(&NetOurGatewayIP, (IPaddr_t *) (ext + 2));
                break;
-       case 4:                 /* Time server - Not yet supported              */
+       case 4:                 /* Time server - Not yet supported */
                break;
-       case 5:                 /* IEN-116 name server - Not yet supported      */
+       case 5:                 /* IEN-116 name server - Not yet supported */
                break;
        case 6:
-               if (NetOurDNSIP == 0) {
-                       NetCopyIP (&NetOurDNSIP, (IPaddr_t *) (ext + 2));
-               }
+               if (NetOurDNSIP == 0)
+                       NetCopyIP(&NetOurDNSIP, (IPaddr_t *) (ext + 2));
 #if defined(CONFIG_BOOTP_DNS2)
-               if ((NetOurDNS2IP == 0) && (size > 4)) {
-                       NetCopyIP (&NetOurDNS2IP, (IPaddr_t *) (ext + 2 + 4));
-               }
+               if ((NetOurDNS2IP == 0) && (size > 4))
+                       NetCopyIP(&NetOurDNS2IP, (IPaddr_t *) (ext + 2 + 4));
 #endif
                break;
-       case 7:                 /* Log server - Not yet supported               */
+       case 7:                 /* Log server - Not yet supported */
                break;
-       case 8:                 /* Cookie/Quote server - Not yet supported      */
+       case 8:                 /* Cookie/Quote server - Not yet supported */
                break;
-       case 9:                 /* LPR server - Not yet supported               */
+       case 9:                 /* LPR server - Not yet supported */
                break;
-       case 10:                /* Impress server - Not yet supported           */
+       case 10:                /* Impress server - Not yet supported */
                break;
-       case 11:                /* RPL server - Not yet supported               */
+       case 11:                /* RPL server - Not yet supported */
                break;
-       case 12:                /* Host name                                    */
+       case 12:                /* Host name */
                if (NetOurHostName[0] == 0) {
-                       size = truncate_sz ("Host Name", sizeof (NetOurHostName), size);
-                       memcpy (&NetOurHostName, ext + 2, size);
+                       size = truncate_sz("Host Name",
+                               sizeof(NetOurHostName), size);
+                       memcpy(&NetOurHostName, ext + 2, size);
                        NetOurHostName[size] = 0;
                }
                break;
-       case 13:                /* Boot file size                               */
+       case 13:                /* Boot file size */
                if (size == 2)
-                       NetBootFileSize = ntohs (*(ushort *) (ext + 2));
+                       NetBootFileSize = ntohs(*(ushort *) (ext + 2));
                else if (size == 4)
-                       NetBootFileSize = ntohl (*(ulong *) (ext + 2));
+                       NetBootFileSize = ntohl(*(ulong *) (ext + 2));
                break;
-       case 14:                /* Merit dump file - Not yet supported          */
+       case 14:                /* Merit dump file - Not yet supported */
                break;
-       case 15:                /* Domain name - Not yet supported              */
+       case 15:                /* Domain name - Not yet supported */
                break;
-       case 16:                /* Swap server - Not yet supported              */
+       case 16:                /* Swap server - Not yet supported */
                break;
-       case 17:                /* Root path                                    */
+       case 17:                /* Root path */
                if (NetOurRootPath[0] == 0) {
-                       size = truncate_sz ("Root Path", sizeof (NetOurRootPath), size);
-                       memcpy (&NetOurRootPath, ext + 2, size);
+                       size = truncate_sz("Root Path",
+                               sizeof(NetOurRootPath), size);
+                       memcpy(&NetOurRootPath, ext + 2, size);
                        NetOurRootPath[size] = 0;
                }
                break;
-       case 18:                /* Extension path - Not yet supported           */
+       case 18:                /* Extension path - Not yet supported */
                /*
                 * This can be used to send the information of the
                 * vendor area in another file that the client can
@@ -233,15 +252,21 @@ static void BootpVendorFieldProcess (u8 * ext)
                 */
                break;
                /* IP host layer fields */
-       case 40:                /* NIS Domain name                              */
+       case 40:                /* NIS Domain name */
                if (NetOurNISDomain[0] == 0) {
-                       size = truncate_sz ("NIS Domain Name", sizeof (NetOurNISDomain), size);
-                       memcpy (&NetOurNISDomain, ext + 2, size);
+                       size = truncate_sz("NIS Domain Name",
+                               sizeof(NetOurNISDomain), size);
+                       memcpy(&NetOurNISDomain, ext + 2, size);
                        NetOurNISDomain[size] = 0;
                }
                break;
+#if defined(CONFIG_CMD_SNTP) && defined(CONFIG_BOOTP_NTPSERVER)
+       case 42:        /* NTP server IP */
+               NetCopyIP(&NetNtpServerIP, (IPaddr_t *) (ext + 2));
+               break;
+#endif
                /* Application layer fields */
-       case 43:                /* Vendor specific info - Not yet supported     */
+       case 43:                /* Vendor specific info - Not yet supported */
                /*
                 * Binary information to exchange specific
                 * product information.
@@ -251,11 +276,11 @@ static void BootpVendorFieldProcess (u8 * ext)
        }
 }
 
-static void BootpVendorProcess (u8 * ext, int size)
+static void BootpVendorProcess(u8 *ext, int size)
 {
        u8 *end = ext + size;
 
-       debug_ext ("[BOOTP] Checking extension (%d bytes)...\n", size);
+       debug("[BOOTP] Checking extension (%d bytes)...\n", size);
 
        while ((ext < end) && (*ext != 0xff)) {
                if (*ext == 0) {
@@ -265,93 +290,75 @@ static void BootpVendorProcess (u8 * ext, int size)
 
                        ext += ext[1] + 2;
                        if (ext <= end)
-                               BootpVendorFieldProcess (opt);
+                               BootpVendorFieldProcess(opt);
                }
        }
 
-#ifdef DEBUG_BOOTP_EXT
-       puts ("[BOOTP] Received fields: \n");
+       debug("[BOOTP] Received fields:\n");
        if (NetOurSubnetMask)
-               printf ("NetOurSubnetMask : %pI4\n", &NetOurSubnetMask);
+               debug("NetOurSubnetMask : %pI4\n", &NetOurSubnetMask);
 
        if (NetOurGatewayIP)
-               printf ("NetOurGatewayIP        : %pI4", &NetOurGatewayIP);
+               debug("NetOurGatewayIP  : %pI4", &NetOurGatewayIP);
 
-       if (NetBootFileSize) {
-               printf ("NetBootFileSize : %d\n", NetBootFileSize);
-       }
+       if (NetBootFileSize)
+               debug("NetBootFileSize : %d\n", NetBootFileSize);
 
-       if (NetOurHostName[0]) {
-               printf ("NetOurHostName  : %s\n", NetOurHostName);
-       }
+       if (NetOurHostName[0])
+               debug("NetOurHostName  : %s\n", NetOurHostName);
 
-       if (NetOurRootPath[0]) {
-               printf ("NetOurRootPath  : %s\n", NetOurRootPath);
-       }
+       if (NetOurRootPath[0])
+               debug("NetOurRootPath  : %s\n", NetOurRootPath);
 
-       if (NetOurNISDomain[0]) {
-               printf ("NetOurNISDomain : %s\n", NetOurNISDomain);
-       }
+       if (NetOurNISDomain[0])
+               debug("NetOurNISDomain : %s\n", NetOurNISDomain);
 
-       if (NetBootFileSize) {
-               printf ("NetBootFileSize: %d\n", NetBootFileSize);
-       }
-#endif /* DEBUG_BOOTP_EXT */
+       if (NetBootFileSize)
+               debug("NetBootFileSize: %d\n", NetBootFileSize);
+
+#if defined(CONFIG_CMD_SNTP) && defined(CONFIG_BOOTP_NTPSERVER)
+       if (NetNtpServerIP)
+               debug("NetNtpServerIP : %pI4\n", &NetNtpServerIP);
+#endif
 }
+
 /*
  *     Handle a BOOTP received packet.
  */
 static void
-BootpHandler(uchar * pkt, unsigned dest, unsigned src, unsigned len)
+BootpHandler(uchar *pkt, unsigned dest, IPaddr_t sip, unsigned src,
+            unsigned len)
 {
-       Bootp_t *bp;
-       char    *s;
+       struct Bootp_t *bp;
 
-       debug ("got BOOTP packet (src=%d, dst=%d, len=%d want_len=%zu)\n",
-               src, dest, len, sizeof (Bootp_t));
+       debug("got BOOTP packet (src=%d, dst=%d, len=%d want_len=%zu)\n",
+               src, dest, len, sizeof(struct Bootp_t));
 
-       bp = (Bootp_t *)pkt;
+       bp = (struct Bootp_t *)pkt;
 
-       if (BootpCheckPkt(pkt, dest, src, len)) /* Filter out pkts we don't want */
+       /* Filter out pkts we don't want */
+       if (BootpCheckPkt(pkt, dest, src, len))
                return;
 
        /*
         *      Got a good BOOTP reply.  Copy the data into our variables.
         */
 #ifdef CONFIG_STATUS_LED
-       status_led_set (STATUS_LED_BOOT, STATUS_LED_OFF);
+       status_led_set(STATUS_LED_BOOT, STATUS_LED_OFF);
 #endif
 
        BootpCopyNetParams(bp);         /* Store net parameters from reply */
 
        /* Retrieve extended information (we must parse the vendor area) */
-       if (NetReadLong((ulong*)&bp->bp_vend[0]) == htonl(BOOTP_VENDOR_MAGIC))
+       if (NetReadLong((ulong *)&bp->bp_vend[0]) == htonl(BOOTP_VENDOR_MAGIC))
                BootpVendorProcess((uchar *)&bp->bp_vend[4], len);
 
        NetSetTimeout(0, (thand_f *)0);
+       bootstage_mark_name(BOOTSTAGE_ID_BOOTP_STOP, "bootp_stop");
 
-       debug ("Got good BOOTP\n");
-
-       if ((s = getenv("autoload")) != NULL) {
-               if (*s == 'n') {
-                       /*
-                        * Just use BOOTP to configure system;
-                        * Do not use TFTP to load the bootfile.
-                        */
-                       NetState = NETLOOP_SUCCESS;
-                       return;
-#if defined(CONFIG_CMD_NFS)
-               } else if (strcmp(s, "NFS") == 0) {
-                       /*
-                        * Use NFS to load the bootfile.
-                        */
-                       NfsStart();
-                       return;
-#endif
-               }
-       }
+       debug("Got good BOOTP\n");
 
-       TftpStart();
+       net_auto_load();
 }
 #endif
 
@@ -361,23 +368,47 @@ BootpHandler(uchar * pkt, unsigned dest, unsigned src, unsigned len)
 static void
 BootpTimeout(void)
 {
-       if (BootpTry >= TIMEOUT_COUNT) {
-               puts ("\nRetry count exceeded; starting again\n");
-               NetStartAgain ();
+       ulong time_taken = get_timer(bootp_start);
+
+       if (time_taken >= TIMEOUT_MS) {
+#ifdef CONFIG_BOOTP_MAY_FAIL
+               puts("\nRetry time exceeded\n");
+               net_set_state(NETLOOP_FAIL);
+#else
+               puts("\nRetry time exceeded; starting again\n");
+               NetStartAgain();
+#endif
        } else {
-               NetSetTimeout (TIMEOUT, BootpTimeout);
-               BootpRequest ();
+               bootp_timeout *= 2;
+               if (bootp_timeout > 2000)
+                       bootp_timeout = 2000;
+               NetSetTimeout(bootp_timeout, BootpTimeout);
+               BootpRequest();
        }
 }
 
+#define put_vci(e, str)                                                \
+       do {                                                    \
+               size_t vci_strlen = strlen(str);                \
+               *e++ = 60;      /* Vendor Class Identifier */   \
+               *e++ = vci_strlen;                              \
+               memcpy(e, str, vci_strlen);                     \
+               e += vci_strlen;                                \
+       } while (0)
+
 /*
  *     Initialize BOOTP extension fields in the request.
  */
 #if defined(CONFIG_CMD_DHCP)
-static int DhcpExtended (u8 * e, int message_type, IPaddr_t ServerID, IPaddr_t RequestedIP)
+static int DhcpExtended(u8 *e, int message_type, IPaddr_t ServerID,
+                       IPaddr_t RequestedIP)
 {
        u8 *start = e;
        u8 *cnt;
+#if defined(CONFIG_BOOTP_PXE)
+       char *uuid;
+       u16 clientarch;
+#endif
 
 #if defined(CONFIG_BOOTP_VENDOREX)
        u8 *x;
@@ -397,11 +428,11 @@ static int DhcpExtended (u8 * e, int message_type, IPaddr_t ServerID, IPaddr_t R
 
        *e++ = 57;              /* Maximum DHCP Message Size */
        *e++ = 2;
-       *e++ = (576 - 312 + OPT_SIZE) >> 8;
-       *e++ = (576 - 312 + OPT_SIZE) & 0xff;
+       *e++ = (576 - 312 + OPT_FIELD_SIZE) >> 8;
+       *e++ = (576 - 312 + OPT_FIELD_SIZE) & 0xff;
 
        if (ServerID) {
-               int tmp = ntohl (ServerID);
+               int tmp = ntohl(ServerID);
 
                *e++ = 54;      /* ServerID */
                *e++ = 4;
@@ -412,7 +443,7 @@ static int DhcpExtended (u8 * e, int message_type, IPaddr_t ServerID, IPaddr_t R
        }
 
        if (RequestedIP) {
-               int tmp = ntohl (RequestedIP);
+               int tmp = ntohl(RequestedIP);
 
                *e++ = 50;      /* Requested IP */
                *e++ = 4;
@@ -422,18 +453,53 @@ static int DhcpExtended (u8 * e, int message_type, IPaddr_t ServerID, IPaddr_t R
                *e++ = tmp & 0xff;
        }
 #if defined(CONFIG_BOOTP_SEND_HOSTNAME)
-       if ((hostname = getenv ("hostname"))) {
-               int hostnamelen = strlen (hostname);
+       hostname = getenv("hostname");
+       if (hostname) {
+               int hostnamelen = strlen(hostname);
 
                *e++ = 12;      /* Hostname */
                *e++ = hostnamelen;
-               memcpy (e, hostname, hostnamelen);
+               memcpy(e, hostname, hostnamelen);
                e += hostnamelen;
        }
 #endif
 
+#if defined(CONFIG_BOOTP_PXE)
+       clientarch = CONFIG_BOOTP_PXE_CLIENTARCH;
+       *e++ = 93;      /* Client System Architecture */
+       *e++ = 2;
+       *e++ = (clientarch >> 8) & 0xff;
+       *e++ = clientarch & 0xff;
+
+       *e++ = 94;      /* Client Network Interface Identifier */
+       *e++ = 3;
+       *e++ = 1;       /* type field for UNDI */
+       *e++ = 0;       /* major revision */
+       *e++ = 0;       /* minor revision */
+
+       uuid = getenv("pxeuuid");
+
+       if (uuid) {
+               if (uuid_str_valid(uuid)) {
+                       *e++ = 97;      /* Client Machine Identifier */
+                       *e++ = 17;
+                       *e++ = 0;       /* type 0 - UUID */
+
+                       uuid_str_to_bin(uuid, e, UUID_STR_FORMAT_STD);
+                       e += 16;
+               } else {
+                       printf("Invalid pxeuuid: %s\n", uuid);
+               }
+       }
+#endif
+
+#ifdef CONFIG_BOOTP_VCI_STRING
+       put_vci(e, CONFIG_BOOTP_VCI_STRING);
+#endif
+
 #if defined(CONFIG_BOOTP_VENDOREX)
-       if ((x = dhcp_vendorex_prep (e)))
+       x = dhcp_vendorex_prep(e);
+       if (x)
                return x - start;
 #endif
 
@@ -476,11 +542,15 @@ static int DhcpExtended (u8 * e, int message_type, IPaddr_t ServerID, IPaddr_t R
        *e++  = 42;
        *cnt += 1;
 #endif
+       /* no options, so back up to avoid sending an empty request list */
+       if (*cnt == 0)
+               e -= 2;
+
        *e++  = 255;            /* End of the list */
 
        /* Pad to minimal length */
 #ifdef CONFIG_DHCP_MIN_EXT_LEN
-       while ((e - start) <= CONFIG_DHCP_MIN_EXT_LEN)
+       while ((e - start) < CONFIG_DHCP_MIN_EXT_LEN)
                *e++ = 0;
 #endif
 
@@ -489,9 +559,9 @@ static int DhcpExtended (u8 * e, int message_type, IPaddr_t ServerID, IPaddr_t R
 
 #else
 /*
- *     Warning: no field size check - change CONFIG_BOOTP_* at your own risk!
+ * Warning: no field size check - change CONFIG_BOOTP_* at your own risk!
  */
-static int BootpExtended (u8 * e)
+static int BootpExtended(u8 *e)
 {
        u8 *start = e;
 
@@ -507,8 +577,17 @@ static int BootpExtended (u8 * e)
 
        *e++ = 57;              /* Maximum DHCP Message Size */
        *e++ = 2;
-       *e++ = (576 - 312 + OPT_SIZE) >> 16;
-       *e++ = (576 - 312 + OPT_SIZE) & 0xff;
+       *e++ = (576 - 312 + OPT_FIELD_SIZE) >> 16;
+       *e++ = (576 - 312 + OPT_FIELD_SIZE) & 0xff;
+#endif
+
+#if defined(CONFIG_BOOTP_VCI_STRING) || \
+       (defined(CONFIG_SPL_BUILD) && defined(CONFIG_SPL_NET_VCI_STRING))
+#ifdef CONFIG_SPL_BUILD
+       put_vci(e, CONFIG_SPL_NET_VCI_STRING);
+#else
+       put_vci(e, CONFIG_BOOTP_VCI_STRING);
+#endif
 #endif
 
 #if defined(CONFIG_BOOTP_SUBNETMASK)
@@ -552,6 +631,11 @@ static int BootpExtended (u8 * e)
        *e++ = 32;
        e   += 32;
 #endif
+#if defined(CONFIG_BOOTP_NTPSERVER)
+       *e++ = 42;
+       *e++ = 4;
+       e   += 4;
+#endif
 
        *e++ = 255;             /* End of the list */
 
@@ -559,99 +643,65 @@ static int BootpExtended (u8 * e)
 }
 #endif
 
+void BootpReset(void)
+{
+       bootp_num_ids = 0;
+       BootpTry = 0;
+       bootp_start = get_timer(0);
+       bootp_timeout = 250;
+}
+
 void
-BootpRequest (void)
+BootpRequest(void)
 {
-       volatile uchar *pkt, *iphdr;
-       Bootp_t *bp;
-       int ext_len, pktlen, iplen;
+       uchar *pkt, *iphdr;
+       struct Bootp_t *bp;
+       int extlen, pktlen, iplen;
+       int eth_hdr_size;
+#ifdef CONFIG_BOOTP_RANDOM_DELAY
+       ulong rand_ms;
+#endif
+       ulong BootpID;
 
+       bootstage_mark_name(BOOTSTAGE_ID_BOOTP_START, "bootp_start");
 #if defined(CONFIG_CMD_DHCP)
        dhcp_state = INIT;
 #endif
 
 #ifdef CONFIG_BOOTP_RANDOM_DELAY               /* Random BOOTP delay */
-       unsigned char bi_enetaddr[6];
-       int   reg;
-       ulong tst1, tst2, sum, m_mask, m_value = 0;
-
-       if (BootpTry ==0) {
-               /* get our mac */
-               eth_getenv_enetaddr("ethaddr", bi_enetaddr);
-
-#ifdef DEBUG
-               puts ("BootpRequest => Our Mac: ");
-               for (reg=0; reg<6; reg++) {
-                       printf ("%x%c",
-                               bi_enetaddr[reg],
-                               reg==5 ? '\n' : ':');
-               }
-#endif /* DEBUG */
-
-               /* Mac-Manipulation 2 get seed1 */
-               tst1=0;
-               tst2=0;
-               for (reg=2; reg<6; reg++) {
-                       tst1 = tst1 << 8;
-                       tst1 = tst1 | bi_enetaddr[reg];
-               }
-               for (reg=0; reg<2; reg++) {
-                       tst2 = tst2 | bi_enetaddr[reg];
-                       tst2 = tst2 << 8;
-               }
-
-               seed1 = tst1^tst2;
-
-               /* Mirror seed1*/
-               m_mask=0x1;
-               for (reg=1;reg<=32;reg++) {
-                       m_value |= (m_mask & seed1);
-                       seed1 = seed1 >> 1;
-                       m_value = m_value << 1;
-               }
-               seed1 = m_value;
-               seed2 = 0xB78D0945;
-       }
-
-       /* Random Number Generator */
+       if (BootpTry == 0)
+               srand_mac();
 
-       for (reg=0;reg<=0;reg++) {
-               sum = seed1 + seed2;
-               if (sum < seed1 || sum < seed2)
-                       sum++;
-               seed2 = seed1;
-               seed1 = sum;
+       if (BootpTry <= 2)      /* Start with max 1024 * 1ms */
+               rand_ms = rand() >> (22 - BootpTry);
+       else            /* After 3rd BOOTP request max 8192 * 1ms */
+               rand_ms = rand() >> 19;
 
-               if (BootpTry<=2) {      /* Start with max 1024 * 1ms */
-                       sum = sum >> (22-BootpTry);
-               } else {                /*After 3rd BOOTP request max 8192 * 1ms */
-                       sum = sum >> 19;
-               }
-       }
+       printf("Random delay: %ld ms...\n", rand_ms);
+       mdelay(rand_ms);
 
-       printf ("Random delay: %ld ms...\n", sum);
-       for (reg=0; reg <sum; reg++) {
-               udelay(1000); /*Wait 1ms*/
-       }
 #endif /* CONFIG_BOOTP_RANDOM_DELAY */
 
        printf("BOOTP broadcast %d\n", ++BootpTry);
        pkt = NetTxPacket;
-       memset ((void*)pkt, 0, PKTSIZE);
+       memset((void *)pkt, 0, PKTSIZE);
 
-       pkt += NetSetEther(pkt, NetBcastAddr, PROT_IP);
+       eth_hdr_size = NetSetEther(pkt, NetBcastAddr, PROT_IP);
+       pkt += eth_hdr_size;
 
        /*
-        * Next line results in incorrect packet size being transmitted, resulting
-        * in errors in some DHCP servers, reporting missing bytes.  Size must be
-        * set in packet header after extension length has been determined.
+        * Next line results in incorrect packet size being transmitted,
+        * resulting in errors in some DHCP servers, reporting missing bytes.
+        * Size must be set in packet header after extension length has been
+        * determined.
         * C. Hallinan, DS4.COM, Inc.
         */
-       /* NetSetIP(pkt, 0xFFFFFFFFL, PORT_BOOTPS, PORT_BOOTPC, sizeof (Bootp_t)); */
-       iphdr = pkt;    /* We need this later for NetSetIP() */
-       pkt += IP_HDR_SIZE;
+       /* net_set_udp_header(pkt, 0xFFFFFFFFL, PORT_BOOTPS, PORT_BOOTPC,
+               sizeof (struct Bootp_t)); */
+       iphdr = pkt;    /* We need this later for net_set_udp_header() */
+       pkt += IP_UDP_HDR_SIZE;
 
-       bp = (Bootp_t *)pkt;
+       bp = (struct Bootp_t *)pkt;
        bp->bp_op = OP_BOOTREQUEST;
        bp->bp_htype = HWT_ETHER;
        bp->bp_hlen = HWL_ETHER;
@@ -661,14 +711,14 @@ BootpRequest (void)
        NetWriteIP(&bp->bp_yiaddr, 0);
        NetWriteIP(&bp->bp_siaddr, 0);
        NetWriteIP(&bp->bp_giaddr, 0);
-       memcpy (bp->bp_chaddr, NetOurEther, 6);
-       copy_filename (bp->bp_file, BootFile, sizeof(bp->bp_file));
+       memcpy(bp->bp_chaddr, NetOurEther, 6);
+       copy_filename(bp->bp_file, BootFile, sizeof(bp->bp_file));
 
        /* Request additional information from the BOOTP/DHCP server */
 #if defined(CONFIG_CMD_DHCP)
-       ext_len = DhcpExtended((u8 *)bp->bp_vend, DHCP_DISCOVER, 0, 0);
+       extlen = DhcpExtended((u8 *)bp->bp_vend, DHCP_DISCOVER, 0, 0);
 #else
-       ext_len = BootpExtended((u8 *)bp->bp_vend);
+       extlen = BootpExtended((u8 *)bp->bp_vend);
 #endif
 
        /*
@@ -680,80 +730,88 @@ BootpRequest (void)
                | ((ulong)NetOurEther[4] << 8)
                | (ulong)NetOurEther[5];
        BootpID += get_timer(0);
-       BootpID  = htonl(BootpID);
+       BootpID = htonl(BootpID);
+       bootp_add_id(BootpID);
        NetCopyLong(&bp->bp_id, &BootpID);
 
        /*
         * Calculate proper packet lengths taking into account the
         * variable size of the options field
         */
-       pktlen = ((int)(pkt-NetTxPacket)) + BOOTP_HDR_SIZE - sizeof(bp->bp_vend) + ext_len;
-       iplen = BOOTP_HDR_SIZE - sizeof(bp->bp_vend) + ext_len;
-       NetSetIP(iphdr, 0xFFFFFFFFL, PORT_BOOTPS, PORT_BOOTPC, iplen);
-       NetSetTimeout(SELECT_TIMEOUT, BootpTimeout);
+       iplen = BOOTP_HDR_SIZE - OPT_FIELD_SIZE + extlen;
+       pktlen = eth_hdr_size + IP_UDP_HDR_SIZE + iplen;
+       net_set_udp_header(iphdr, 0xFFFFFFFFL, PORT_BOOTPS, PORT_BOOTPC, iplen);
+       NetSetTimeout(bootp_timeout, BootpTimeout);
 
 #if defined(CONFIG_CMD_DHCP)
        dhcp_state = SELECTING;
-       NetSetHandler(DhcpHandler);
+       net_set_udp_handler(DhcpHandler);
 #else
-       NetSetHandler(BootpHandler);
+       net_set_udp_handler(BootpHandler);
 #endif
        NetSendPacket(NetTxPacket, pktlen);
 }
 
 #if defined(CONFIG_CMD_DHCP)
-static void DhcpOptionsProcess (uchar * popt, Bootp_t *bp)
+static void DhcpOptionsProcess(uchar *popt, struct Bootp_t *bp)
 {
        uchar *end = popt + BOOTP_HDR_SIZE;
        int oplen, size;
+#if defined(CONFIG_CMD_SNTP) && defined(CONFIG_BOOTP_TIMEOFFSET)
+       int *to_ptr;
+#endif
 
        while (popt < end && *popt != 0xff) {
                oplen = *(popt + 1);
                switch (*popt) {
                case 1:
-                       NetCopyIP (&NetOurSubnetMask, (popt + 2));
+                       NetCopyIP(&NetOurSubnetMask, (popt + 2));
                        break;
 #if defined(CONFIG_CMD_SNTP) && defined(CONFIG_BOOTP_TIMEOFFSET)
                case 2:         /* Time offset  */
-                       NetCopyLong ((ulong *)&NetTimeOffset, (ulong *) (popt + 2));
-                       NetTimeOffset = ntohl (NetTimeOffset);
+                       to_ptr = &NetTimeOffset;
+                       NetCopyLong((ulong *)to_ptr, (ulong *)(popt + 2));
+                       NetTimeOffset = ntohl(NetTimeOffset);
                        break;
 #endif
                case 3:
-                       NetCopyIP (&NetOurGatewayIP, (popt + 2));
+                       NetCopyIP(&NetOurGatewayIP, (popt + 2));
                        break;
                case 6:
-                       NetCopyIP (&NetOurDNSIP, (popt + 2));
+                       NetCopyIP(&NetOurDNSIP, (popt + 2));
 #if defined(CONFIG_BOOTP_DNS2)
-                       if (*(popt + 1) > 4) {
-                               NetCopyIP (&NetOurDNS2IP, (popt + 2 + 4));
-                       }
+                       if (*(popt + 1) > 4)
+                               NetCopyIP(&NetOurDNS2IP, (popt + 2 + 4));
 #endif
                        break;
                case 12:
-                       size = truncate_sz ("Host Name", sizeof (NetOurHostName), oplen);
-                       memcpy (&NetOurHostName, popt + 2, size);
+                       size = truncate_sz("Host Name",
+                               sizeof(NetOurHostName), oplen);
+                       memcpy(&NetOurHostName, popt + 2, size);
                        NetOurHostName[size] = 0;
                        break;
                case 15:        /* Ignore Domain Name Option */
                        break;
                case 17:
-                       size = truncate_sz ("Root Path", sizeof (NetOurRootPath), oplen);
-                       memcpy (&NetOurRootPath, popt + 2, size);
+                       size = truncate_sz("Root Path",
+                               sizeof(NetOurRootPath), oplen);
+                       memcpy(&NetOurRootPath, popt + 2, size);
                        NetOurRootPath[size] = 0;
                        break;
+               case 28:        /* Ignore Broadcast Address Option */
+                       break;
 #if defined(CONFIG_CMD_SNTP) && defined(CONFIG_BOOTP_NTPSERVER)
                case 42:        /* NTP server IP */
-                       NetCopyIP (&NetNtpServerIP, (popt + 2));
+                       NetCopyIP(&NetNtpServerIP, (popt + 2));
                        break;
 #endif
                case 51:
-                       NetCopyLong (&dhcp_leasetime, (ulong *) (popt + 2));
+                       NetCopyLong(&dhcp_leasetime, (ulong *) (popt + 2));
                        break;
                case 53:        /* Ignore Message Type Option */
                        break;
                case 54:
-                       NetCopyIP (&NetDHCPServerIP, (popt + 2));
+                       NetCopyIP(&NetDHCPServerIP, (popt + 2));
                        break;
                case 58:        /* Ignore Renewal Time Option */
                        break;
@@ -768,7 +826,7 @@ static void DhcpOptionsProcess (uchar * popt, Bootp_t *bp)
                         * pass the bootp packet pointer into here as the
                         * second arg
                         */
-                       size = truncate_sz ("Opt Boot File",
+                       size = truncate_sz("Opt Boot File",
                                            sizeof(bp->bp_file),
                                            oplen);
                        if (bp->bp_file[0] == '\0' && size > 0) {
@@ -789,10 +847,11 @@ static void DhcpOptionsProcess (uchar * popt, Bootp_t *bp)
                        break;
                default:
 #if defined(CONFIG_BOOTP_VENDOREX)
-                       if (dhcp_vendorex_proc (popt))
+                       if (dhcp_vendorex_proc(popt))
                                break;
 #endif
-                       printf ("*** Unhandled DHCP Option in OFFER/ACK: %d\n", *popt);
+                       printf("*** Unhandled DHCP Option in OFFER/ACK:"
+                               " %d\n", *popt);
                        break;
                }
                popt += oplen + 2;      /* Process next option */
@@ -801,42 +860,44 @@ static void DhcpOptionsProcess (uchar * popt, Bootp_t *bp)
 
 static int DhcpMessageType(unsigned char *popt)
 {
-       if (NetReadLong((ulong*)popt) != htonl(BOOTP_VENDOR_MAGIC))
+       if (NetReadLong((ulong *)popt) != htonl(BOOTP_VENDOR_MAGIC))
                return -1;
 
        popt += 4;
-       while ( *popt != 0xff ) {
-               if ( *popt == 53 )      /* DHCP Message Type */
+       while (*popt != 0xff) {
+               if (*popt == 53)        /* DHCP Message Type */
                        return *(popt + 2);
                popt += *(popt + 1) + 2;        /* Scan through all options */
        }
        return -1;
 }
 
-static void DhcpSendRequestPkt(Bootp_t *bp_offer)
+static void DhcpSendRequestPkt(struct Bootp_t *bp_offer)
 {
-       volatile uchar *pkt, *iphdr;
-       Bootp_t *bp;
+       uchar *pkt, *iphdr;
+       struct Bootp_t *bp;
        int pktlen, iplen, extlen;
+       int eth_hdr_size;
        IPaddr_t OfferedIP;
 
-       debug ("DhcpSendRequestPkt: Sending DHCPREQUEST\n");
+       debug("DhcpSendRequestPkt: Sending DHCPREQUEST\n");
        pkt = NetTxPacket;
-       memset ((void*)pkt, 0, PKTSIZE);
+       memset((void *)pkt, 0, PKTSIZE);
 
-       pkt += NetSetEther(pkt, NetBcastAddr, PROT_IP);
+       eth_hdr_size = NetSetEther(pkt, NetBcastAddr, PROT_IP);
+       pkt += eth_hdr_size;
 
-       iphdr = pkt;            /* We'll need this later to set proper pkt size */
-       pkt += IP_HDR_SIZE;
+       iphdr = pkt;    /* We'll need this later to set proper pkt size */
+       pkt += IP_UDP_HDR_SIZE;
 
-       bp = (Bootp_t *)pkt;
+       bp = (struct Bootp_t *)pkt;
        bp->bp_op = OP_BOOTREQUEST;
        bp->bp_htype = HWT_ETHER;
        bp->bp_hlen = HWL_ETHER;
        bp->bp_hops = 0;
        bp->bp_secs = htons(get_timer(0) / 1000);
-       /* Do not set the client IP, your IP, or server IP yet, since it hasn't been ACK'ed by
-        * the server yet */
+       /* Do not set the client IP, your IP, or server IP yet, since it
+        * hasn't been ACK'ed by the server yet */
 
        /*
         * RFC3046 requires Relay Agents to discard packets with
@@ -844,7 +905,7 @@ static void DhcpSendRequestPkt(Bootp_t *bp_offer)
         */
        NetWriteIP(&bp->bp_giaddr, 0);
 
-       memcpy (bp->bp_chaddr, NetOurEther, 6);
+       memcpy(bp->bp_chaddr, NetOurEther, 6);
 
        /*
         * ID is the id of the OFFER packet
@@ -858,16 +919,17 @@ static void DhcpSendRequestPkt(Bootp_t *bp_offer)
 
        /* Copy offered IP into the parameters request list */
        NetCopyIP(&OfferedIP, &bp_offer->bp_yiaddr);
-       extlen = DhcpExtended((u8 *)bp->bp_vend, DHCP_REQUEST, NetDHCPServerIP, OfferedIP);
+       extlen = DhcpExtended((u8 *)bp->bp_vend, DHCP_REQUEST,
+               NetDHCPServerIP, OfferedIP);
 
-       pktlen = ((int)(pkt-NetTxPacket)) + BOOTP_HDR_SIZE - sizeof(bp->bp_vend) + extlen;
-       iplen = BOOTP_HDR_SIZE - sizeof(bp->bp_vend) + extlen;
-       NetSetIP(iphdr, 0xFFFFFFFFL, PORT_BOOTPS, PORT_BOOTPC, iplen);
+       iplen = BOOTP_HDR_SIZE - OPT_FIELD_SIZE + extlen;
+       pktlen = eth_hdr_size + IP_UDP_HDR_SIZE + iplen;
+       net_set_udp_header(iphdr, 0xFFFFFFFFL, PORT_BOOTPS, PORT_BOOTPC, iplen);
 
-       debug ("Transmitting DHCPREQUEST packet: len = %d\n", pktlen);
 #ifdef CONFIG_BOOTP_DHCP_REQUEST_DELAY
        udelay(CONFIG_BOOTP_DHCP_REQUEST_DELAY);
 #endif /* CONFIG_BOOTP_DHCP_REQUEST_DELAY */
+       debug("Transmitting DHCPREQUEST packet: len = %d\n", pktlen);
        NetSendPacket(NetTxPacket, pktlen);
 }
 
@@ -875,41 +937,44 @@ static void DhcpSendRequestPkt(Bootp_t *bp_offer)
  *     Handle DHCP received packets.
  */
 static void
-DhcpHandler(uchar * pkt, unsigned dest, unsigned src, unsigned len)
+DhcpHandler(uchar *pkt, unsigned dest, IPaddr_t sip, unsigned src,
+           unsigned len)
 {
-       Bootp_t *bp = (Bootp_t *)pkt;
+       struct Bootp_t *bp = (struct Bootp_t *)pkt;
 
-       debug ("DHCPHandler: got packet: (src=%d, dst=%d, len=%d) state: %d\n",
+       debug("DHCPHandler: got packet: (src=%d, dst=%d, len=%d) state: %d\n",
                src, dest, len, dhcp_state);
 
-       if (BootpCheckPkt(pkt, dest, src, len)) /* Filter out pkts we don't want */
+       /* Filter out pkts we don't want */
+       if (BootpCheckPkt(pkt, dest, src, len))
                return;
 
-       debug ("DHCPHandler: got DHCP packet: (src=%d, dst=%d, len=%d) state: %d\n",
-               src, dest, len, dhcp_state);
+       debug("DHCPHandler: got DHCP packet: (src=%d, dst=%d, len=%d) state:"
+               " %d\n", src, dest, len, dhcp_state);
 
        switch (dhcp_state) {
        case SELECTING:
                /*
                 * Wait an appropriate time for any potential DHCPOFFER packets
-                * to arrive.  Then select one, and generate DHCPREQUEST response.
-                * If filename is in format we recognize, assume it is a valid
-                * OFFER from a server we want.
+                * to arrive.  Then select one, and generate DHCPREQUEST
+                * response.  If filename is in format we recognize, assume it
+                * is a valid OFFER from a server we want.
                 */
-               debug ("DHCP: state=SELECTING bp_file: \"%s\"\n", bp->bp_file);
+               debug("DHCP: state=SELECTING bp_file: \"%s\"\n", bp->bp_file);
 #ifdef CONFIG_SYS_BOOTFILE_PREFIX
                if (strncmp(bp->bp_file,
                            CONFIG_SYS_BOOTFILE_PREFIX,
-                           strlen(CONFIG_SYS_BOOTFILE_PREFIX)) == 0 ) {
+                           strlen(CONFIG_SYS_BOOTFILE_PREFIX)) == 0) {
 #endif /* CONFIG_SYS_BOOTFILE_PREFIX */
 
-                       debug ("TRANSITIONING TO REQUESTING STATE\n");
+                       debug("TRANSITIONING TO REQUESTING STATE\n");
                        dhcp_state = REQUESTING;
 
-                       if (NetReadLong((ulong*)&bp->bp_vend[0]) == htonl(BOOTP_VENDOR_MAGIC))
+                       if (NetReadLong((ulong *)&bp->bp_vend[0]) ==
+                                               htonl(BOOTP_VENDOR_MAGIC))
                                DhcpOptionsProcess((u8 *)&bp->bp_vend[4], bp);
 
-                       NetSetTimeout(TIMEOUT, BootpTimeout);
+                       NetSetTimeout(5000, BootpTimeout);
                        DhcpSendRequestPkt(bp);
 #ifdef CONFIG_SYS_BOOTFILE_PREFIX
                }
@@ -918,37 +983,21 @@ DhcpHandler(uchar * pkt, unsigned dest, unsigned src, unsigned len)
                return;
                break;
        case REQUESTING:
-               debug ("DHCP State: REQUESTING\n");
-
-               if ( DhcpMessageType((u8 *)bp->bp_vend) == DHCP_ACK ) {
-                       char *s;
+               debug("DHCP State: REQUESTING\n");
 
-                       if (NetReadLong((ulong*)&bp->bp_vend[0]) == htonl(BOOTP_VENDOR_MAGIC))
+               if (DhcpMessageType((u8 *)bp->bp_vend) == DHCP_ACK) {
+                       if (NetReadLong((ulong *)&bp->bp_vend[0]) ==
+                                               htonl(BOOTP_VENDOR_MAGIC))
                                DhcpOptionsProcess((u8 *)&bp->bp_vend[4], bp);
-                       BootpCopyNetParams(bp); /* Store net params from reply */
+                       /* Store net params from reply */
+                       BootpCopyNetParams(bp);
                        dhcp_state = BOUND;
-                       printf ("DHCP client bound to address %pI4\n", &NetOurIP);
-
-                       /* Obey the 'autoload' setting */
-                       if ((s = getenv("autoload")) != NULL) {
-                               if (*s == 'n') {
-                                       /*
-                                        * Just use BOOTP to configure system;
-                                        * Do not use TFTP to load the bootfile.
-                                        */
-                                       NetState = NETLOOP_SUCCESS;
-                                       return;
-#if defined(CONFIG_CMD_NFS)
-                               } else if (strcmp(s, "NFS") == 0) {
-                                       /*
-                                        * Use NFS to load the bootfile.
-                                        */
-                                       NfsStart();
-                                       return;
-#endif
-                               }
-                       }
-                       TftpStart();
+                       printf("DHCP client bound to address %pI4 (%lu ms)\n",
+                               &NetOurIP, get_timer(bootp_start));
+                       bootstage_mark_name(BOOTSTAGE_ID_BOOTP_STOP,
+                               "bootp_stop");
+
+                       net_auto_load();
                        return;
                }
                break;
@@ -956,7 +1005,7 @@ DhcpHandler(uchar * pkt, unsigned dest, unsigned src, unsigned len)
                /* DHCP client bound to address */
                break;
        default:
-               puts ("DHCP: INVALID STATE\n");
+               puts("DHCP: INVALID STATE\n");
                break;
        }
 
@@ -967,5 +1016,3 @@ void DhcpRequest(void)
        BootpRequest();
 }
 #endif /* CONFIG_CMD_DHCP */
-
-#endif /* CONFIG_CMD_NET */