m68k/amiga,atari: Fix specifying multiple debug= parameters

Since commit d6713b4091a99fa2af2fabdcd2f3fb97f32ecf2e ("m68k: early
parameter support"), the user can specify multiple debug consoles using the
"debug=" kernel command line parameter.
However, as there's only a single struct console object, which is reused,
it would actually register the same console object multiple times, causing
the following warning:

WARNING: CPU: 0 PID: 0 at kernel/printk/printk.c:2233 register_console+0x36/
console 'debug0' already registered

Make sure to register the console object only once, to avoid the warning.

Note that still only one console (the one corresponding to the last
"debug=" parameter) will be active at the same time, as the .write() method
of the already registered console object is overwritten by a subsequent
"debug=" parameter.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>

1 files changed, 14 insertions, 5 deletions

diff --git a/arch/m68k/amiga/config.c b/arch/m68k/amiga/config.c
index c425fa6c2795..9625b7132227 100644
--- a/arch/m68k/amiga/config.c
+++ b/arch/m68k/amiga/config.c
@@ -620,6 +620,8 @@ static void amiga_mem_console_write(struct console *co, const char *s,
 
 static int __init amiga_savekmsg_setup(char *arg)
 {
+        bool registered;
+
         if (!MACH_IS_AMIGA || strcmp(arg, "mem"))
                 return 0;
 
@@ -636,8 +638,10 @@ static int __init amiga_savekmsg_setup(char *arg)
         savekmsg->magicptr = ZTWO_PADDR(savekmsg);
         savekmsg->size = 0;
 
+        registered = !!amiga_console_driver.write;
         amiga_console_driver.write = amiga_mem_console_write;
-        register_console(&amiga_console_driver);
+        if (!registered)
+                register_console(&amiga_console_driver);
         return 0;
 }
 
@@ -719,11 +723,16 @@ void amiga_serial_gets(struct console *co, char *s, int len)
 
 static int __init amiga_debug_setup(char *arg)
 {
-        if (MACH_IS_AMIGA && !strcmp(arg, "ser")) {
-                /* no initialization required (?) */
-                amiga_console_driver.write = amiga_serial_console_write;
+        bool registered;
+
+        if (!MACH_IS_AMIGA || strcmp(arg, "ser"))
+                return 0;
+
+        /* no initialization required (?) */
+        registered = !!amiga_console_driver.write;
+        amiga_console_driver.write = amiga_serial_console_write;
+        if (!registered)
                 register_console(&amiga_console_driver);
-        }
         return 0;
 }