Re: [PATCH 1/6] kconfig: print symbol type in help text

[Randy Dunlap]

On Tue, 27 Apr 2010 15:48:16 +0800 Li Zefan wrote:

> Randy suggested to print out the symbol type in gconfig.
> 
> Note this change does more than Randy's suggestion, that it also
> affects the display of help text in other config tools.
> 
>   â Symbol: BLOCK [=y]
>   â Type  : boolean
>   â Prompt: Enable the block layer
>   â   Defined at block/Kconfig:4
>   â   Depends on: EMBEDDED [=n]
> 
> Signed-off-by: Li Zefan <lizf@xxxxxxxxxxxxxx>

Acked-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>

Thanks.

> ---
>  scripts/kconfig/menu.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/scripts/kconfig/menu.c b/scripts/kconfig/menu.c
> index e150176..b5d15fa 100644
> --- a/scripts/kconfig/menu.c
> +++ b/scripts/kconfig/menu.c
> @@ -501,9 +501,11 @@ void get_symbol_str(struct gstr *r, struct symbol *sym)
>  	bool hit;
>  	struct property *prop;
>  
> -	if (sym && sym->name)
> +	if (sym && sym->name) {
>  		str_printf(r, "Symbol: %s [=%s]\n", sym->name,
>  			   sym_get_string_value(sym));
> +		str_printf(r, "Type  : %s\n", sym_type_name(sym->type));
> +	}
>  	for_all_prompts(sym, prop)
>  		get_prompt_str(r, prop);
>  	hit = false;
> -- 
> 1.6.3


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/