Re: [PATCH] bootconfig: Update prototype of setup_boot_config()

From: Masami Hiramatsu
Date: Thu Mar 11 2021 - 04:51:19 EST


On Thu, 11 Mar 2021 16:52:13 +0800
Cao jin <jojing64@xxxxxxxxx> wrote:

> Parameter "cmdline" has no use, drop it.

OK, this looks good to me.

Acked-by: Masami Hiramatsu <mhiramat@xxxxxxxxxx>

Thank you,

>
> Signed-off-by: Cao jin <jojing64@xxxxxxxxx>
> ---
> init/main.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/init/main.c b/init/main.c
> index c68d784376ca..621a11ed18fb 100644
> --- a/init/main.c
> +++ b/init/main.c
> @@ -404,7 +404,7 @@ static int __init bootconfig_params(char *param, char *val,
> return 0;
> }
>
> -static void __init setup_boot_config(const char *cmdline)
> +static void __init setup_boot_config(void)
> {
> static char tmp_cmdline[COMMAND_LINE_SIZE] __initdata;
> const char *msg;
> @@ -471,7 +471,7 @@ static void __init setup_boot_config(const char *cmdline)
>
> #else
>
> -static void __init setup_boot_config(const char *cmdline)
> +static void __init setup_boot_config(void)
> {
> /* Remove bootconfig data from initrd */
> get_boot_config_from_initrd(NULL, NULL);
> @@ -869,7 +869,7 @@ asmlinkage __visible void __init __no_sanitize_address start_kernel(void)
> pr_notice("%s", linux_banner);
> early_security_init();
> setup_arch(&command_line);
> - setup_boot_config(command_line);
> + setup_boot_config();
> setup_command_line(command_line);
> setup_nr_cpu_ids();
> setup_per_cpu_areas();
> --
> 2.29.2
>


--
Masami Hiramatsu <mhiramat@xxxxxxxxxx>