linux-toradex.git/scripts/kconfig/conf.c, branch v5.16-rc3

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