kernel/git/stable/linux.git/scripts/kconfig/conf.c, branch linux-5.16.y

kconfig: refactor listnewconfig code

2021-10-11T14:13:23+00:00

We can reuse __print_symbol() helper to print symbols for listnewconfig. Only the difference is the format for "n" symbols. This prints "CONFIG_FOO=n" instead of "# CONFIG_FOO is not set". Signed-off-by: Masahiro Yamada

kconfig: remove 'const' from the return type of sym_escape_string_value()

2021-10-01T08:28:17+00:00

sym_escape_string_value() returns a malloc'ed memory, but as (const char *). So, it must be casted to (void *) when it is free'd. This is odd. The return type of sym_escape_string_value() should be (char *). I exploited that free(NULL) has no effect. Signed-off-by: Masahiro Yamada

kconfig: constify long_opts

2021-06-05T14:49:46+00:00

getopt_long() does not modify the long_opts structure. Signed-off-by: Masahiro Yamada

kconfig: remove allnoconfig_y option

2021-04-14T06:22:49+00:00

Now that the only user, CONFIG_EMBEDDED has stopped using this option, remove it entirely. Signed-off-by: Masahiro Yamada

kconfig: move conf_set_all_new_symbols() to conf.c

2021-04-14T06:20:14+00:00

This function is only used in conf.c. Move it there together with the randomize_choice_values() helper. Define 'enum conf_def_mode' locally in conf.c as well. Signed-off-by: Masahiro Yamada

kconfig: move conf_rewrite_mod_or_yes() to conf.c

2021-03-25T04:35:29+00:00

This function is only used in conf.c. Signed-off-by: Masahiro Yamada

kconfig: remove assignment for Kconfig file

2021-03-25T04:35:20+00:00

Pass av[optind] to conf_parse() directly. Signed-off-by: Masahiro Yamada

kconfig: add help messages for --help (-h) and --silent (-s)

2021-03-25T04:35:12+00:00

Add missing options and make the help message more readable. Signed-off-by: Masahiro Yamada

kconfig: add long options --help and --silent

2021-03-25T04:35:03+00:00

They are long options for -h and -s, respectively. Signed-off-by: Masahiro Yamada

kconfig: refactor option parse code

2021-03-25T04:34:37+00:00

The current option parse code is clumsy. The 's' option is separately handled in an if-conditional due to the following code: input_mode = (enum input_mode)opt; If 's' is moved to the switch statement, the invalid value 's' would be assigned to the input_mode. Another potential problem is that we are mixing 'enum input_mode' and ASCII characters. They could overwrap if we add more input modes. To separate them out, set the flag field of long options to a pointer of input_mode_opt. For mode select options, getopt_long() returns 0, which never causes overwrap with ASCII characters that represent short options. Signed-off-by: Masahiro Yamada