linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MIPS: Fix CONFIG_MIPS_CMDLINE_DTB_EXTEND handling
@ 2020-02-25 15:28 Paul Cercueil
  2020-02-26 14:59 ` Thomas Bogendoerfer
  0 siblings, 1 reply; 2+ messages in thread
From: Paul Cercueil @ 2020-02-25 15:28 UTC (permalink / raw)
  To: Paul Burton, Thomas Bogendoerfer
  Cc: od, linux-mips, linux-kernel, Paul Cercueil, stable

The CONFIG_MIPS_CMDLINE_DTB_EXTEND option is used so that the kernel
arguments provided in the 'bootargs' property in devicetree are extended
with the kernel arguments provided by the bootloader.

The code was broken, as it didn't actually take any of the kernel
arguments provided in devicetree when that option was set.

Fixes: 7784cac69735 ("MIPS: cmdline: Clean up boot_command_line
initialization")
Cc: stable@vger.kernel.org
Signed-off-by: Paul Cercueil <paul@crapouillou.net>
---
 arch/mips/kernel/setup.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/mips/kernel/setup.c b/arch/mips/kernel/setup.c
index 1ac2752fb791..a7b469d89e2c 100644
--- a/arch/mips/kernel/setup.c
+++ b/arch/mips/kernel/setup.c
@@ -605,7 +605,8 @@ static void __init bootcmdline_init(char **cmdline_p)
 	 * If we're configured to take boot arguments from DT, look for those
 	 * now.
 	 */
-	if (IS_ENABLED(CONFIG_MIPS_CMDLINE_FROM_DTB))
+	if (IS_ENABLED(CONFIG_MIPS_CMDLINE_FROM_DTB) ||
+	    IS_ENABLED(CONFIG_MIPS_CMDLINE_DTB_EXTEND))
 		of_scan_flat_dt(bootcmdline_scan_chosen, &dt_bootargs);
 #endif
 
-- 
2.25.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] MIPS: Fix CONFIG_MIPS_CMDLINE_DTB_EXTEND handling
  2020-02-25 15:28 [PATCH] MIPS: Fix CONFIG_MIPS_CMDLINE_DTB_EXTEND handling Paul Cercueil
@ 2020-02-26 14:59 ` Thomas Bogendoerfer
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Bogendoerfer @ 2020-02-26 14:59 UTC (permalink / raw)
  To: Paul Cercueil; +Cc: Paul Burton, od, linux-mips, linux-kernel, stable

On Tue, Feb 25, 2020 at 12:28:09PM -0300, Paul Cercueil wrote:
> The CONFIG_MIPS_CMDLINE_DTB_EXTEND option is used so that the kernel
> arguments provided in the 'bootargs' property in devicetree are extended
> with the kernel arguments provided by the bootloader.
> 
> The code was broken, as it didn't actually take any of the kernel
> arguments provided in devicetree when that option was set.
> 
> Fixes: 7784cac69735 ("MIPS: cmdline: Clean up boot_command_line
> initialization")
> Cc: stable@vger.kernel.org
> Signed-off-by: Paul Cercueil <paul@crapouillou.net>
> ---
>  arch/mips/kernel/setup.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

applied to mips-fixes.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-02-26 14:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-25 15:28 [PATCH] MIPS: Fix CONFIG_MIPS_CMDLINE_DTB_EXTEND handling Paul Cercueil
2020-02-26 14:59 ` Thomas Bogendoerfer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).