Re: [PATCH] RISC-V: Kconfig: Fix help text of CMDLINE_EXTEND

[Alexandre Ghiti]

Hi XIE Zhibang,

On 28/03/2025 11:14, Yeking@xxxxxxxxx wrote:
> From: 谢致邦 (XIE Zhibang) <Yeking@xxxxxxxxx>
>
> It is the built-in command line appended to the bootloader command line,
> not the bootloader command line appended to the built-in command line.
>
> Fixes: 3aed8c43267e ("RISC-V: Update Kconfig to better handle CMDLINE")
> Signed-off-by: 谢致邦 (XIE Zhibang) <Yeking@xxxxxxxxx>
> ---
>   arch/riscv/Kconfig | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index 54ed2a8cd7af..4cd18d0d07b3 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -1144,8 +1144,8 @@ config CMDLINE_FALLBACK
>   config CMDLINE_EXTEND
>   	bool "Extend bootloader kernel arguments"
>   	help
> -	  The command-line arguments provided during boot will be
> -	  appended to the built-in command line. This is useful in
> +	  The built-in command line will be appended to the command-
> +	  line arguments provided during boot. This is useful in
>   	  cases where the provided arguments are insufficient and
>   	  you don't want to or cannot modify them.
>   


It seems that you're correct 
(https://elixir.bootlin.com/linux/v6.14-rc6/source/drivers/of/fdt.c#L1065).

I noticed that all the architectures that define this config have the 
same text, so I guess the behavior changed at some point.

I would suggest to fix all the architectures and find which commit 
changed this behavior.

Thanks,

Alex