Re: [PATCH] powerpc: Remove inaccessible CMDLINE default
On Fri, 2019-08-02 at 07:18 +0200, Christophe Leroy wrote: > > Le 02/08/2019 à 07:02, Chris Packham a écrit : > > > > Since commit cbe46bd4f510 ("powerpc: remove CONFIG_CMDLINE #ifdef > > mess") > > CONFIG_CMDLINE has always had a value regardless of > > CONNIG_CMDLINE_BOOL. > s/CONNIG/CONFIG/ > > > > > > > For example: > > > > $ make ARCH=powerpc defconfig > > $ cat .config > > # CONFIG_CMDLINE_BOOL is not set > > CONFIG_CMDLINE="" > > > > When enabling CONNIG_CMDLINE_BOOL this value is kept making the > > 'default > > "..." if CONNIG_CMDLINE_BOOL' ineffective. > s/CONNIG/CONFIG/ > Will fix in v2. > > > > > > $ ./scripts/config --enable CONFIG_CMDLINE_BOOL > > $ cat .config > > CONFIG_CMDLINE_BOOL=y > > CONFIG_CMDLINE="" > > > > Additionally all the in-tree powerpc defconfigs that set > > CONFIG_CMDLINE_BOOL=y also set CONFIG_CMDLINE to something else. > > For > > these reasons remove the inaccessible default. > > > > Signed-off-by: Chris Packham > Reviewed-by: Christophe Leroy > > > > > --- > > This should be independent of http://patchwork.ozlabs.org/patch/114 > > 0811/ but > > I've generated this patch on a stream that has it applied locally. > > > > arch/powerpc/Kconfig | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig > > index d413fe1b4058..6fca6eba6aee 100644 > > --- a/arch/powerpc/Kconfig > > +++ b/arch/powerpc/Kconfig > > @@ -844,7 +844,6 @@ config CMDLINE_BOOL > > > > config CMDLINE > > string "Initial kernel command string" if CMDLINE_BOOL > > - default "console=ttyS0,9600 console=tty0 root=/dev/sda2" > > if CMDLINE_BOOL > > default "" > > help > > On some platforms, there is currently no way for the > > boot loader to > > > I think we could also get rid of CMDLINE_BOOL totally and use CMDLINE > != > "" instead. The only reason I can see to keep CMDLINE_BOOL is that it hides the text input for CMDLINE which seems to be a pattern in Kconfig. Happy to remove it if that's the consensus. I'll wait for the dust to settle on my other patch before sending a v2 of this one. > > Christophe
Re: [PATCH] powerpc: Remove inaccessible CMDLINE default
Le 02/08/2019 à 07:02, Chris Packham a écrit : Since commit cbe46bd4f510 ("powerpc: remove CONFIG_CMDLINE #ifdef mess") CONFIG_CMDLINE has always had a value regardless of CONNIG_CMDLINE_BOOL. s/CONNIG/CONFIG/ For example: $ make ARCH=powerpc defconfig $ cat .config # CONFIG_CMDLINE_BOOL is not set CONFIG_CMDLINE="" When enabling CONNIG_CMDLINE_BOOL this value is kept making the 'default "..." if CONNIG_CMDLINE_BOOL' ineffective. s/CONNIG/CONFIG/ $ ./scripts/config --enable CONFIG_CMDLINE_BOOL $ cat .config CONFIG_CMDLINE_BOOL=y CONFIG_CMDLINE="" Additionally all the in-tree powerpc defconfigs that set CONFIG_CMDLINE_BOOL=y also set CONFIG_CMDLINE to something else. For these reasons remove the inaccessible default. Signed-off-by: Chris Packham Reviewed-by: Christophe Leroy --- This should be independent of http://patchwork.ozlabs.org/patch/1140811/ but I've generated this patch on a stream that has it applied locally. arch/powerpc/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index d413fe1b4058..6fca6eba6aee 100644 --- a/arch/powerpc/Kconfig +++ b/arch/powerpc/Kconfig @@ -844,7 +844,6 @@ config CMDLINE_BOOL config CMDLINE string "Initial kernel command string" if CMDLINE_BOOL - default "console=ttyS0,9600 console=tty0 root=/dev/sda2" if CMDLINE_BOOL default "" help On some platforms, there is currently no way for the boot loader to I think we could also get rid of CMDLINE_BOOL totally and use CMDLINE != "" instead. Christophe
[PATCH] powerpc: Remove inaccessible CMDLINE default
Since commit cbe46bd4f510 ("powerpc: remove CONFIG_CMDLINE #ifdef mess") CONFIG_CMDLINE has always had a value regardless of CONNIG_CMDLINE_BOOL. For example: $ make ARCH=powerpc defconfig $ cat .config # CONFIG_CMDLINE_BOOL is not set CONFIG_CMDLINE="" When enabling CONNIG_CMDLINE_BOOL this value is kept making the 'default "..." if CONNIG_CMDLINE_BOOL' ineffective. $ ./scripts/config --enable CONFIG_CMDLINE_BOOL $ cat .config CONFIG_CMDLINE_BOOL=y CONFIG_CMDLINE="" Additionally all the in-tree powerpc defconfigs that set CONFIG_CMDLINE_BOOL=y also set CONFIG_CMDLINE to something else. For these reasons remove the inaccessible default. Signed-off-by: Chris Packham --- This should be independent of http://patchwork.ozlabs.org/patch/1140811/ but I've generated this patch on a stream that has it applied locally. arch/powerpc/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index d413fe1b4058..6fca6eba6aee 100644 --- a/arch/powerpc/Kconfig +++ b/arch/powerpc/Kconfig @@ -844,7 +844,6 @@ config CMDLINE_BOOL config CMDLINE string "Initial kernel command string" if CMDLINE_BOOL - default "console=ttyS0,9600 console=tty0 root=/dev/sda2" if CMDLINE_BOOL default "" help On some platforms, there is currently no way for the boot loader to -- 2.22.0