MIPS: TXx9: Improve handling of built-in and command-line args
authorAtsushi Nemoto <anemo@mba.ocn.ne.jp>
Tue, 19 Aug 2008 13:55:06 +0000 (22:55 +0900)
committerRalf Baechle <ralf@linux-mips.org>
Sat, 11 Oct 2008 15:18:41 +0000 (16:18 +0100)
* Make prom_init_cmdline() static and be called from prom_init.
* Append built-in args if the first character was '+'.
* Drop command-line args if the first character of built-in was '-'.
* Enclose args include spaces by quotes.
* TX4938_NAND_BOOT is no longer needed.

Signed-off-by: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
arch/mips/txx9/Kconfig
arch/mips/txx9/generic/setup.c
arch/mips/txx9/jmr3927/prom.c
arch/mips/txx9/rbtx4927/prom.c
arch/mips/txx9/rbtx4938/prom.c
include/asm-mips/txx9/generic.h

index 840fe757c48d836faa36b77b1c94e32585552b70..6fb2ef09d5dd858ed428a04835776cc3d3b87d05 100644 (file)
@@ -97,13 +97,6 @@ config TOSHIBA_RBTX4938_MPLEX_ATA
 
 endchoice
 
-config TX4938_NAND_BOOT
-       depends on EXPERIMENTAL && TOSHIBA_RBTX4938_MPLEX_NAND
-       bool "NAND Boot Support (EXPERIMENTAL)"
-       help
-         This is only for Toshiba RBTX4938 reference board, which has NAND IPL.
-         Select this option if you need to use NAND boot.
-
 endif
 
 config PCI_TX4927
index fe6bee09cecea23e640a40861a9dcbc4e4adfe94..33d8c9e3223d37ac90e0b756f22c85b07d38f833 100644 (file)
@@ -127,31 +127,51 @@ extern struct txx9_board_vec rbtx4938_vec;
 struct txx9_board_vec *txx9_board_vec __initdata;
 static char txx9_system_type[32];
 
-void __init prom_init_cmdline(void)
+static void __init prom_init_cmdline(void)
 {
        int argc = (int)fw_arg0;
-       char **argv = (char **)fw_arg1;
+       int *argv32 = (int *)fw_arg1;
        int i;                  /* Always ignore the "-c" at argv[0] */
-#ifdef CONFIG_64BIT
-       char *fixed_argv[32];
-       for (i = 0; i < argc; i++)
-               fixed_argv[i] = (char *)(long)(*((__s32 *)argv + i));
-       argv = fixed_argv;
-#endif
+       char builtin[CL_SIZE];
 
        /* ignore all built-in args if any f/w args given */
-       if (argc > 1)
-               *arcs_cmdline = '\0';
+       /*
+        * But if built-in strings was started with '+', append them
+        * to command line args.  If built-in was started with '-',
+        * ignore all f/w args.
+        */
+       builtin[0] = '\0';
+       if (arcs_cmdline[0] == '+')
+               strcpy(builtin, arcs_cmdline + 1);
+       else if (arcs_cmdline[0] == '-') {
+               strcpy(builtin, arcs_cmdline + 1);
+               argc = 0;
+       } else if (argc <= 1)
+               strcpy(builtin, arcs_cmdline);
+       arcs_cmdline[0] = '\0';
 
        for (i = 1; i < argc; i++) {
+               char *str = (char *)(long)argv32[i];
                if (i != 1)
                        strcat(arcs_cmdline, " ");
-               strcat(arcs_cmdline, argv[i]);
+               if (strchr(str, ' ')) {
+                       strcat(arcs_cmdline, "\"");
+                       strcat(arcs_cmdline, str);
+                       strcat(arcs_cmdline, "\"");
+               } else
+                       strcat(arcs_cmdline, str);
+       }
+       /* append saved builtin args */
+       if (builtin[0]) {
+               if (arcs_cmdline[0])
+                       strcat(arcs_cmdline, " ");
+               strcat(arcs_cmdline, builtin);
        }
 }
 
 void __init prom_init(void)
 {
+       prom_init_cmdline();
 #ifdef CONFIG_CPU_TX39XX
        txx9_board_vec = &jmr3927_vec;
 #endif
index 70c4c8ec3e84b36d57b241f68a19c9a38bc05303..c899c0c087a0fa846387061c532f2441adaca599 100644 (file)
@@ -47,7 +47,6 @@ void __init jmr3927_prom_init(void)
        if ((tx3927_ccfgptr->ccfg & TX3927_CCFG_TLBOFF) == 0)
                printk(KERN_ERR "TX3927 TLB off\n");
 
-       prom_init_cmdline();
        add_memory_region(0, JMR3927_SDRAM_SIZE, BOOT_MEM_RAM);
        txx9_sio_putchar_init(TX3927_SIO_REG(1));
 }
index 1dc0a5b1956b2ec425e790adfe4f10d58b4e3bd9..cc97c6a6011b0109a67e92c559d9d262e39a17e7 100644 (file)
@@ -36,7 +36,6 @@
 
 void __init rbtx4927_prom_init(void)
 {
-       prom_init_cmdline();
        add_memory_region(0, tx4927_get_mem_size(), BOOT_MEM_RAM);
        txx9_sio_putchar_init(TX4927_SIO_REG(0) & 0xfffffffffULL);
 }
index d73123cd2ab9441a68fcc65e31251a214ab16b54..bcb469247e8c4ffa024a616ab9f5154e9ef616a1 100644 (file)
@@ -18,9 +18,6 @@
 
 void __init rbtx4938_prom_init(void)
 {
-#ifndef CONFIG_TX4938_NAND_BOOT
-       prom_init_cmdline();
-#endif
        add_memory_region(0, tx4938_get_mem_size(), BOOT_MEM_RAM);
        txx9_sio_putchar_init(TX4938_SIO_REG(0) & 0xfffffffffULL);
 }
index 5b1ccf901c625a99931e6ed8e6e3a34c622c3312..c9eed7ea148c79b34052b235b2712ae0c006469a 100644 (file)
@@ -42,7 +42,6 @@ struct txx9_board_vec {
 };
 extern struct txx9_board_vec *txx9_board_vec;
 extern int (*txx9_irq_dispatch)(int pending);
-void prom_init_cmdline(void);
 char *prom_getcmdline(void);
 void txx9_wdt_init(unsigned long base);
 void txx9_spi_init(int busid, unsigned long base, int irq);