1 files changed, 133 insertions, 95 deletions

diff --git a/scripts/kconfig/zconf.y b/scripts/kconfig/zconf.y
index 996b10a..2468068 100644
--- a/scripts/kconfig/zconf.y
+++ b/scripts/kconfig/zconf.y
@@ -28,5 +28,5 @@ struct symbol *symbol_hash[257];
 #define YYERROR_VERBOSE
 %}
-%expect 36
+%expect 40
 
 %union
@@ -47,4 +47,5 @@ struct symbol *symbol_hash[257];
 %token T_COMMENT
 %token T_CONFIG
+%token T_MENUCONFIG
 %token T_HELP
 %token <string> T_HELPTEXT
@@ -57,9 +58,12 @@ struct symbol *symbol_hash[257];
 %token T_DEFAULT
 %token T_TRISTATE
+%token T_DEF_TRISTATE
 %token T_BOOLEAN
+%token T_DEF_BOOLEAN
+%token T_STRING
 %token T_INT
 %token T_HEX
 %token <string> T_WORD
-%token <string> T_STRING
+%token <string> T_WORD_QUOTE
 %token T_UNEQUAL
 %token T_EOF
@@ -68,4 +72,6 @@ struct symbol *symbol_hash[257];
 %token T_OPEN_PAREN
 %token T_ON
+%token T_SELECT
+%token T_RANGE
 
 %left T_OR
@@ -104,4 +110,5 @@ common_block:
         | comment_stmt
         | config_stmt
+        | menuconfig_stmt
         | source_stmt
         | nl_or_eof
@@ -109,7 +116,7 @@ common_block:
 
 
-/* config entry */
+/* config/menuconfig entry */
 
-config_entry_start: T_CONFIG T_WORD
+config_entry_start: T_CONFIG T_WORD T_EOL
 {
         struct symbol *sym = sym_lookup($2, 0);
@@ -119,5 +126,5 @@ config_entry_start: T_CONFIG T_WORD
 };
 
-config_stmt: config_entry_start T_EOL config_option_list
+config_stmt: config_entry_start config_option_list
 {
         menu_end_entry();
@@ -125,13 +132,31 @@ config_stmt: config_entry_start T_EOL config_option_list
 };
 
+menuconfig_entry_start: T_MENUCONFIG T_WORD T_EOL
+{
+        struct symbol *sym = sym_lookup($2, 0);
+        sym->flags |= SYMBOL_OPTIONAL;
+        menu_add_entry(sym);
+        printd(DEBUG_PARSE, "%s:%d:menuconfig %s\n", zconf_curname(), zconf_lineno(), $2);
+};
+
+menuconfig_stmt: menuconfig_entry_start config_option_list
+{
+        if (current_entry->prompt)
+                current_entry->prompt->type = P_MENU;
+        else
+                zconfprint("warning: menuconfig statement without prompt");
+        menu_end_entry();
+        printd(DEBUG_PARSE, "%s:%d:endconfig\n", zconf_curname(), zconf_lineno());
+};
+
 config_option_list:
           /* empty */
-        | config_option_list config_option T_EOL
-        | config_option_list depends T_EOL
+        | config_option_list config_option
+        | config_option_list depends
         | config_option_list help
         | config_option_list T_EOL
-{ };
+;
 
-config_option: T_TRISTATE prompt_stmt_opt
+config_option: T_TRISTATE prompt_stmt_opt T_EOL
 {
         menu_set_type(S_TRISTATE);
@@ -139,5 +164,12 @@ config_option: T_TRISTATE prompt_stmt_opt
 };
 
-config_option: T_BOOLEAN prompt_stmt_opt
+config_option: T_DEF_TRISTATE expr if_expr T_EOL
+{
+        menu_add_expr(P_DEFAULT, $2, $3);
+        menu_set_type(S_TRISTATE);
+        printd(DEBUG_PARSE, "%s:%d:def_boolean\n", zconf_curname(), zconf_lineno());
+};
+
+config_option: T_BOOLEAN prompt_stmt_opt T_EOL
 {
         menu_set_type(S_BOOLEAN);
@@ -145,5 +177,12 @@ config_option: T_BOOLEAN prompt_stmt_opt
 };
 
-config_option: T_INT prompt_stmt_opt
+config_option: T_DEF_BOOLEAN expr if_expr T_EOL
+{
+        menu_add_expr(P_DEFAULT, $2, $3);
+        menu_set_type(S_BOOLEAN);
+        printd(DEBUG_PARSE, "%s:%d:def_boolean\n", zconf_curname(), zconf_lineno());
+};
+
+config_option: T_INT prompt_stmt_opt T_EOL
 {
         menu_set_type(S_INT);
@@ -151,5 +190,5 @@ config_option: T_INT prompt_stmt_opt
 };
 
-config_option: T_HEX prompt_stmt_opt
+config_option: T_HEX prompt_stmt_opt T_EOL
 {
         menu_set_type(S_HEX);
@@ -157,5 +196,5 @@ config_option: T_HEX prompt_stmt_opt
 };
 
-config_option: T_STRING prompt_stmt_opt
+config_option: T_STRING prompt_stmt_opt T_EOL
 {
         menu_set_type(S_STRING);
@@ -163,28 +202,40 @@ config_option: T_STRING prompt_stmt_opt
 };
 
-config_option: T_PROMPT prompt if_expr
+config_option: T_PROMPT prompt if_expr T_EOL
 {
-        menu_add_prop(P_PROMPT, $2, NULL, $3);
+        menu_add_prompt(P_PROMPT, $2, $3);
         printd(DEBUG_PARSE, "%s:%d:prompt\n", zconf_curname(), zconf_lineno());
 };
 
-config_option: T_DEFAULT symbol if_expr
+config_option: T_DEFAULT expr if_expr T_EOL
 {
-        menu_add_prop(P_DEFAULT, NULL, $2, $3);
+        menu_add_expr(P_DEFAULT, $2, $3);
         printd(DEBUG_PARSE, "%s:%d:default\n", zconf_curname(), zconf_lineno());
 };
 
+config_option: T_SELECT T_WORD if_expr T_EOL
+{
+        menu_add_symbol(P_SELECT, sym_lookup($2, 0), $3);
+        printd(DEBUG_PARSE, "%s:%d:select\n", zconf_curname(), zconf_lineno());
+};
+
+config_option: T_RANGE symbol symbol if_expr T_EOL
+{
+        menu_add_expr(P_RANGE, expr_alloc_comp(E_RANGE,$2, $3), $4);
+        printd(DEBUG_PARSE, "%s:%d:range\n", zconf_curname(), zconf_lineno());
+};
+
 /* choice entry */
 
-choice: T_CHOICE
+choice: T_CHOICE T_EOL
 {
         struct symbol *sym = sym_lookup(NULL, 0);
         sym->flags |= SYMBOL_CHOICE;
         menu_add_entry(sym);
-        menu_add_prop(P_CHOICE, NULL, NULL, NULL);
+        menu_add_expr(P_CHOICE, NULL, NULL);
         printd(DEBUG_PARSE, "%s:%d:choice\n", zconf_curname(), zconf_lineno());
 };
 
-choice_entry: choice T_EOL choice_option_list
+choice_entry: choice choice_option_list
 {
         menu_end_entry();
@@ -201,5 +252,5 @@ choice_end: end
 
 choice_stmt:
-          choice_entry choice_block choice_end T_EOL
+          choice_entry choice_block choice_end
         | choice_entry choice_block
 {
@@ -210,17 +261,29 @@ choice_stmt:
 choice_option_list:
           /* empty */
-        | choice_option_list choice_option T_EOL
-        | choice_option_list depends T_EOL
+        | choice_option_list choice_option
+        | choice_option_list depends
         | choice_option_list help
         | choice_option_list T_EOL
 ;
 
-choice_option: T_PROMPT prompt if_expr
+choice_option: T_PROMPT prompt if_expr T_EOL
 {
-        menu_add_prop(P_PROMPT, $2, NULL, $3);
+        menu_add_prompt(P_PROMPT, $2, $3);
         printd(DEBUG_PARSE, "%s:%d:prompt\n", zconf_curname(), zconf_lineno());
 };
 
-choice_option: T_OPTIONAL
+choice_option: T_TRISTATE prompt_stmt_opt T_EOL
+{
+        menu_set_type(S_TRISTATE);
+        printd(DEBUG_PARSE, "%s:%d:tristate\n", zconf_curname(), zconf_lineno());
+};
+
+choice_option: T_BOOLEAN prompt_stmt_opt T_EOL
+{
+        menu_set_type(S_BOOLEAN);
+        printd(DEBUG_PARSE, "%s:%d:boolean\n", zconf_curname(), zconf_lineno());
+};
+
+choice_option: T_OPTIONAL T_EOL
 {
         current_entry->sym->flags |= SYMBOL_OPTIONAL;
@@ -228,7 +291,7 @@ choice_option: T_OPTIONAL
 };
 
-choice_option: T_DEFAULT symbol if_expr
+choice_option: T_DEFAULT T_WORD if_expr T_EOL
 {
-        menu_add_prop(P_DEFAULT, NULL, $2, $3);
+        menu_add_symbol(P_DEFAULT, sym_lookup($2, 0), $3);
         printd(DEBUG_PARSE, "%s:%d:default\n", zconf_curname(), zconf_lineno());
 };
@@ -241,5 +304,5 @@ choice_block:
 /* if entry */
 
-if: T_IF expr
+if: T_IF expr T_EOL
 {
         printd(DEBUG_PARSE, "%s:%d:if\n", zconf_curname(), zconf_lineno());
@@ -259,6 +322,6 @@ if_end: end
 
 if_stmt:
-          if T_EOL if_block if_end T_EOL
-        | if T_EOL if_block
+          if if_block if_end
+        | if if_block
 {
         printf("%s:%d: missing 'endif' for this 'if' statement\n", current_menu->file->name, current_menu->lineno);
@@ -275,5 +338,5 @@ if_block:
 /* menu entry */
 
-menu: T_MENU prompt
+menu: T_MENU prompt T_EOL
 {
         menu_add_entry(NULL);
@@ -282,5 +345,5 @@ menu: T_MENU prompt
 };
 
-menu_entry: menu T_EOL depends_list
+menu_entry: menu depends_list
 {
         menu_end_entry();
@@ -297,5 +360,5 @@ menu_end: end
 
 menu_stmt:
-          menu_entry menu_block menu_end T_EOL
+          menu_entry menu_block menu_end
         | menu_entry menu_block
 {
@@ -312,5 +375,5 @@ menu_block:
 ;
 
-source: T_SOURCE prompt
+source: T_SOURCE prompt T_EOL
 {
         $$ = $2;
@@ -318,5 +381,5 @@ source: T_SOURCE prompt
 };
 
-source_stmt: source T_EOL
+source_stmt: source
 {
         zconf_nextfile($1);
@@ -325,5 +388,5 @@ source_stmt: source T_EOL
 /* comment entry */
 
-comment: T_COMMENT prompt
+comment: T_COMMENT prompt T_EOL
 {
         menu_add_entry(NULL);
@@ -332,5 +395,5 @@ comment: T_COMMENT prompt
 };
 
-comment_stmt: comment T_EOL depends_list
+comment_stmt: comment depends_list
 {
         menu_end_entry();
@@ -353,19 +416,19 @@ help: help_start T_HELPTEXT
 
         depends_list:  /* empty */
-                | depends_list depends T_EOL
+                | depends_list depends
                 | depends_list T_EOL
-{ };
+;
 
-depends: T_DEPENDS T_ON expr
+depends: T_DEPENDS T_ON expr T_EOL
 {
         menu_add_dep($3);
         printd(DEBUG_PARSE, "%s:%d:depends on\n", zconf_curname(), zconf_lineno());
 }
-        | T_DEPENDS expr
+        | T_DEPENDS expr T_EOL
 {
         menu_add_dep($2);
         printd(DEBUG_PARSE, "%s:%d:depends\n", zconf_curname(), zconf_lineno());
 }
-        | T_REQUIRES expr
+        | T_REQUIRES expr T_EOL
 {
         menu_add_dep($2);
@@ -377,20 +440,16 @@ depends: T_DEPENDS T_ON expr
 prompt_stmt_opt:
           /* empty */
-        | prompt
-{
-        menu_add_prop(P_PROMPT, $1, NULL, NULL);
-}
-        | prompt T_IF expr
+        | prompt if_expr
 {
-        menu_add_prop(P_PROMPT, $1, NULL, $3);
+        menu_add_prop(P_PROMPT, $1, NULL, $2);
 };
 
         prompt:  T_WORD
-        | T_STRING
+        | T_WORD_QUOTE
 ;
 
-        end:      T_ENDMENU     { $$ = T_ENDMENU; }
-                | T_ENDCHOICE   { $$ = T_ENDCHOICE; }
-                | T_ENDIF       { $$ = T_ENDIF; }
+        end:      T_ENDMENU nl_or_eof{ $$ = T_ENDMENU; }
+                | T_ENDCHOICE nl_or_eof{ $$ = T_ENDCHOICE; }
+                | T_ENDIF nl_or_eof{ $$ = T_ENDIF; }
 ;
 
@@ -412,5 +471,5 @@ expr:	  symbol				{ $$ = expr_alloc_symbol($1); }
 
         symbol:           T_WORD{ $$ = sym_lookup($1, 0); free($1); }
-                | T_STRING{ $$ = sym_lookup($1, 1); free($1); }
+                | T_WORD_QUOTE{ $$ = sym_lookup($1, 1); free($1); }
 ;
 
@@ -419,9 +478,13 @@ symbol:	  T_WORD	{ $$ = sym_lookup($1, 0); free($1); }
 void conf_parse(const char *name)
 {
+        struct symbol *sym;
+        int i;
+
         zconf_initscan(name);
 
         sym_init();
         menu_init();
-        rootmenu.prompt = menu_add_prop(P_MENU, "Configuration", NULL, NULL);
+        modules_sym = sym_lookup("MODULES", 0);
+        rootmenu.prompt = menu_add_prop(P_MENU, "Build Configuration", NULL, NULL);
 
         //zconfdebug = 1;
@@ -430,6 +493,10 @@ void conf_parse(const char *name)
                 exit(1);
         menu_finalize(&rootmenu);
-
-        modules_sym = sym_lookup("MODULES", 0);
+        for_all_symbols(i, sym) {
+                if (!(sym->flags & SYMBOL_CHECKED) && sym_check_deps(sym))
+                        printf("\n");
+                else
+                        sym->flags |= SYMBOL_CHECK_DONE;
+        }
 
         sym_change_count = 1;
@@ -469,5 +536,5 @@ static void zconfprint(const char *err, ...)
         va_list ap;
 
-        fprintf(stderr, "%s:%d: ", zconf_curname(), zconf_lineno());
+        fprintf(stderr, "%s:%d: ", zconf_curname(), zconf_lineno() + 1);
         va_start(ap, err);
         vfprintf(stderr, err, ap);
@@ -478,5 +545,5 @@ static void zconfprint(const char *err, ...)
 static void zconferror(const char *err)
 {
-        fprintf(stderr, "%s:%d: %s\n", zconf_curname(), zconf_lineno(), err);
+        fprintf(stderr, "%s:%d: %s\n", zconf_curname(), zconf_lineno() + 1, err);
 }
 
@@ -503,6 +570,4 @@ void print_symbol(FILE *out, struct menu *menu)
         struct property *prop;
 
-        //sym->flags |= SYMBOL_PRINTED;
-
         if (sym_is_choice(sym))
                 fprintf(out, "choice\n");
@@ -529,11 +594,4 @@ void print_symbol(FILE *out, struct menu *menu)
                 break;
         }
-#if 0
-        if (!expr_is_yes(sym->dep)) {
-                fputs("  depends ", out);
-                expr_fprint(sym->dep, out);
-                fputc('\n', out);
-        }
-#endif
         for (prop = sym->prop; prop; prop = prop->next) {
                 if (prop->menu != menu)
@@ -543,14 +601,7 @@ void print_symbol(FILE *out, struct menu *menu)
                         fputs("  prompt ", out);
                         print_quoted_string(out, prop->text);
-                        if (prop->def) {
-                                fputc(' ', out);
-                                if (prop->def->flags & SYMBOL_CONST)
-                                        print_quoted_string(out, prop->def->name);
-                                else
-                                        fputs(prop->def->name, out);
-                        }
-                        if (!expr_is_yes(E_EXPR(prop->visible))) {
+                        if (!expr_is_yes(prop->visible.expr)) {
                                 fputs(" if ", out);
-                                expr_fprint(E_EXPR(prop->visible), out);
+                                expr_fprint(prop->visible.expr, out);
                         }
                         fputc('\n', out);
@@ -558,8 +609,8 @@ void print_symbol(FILE *out, struct menu *menu)
                 case P_DEFAULT:
                         fputs( "  default ", out);
-                        print_quoted_string(out, prop->def->name);
-                        if (!expr_is_yes(E_EXPR(prop->visible))) {
+                        expr_fprint(prop->expr, out);
+                        if (!expr_is_yes(prop->visible.expr)) {
                                 fputs(" if ", out);
-                                expr_fprint(E_EXPR(prop->visible), out);
+                                expr_fprint(prop->visible.expr, out);
                         }
                         fputc('\n', out);
@@ -584,5 +635,4 @@ void print_symbol(FILE *out, struct menu *menu)
 void zconfdump(FILE *out)
 {
-        //struct file *file;
         struct property *prop;
         struct symbol *sym;
@@ -595,9 +645,4 @@ void zconfdump(FILE *out)
                 else if ((prop = menu->prompt)) {
                         switch (prop->type) {
-                        //case T_MAINMENU:
-                                //fputs("\nmainmenu ", out);
-                                //print_quoted_string(out, prop->text);
-                                //fputs("\n", out);
-                                //break;
                         case P_COMMENT:
                                 fputs("\ncomment ", out);
@@ -610,17 +655,10 @@ void zconfdump(FILE *out)
                                 fputs("\n", out);
                                 break;
-                        //case T_SOURCE:
-                                //fputs("\nsource ", out);
-                                //print_quoted_string(out, prop->text);
-                                //fputs("\n", out);
-                                //break;
-                        //case T_IF:
-                                //fputs("\nif\n", out);
                         default:
                                 ;
                         }
-                        if (!expr_is_yes(E_EXPR(prop->visible))) {
+                        if (!expr_is_yes(prop->visible.expr)) {
                                 fputs("  depends ", out);
-                                expr_fprint(E_EXPR(prop->visible), out);
+                                expr_fprint(prop->visible.expr, out);
                                 fputc('\n', out);
                         }