In a multi-platform kernel binary we only need one early console instance.   

The difficulty here is that the common early console is started by
early_param(), whereas the MVME16x instance is started later by
config_mvme16x(). That means some interrupt setup must be done earlier.

Signed-off-by: Finn Thain <fth...@telegraphics.com.au>
Tested-by: Stephen N Chivers <schiv...@csc.com.au>

---
Changes in v2:
- place MACH_IS_MVME16x tests inside #ifdef platform tests
- eliminate #ifdef CONFIG_MVME16x

---
 arch/m68k/kernel/early_printk.c |   17 +++++++++++------
 arch/m68k/kernel/head.S         |   11 +++++++++++
 arch/m68k/mvme16x/config.c      |   26 +-------------------------
 3 files changed, 23 insertions(+), 31 deletions(-)

Index: linux-m68k/arch/m68k/kernel/early_printk.c
===================================================================
--- linux-m68k.orig/arch/m68k/kernel/early_printk.c     2014-04-12 
22:20:45.000000000 +1000
+++ linux-m68k/arch/m68k/kernel/early_printk.c  2014-04-12 22:20:48.000000000 
+1000
@@ -12,12 +12,21 @@
 #include <linux/string.h>
 #include <asm/setup.h>
 
+extern void mvme16x_cons_write(struct console *co,
+                               const char *str, unsigned count);
+
 asmlinkage void __init debug_cons_nputs(const char *s, unsigned n);
 
 static void debug_cons_write(struct console *c,
                              const char *s, unsigned n)
 {
-       debug_cons_nputs(s, n);
+#if !(defined(CONFIG_SUN3)   || defined(CONFIG_M68360) || \
+      defined(CONFIG_M68000) || defined(CONFIG_COLDFIRE))
+       if (MACH_IS_MVME16x)
+               mvme16x_cons_write(c, s, n);
+       else
+               debug_cons_nputs(s, n);
+#endif
 }
 
 static struct console early_console_instance = {
@@ -29,10 +38,6 @@ static struct console early_console_inst
 
 static int __init setup_early_printk(char *buf)
 {
-       /* MVME16x registers an early console after interrupt setup. */
-       if (MACH_IS_MVME16x)
-               return 0;
-
        if (early_console || buf)
                return 0;
 
@@ -53,7 +58,7 @@ early_param("earlyprintk", setup_early_p
 
 static int __init unregister_early_console(void)
 {
-       if (!early_console)
+       if (!early_console || MACH_IS_MVME16x)
                return 0;
 
        return unregister_console(early_console);
Index: linux-m68k/arch/m68k/mvme16x/config.c
===================================================================
--- linux-m68k.orig/arch/m68k/mvme16x/config.c  2014-04-12 22:20:10.000000000 
+1000
+++ linux-m68k/arch/m68k/mvme16x/config.c       2014-04-12 22:20:48.000000000 
+1000
@@ -213,7 +213,7 @@ static void __init mvme16x_init_IRQ (voi
 #define CySCRH         (0x22)
 #define CyTFTC         (0x80)
 
-static void cons_write(struct console *co, const char *str, unsigned count)
+void mvme16x_cons_write(struct console *co, const char *str, unsigned count)
 {
        volatile unsigned char *base_addr = (u_char *)CD2401_ADDR;
        volatile u_char sink;
@@ -268,20 +268,6 @@ static void cons_write(struct console *c
        base_addr[CyIER] = ier;
 }
 
-static struct console cons_info =
-{
-       .name   = "sercon",
-       .write  = cons_write,
-       .flags  = CON_PRINTBUFFER | CON_BOOT,
-       .index  = -1,
-};
-
-static void __init mvme16x_early_console(void)
-{
-       register_console(&cons_info);
-
-       printk(KERN_INFO "MVME16x: early console registered\n");
-}
 #endif
 
 void __init config_mvme16x(void)
@@ -336,16 +322,6 @@ void __init config_mvme16x(void)
     else
     {
        mvme16x_config = MVME16x_CONFIG_GOT_LP | MVME16x_CONFIG_GOT_CD2401;
-
-       /* Dont allow any interrupts from the CD2401 until the interrupt */
-       /* handlers are installed                                        */
-
-       pcc2chip[PccSCCMICR] = 0x10;
-       pcc2chip[PccSCCTICR] = 0x10;
-       pcc2chip[PccSCCRICR] = 0x10;
-#ifdef CONFIG_EARLY_PRINTK
-       mvme16x_early_console();
-#endif
     }
 }
 
Index: linux-m68k/arch/m68k/kernel/head.S
===================================================================
--- linux-m68k.orig/arch/m68k/kernel/head.S     2014-04-12 22:20:45.000000000 
+1000
+++ linux-m68k/arch/m68k/kernel/head.S  2014-04-12 22:20:48.000000000 +1000
@@ -2936,6 +2936,15 @@ L(serial_init_not_mac):
 2:
 #endif
 
+#ifdef CONFIG_MVME16x
+       is_not_mvme16x(L(serial_init_not_mvme16x))
+       moveb   #0x10,M167_PCSCCMICR
+       moveb   #0x10,M167_PCSCCTICR
+       moveb   #0x10,M167_PCSCCRICR
+       jra     L(serial_init_done)
+L(serial_init_not_mvme16x):
+#endif
+
 #ifdef CONFIG_APOLLO
 /* We count on the PROM initializing SIO1 */
 #endif
@@ -3768,7 +3777,9 @@ M167_CYIER = 0xfff45011
 M167_CYLICR = 0xfff45026
 M167_CYTEOIR = 0xfff45085
 M167_CYTDR = 0xfff450f8
+M167_PCSCCMICR = 0xfff4201d
 M167_PCSCCTICR = 0xfff4201e
+M167_PCSCCRICR = 0xfff4201f
 M167_PCTPIACKR = 0xfff42025
 #endif
 
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to