[IA64-SGI] sn_console.c minor cleanup
authorJes Sorensen <jes@sgi.com>
Tue, 17 Jan 2006 18:05:12 +0000 (13:05 -0500)
committerTony Luck <tony.luck@intel.com>
Tue, 17 Jan 2006 21:54:41 +0000 (13:54 -0800)
Fix printk level and remove unnecessary CONFIG_SMP|CONFIG_PREEMPT tests
as this is taken care through the spinlock macros anyway.

Signed-off-by: Jes Sorensen <jes@sgi.com>
Signed-off-by: Pat Gefre <pfg@sgi.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
drivers/serial/sn_console.c

index 5468e5a767e21bf9bcbf951f21b05dbbdb17b3b7..43e67d6c29d448a14923ec8c4df45973ac252030 100644 (file)
@@ -6,7 +6,7 @@
  * driver for that.
  *
  *
- * Copyright (c) 2004-2005 Silicon Graphics, Inc.  All Rights Reserved.
+ * Copyright (c) 2004-2006 Silicon Graphics, Inc.  All Rights Reserved.
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms of version 2 of the GNU General Public License
@@ -829,8 +829,8 @@ static int __init sn_sal_module_init(void)
                misc.name = DEVICE_NAME_DYNAMIC;
                retval = misc_register(&misc);
                if (retval != 0) {
-                       printk
-                           ("Failed to register console device using misc_register.\n");
+                       printk(KERN_WARNING "Failed to register console "
+                              "device using misc_register.\n");
                        return -ENODEV;
                }
                sal_console_uart.major = MISC_MAJOR;
@@ -942,88 +942,75 @@ sn_sal_console_write(struct console *co, const char *s, unsigned count)
 {
        unsigned long flags = 0;
        struct sn_cons_port *port = &sal_console_port;
-#if defined(CONFIG_SMP) || defined(CONFIG_PREEMPT)
        static int stole_lock = 0;
-#endif
 
        BUG_ON(!port->sc_is_asynch);
 
        /* We can't look at the xmit buffer if we're not registered with serial core
         *  yet.  So only do the fancy recovery after registering
         */
-       if (port->sc_port.info) {
-
-               /* somebody really wants this output, might be an
-                * oops, kdb, panic, etc.  make sure they get it. */
-#if defined(CONFIG_SMP) || defined(CONFIG_PREEMPT)
-               if (spin_is_locked(&port->sc_port.lock)) {
-                       int lhead = port->sc_port.info->xmit.head;
-                       int ltail = port->sc_port.info->xmit.tail;
-                       int counter, got_lock = 0;
+       if (!port->sc_port.info) {
+               /* Not yet registered with serial core - simple case */
+               puts_raw_fixed(port->sc_ops->sal_puts_raw, s, count);
+               return;
+       }
 
-                       /*
-                        * We attempt to determine if someone has died with the
-                        * lock. We wait ~20 secs after the head and tail ptrs
-                        * stop moving and assume the lock holder is not functional
-                        * and plow ahead. If the lock is freed within the time out
-                        * period we re-get the lock and go ahead normally. We also
-                        * remember if we have plowed ahead so that we don't have
-                        * to wait out the time out period again - the asumption
-                        * is that we will time out again.
-                        */
+       /* somebody really wants this output, might be an
+        * oops, kdb, panic, etc.  make sure they get it. */
+       if (spin_is_locked(&port->sc_port.lock)) {
+               int lhead = port->sc_port.info->xmit.head;
+               int ltail = port->sc_port.info->xmit.tail;
+               int counter, got_lock = 0;
+
+               /*
+                * We attempt to determine if someone has died with the
+                * lock. We wait ~20 secs after the head and tail ptrs
+                * stop moving and assume the lock holder is not functional
+                * and plow ahead. If the lock is freed within the time out
+                * period we re-get the lock and go ahead normally. We also
+                * remember if we have plowed ahead so that we don't have
+                * to wait out the time out period again - the asumption
+                * is that we will time out again.
+                */
 
-                       for (counter = 0; counter < 150; mdelay(125), counter++) {
-                               if (!spin_is_locked(&port->sc_port.lock)
-                                   || stole_lock) {
-                                       if (!stole_lock) {
-                                               spin_lock_irqsave(&port->
-                                                                 sc_port.lock,
-                                                                 flags);
-                                               got_lock = 1;
-                                       }
-                                       break;
-                               } else {
-                                       /* still locked */
-                                       if ((lhead !=
-                                            port->sc_port.info->xmit.head)
-                                           || (ltail !=
-                                               port->sc_port.info->xmit.
-                                               tail)) {
-                                               lhead =
-                                                   port->sc_port.info->xmit.
-                                                   head;
-                                               ltail =
-                                                   port->sc_port.info->xmit.
-                                                   tail;
-                                               counter = 0;
-                                       }
+               for (counter = 0; counter < 150; mdelay(125), counter++) {
+                       if (!spin_is_locked(&port->sc_port.lock)
+                           || stole_lock) {
+                               if (!stole_lock) {
+                                       spin_lock_irqsave(&port->sc_port.lock,
+                                                         flags);
+                                       got_lock = 1;
                                }
-                       }
-                       /* flush anything in the serial core xmit buffer, raw */
-                       sn_transmit_chars(port, 1);
-                       if (got_lock) {
-                               spin_unlock_irqrestore(&port->sc_port.lock,
-                                                      flags);
-                               stole_lock = 0;
+                               break;
                        } else {
-                               /* fell thru */
-                               stole_lock = 1;
+                               /* still locked */
+                               if ((lhead != port->sc_port.info->xmit.head)
+                                   || (ltail !=
+                                       port->sc_port.info->xmit.tail)) {
+                                       lhead =
+                                               port->sc_port.info->xmit.head;
+                                       ltail =
+                                               port->sc_port.info->xmit.tail;
+                                       counter = 0;
+                               }
                        }
-                       puts_raw_fixed(port->sc_ops->sal_puts_raw, s, count);
-               } else {
-                       stole_lock = 0;
-#endif
-                       spin_lock_irqsave(&port->sc_port.lock, flags);
-                       sn_transmit_chars(port, 1);
+               }
+               /* flush anything in the serial core xmit buffer, raw */
+               sn_transmit_chars(port, 1);
+               if (got_lock) {
                        spin_unlock_irqrestore(&port->sc_port.lock, flags);
-
-                       puts_raw_fixed(port->sc_ops->sal_puts_raw, s, count);
-#if defined(CONFIG_SMP) || defined(CONFIG_PREEMPT)
+                       stole_lock = 0;
+               } else {
+                       /* fell thru */
+                       stole_lock = 1;
                }
-#endif
-       }
-       else {
-               /* Not yet registered with serial core - simple case */
+               puts_raw_fixed(port->sc_ops->sal_puts_raw, s, count);
+       } else {
+               stole_lock = 0;
+               spin_lock_irqsave(&port->sc_port.lock, flags);
+               sn_transmit_chars(port, 1);
+               spin_unlock_irqrestore(&port->sc_port.lock, flags);
+
                puts_raw_fixed(port->sc_ops->sal_puts_raw, s, count);
        }
 }