Browse Source

hush: complete renaming CONFIG_SYS_HUSH_PARSER to CONFIG_HUSH_PARSER

There is no more define of CONFIG_SYS_HUSH_PARSER.  Rename some
remaining references and drop the backward compatible Kconfig entry.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Masahiro Yamada 9 years ago
parent
commit
f1f9d4fac5
6 changed files with 13 additions and 22 deletions
  1. 2 2
      board/work-microwave/work_92105/work_92105_display.c
  2. 0 6
      cmd/Kconfig
  3. 2 2
      cmd/Makefile
  4. 1 4
      common/Makefile
  5. 7 7
      common/cli.c
  6. 1 1
      test/command_ut.c

+ 2 - 2
board/work-microwave/work_92105/work_92105_display.c

@@ -311,8 +311,8 @@ U_BOOT_CMD(
  * only HUSH can understand them.
  * only HUSH can understand them.
  */
  */
 
 
-#if !defined(CONFIG_SYS_HUSH_PARSER)
-#error CONFIG_CMD_HD44760 requires CONFIG_SYS_HUSH_PARSER
+#if !defined(CONFIG_HUSH_PARSER)
+#error CONFIG_CMD_HD44760 requires CONFIG_HUSH_PARSER
 #endif
 #endif
 
 
 static int do_hd44780(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])
 static int do_hd44780(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])

+ 0 - 6
cmd/Kconfig

@@ -13,7 +13,6 @@ config CMDLINE
 
 
 config HUSH_PARSER
 config HUSH_PARSER
 	bool "Use hush shell"
 	bool "Use hush shell"
-	select SYS_HUSH_PARSER
 	depends on CMDLINE
 	depends on CMDLINE
 	help
 	help
 	  This option enables the "hush" shell (from Busybox) as command line
 	  This option enables the "hush" shell (from Busybox) as command line
@@ -24,11 +23,6 @@ config HUSH_PARSER
 	  If disabled, you get the old, much simpler behaviour with a somewhat
 	  If disabled, you get the old, much simpler behaviour with a somewhat
 	  smaller memory footprint.
 	  smaller memory footprint.
 
 
-config SYS_HUSH_PARSER
-	bool
-	help
-	  Backward compatibility.
-
 config SYS_PROMPT
 config SYS_PROMPT
 	string "Shell prompt"
 	string "Shell prompt"
 	default "=> "
 	default "=> "

+ 2 - 2
cmd/Makefile

@@ -49,7 +49,7 @@ obj-$(CONFIG_ENV_IS_IN_EEPROM) += eeprom.o
 obj-$(CONFIG_CMD_EEPROM) += eeprom.o
 obj-$(CONFIG_CMD_EEPROM) += eeprom.o
 obj-$(CONFIG_EFI_STUB) += efi.o
 obj-$(CONFIG_EFI_STUB) += efi.o
 obj-$(CONFIG_CMD_ELF) += elf.o
 obj-$(CONFIG_CMD_ELF) += elf.o
-obj-$(CONFIG_SYS_HUSH_PARSER) += exit.o
+obj-$(CONFIG_HUSH_PARSER) += exit.o
 obj-$(CONFIG_CMD_EXT4) += ext4.o
 obj-$(CONFIG_CMD_EXT4) += ext4.o
 obj-$(CONFIG_CMD_EXT2) += ext2.o
 obj-$(CONFIG_CMD_EXT2) += ext2.o
 obj-$(CONFIG_CMD_FAT) += fat.o
 obj-$(CONFIG_CMD_FAT) += fat.o
@@ -123,7 +123,7 @@ obj-$(CONFIG_CMD_STRINGS) += strings.o
 obj-$(CONFIG_CMD_TERMINAL) += terminal.o
 obj-$(CONFIG_CMD_TERMINAL) += terminal.o
 obj-$(CONFIG_CMD_TIME) += time.o
 obj-$(CONFIG_CMD_TIME) += time.o
 obj-$(CONFIG_CMD_TRACE) += trace.o
 obj-$(CONFIG_CMD_TRACE) += trace.o
-obj-$(CONFIG_SYS_HUSH_PARSER) += test.o
+obj-$(CONFIG_HUSH_PARSER) += test.o
 obj-$(CONFIG_CMD_TPM) += tpm.o
 obj-$(CONFIG_CMD_TPM) += tpm.o
 obj-$(CONFIG_CMD_TPM_TEST) += tpm_test.o
 obj-$(CONFIG_CMD_TPM_TEST) += tpm_test.o
 obj-$(CONFIG_CMD_TSI148) += tsi148.o
 obj-$(CONFIG_CMD_TSI148) += tsi148.o

+ 1 - 4
common/Makefile

@@ -11,10 +11,7 @@ obj-y += init/
 obj-y += main.o
 obj-y += main.o
 obj-y += exports.o
 obj-y += exports.o
 obj-y += hash.o
 obj-y += hash.o
-ifdef CONFIG_SYS_HUSH_PARSER
-obj-y += cli_hush.o
-endif
-
+obj-$(CONFIG_HUSH_PARSER) += cli_hush.o
 obj-$(CONFIG_AUTOBOOT) += autoboot.o
 obj-$(CONFIG_AUTOBOOT) += autoboot.o
 
 
 # This option is not just y/n - it can have a numeric value
 # This option is not just y/n - it can have a numeric value

+ 7 - 7
common/cli.c

@@ -28,7 +28,7 @@ DECLARE_GLOBAL_DATA_PTR;
  */
  */
 int run_command(const char *cmd, int flag)
 int run_command(const char *cmd, int flag)
 {
 {
-#ifndef CONFIG_SYS_HUSH_PARSER
+#ifndef CONFIG_HUSH_PARSER
 	/*
 	/*
 	 * cli_run_command can return 0 or 1 for success, so clean up
 	 * cli_run_command can return 0 or 1 for success, so clean up
 	 * its result.
 	 * its result.
@@ -55,7 +55,7 @@ int run_command(const char *cmd, int flag)
  */
  */
 int run_command_repeatable(const char *cmd, int flag)
 int run_command_repeatable(const char *cmd, int flag)
 {
 {
-#ifndef CONFIG_SYS_HUSH_PARSER
+#ifndef CONFIG_HUSH_PARSER
 	return cli_simple_run_command(cmd, flag);
 	return cli_simple_run_command(cmd, flag);
 #else
 #else
 	/*
 	/*
@@ -79,7 +79,7 @@ int run_command_list(const char *cmd, int len, int flag)
 
 
 	if (len == -1) {
 	if (len == -1) {
 		len = strlen(cmd);
 		len = strlen(cmd);
-#ifdef CONFIG_SYS_HUSH_PARSER
+#ifdef CONFIG_HUSH_PARSER
 		/* hush will never change our string */
 		/* hush will never change our string */
 		need_buff = 0;
 		need_buff = 0;
 #else
 #else
@@ -94,7 +94,7 @@ int run_command_list(const char *cmd, int len, int flag)
 		memcpy(buff, cmd, len);
 		memcpy(buff, cmd, len);
 		buff[len] = '\0';
 		buff[len] = '\0';
 	}
 	}
-#ifdef CONFIG_SYS_HUSH_PARSER
+#ifdef CONFIG_HUSH_PARSER
 	rcode = parse_string_outer(buff, FLAG_PARSE_SEMICOLON);
 	rcode = parse_string_outer(buff, FLAG_PARSE_SEMICOLON);
 #else
 #else
 	/*
 	/*
@@ -214,7 +214,7 @@ err:
 
 
 void cli_loop(void)
 void cli_loop(void)
 {
 {
-#ifdef CONFIG_SYS_HUSH_PARSER
+#ifdef CONFIG_HUSH_PARSER
 	parse_file_outer();
 	parse_file_outer();
 	/* This point is never reached */
 	/* This point is never reached */
 	for (;;);
 	for (;;);
@@ -222,12 +222,12 @@ void cli_loop(void)
 	cli_simple_loop();
 	cli_simple_loop();
 #else
 #else
 	printf("## U-Boot command line is disabled. Please enable CONFIG_CMDLINE\n");
 	printf("## U-Boot command line is disabled. Please enable CONFIG_CMDLINE\n");
-#endif /*CONFIG_SYS_HUSH_PARSER*/
+#endif /*CONFIG_HUSH_PARSER*/
 }
 }
 
 
 void cli_init(void)
 void cli_init(void)
 {
 {
-#ifdef CONFIG_SYS_HUSH_PARSER
+#ifdef CONFIG_HUSH_PARSER
 	u_boot_hush_start();
 	u_boot_hush_start();
 #endif
 #endif
 
 

+ 1 - 1
test/command_ut.c

@@ -43,7 +43,7 @@ static int do_ut_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 	assert(run_command_list("false", -1, 0) == 1);
 	assert(run_command_list("false", -1, 0) == 1);
 	assert(run_command_list("echo", -1, 0) == 0);
 	assert(run_command_list("echo", -1, 0) == 0);
 
 
-#ifdef CONFIG_SYS_HUSH_PARSER
+#ifdef CONFIG_HUSH_PARSER
 	run_command("setenv foo 'setenv black 1\nsetenv adder 2'", 0);
 	run_command("setenv foo 'setenv black 1\nsetenv adder 2'", 0);
 	run_command("run foo", 0);
 	run_command("run foo", 0);
 	assert(getenv("black") != NULL);
 	assert(getenv("black") != NULL);