]> git.kernelconcepts.de Git - karo-tx-uboot.git/blobdiff - common/cmd_elf.c
Make sure that argv[] argument pointers are not modified.
[karo-tx-uboot.git] / common / cmd_elf.c
index 4d8e1d2762df420ea55017d6f2ec08012b978a6f..104d6e68b3cf308930c4ae8ef4613469459ed16a 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/ctype.h>
 #include <net.h>
 #include <elf.h>
+#include <vxworks.h>
 
 #if defined(CONFIG_WALNUT) || defined(CONFIG_SYS_VXWORKS_MAC_PTR)
 DECLARE_GLOBAL_DATA_PTR;
@@ -28,7 +29,8 @@ unsigned long load_elf_image (unsigned long addr);
 
 /* Allow ports to override the default behavior */
 __attribute__((weak))
-unsigned long do_bootelf_exec (ulong (*entry)(int, char *[]), int argc, char *argv[])
+unsigned long do_bootelf_exec (ulong (*entry)(int, char * const[]),
+                              int argc, char * const argv[])
 {
        unsigned long ret;
 
@@ -55,7 +57,7 @@ unsigned long do_bootelf_exec (ulong (*entry)(int, char *[]), int argc, char *ar
 /* ======================================================================
  * Interpreter command to boot an arbitrary ELF image from memory.
  * ====================================================================== */
-int do_bootelf (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
+int do_bootelf (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
        unsigned long addr;             /* Address of the ELF image     */
        unsigned long rc;               /* Return value from user code  */
@@ -92,19 +94,16 @@ int do_bootelf (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
  * be either an ELF image or a raw binary.  Will attempt to setup the
  * bootline and other parameters correctly.
  * ====================================================================== */
-int do_bootvx (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
+int do_bootvx (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 {
        unsigned long addr;             /* Address of image            */
        unsigned long bootaddr;         /* Address to put the bootline */
        char *bootline;                 /* Text of the bootline        */
        char *tmp;                      /* Temporary char pointer      */
+       char build_buf[128];            /* Buffer for building the bootline */
 
-#if defined(CONFIG_4xx) || defined(CONFIG_IOP480)
-       char build_buf[80];             /* Buffer for building the bootline */
-#endif
-       /* -------------------------------------------------- */
-
-       /*
+       /* ---------------------------------------------------
+        *
         * Check the loadaddr variable.
         * If we don't know where the image is then we're done.
         */
@@ -120,7 +119,8 @@ int do_bootvx (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
        if ((argc == 2) && (strcmp (argv[1], "tftp") == 0)) {
                if (NetLoop (TFTP) <= 0)
                        return 1;
-               printf ("Automatic boot of VxWorks image at address 0x%08lx ... \n", addr);
+               printf ("Automatic boot of VxWorks image at address 0x%08lx ... \n",
+                    addr);
        }
 #endif
 
@@ -132,10 +132,12 @@ int do_bootvx (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 
 #if defined(CONFIG_WALNUT)
        tmp = (char *) CONFIG_SYS_NVRAM_BASE_ADDR + 0x500;
-       memcpy ((char *) tmp, (char *) &gd->bd->bi_enetaddr[3], 3);
+       eth_getenv_enetaddr("ethaddr", (uchar *)build_buf);
+       memcpy(tmp, &build_buf[3], 3);
 #elif defined(CONFIG_SYS_VXWORKS_MAC_PTR)
        tmp = (char *) CONFIG_SYS_VXWORKS_MAC_PTR;
-       memcpy ((char *) tmp, (char *) &gd->bd->bi_enetaddr[0], 6);
+       eth_getenv_enetaddr("ethaddr", (uchar *)build_buf);
+       memcpy(tmp, build_buf, 6);
 #else
        puts ("## Ethernet MAC address not copied to NV RAM\n");
 #endif
@@ -148,7 +150,7 @@ int do_bootvx (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
         */
 
        if ((tmp = getenv ("bootaddr")) == NULL)
-               bootaddr = 0x4200;
+               bootaddr = CONFIG_SYS_VXWORKS_BOOT_ADDR;
        else
                bootaddr = simple_strtoul (tmp, NULL, 16);
 
@@ -159,54 +161,40 @@ int do_bootvx (cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
         */
 
        if ((bootline = getenv ("bootargs")) != NULL) {
-               memcpy ((void *) bootaddr, bootline, MAX(strlen(bootline), 255));
-               flush_cache (bootaddr, MAX(strlen(bootline), 255));
+               memcpy ((void *) bootaddr, bootline,
+                       max (strlen (bootline), 255));
+               flush_cache (bootaddr, max (strlen (bootline), 255));
        } else {
-#if defined(CONFIG_4xx)
-               sprintf (build_buf, "ibmEmac(0,0)");
 
-               if ((tmp = getenv ("hostname")) != NULL) {
-                       sprintf (&build_buf[strlen (build_buf - 1)],
-                               "host:%s ", tmp);
+
+               sprintf (build_buf, CONFIG_SYS_VXWORKS_BOOT_DEVICE);
+               if ((tmp = getenv ("bootfile")) != NULL) {
+                       sprintf (&build_buf[strlen (build_buf)],
+                                "%s:%s ", CONFIG_SYS_VXWORKS_SERVERNAME, tmp);
                } else {
-                       sprintf (&build_buf[strlen (build_buf - 1)],
-                               ": ");
+                       sprintf (&build_buf[strlen (build_buf)],
+                                "%s:file ", CONFIG_SYS_VXWORKS_SERVERNAME);
                }
 
                if ((tmp = getenv ("ipaddr")) != NULL) {
-                       sprintf (&build_buf[strlen (build_buf - 1)],
-                               "e=%s ", tmp);
+                       sprintf (&build_buf[strlen (build_buf)], "e=%s ", tmp);
                }
-               memcpy ((void *)bootaddr, build_buf, MAX(strlen(build_buf), 255));
-               flush_cache (bootaddr, MAX(strlen(build_buf), 255));
-#elif defined(CONFIG_IOP480)
-               sprintf (build_buf, "dc(0,0)");
 
-               if ((tmp = getenv ("hostname")) != NULL) {
-                       sprintf (&build_buf[strlen (build_buf - 1)],
-                               "host:%s ", tmp);
-               } else {
-                       sprintf (&build_buf[strlen (build_buf - 1)],
-                               ": ");
+               if ((tmp = getenv ("serverip")) != NULL) {
+                       sprintf (&build_buf[strlen (build_buf)], "h=%s ", tmp);
                }
 
-               if ((tmp = getenv ("ipaddr")) != NULL) {
-                       sprintf (&build_buf[strlen (build_buf - 1)],
-                               "e=%s ", tmp);
+               if ((tmp = getenv ("hostname")) != NULL) {
+                       sprintf (&build_buf[strlen (build_buf)], "tn=%s ", tmp);
                }
-               memcpy ((void *) bootaddr, build_buf, MAX(strlen(build_buf), 255));
-               flush_cache (bootaddr, MAX(strlen(build_buf), 255));
-#else
-
-               /*
-                * I'm not sure what the device should be for other
-                * PPC flavors, the hostname and ipaddr should be ok
-                * to just copy
-                */
-
-               puts ("No bootargs defined\n");
-               return 1;
+#ifdef CONFIG_SYS_VXWORKS_ADD_PARAMS
+               sprintf (&build_buf[strlen (build_buf)],
+                        CONFIG_SYS_VXWORKS_ADD_PARAMS);
 #endif
+
+               memcpy ((void *) bootaddr, build_buf,
+                       max (strlen (build_buf), 255));
+               flush_cache (bootaddr, max (strlen (build_buf), 255));
        }
 
        /*
@@ -251,8 +239,7 @@ int valid_elf_image (unsigned long addr)
        }
 
        if (ehdr->e_type != ET_EXEC) {
-               printf ("## Not a 32-bit elf image at address 0x%08lx\n",
-                       addr);
+               printf ("## Not a 32-bit elf image at address 0x%08lx\n", addr);
                return 0;
        }
 
@@ -267,7 +254,6 @@ int valid_elf_image (unsigned long addr)
        return 1;
 }
 
-
 /* ======================================================================
  * A very simple elf loader, assumes the image is valid, returns the
  * entry point address.
@@ -302,7 +288,7 @@ unsigned long load_elf_image (unsigned long addr)
                }
 
                if (strtab) {
-                       printf ("%sing %s @ 0x%08lx (%ld bytes)\n",
+                       debug("%sing %s @ 0x%08lx (%ld bytes)\n",
                                (shdr->sh_type == SHT_NOBITS) ?
                                        "Clear" : "Load",
                                &strtab[shdr->sh_name],
@@ -327,12 +313,12 @@ unsigned long load_elf_image (unsigned long addr)
 /* ====================================================================== */
 U_BOOT_CMD(
        bootelf,      2,      0,      do_bootelf,
-       "bootelf - Boot from an ELF image in memory\n",
-       " [address] - load address of ELF image.\n"
+       "Boot from an ELF image in memory",
+       " [address] - load address of ELF image."
 );
 
 U_BOOT_CMD(
        bootvx,      2,      0,      do_bootvx,
-       "bootvx  - Boot vxWorks from an ELF image\n",
-       " [address] - load address of vxWorks ELF image.\n"
+       "Boot vxWorks from an ELF image",
+       " [address] - load address of vxWorks ELF image."
 );