]> pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - drivers/char/hvc_console.c
[PATCH] handle errors returned by platform_get_irq*()
[linux-2.6-omap-h63xx.git] / drivers / char / hvc_console.c
index 4b776f4eb467b57707d2ab1f02532d41daa775ac..f65b2e14a48561adccb082815e1cadfced89a042 100644 (file)
@@ -22,6 +22,7 @@
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
  */
 
+#include <linux/config.h>
 #include <linux/console.h>
 #include <linux/cpumask.h>
 #include <linux/init.h>
@@ -40,7 +41,6 @@
 #include <linux/delay.h>
 #include <asm/uaccess.h>
 #include <asm/hvconsole.h>
-#include <asm/vio.h>
 
 #define HVC_MAJOR      229
 #define HVC_MINOR      0
@@ -85,11 +85,11 @@ struct hvc_struct {
        char outbuf[N_OUTBUF] __ALIGNED__;
        int n_outbuf;
        uint32_t vtermno;
+       struct hv_ops *ops;
        int irq_requested;
        int irq;
        struct list_head next;
        struct kobject kobj; /* ref count & hvc_struct lifetime */
-       struct vio_dev *vdev;
 };
 
 /* dynamic list of hvc_struct instances */
@@ -143,10 +143,9 @@ struct hvc_struct *hvc_get_by_index(int index)
  * console interfaces but can still be used as a tty device.  This has to be
  * static because kmalloc will not work during early console init.
  */
-static uint32_t vtermnos[MAX_NR_HVC_CONSOLES];
-
-/* Used for accounting purposes */
-static int num_vterms = 0;
+static struct hv_ops *cons_ops[MAX_NR_HVC_CONSOLES];
+static uint32_t vtermnos[MAX_NR_HVC_CONSOLES] =
+       {[0 ... MAX_NR_HVC_CONSOLES - 1] = -1};
 
 /*
  * Console APIs, NOT TTY.  These APIs are available immediately when
@@ -157,14 +156,14 @@ void hvc_console_print(struct console *co, const char *b, unsigned count)
 {
        char c[16] __ALIGNED__;
        unsigned i = 0, n = 0;
-       int r, donecr = 0;
+       int r, donecr = 0, index = co->index;
 
        /* Console access attempt outside of acceptable console range. */
-       if (co->index >= MAX_NR_HVC_CONSOLES)
+       if (index >= MAX_NR_HVC_CONSOLES)
                return;
 
        /* This console adapter was removed so it is not useable. */
-       if (vtermnos[co->index] < 0)
+       if (vtermnos[index] < 0)
                return;
 
        while (count > 0 || i > 0) {
@@ -178,7 +177,7 @@ void hvc_console_print(struct console *co, const char *b, unsigned count)
                                --count;
                        }
                } else {
-                       r = hvc_put_chars(vtermnos[co->index], c, i);
+                       r = cons_ops[index]->put_chars(vtermnos[index], c, i);
                        if (r < 0) {
                                /* throw away chars on error */
                                i = 0;
@@ -193,12 +192,21 @@ void hvc_console_print(struct console *co, const char *b, unsigned count)
 
 static struct tty_driver *hvc_console_device(struct console *c, int *index)
 {
+       if (vtermnos[c->index] == -1)
+               return NULL;
+
        *index = c->index;
        return hvc_driver;
 }
 
 static int __init hvc_console_setup(struct console *co, char *options)
 {
+       if (co->index < 0 || co->index >= MAX_NR_HVC_CONSOLES)
+               return -ENODEV;
+
+       if (vtermnos[co->index] == -1)
+               return -ENODEV;
+
        return 0;
 }
 
@@ -211,14 +219,23 @@ struct console hvc_con_driver = {
        .index          = -1,
 };
 
-/* Early console initialization.  Preceeds driver initialization. */
+/*
+ * Early console initialization.  Preceeds driver initialization.
+ *
+ * (1) we are first, and the user specified another driver
+ * -- index will remain -1
+ * (2) we are first and the user specified no driver
+ * -- index will be set to 0, then we will fail setup.
+ * (3)  we are first and the user specified our driver
+ * -- index will be set to user specified driver, and we will fail
+ * (4) we are after driver, and this initcall will register us
+ * -- if the user didn't specify a driver then the console will match
+ *
+ * Note that for cases 2 and 3, we will match later when the io driver
+ * calls hvc_instantiate() and call register again.
+ */
 static int __init hvc_console_init(void)
 {
-       int i;
-
-       for (i=0; i<MAX_NR_HVC_CONSOLES; i++)
-               vtermnos[i] = -1;
-       num_vterms = hvc_find_vtys();
        register_console(&hvc_con_driver);
        return 0;
 }
@@ -230,22 +247,40 @@ console_initcall(hvc_console_init);
  * vty adapters do NOT get an hvc_instantiate() callback since they
  * appear after early console init.
  */
-int hvc_instantiate(uint32_t vtermno, int index)
+int hvc_instantiate(uint32_t vtermno, int index, struct hv_ops *ops)
 {
+       struct hvc_struct *hp;
+
        if (index < 0 || index >= MAX_NR_HVC_CONSOLES)
                return -1;
 
        if (vtermnos[index] != -1)
                return -1;
 
+       /* make sure no no tty has been registerd in this index */
+       hp = hvc_get_by_index(index);
+       if (hp) {
+               kobject_put(&hp->kobj);
+               return -1;
+       }
+
        vtermnos[index] = vtermno;
+       cons_ops[index] = ops;
 
        /* reserve all indices upto and including this index */
        if (last_hvc < index)
                last_hvc = index;
 
+       /* if this index is what the user requested, then register
+        * now (setup won't fail at this point).  It's ok to just
+        * call register again if previously .setup failed.
+        */
+       if (index == hvc_con_driver.index)
+               register_console(&hvc_con_driver);
+
        return 0;
 }
+EXPORT_SYMBOL(hvc_instantiate);
 
 /* Wake the sleeping khvcd */
 static void hvc_kick(void)
@@ -254,13 +289,17 @@ static void hvc_kick(void)
        wake_up_process(hvc_task);
 }
 
+static int hvc_poll(struct hvc_struct *hp);
+
 /*
  * NOTE: This API isn't used if the console adapter doesn't support interrupts.
  * In this case the console is poll driven.
  */
 static irqreturn_t hvc_handle_interrupt(int irq, void *dev_instance, struct pt_regs *regs)
 {
-       hvc_kick();
+       /* if hvc_poll request a repoll, then kick the hvcd thread */
+       if (hvc_poll(dev_instance))
+               hvc_kick();
        return IRQ_HANDLED;
 }
 
@@ -296,6 +335,8 @@ static int hvc_open(struct tty_struct *tty, struct file * filp)
        } /* else count == 0 */
 
        tty->driver_data = hp;
+       tty->low_latency = 1; /* Makes flushes to ldisc synchronous. */
+
        hp->tty = tty;
        /* Save for request_irq outside of spin_lock. */
        irq = hp->irq;
@@ -430,7 +471,7 @@ static void hvc_push(struct hvc_struct *hp)
 {
        int n;
 
-       n = hvc_put_chars(hp->vtermno, hp->outbuf, hp->n_outbuf);
+       n = hp->ops->put_chars(hp->vtermno, hp->outbuf, hp->n_outbuf);
        if (n <= 0) {
                if (n == 0)
                        return;
@@ -558,9 +599,7 @@ static int hvc_poll(struct hvc_struct *hp)
 
        /* Read data if any */
        for (;;) {
-               int count = N_INBUF;
-               if (count > (TTY_FLIPBUF_SIZE - tty->flip.count))
-                       count = TTY_FLIPBUF_SIZE - tty->flip.count;
+               int count = tty_buffer_request_room(tty, N_INBUF);
 
                /* If flip is full, just reschedule a later read */
                if (count == 0) {
@@ -568,7 +607,7 @@ static int hvc_poll(struct hvc_struct *hp)
                        break;
                }
 
-               n = hvc_get_chars(hp->vtermno, buf, count);
+               n = hp->ops->get_chars(hp->vtermno, buf, count);
                if (n <= 0) {
                        /* Hangup the tty when disconnected from host */
                        if (n == -EPIPE) {
@@ -580,26 +619,26 @@ static int hvc_poll(struct hvc_struct *hp)
                }
                for (i = 0; i < n; ++i) {
 #ifdef CONFIG_MAGIC_SYSRQ
-                       /* Handle the SysRq Hack */
-                       if (buf[i] == '\x0f') { /* ^O -- should support a sequence */
-                               sysrq_pressed = 1;
-                               continue;
-                       } else if (sysrq_pressed) {
-                               handle_sysrq(buf[i], NULL, tty);
-                               sysrq_pressed = 0;
-                               continue;
+                       if (hp->index == hvc_con_driver.index) {
+                               /* Handle the SysRq Hack */
+                               /* XXX should support a sequence */
+                               if (buf[i] == '\x0f') { /* ^O */
+                                       sysrq_pressed = 1;
+                                       continue;
+                               } else if (sysrq_pressed) {
+                                       handle_sysrq(buf[i], NULL, tty);
+                                       sysrq_pressed = 0;
+                                       continue;
+                               }
                        }
 #endif /* CONFIG_MAGIC_SYSRQ */
                        tty_insert_flip_char(tty, buf[i], 0);
                }
 
-               if (tty->flip.count)
-                       tty_schedule_flip(tty);
-
                /*
                 * Account for the total amount read in one loop, and if above
-                * 64 bytes, we do a quick schedule loop to let the tty grok the
-                * data and eventually throttle us.
+                * 64 bytes, we do a quick schedule loop to let the tty grok
+                * the data and eventually throttle us.
                 */
                read_total += n;
                if (read_total >= 64) {
@@ -616,6 +655,9 @@ static int hvc_poll(struct hvc_struct *hp)
  bail:
        spin_unlock_irqrestore(&hp->lock, flags);
 
+       if (read_total)
+               tty_flip_buffer_push(tty);
+       
        return poll_mask;
 }
 
@@ -643,7 +685,6 @@ int khvcd(void *unused)
                if (cpus_empty(cpus_in_xmon)) {
                        spin_lock(&hvc_structs_lock);
                        list_for_each_entry(hp, &hvc_structs, next) {
-                               /*hp = list_entry(node, struct hvc_struct, * next); */
                                poll_mask |= hvc_poll(hp);
                        }
                        spin_unlock(&hvc_structs_lock);
@@ -699,26 +740,21 @@ static struct kobj_type hvc_kobj_type = {
        .release = destroy_hvc_struct,
 };
 
-static int __devinit hvc_probe(
-               struct vio_dev *dev,
-               const struct vio_device_id *id)
+struct hvc_struct __devinit *hvc_alloc(uint32_t vtermno, int irq,
+                                       struct hv_ops *ops)
 {
        struct hvc_struct *hp;
        int i;
 
-       /* probed with invalid parameters. */
-       if (!dev || !id)
-               return -EPERM;
-
        hp = kmalloc(sizeof(*hp), GFP_KERNEL);
        if (!hp)
-               return -ENOMEM;
+               return ERR_PTR(-ENOMEM);
 
        memset(hp, 0x00, sizeof(*hp));
-       hp->vtermno = dev->unit_address;
-       hp->vdev = dev;
-       hp->vdev->dev.driver_data = hp;
-       hp->irq = dev->irq;
+
+       hp->vtermno = vtermno;
+       hp->irq = irq;
+       hp->ops = ops;
 
        kobject_init(&hp->kobj);
        hp->kobj.ktype = &hvc_kobj_type;
@@ -743,12 +779,12 @@ static int __devinit hvc_probe(
        list_add_tail(&(hp->next), &hvc_structs);
        spin_unlock(&hvc_structs_lock);
 
-       return 0;
+       return hp;
 }
+EXPORT_SYMBOL(hvc_alloc);
 
-static int __devexit hvc_remove(struct vio_dev *dev)
+int __devexit hvc_remove(struct hvc_struct *hp)
 {
-       struct hvc_struct *hp = dev->dev.driver_data;
        unsigned long flags;
        struct kobject *kobjp;
        struct tty_struct *tty;
@@ -781,31 +817,14 @@ static int __devexit hvc_remove(struct vio_dev *dev)
                tty_hangup(tty);
        return 0;
 }
-
-char hvc_driver_name[] = "hvc_console";
-
-static struct vio_device_id hvc_driver_table[] __devinitdata= {
-       {"serial", "hvterm1"},
-       { NULL, }
-};
-MODULE_DEVICE_TABLE(vio, hvc_driver_table);
-
-static struct vio_driver hvc_vio_driver = {
-       .name           = hvc_driver_name,
-       .id_table       = hvc_driver_table,
-       .probe          = hvc_probe,
-       .remove         = hvc_remove,
-};
+EXPORT_SYMBOL(hvc_remove);
 
 /* Driver initialization.  Follow console initialization.  This is where the TTY
  * interfaces start to become available. */
 int __init hvc_init(void)
 {
-       int rc;
-
-       /* We need more than num_vterms adapters due to hotplug additions. */
+       /* We need more than hvc_count adapters due to hotplug additions. */
        hvc_driver = alloc_tty_driver(HVC_ALLOC_TTY_ADAPTERS);
-       /* hvc_driver = alloc_tty_driver(num_vterms); */
        if (!hvc_driver)
                return -ENOMEM;
 
@@ -820,9 +839,6 @@ int __init hvc_init(void)
        hvc_driver->flags = TTY_DRIVER_REAL_RAW;
        tty_set_operations(hvc_driver, &hvc_ops);
 
-       if (tty_register_driver(hvc_driver))
-               panic("Couldn't register hvc console driver\n");
-
        /* Always start the kthread because there can be hotplug vty adapters
         * added later. */
        hvc_task = kthread_run(khvcd, NULL, "khvcd");
@@ -832,22 +848,23 @@ int __init hvc_init(void)
                return -EIO;
        }
 
-       /* Register as a vio device to receive callbacks */
-       rc = vio_register_driver(&hvc_vio_driver);
+       if (tty_register_driver(hvc_driver))
+               panic("Couldn't register hvc console driver\n");
 
-       return rc;
+       return 0;
 }
 module_init(hvc_init);
 
-/* This isn't particularily necessary due to this being a console driver but it
- * is nice to be thorough */
+/* This isn't particularily necessary due to this being a console driver
+ * but it is nice to be thorough.
+ */
 static void __exit hvc_exit(void)
 {
        kthread_stop(hvc_task);
 
-       vio_unregister_driver(&hvc_vio_driver);
        tty_unregister_driver(hvc_driver);
        /* return tty_struct instances allocated in hvc_init(). */
        put_tty_driver(hvc_driver);
+       unregister_console(&hvc_con_driver);
 }
 module_exit(hvc_exit);