fedora core 6 1.2949 + vserver 2.2.0
[linux-2.6.git] / drivers / input / serio / i8042.c
index e9d8dab..3b12996 100644 (file)
@@ -1,7 +1,7 @@
 /*
  *  i8042 keyboard and mouse controller driver for Linux
  *
- *  Copyright (c) 1999-2002 Vojtech Pavlik
+ *  Copyright (c) 1999-2004 Vojtech Pavlik
  */
 
 /*
 #include <linux/moduleparam.h>
 #include <linux/interrupt.h>
 #include <linux/ioport.h>
-#include <linux/config.h>
-#include <linux/reboot.h>
 #include <linux/init.h>
-#include <linux/sysdev.h>
-#include <linux/pm.h>
 #include <linux/serio.h>
+#include <linux/err.h>
+#include <linux/rcupdate.h>
+#include <linux/platform_device.h>
 
 #include <asm/io.h>
 
@@ -28,6 +27,10 @@ MODULE_AUTHOR("Vojtech Pavlik <vojtech@suse.cz>");
 MODULE_DESCRIPTION("i8042 keyboard and mouse controller driver");
 MODULE_LICENSE("GPL");
 
+static unsigned int i8042_nokbd;
+module_param_named(nokbd, i8042_nokbd, bool, 0);
+MODULE_PARM_DESC(nokbd, "Do not probe or use KBD port.");
+
 static unsigned int i8042_noaux;
 module_param_named(noaux, i8042_noaux, bool, 0);
 MODULE_PARM_DESC(noaux, "Do not probe or use AUX (mouse) port.");
@@ -52,6 +55,27 @@ static unsigned int i8042_dumbkbd;
 module_param_named(dumbkbd, i8042_dumbkbd, bool, 0);
 MODULE_PARM_DESC(dumbkbd, "Pretend that controller can only read data from keyboard");
 
+static unsigned int i8042_noloop;
+module_param_named(noloop, i8042_noloop, bool, 0);
+MODULE_PARM_DESC(noloop, "Disable the AUX Loopback command while probing for the AUX port");
+
+static unsigned int i8042_blink_frequency = 500;
+module_param_named(panicblink, i8042_blink_frequency, uint, 0600);
+MODULE_PARM_DESC(panicblink, "Frequency with which keyboard LEDs should blink when kernel panics");
+
+#ifdef CONFIG_PNP
+static int i8042_nopnp;
+module_param_named(nopnp, i8042_nopnp, bool, 0);
+MODULE_PARM_DESC(nopnp, "Do not use PNP to detect controller settings");
+#endif
+
+#define DEBUG
+#ifdef DEBUG
+static int i8042_debug;
+module_param_named(debug, i8042_debug, bool, 0600);
+MODULE_PARM_DESC(debug, "Turn i8042 debugging mode on and off");
+#endif
+
 __obsolete_setup("i8042_noaux");
 __obsolete_setup("i8042_nomux");
 __obsolete_setup("i8042_unlock");
@@ -59,47 +83,44 @@ __obsolete_setup("i8042_reset");
 __obsolete_setup("i8042_direct");
 __obsolete_setup("i8042_dumbkbd");
 
-#undef DEBUG
 #include "i8042.h"
 
-spinlock_t i8042_lock = SPIN_LOCK_UNLOCKED;
+static DEFINE_SPINLOCK(i8042_lock);
 
-struct i8042_values {
+struct i8042_port {
+       struct serio *serio;
        int irq;
-       unsigned char disable;
-       unsigned char irqen;
        unsigned char exists;
        signed char mux;
-       unsigned char *name;
-       unsigned char *phys;
 };
 
-static struct serio i8042_kbd_port;
-static struct serio i8042_aux_port;
+#define I8042_KBD_PORT_NO      0
+#define I8042_AUX_PORT_NO      1
+#define I8042_MUX_PORT_NO      2
+#define I8042_NUM_PORTS                (I8042_NUM_MUX_PORTS + 2)
+
+static struct i8042_port i8042_ports[I8042_NUM_PORTS];
+
 static unsigned char i8042_initial_ctr;
 static unsigned char i8042_ctr;
-static unsigned char i8042_mux_open;
 static unsigned char i8042_mux_present;
-static unsigned char i8042_sysdev_initialized;
-static struct pm_dev *i8042_pm_dev;
-struct timer_list i8042_timer;
-
-/*
- * Shared IRQ's require a device pointer, but this driver doesn't support
- * multiple devices
- */
-#define i8042_request_irq_cookie (&i8042_timer)
+static unsigned char i8042_kbd_irq_registered;
+static unsigned char i8042_aux_irq_registered;
+static unsigned char i8042_suppress_kbd_ack;
+static struct platform_device *i8042_platform_device;
 
-static irqreturn_t i8042_interrupt(int irq, void *dev_id, struct pt_regs *regs);
+static irqreturn_t i8042_interrupt(int irq, void *dev_id);
 
 /*
  * The i8042_wait_read() and i8042_wait_write functions wait for the i8042 to
  * be ready for reading values from it / writing values to it.
+ * Called always with i8042_lock held.
  */
 
 static int i8042_wait_read(void)
 {
        int i = 0;
+
        while ((~i8042_read_status() & I8042_STR_OBF) && (i < I8042_CTL_TIMEOUT)) {
                udelay(50);
                i++;
@@ -110,6 +131,7 @@ static int i8042_wait_read(void)
 static int i8042_wait_write(void)
 {
        int i = 0;
+
        while ((i8042_read_status() & I8042_STR_IBF) && (i < I8042_CTL_TIMEOUT)) {
                udelay(50);
                i++;
@@ -125,15 +147,17 @@ static int i8042_wait_write(void)
 static int i8042_flush(void)
 {
        unsigned long flags;
-       unsigned char data;
+       unsigned char data, str;
        int i = 0;
 
        spin_lock_irqsave(&i8042_lock, flags);
 
-       while ((i8042_read_status() & I8042_STR_OBF) && (i++ < I8042_BUFFER_SIZE)) {
+       while (((str = i8042_read_status()) & I8042_STR_OBF) && (i < I8042_BUFFER_SIZE)) {
+               udelay(50);
                data = i8042_read_data();
+               i++;
                dbg("%02x <- i8042 (flush, %s)", data,
-                       i8042_read_status() & I8042_STR_AUXDATA ? "aux" : "kbd");
+                       str & I8042_STR_AUXDATA ? "aux" : "kbd");
        }
 
        spin_unlock_irqrestore(&i8042_lock, flags);
@@ -149,40 +173,56 @@ static int i8042_flush(void)
  * encoded in bits 8-11 of the command number.
  */
 
-static int i8042_command(unsigned char *param, int command)
+static int __i8042_command(unsigned char *param, int command)
 {
-       unsigned long flags;
-       int retval = 0, i = 0;
+       int i, error;
 
-       spin_lock_irqsave(&i8042_lock, flags);
+       if (i8042_noloop && command == I8042_CMD_AUX_LOOP)
+               return -1;
 
-       retval = i8042_wait_write();
-       if (!retval) {
-               dbg("%02x -> i8042 (command)", command & 0xff);
-               i8042_write_command(command & 0xff);
+       error = i8042_wait_write();
+       if (error)
+               return error;
+
+       dbg("%02x -> i8042 (command)", command & 0xff);
+       i8042_write_command(command & 0xff);
+
+       for (i = 0; i < ((command >> 12) & 0xf); i++) {
+               error = i8042_wait_write();
+               if (error)
+                       return error;
+               dbg("%02x -> i8042 (parameter)", param[i]);
+               i8042_write_data(param[i]);
        }
 
-       if (!retval)
-               for (i = 0; i < ((command >> 12) & 0xf); i++) {
-                       if ((retval = i8042_wait_write())) break;
-                       dbg("%02x -> i8042 (parameter)", param[i]);
-                       i8042_write_data(param[i]);
+       for (i = 0; i < ((command >> 8) & 0xf); i++) {
+               error = i8042_wait_read();
+               if (error) {
+                       dbg("     -- i8042 (timeout)");
+                       return error;
                }
 
-       if (!retval)
-               for (i = 0; i < ((command >> 8) & 0xf); i++) {
-                       if ((retval = i8042_wait_read())) break;
-                       if (i8042_read_status() & I8042_STR_AUXDATA)
-                               param[i] = ~i8042_read_data();
-                       else
-                               param[i] = i8042_read_data();
-                       dbg("%02x <- i8042 (return)", param[i]);
+               if (command == I8042_CMD_AUX_LOOP &&
+                   !(i8042_read_status() & I8042_STR_AUXDATA)) {
+                       dbg("     -- i8042 (auxerr)");
+                       return -1;
                }
 
-       spin_unlock_irqrestore(&i8042_lock, flags);
+               param[i] = i8042_read_data();
+               dbg("%02x <- i8042 (return)", param[i]);
+       }
 
-       if (retval)
-               dbg("     -- i8042 (timeout)");
+       return 0;
+}
+
+static int i8042_command(unsigned char *param, int command)
+{
+       unsigned long flags;
+       int retval;
+
+       spin_lock_irqsave(&i8042_lock, flags);
+       retval = __i8042_command(param, command);
+       spin_unlock_irqrestore(&i8042_lock, flags);
 
        return retval;
 }
@@ -198,7 +238,7 @@ static int i8042_kbd_write(struct serio *port, unsigned char c)
 
        spin_lock_irqsave(&i8042_lock, flags);
 
-       if(!(retval = i8042_wait_write())) {
+       if (!(retval = i8042_wait_write())) {
                dbg("%02x -> i8042 (kbd-data)", c);
                i8042_write_data(c);
        }
@@ -212,246 +252,194 @@ static int i8042_kbd_write(struct serio *port, unsigned char c)
  * i8042_aux_write() sends a byte out through the aux interface.
  */
 
-static int i8042_aux_write(struct serio *port, unsigned char c)
+static int i8042_aux_write(struct serio *serio, unsigned char c)
 {
-       struct i8042_values *values = port->driver;
-       int retval;
+       struct i8042_port *port = serio->port_data;
+
+       return i8042_command(&c, port->mux == -1 ?
+                                       I8042_CMD_AUX_SEND :
+                                       I8042_CMD_MUX_SEND + port->mux);
+}
 
 /*
- * Send the byte out.
+ * i8042_start() is called by serio core when port is about to finish
+ * registering. It will mark port as existing so i8042_interrupt can
+ * start sending data through it.
  */
+static int i8042_start(struct serio *serio)
+{
+       struct i8042_port *port = serio->port_data;
 
-       if (values->mux == -1)
-               retval = i8042_command(&c, I8042_CMD_AUX_SEND);
-       else
-               retval = i8042_command(&c, I8042_CMD_MUX_SEND + values->mux);
+       port->exists = 1;
+       mb();
+       return 0;
+}
 
 /*
- * Make sure the interrupt happens and the character is received even
- * in the case the IRQ isn't wired, so that we can receive further
- * characters later.
+ * i8042_stop() marks serio port as non-existing so i8042_interrupt
+ * will not try to send data to the port that is about to go away.
+ * The function is called by serio core as part of unregister procedure.
  */
+static void i8042_stop(struct serio *serio)
+{
+       struct i8042_port *port = serio->port_data;
 
-       i8042_interrupt(0, NULL, NULL);
-       return retval;
+       port->exists = 0;
+       synchronize_sched();
+       port->serio = NULL;
 }
 
 /*
- * i8042_activate_port() enables port on a chip.
+ * i8042_interrupt() is the most important function in this driver -
+ * it handles the interrupts from the i8042, and sends incoming bytes
+ * to the upper layers.
  */
 
-static int i8042_activate_port(struct serio *port)
+static irqreturn_t i8042_interrupt(int irq, void *dev_id)
 {
-       struct i8042_values *values = port->driver;
-
-       i8042_flush();
-
-       /*
-        * Enable port again here because it is disabled if we are
-        * resuming (normally it is enabled already).
-        */
-       i8042_ctr &= ~values->disable;
-
-       i8042_ctr |= values->irqen;
+       struct i8042_port *port;
+       unsigned long flags;
+       unsigned char str, data;
+       unsigned int dfl;
+       unsigned int port_no;
+       int ret = 1;
 
-       if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR)) {
-               i8042_ctr &= ~values->irqen;
-               return -1;
+       spin_lock_irqsave(&i8042_lock, flags);
+       str = i8042_read_status();
+       if (unlikely(~str & I8042_STR_OBF)) {
+               spin_unlock_irqrestore(&i8042_lock, flags);
+               if (irq) dbg("Interrupt %d, without any data", irq);
+               ret = 0;
+               goto out;
        }
+       data = i8042_read_data();
+       spin_unlock_irqrestore(&i8042_lock, flags);
 
-       return 0;
-}
-
+       if (i8042_mux_present && (str & I8042_STR_AUXDATA)) {
+               static unsigned long last_transmit;
+               static unsigned char last_str;
 
+               dfl = 0;
+               if (str & I8042_STR_MUXERR) {
+                       dbg("MUX error, status is %02x, data is %02x", str, data);
 /*
- * i8042_open() is called when a port is open by the higher layer.
- * It allocates the interrupt and calls i8042_enable_port.
+ * When MUXERR condition is signalled the data register can only contain
+ * 0xfd, 0xfe or 0xff if implementation follows the spec. Unfortunately
+ * it is not always the case. Some KBCs also report 0xfc when there is
+ * nothing connected to the port while others sometimes get confused which
+ * port the data came from and signal error leaving the data intact. They
+ * _do not_ revert to legacy mode (actually I've never seen KBC reverting
+ * to legacy mode yet, when we see one we'll add proper handling).
+ * Anyway, we process 0xfc, 0xfd, 0xfe and 0xff as timeouts, and for the
+ * rest assume that the data came from the same serio last byte
+ * was transmitted (if transmission happened not too long ago).
  */
 
-static int i8042_open(struct serio *port)
-{
-       struct i8042_values *values = port->driver;
+                       switch (data) {
+                               default:
+                                       if (time_before(jiffies, last_transmit + HZ/10)) {
+                                               str = last_str;
+                                               break;
+                                       }
+                                       /* fall through - report timeout */
+                               case 0xfc:
+                               case 0xfd:
+                               case 0xfe: dfl = SERIO_TIMEOUT; data = 0xfe; break;
+                               case 0xff: dfl = SERIO_PARITY;  data = 0xfe; break;
+                       }
+               }
 
-       if (values->mux != -1)
-               if (i8042_mux_open++)
-                       return 0;
+               port_no = I8042_MUX_PORT_NO + ((str >> 6) & 3);
+               last_str = str;
+               last_transmit = jiffies;
+       } else {
 
-       if (request_irq(values->irq, i8042_interrupt,
-                       SA_SHIRQ, "i8042", i8042_request_irq_cookie)) {
-               printk(KERN_ERR "i8042.c: Can't get irq %d for %s, unregistering the port.\n", values->irq, values->name);
-               goto irq_fail;
-       }
+               dfl = ((str & I8042_STR_PARITY) ? SERIO_PARITY : 0) |
+                     ((str & I8042_STR_TIMEOUT) ? SERIO_TIMEOUT : 0);
 
-       if (i8042_activate_port(port)) {
-               printk(KERN_ERR "i8042.c: Can't activate %s, unregistering the port\n", values->name);
-               goto activate_fail;
+               port_no = (str & I8042_STR_AUXDATA) ?
+                               I8042_AUX_PORT_NO : I8042_KBD_PORT_NO;
        }
 
-       i8042_interrupt(0, NULL, NULL);
+       port = &i8042_ports[port_no];
 
-       return 0;
+       dbg("%02x <- i8042 (interrupt, %d, %d%s%s)",
+           data, port_no, irq,
+           dfl & SERIO_PARITY ? ", bad parity" : "",
+           dfl & SERIO_TIMEOUT ? ", timeout" : "");
 
-activate_fail:
-       free_irq(values->irq, i8042_request_irq_cookie);
+       if (unlikely(i8042_suppress_kbd_ack))
+               if (port_no == I8042_KBD_PORT_NO &&
+                   (data == 0xfa || data == 0xfe)) {
+                       i8042_suppress_kbd_ack--;
+                       goto out;
+               }
 
-irq_fail:
-       values->exists = 0;
-       serio_unregister_port_delayed(port);
+       if (likely(port->exists))
+               serio_interrupt(port->serio, data, dfl);
 
-       return -1;
+ out:
+       return IRQ_RETVAL(ret);
 }
 
 /*
- * i8042_close() frees the interrupt, so that it can possibly be used
- * by another driver. We never know - if the user doesn't have a mouse,
- * the BIOS could have used the AUX interrupt for PCI.
+ * i8042_enable_kbd_port enables keybaord port on chip
  */
 
-static void i8042_close(struct serio *port)
+static int i8042_enable_kbd_port(void)
 {
-       struct i8042_values *values = port->driver;
-
-       if (values->mux != -1)
-               if (--i8042_mux_open)
-                       return;
-
-       i8042_ctr &= ~values->irqen;
+       i8042_ctr &= ~I8042_CTR_KBDDIS;
+       i8042_ctr |= I8042_CTR_KBDINT;
 
        if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR)) {
-               printk(KERN_ERR "i8042.c: Can't write CTR while closing %s.\n", values->name);
-               return;
+               printk(KERN_ERR "i8042.c: Failed to enable KBD port.\n");
+               return -EIO;
        }
 
-       free_irq(values->irq, i8042_request_irq_cookie);
-
-       i8042_flush();
+       return 0;
 }
 
 /*
- * Structures for registering the devices in the serio.c module.
+ * i8042_enable_aux_port enables AUX (mouse) port on chip
  */
 
-static struct i8042_values i8042_kbd_values = {
-       .irqen =        I8042_CTR_KBDINT,
-       .disable =      I8042_CTR_KBDDIS,
-       .name =         "KBD",
-       .mux =          -1,
-};
-
-static struct serio i8042_kbd_port =
+static int i8042_enable_aux_port(void)
 {
-       .type =         SERIO_8042_XL,
-       .write =        i8042_kbd_write,
-       .open =         i8042_open,
-       .close =        i8042_close,
-       .driver =       &i8042_kbd_values,
-       .name =         "i8042 Kbd Port",
-       .phys =         I8042_KBD_PHYS_DESC,
-};
+       i8042_ctr &= ~I8042_CTR_AUXDIS;
+       i8042_ctr |= I8042_CTR_AUXINT;
 
-static struct i8042_values i8042_aux_values = {
-       .irqen =        I8042_CTR_AUXINT,
-       .disable =      I8042_CTR_AUXDIS,
-       .name =         "AUX",
-       .mux =          -1,
-};
-
-static struct serio i8042_aux_port =
-{
-       .type =         SERIO_8042,
-       .write =        i8042_aux_write,
-       .open =         i8042_open,
-       .close =        i8042_close,
-       .driver =       &i8042_aux_values,
-       .name =         "i8042 Aux Port",
-       .phys =         I8042_AUX_PHYS_DESC,
-};
+       if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR)) {
+               printk(KERN_ERR "i8042.c: Failed to enable AUX port.\n");
+               return -EIO;
+       }
 
-static struct i8042_values i8042_mux_values[4];
-static struct serio i8042_mux_port[4];
-static char i8042_mux_names[4][32];
-static char i8042_mux_short[4][16];
-static char i8042_mux_phys[4][32];
+       return 0;
+}
 
 /*
- * i8042_interrupt() is the most important function in this driver -
- * it handles the interrupts from the i8042, and sends incoming bytes
- * to the upper layers.
+ * i8042_enable_mux_ports enables 4 individual AUX ports after
+ * the controller has been switched into Multiplexed mode
  */
 
-static irqreturn_t i8042_interrupt(int irq, void *dev_id, struct pt_regs *regs)
+static int i8042_enable_mux_ports(void)
 {
-       unsigned long flags;
-       unsigned char str, data = 0;
-       unsigned int dfl;
-       int ret;
-
-       mod_timer(&i8042_timer, jiffies + I8042_POLL_PERIOD);
-
-       spin_lock_irqsave(&i8042_lock, flags);
-       str = i8042_read_status();
-       if (str & I8042_STR_OBF)
-               data = i8042_read_data();
-       spin_unlock_irqrestore(&i8042_lock, flags);
-
-       if (~str & I8042_STR_OBF) {
-               if (irq) dbg("Interrupt %d, without any data", irq);
-               ret = 0;
-               goto out;
-       }
-
-       dfl = ((str & I8042_STR_PARITY) ? SERIO_PARITY : 0) |
-             ((str & I8042_STR_TIMEOUT) ? SERIO_TIMEOUT : 0);
-
-       if (i8042_mux_values[0].exists && (str & I8042_STR_AUXDATA)) {
-
-               if (str & I8042_STR_MUXERR) {
-                       switch (data) {
-                               case 0xfd:
-                               case 0xfe: dfl = SERIO_TIMEOUT; break;
-                               case 0xff: dfl = SERIO_PARITY; break;
-                       }
-                       data = 0xfe;
-               } else dfl = 0;
-
-               dbg("%02x <- i8042 (interrupt, aux%d, %d%s%s)",
-                       data, (str >> 6), irq,
-                       dfl & SERIO_PARITY ? ", bad parity" : "",
-                       dfl & SERIO_TIMEOUT ? ", timeout" : "");
-
-               serio_interrupt(i8042_mux_port + ((str >> 6) & 3), data, dfl, regs);
-
-               goto irq_ret;
-       }
-
-       dbg("%02x <- i8042 (interrupt, %s, %d%s%s)",
-               data, (str & I8042_STR_AUXDATA) ? "aux" : "kbd", irq,
-               dfl & SERIO_PARITY ? ", bad parity" : "",
-               dfl & SERIO_TIMEOUT ? ", timeout" : "");
+       unsigned char param;
+       int i;
 
-       if (i8042_aux_values.exists && (str & I8042_STR_AUXDATA)) {
-               serio_interrupt(&i8042_aux_port, data, dfl, regs);
-               goto irq_ret;
+       for (i = 0; i < I8042_NUM_MUX_PORTS; i++) {
+               i8042_command(&param, I8042_CMD_MUX_PFX + i);
+               i8042_command(&param, I8042_CMD_AUX_ENABLE);
        }
 
-       if (!i8042_kbd_values.exists)
-               goto irq_ret;
-
-       serio_interrupt(&i8042_kbd_port, data, dfl, regs);
-
-irq_ret:
-       ret = 1;
-out:
-       return IRQ_RETVAL(ret);
+       return i8042_enable_aux_port();
 }
 
 /*
- * i8042_enable_mux_mode checks whether the controller has an active
- * multiplexor and puts the chip into Multiplexed (as opposed to
- * Legacy) mode.
+ * i8042_set_mux_mode checks whether the controller has an active
+ * multiplexor and puts the chip into Multiplexed (1) or Legacy (0) mode.
  */
 
-static int i8042_enable_mux_mode(struct i8042_values *values, unsigned char *mux_version)
+static int i8042_set_mux_mode(unsigned int mode, unsigned char *mux_version)
 {
 
        unsigned char param;
@@ -463,96 +451,86 @@ static int i8042_enable_mux_mode(struct i8042_values *values, unsigned char *mux
 
 /*
  * Internal loopback test - send three bytes, they should come back from the
- * mouse interface, the last should be version. Note that we negate mouseport
- * command responses for the i8042_check_aux() routine.
+ * mouse interface, the last should be version.
  */
 
        param = 0xf0;
-       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param != 0x0f)
+       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param != 0xf0)
                return -1;
-       param = 0x56;
-       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param != 0xa9)
+       param = mode ? 0x56 : 0xf6;
+       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param != (mode ? 0x56 : 0xf6))
                return -1;
-       param = 0xa4;
-       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param == 0x5b) {
-
-/*
- * Do another loop test with the 0x5a value. Doing anything else upsets
- * Profusion/ServerWorks OSB4 chipsets.
- */
-
-               param = 0x5a;
-               i8042_command(&param, I8042_CMD_AUX_LOOP);
+       param = mode ? 0xa4 : 0xa5;
+       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param == (mode ? 0xa4 : 0xa5))
                return -1;
-       }
 
        if (mux_version)
-               *mux_version = ~param;
+               *mux_version = param;
 
        return 0;
 }
 
-
 /*
- * i8042_enable_mux_ports enables 4 individual AUX ports after
- * the controller has been switched into Multiplexed mode
+ * i8042_check_mux() checks whether the controller supports the PS/2 Active
+ * Multiplexing specification by Synaptics, Phoenix, Insyde and
+ * LCS/Telegraphics.
  */
 
-static int i8042_enable_mux_ports(struct i8042_values *values)
+static int __devinit i8042_check_mux(void)
 {
-       unsigned char param;
-       int i;
+       unsigned char mux_version;
+
+       if (i8042_set_mux_mode(1, &mux_version))
+               return -1;
+
 /*
- * Disable all muxed ports by disabling AUX.
+ * Workaround for interference with USB Legacy emulation
+ * that causes a v10.12 MUX to be found.
  */
+       if (mux_version == 0xAC)
+               return -1;
 
+       printk(KERN_INFO "i8042.c: Detected active multiplexing controller, rev %d.%d.\n",
+               (mux_version >> 4) & 0xf, mux_version & 0xf);
+
+/*
+ * Disable all muxed ports by disabling AUX.
+ */
        i8042_ctr |= I8042_CTR_AUXDIS;
        i8042_ctr &= ~I8042_CTR_AUXINT;
 
        if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR)) {
                printk(KERN_ERR "i8042.c: Failed to disable AUX port, can't use MUX.\n");
-               return -1;
+               return -EIO;
        }
 
-/*
- * Enable all muxed ports.
- */
-
-       for (i = 0; i < 4; i++) {
-               i8042_command(&param, I8042_CMD_MUX_PFX + i);
-               i8042_command(&param, I8042_CMD_AUX_ENABLE);
-       }
+       i8042_mux_present = 1;
 
        return 0;
 }
 
-
 /*
- * i8042_check_mux() checks whether the controller supports the PS/2 Active
- * Multiplexing specification by Synaptics, Phoenix, Insyde and
- * LCS/Telegraphics.
+ * The following is used to test AUX IRQ delivery.
  */
+static struct completion i8042_aux_irq_delivered __devinitdata;
+static int i8042_irq_being_tested __devinitdata;
 
-static int __init i8042_check_mux(struct i8042_values *values)
+static irqreturn_t __devinit i8042_aux_test_irq(int irq, void *dev_id)
 {
-       unsigned char mux_version;
-
-       if (i8042_enable_mux_mode(values, &mux_version))
-               return -1;
-
-       /* Workaround for broken chips which seem to support MUX, but in reality don't. */
-       /* They all report version 10.12 */
-       if (mux_version == 0xAC)
-               return -1;
-
-       printk(KERN_INFO "i8042.c: Detected active multiplexing controller, rev %d.%d.\n",
-               (mux_version >> 4) & 0xf, mux_version & 0xf);
+       unsigned long flags;
+       unsigned char str, data;
 
-       if (i8042_enable_mux_ports(values))
-               return -1;
+       spin_lock_irqsave(&i8042_lock, flags);
+       str = i8042_read_status();
+       if (str & I8042_STR_OBF) {
+               data = i8042_read_data();
+               if (i8042_irq_being_tested &&
+                   data == 0xa5 && (str & I8042_STR_AUXDATA))
+                       complete(&i8042_aux_irq_delivered);
+       }
+       spin_unlock_irqrestore(&i8042_lock, flags);
 
-       i8042_mux_present = 1;
-       return 0;
+       return IRQ_HANDLED;
 }
 
 
@@ -561,20 +539,13 @@ static int __init i8042_check_mux(struct i8042_values *values)
  * the presence of an AUX interface.
  */
 
-static int __init i8042_check_aux(struct i8042_values *values)
+static int __devinit i8042_check_aux(void)
 {
+       int retval = -1;
+       int irq_registered = 0;
+       int aux_loop_broken = 0;
+       unsigned long flags;
        unsigned char param;
-       static int i8042_check_aux_cookie;
-
-/*
- * Check if AUX irq is available. If it isn't, then there is no point
- * in trying to detect AUX presence.
- */
-
-       if (request_irq(values->irq, i8042_interrupt, SA_SHIRQ,
-                               "i8042", &i8042_check_aux_cookie))
-                return -1;
-       free_irq(values->irq, &i8042_check_aux_cookie);
 
 /*
  * Get rid of bytes in the queue.
@@ -589,7 +560,8 @@ static int __init i8042_check_aux(struct i8042_values *values)
  */
 
        param = 0x5a;
-       if (i8042_command(&param, I8042_CMD_AUX_LOOP) || param != 0xa5) {
+       retval = i8042_command(&param, I8042_CMD_AUX_LOOP);
+       if (retval || param != 0x5a) {
 
 /*
  * External connection test - filters out AT-soldered PS/2 i8042's
@@ -599,9 +571,16 @@ static int __init i8042_check_aux(struct i8042_values *values)
  * AUX ports, we test for this only when the LOOP command failed.
  */
 
-               if (i8042_command(&param, I8042_CMD_AUX_TEST)
-                       || (param && param != 0xfa && param != 0xff))
-                               return -1;
+               if (i8042_command(&param, I8042_CMD_AUX_TEST) ||
+                   (param && param != 0xfa && param != 0xff))
+                       return -1;
+
+/*
+ * If AUX_LOOP completed without error but returned unexpected data
+ * mark it as broken
+ */
+               if (!retval)
+                       aux_loop_broken = 1;
        }
 
 /*
@@ -620,6 +599,54 @@ static int __init i8042_check_aux(struct i8042_values *values)
        if (i8042_command(&param, I8042_CMD_CTL_RCTR) || (param & I8042_CTR_AUXDIS))
                return -1;
 
+/*
+ * Test AUX IRQ delivery to make sure BIOS did not grab the IRQ and
+ * used it for a PCI card or somethig else.
+ */
+
+       if (i8042_noloop || aux_loop_broken) {
+/*
+ * Without LOOP command we can't test AUX IRQ delivery. Assume the port
+ * is working and hope we are right.
+ */
+               retval = 0;
+               goto out;
+       }
+
+       if (request_irq(I8042_AUX_IRQ, i8042_aux_test_irq, IRQF_SHARED,
+                       "i8042", i8042_platform_device))
+               goto out;
+
+       irq_registered = 1;
+
+       if (i8042_enable_aux_port())
+               goto out;
+
+       spin_lock_irqsave(&i8042_lock, flags);
+
+       init_completion(&i8042_aux_irq_delivered);
+       i8042_irq_being_tested = 1;
+
+       param = 0xa5;
+       retval = __i8042_command(&param, I8042_CMD_AUX_LOOP & 0xf0ff);
+
+       spin_unlock_irqrestore(&i8042_lock, flags);
+
+       if (retval)
+               goto out;
+
+       if (wait_for_completion_timeout(&i8042_aux_irq_delivered,
+                                       msecs_to_jiffies(250)) == 0) {
+/*
+ * AUX IRQ was never delivered so we need to flush the controller to
+ * get rid of the byte we put there; otherwise keyboard may not work.
+ */
+               i8042_flush();
+               retval = -1;
+       }
+
+ out:
+
 /*
  * Disable the interface.
  */
@@ -628,47 +655,45 @@ static int __init i8042_check_aux(struct i8042_values *values)
        i8042_ctr &= ~I8042_CTR_AUXINT;
 
        if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR))
-               return -1;
+               retval = -1;
 
-       return 0;
+       if (irq_registered)
+               free_irq(I8042_AUX_IRQ, i8042_platform_device);
+
+       return retval;
 }
 
+static int i8042_controller_check(void)
+{
+       if (i8042_flush() == I8042_BUFFER_SIZE) {
+               printk(KERN_ERR "i8042.c: No controller found.\n");
+               return -ENODEV;
+       }
 
-/*
- * i8042_port_register() marks the device as existing,
- * registers it, and reports to the user.
- */
+       return 0;
+}
 
-static int __init i8042_port_register(struct i8042_values *values, struct serio *port)
+static int i8042_controller_selftest(void)
 {
-       values->exists = 1;
+       unsigned char param;
 
-       i8042_ctr &= ~values->disable;
+       if (!i8042_reset)
+               return 0;
 
-       if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR)) {
-               printk(KERN_WARNING "i8042.c: Can't write CTR while registering.\n");
-               values->exists = 0;
-               return -1;
+       if (i8042_command(&param, I8042_CMD_CTL_TEST)) {
+               printk(KERN_ERR "i8042.c: i8042 controller self test timeout.\n");
+               return -ENODEV;
        }
 
-       printk(KERN_INFO "serio: i8042 %s port at %#lx,%#lx irq %d\n",
-              values->name,
-              (unsigned long) I8042_DATA_REG,
-              (unsigned long) I8042_COMMAND_REG,
-              values->irq);
-
-       serio_register_port(port);
+       if (param != I8042_RET_CTL_TEST) {
+               printk(KERN_ERR "i8042.c: i8042 controller selftest failed. (%#x != %#x)\n",
+                        param, I8042_RET_CTL_TEST);
+               return -EIO;
+       }
 
        return 0;
 }
 
-
-static void i8042_timer_func(unsigned long data)
-{
-       i8042_interrupt(0, NULL, NULL);
-}
-
-
 /*
  * i8042_controller init initializes the i8042 controller, and,
  * most importantly, sets it into non-xlated mode if that's
@@ -677,29 +702,7 @@ static void i8042_timer_func(unsigned long data)
 
 static int i8042_controller_init(void)
 {
-
-/*
- * Test the i8042. We need to know if it thinks it's working correctly
- * before doing anything else.
- */
-
-       i8042_flush();
-
-       if (i8042_reset) {
-
-               unsigned char param;
-
-               if (i8042_command(&param, I8042_CMD_CTL_TEST)) {
-                       printk(KERN_ERR "i8042.c: i8042 controller self test timeout.\n");
-                       return -1;
-               }
-
-               if (param != I8042_RET_CTL_TEST) {
-                       printk(KERN_ERR "i8042.c: i8042 controller selftest failed. (%#x != %#x)\n",
-                                param, I8042_RET_CTL_TEST);
-                       return -1;
-               }
-       }
+       unsigned long flags;
 
 /*
  * Save the CTR for restoral on unload / reboot.
@@ -707,7 +710,7 @@ static int i8042_controller_init(void)
 
        if (i8042_command(&i8042_ctr, I8042_CMD_CTL_RCTR)) {
                printk(KERN_ERR "i8042.c: Can't read CTR while initializing i8042.\n");
-               return -1;
+               return -EIO;
        }
 
        i8042_initial_ctr = i8042_ctr;
@@ -723,12 +726,14 @@ static int i8042_controller_init(void)
  * Handle keylock.
  */
 
+       spin_lock_irqsave(&i8042_lock, flags);
        if (~i8042_read_status() & I8042_STR_KEYLOCK) {
                if (i8042_unlock)
                        i8042_ctr |= I8042_CTR_IGNKEYLOCK;
                 else
                        printk(KERN_WARNING "i8042.c: Warning: Keylock active.\n");
        }
+       spin_unlock_irqrestore(&i8042_lock, flags);
 
 /*
  * If the chip is configured into nontranslated mode by the BIOS, don't
@@ -745,10 +750,8 @@ static int i8042_controller_init(void)
  * BIOSes.
  */
 
-       if (i8042_direct) {
+       if (i8042_direct)
                i8042_ctr &= ~I8042_CTR_XLATE;
-               i8042_kbd_port.type = SERIO_8042;
-       }
 
 /*
  * Write CTR back.
@@ -756,7 +759,7 @@ static int i8042_controller_init(void)
 
        if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR)) {
                printk(KERN_ERR "i8042.c: Can't write CTR while initializing i8042.\n");
-               return -1;
+               return -EIO;
        }
 
        return 0;
@@ -764,24 +767,31 @@ static int i8042_controller_init(void)
 
 
 /*
- * Reset the controller.
+ * Reset the controller and reset CRT to the original value set by BIOS.
  */
-void i8042_controller_reset(void)
+
+static void i8042_controller_reset(void)
 {
-       if (i8042_reset) {
-               unsigned char param;
+       i8042_flush();
 
-               if (i8042_command(&param, I8042_CMD_CTL_TEST))
-                       printk(KERN_ERR "i8042.c: i8042 controller reset timeout.\n");
-       }
+/*
+ * Disable MUX mode if present.
+ */
+
+       if (i8042_mux_present)
+               i8042_set_mux_mode(0, NULL);
 
 /*
- * Restore the original control register setting.
+ * Reset the controller if requested.
  */
 
-       i8042_ctr = i8042_initial_ctr;
+       i8042_controller_selftest();
 
-       if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR))
+/*
+ * Restore the original control register setting.
+ */
+
+       if (i8042_command(&i8042_initial_ctr, I8042_CMD_CTL_WCTR))
                printk(KERN_WARNING "i8042.c: Can't restore CTR.\n");
 }
 
@@ -791,38 +801,75 @@ void i8042_controller_reset(void)
  * able to talk to the hardware when rebooting.
  */
 
-void i8042_controller_cleanup(void)
+static void i8042_controller_cleanup(void)
 {
        int i;
 
-       i8042_flush();
-
 /*
  * Reset anything that is connected to the ports.
  */
 
-       if (i8042_kbd_values.exists)
-               serio_cleanup(&i8042_kbd_port);
+       for (i = 0; i < I8042_NUM_PORTS; i++)
+               if (i8042_ports[i].serio)
+                       serio_cleanup(i8042_ports[i].serio);
 
-       if (i8042_aux_values.exists)
-               serio_cleanup(&i8042_aux_port);
+       i8042_controller_reset();
+}
 
-       for (i = 0; i < 4; i++)
-               if (i8042_mux_values[i].exists)
-                       serio_cleanup(i8042_mux_port + i);
 
-       i8042_controller_reset();
+/*
+ * i8042_panic_blink() will flash the keyboard LEDs and is called when
+ * kernel panics. Flashing LEDs is useful for users running X who may
+ * not see the console and will help distingushing panics from "real"
+ * lockups.
+ *
+ * Note that DELAY has a limit of 10ms so we will not get stuck here
+ * waiting for KBC to free up even if KBD interrupt is off
+ */
+
+#define DELAY do { mdelay(1); if (++delay > 10) return delay; } while(0)
+
+static long i8042_panic_blink(long count)
+{
+       long delay = 0;
+       static long last_blink;
+       static char led;
+
+       /*
+        * We expect frequency to be about 1/2s. KDB uses about 1s.
+        * Make sure they are different.
+        */
+       if (!i8042_blink_frequency)
+               return 0;
+       if (count - last_blink < i8042_blink_frequency)
+               return 0;
+
+       led ^= 0x01 | 0x04;
+       while (i8042_read_status() & I8042_STR_IBF)
+               DELAY;
+       dbg("%02x -> i8042 (panic blink)", 0xed);
+       i8042_suppress_kbd_ack = 2;
+       i8042_write_data(0xed); /* set leds */
+       DELAY;
+       while (i8042_read_status() & I8042_STR_IBF)
+               DELAY;
+       DELAY;
+       dbg("%02x -> i8042 (panic blink)", led);
+       i8042_write_data(led);
+       DELAY;
+       last_blink = count;
+       return delay;
 }
 
+#undef DELAY
 
 /*
  * Here we try to restore the original BIOS settings
  */
 
-static int i8042_controller_suspend(void)
+static int i8042_suspend(struct platform_device *dev, pm_message_t state)
 {
-       del_timer_sync(&i8042_timer);
-       i8042_controller_reset();
+       i8042_controller_cleanup();
 
        return 0;
 }
@@ -832,194 +879,341 @@ static int i8042_controller_suspend(void)
  * Here we try to reset everything back to a state in which suspended
  */
 
-static int i8042_controller_resume(void)
+static int i8042_resume(struct platform_device *dev)
 {
-       int i;
+       int error;
 
-       if (i8042_controller_init()) {
-               printk(KERN_ERR "i8042: resume failed\n");
-               return -1;
-       }
+       error = i8042_controller_check();
+       if (error)
+               return error;
 
-       if (i8042_mux_present)
-               if (i8042_enable_mux_mode(&i8042_aux_values, NULL) ||
-                   i8042_enable_mux_ports(&i8042_aux_values)) {
-                       printk(KERN_WARNING "i8042: failed to resume active multiplexor, mouse won't work.\n");
-               }
+       error = i8042_controller_selftest();
+       if (error)
+               return error;
 
 /*
- * Reconnect anything that was connected to the ports.
+ * Restore pre-resume CTR value and disable all ports
  */
 
-       if (i8042_kbd_values.exists && i8042_activate_port(&i8042_kbd_port) == 0)
-               serio_reconnect(&i8042_kbd_port);
+       i8042_ctr |= I8042_CTR_AUXDIS | I8042_CTR_KBDDIS;
+       i8042_ctr &= ~(I8042_CTR_AUXINT | I8042_CTR_KBDINT);
+       if (i8042_command(&i8042_ctr, I8042_CMD_CTL_WCTR)) {
+               printk(KERN_ERR "i8042: Can't write CTR to resume\n");
+               return -EIO;
+       }
+
+       if (i8042_mux_present) {
+               if (i8042_set_mux_mode(1, NULL) || i8042_enable_mux_ports())
+                       printk(KERN_WARNING
+                               "i8042: failed to resume active multiplexor, "
+                               "mouse won't work.\n");
+       } else if (i8042_ports[I8042_AUX_PORT_NO].serio)
+               i8042_enable_aux_port();
 
-       if (i8042_aux_values.exists && i8042_activate_port(&i8042_aux_port) == 0)
-               serio_reconnect(&i8042_aux_port);
+       if (i8042_ports[I8042_KBD_PORT_NO].serio)
+               i8042_enable_kbd_port();
 
-       for (i = 0; i < 4; i++)
-               if (i8042_mux_values[i].exists && i8042_activate_port(i8042_mux_port + i) == 0)
-                       serio_reconnect(i8042_mux_port + i);
-/*
- * Restart timer (for polling "stuck" data)
- */
-       mod_timer(&i8042_timer, jiffies + I8042_POLL_PERIOD);
+       i8042_interrupt(0, NULL);
 
        return 0;
 }
 
-
 /*
  * We need to reset the 8042 back to original mode on system shutdown,
  * because otherwise BIOSes will be confused.
  */
 
-static int i8042_notify_sys(struct notifier_block *this, unsigned long code,
-                           void *unused)
+static void i8042_shutdown(struct platform_device *dev)
 {
-        if (code == SYS_DOWN || code == SYS_HALT)
-               i8042_controller_cleanup();
-        return NOTIFY_DONE;
+       i8042_controller_cleanup();
 }
 
-static struct notifier_block i8042_notifier=
+static int __devinit i8042_create_kbd_port(void)
 {
-        i8042_notify_sys,
-        NULL,
-        0
-};
+       struct serio *serio;
+       struct i8042_port *port = &i8042_ports[I8042_KBD_PORT_NO];
 
-/*
- * Suspend/resume handlers for the new PM scheme (driver model)
- */
-static int i8042_suspend(struct sys_device *dev, u32 state)
+       serio = kzalloc(sizeof(struct serio), GFP_KERNEL);
+       if (!serio)
+               return -ENOMEM;
+
+       serio->id.type          = i8042_direct ? SERIO_8042 : SERIO_8042_XL;
+       serio->write            = i8042_dumbkbd ? NULL : i8042_kbd_write;
+       serio->start            = i8042_start;
+       serio->stop             = i8042_stop;
+       serio->port_data        = port;
+       serio->dev.parent       = &i8042_platform_device->dev;
+       strlcpy(serio->name, "i8042 KBD port", sizeof(serio->name));
+       strlcpy(serio->phys, I8042_KBD_PHYS_DESC, sizeof(serio->phys));
+
+       port->serio = serio;
+       port->irq = I8042_KBD_IRQ;
+
+       return 0;
+}
+
+static int __devinit i8042_create_aux_port(int idx)
 {
-       return i8042_controller_suspend();
+       struct serio *serio;
+       int port_no = idx < 0 ? I8042_AUX_PORT_NO : I8042_MUX_PORT_NO + idx;
+       struct i8042_port *port = &i8042_ports[port_no];
+
+       serio = kzalloc(sizeof(struct serio), GFP_KERNEL);
+       if (!serio)
+               return -ENOMEM;
+
+       serio->id.type          = SERIO_8042;
+       serio->write            = i8042_aux_write;
+       serio->start            = i8042_start;
+       serio->stop             = i8042_stop;
+       serio->port_data        = port;
+       serio->dev.parent       = &i8042_platform_device->dev;
+       if (idx < 0) {
+               strlcpy(serio->name, "i8042 AUX port", sizeof(serio->name));
+               strlcpy(serio->phys, I8042_AUX_PHYS_DESC, sizeof(serio->phys));
+       } else {
+               snprintf(serio->name, sizeof(serio->name), "i8042 AUX%d port", idx);
+               snprintf(serio->phys, sizeof(serio->phys), I8042_MUX_PHYS_DESC, idx + 1);
+       }
+
+       port->serio = serio;
+       port->mux = idx;
+       port->irq = I8042_AUX_IRQ;
+
+       return 0;
 }
 
-static int i8042_resume(struct sys_device *dev)
+static void __devinit i8042_free_kbd_port(void)
 {
-       return i8042_controller_resume();
+       kfree(i8042_ports[I8042_KBD_PORT_NO].serio);
+       i8042_ports[I8042_KBD_PORT_NO].serio = NULL;
 }
 
-static struct sysdev_class kbc_sysclass = {
-       set_kset_name("i8042"),
-       .suspend = i8042_suspend,
-       .resume = i8042_resume,
-};
+static void __devinit i8042_free_aux_ports(void)
+{
+       int i;
 
-static struct sys_device device_i8042 = {
-       .id     = 0,
-       .cls    = &kbc_sysclass,
-};
+       for (i = I8042_AUX_PORT_NO; i < I8042_NUM_PORTS; i++) {
+               kfree(i8042_ports[i].serio);
+               i8042_ports[i].serio = NULL;
+       }
+}
 
-/*
- * Suspend/resume handler for the old PM scheme (APM)
- */
-static int i8042_pm_callback(struct pm_dev *dev, pm_request_t request, void *dummy)
+static void __devinit i8042_register_ports(void)
 {
-       switch (request) {
-               case PM_SUSPEND:
-                       return i8042_controller_suspend();
+       int i;
 
-               case PM_RESUME:
-                       return i8042_controller_resume();
+       for (i = 0; i < I8042_NUM_PORTS; i++) {
+               if (i8042_ports[i].serio) {
+                       printk(KERN_INFO "serio: %s at %#lx,%#lx irq %d\n",
+                               i8042_ports[i].serio->name,
+                               (unsigned long) I8042_DATA_REG,
+                               (unsigned long) I8042_COMMAND_REG,
+                               i8042_ports[i].irq);
+                       serio_register_port(i8042_ports[i].serio);
+               }
        }
+}
 
-       return 0;
+static void __devinit i8042_unregister_ports(void)
+{
+       int i;
+
+       for (i = 0; i < I8042_NUM_PORTS; i++) {
+               if (i8042_ports[i].serio) {
+                       serio_unregister_port(i8042_ports[i].serio);
+                       i8042_ports[i].serio = NULL;
+               }
+       }
 }
 
-static void __init i8042_init_mux_values(struct i8042_values *values, struct serio *port, int index)
+static void i8042_free_irqs(void)
 {
-       memcpy(port, &i8042_aux_port, sizeof(struct serio));
-       memcpy(values, &i8042_aux_values, sizeof(struct i8042_values));
-       sprintf(i8042_mux_names[index], "i8042 Aux-%d Port", index);
-       sprintf(i8042_mux_phys[index], I8042_MUX_PHYS_DESC, index + 1);
-       sprintf(i8042_mux_short[index], "AUX%d", index);
-       port->name = i8042_mux_names[index];
-       port->phys = i8042_mux_phys[index];
-       port->driver = values;
-       values->name = i8042_mux_short[index];
-       values->mux = index;
+       if (i8042_aux_irq_registered)
+               free_irq(I8042_AUX_IRQ, i8042_platform_device);
+       if (i8042_kbd_irq_registered)
+               free_irq(I8042_KBD_IRQ, i8042_platform_device);
+
+       i8042_aux_irq_registered = i8042_kbd_irq_registered = 0;
 }
 
-int __init i8042_init(void)
+static int __devinit i8042_setup_aux(void)
 {
+       int (*aux_enable)(void);
+       int error;
        int i;
 
-       dbg_init();
+       if (i8042_check_aux())
+               return -ENODEV;
 
-       init_timer(&i8042_timer);
-       i8042_timer.function = i8042_timer_func;
+       if (i8042_nomux || i8042_check_mux()) {
+               error = i8042_create_aux_port(-1);
+               if (error)
+                       goto err_free_ports;
+               aux_enable = i8042_enable_aux_port;
+       } else {
+               for (i = 0; i < I8042_NUM_MUX_PORTS; i++) {
+                       error = i8042_create_aux_port(i);
+                       if (error)
+                               goto err_free_ports;
+               }
+               aux_enable = i8042_enable_mux_ports;
+       }
 
-       if (i8042_platform_init())
-               return -EBUSY;
+       error = request_irq(I8042_AUX_IRQ, i8042_interrupt, IRQF_SHARED,
+                           "i8042", i8042_platform_device);
+       if (error)
+               goto err_free_ports;
 
-       i8042_aux_values.irq = I8042_AUX_IRQ;
-       i8042_kbd_values.irq = I8042_KBD_IRQ;
+       if (aux_enable())
+               goto err_free_irq;
 
-       if (i8042_controller_init())
-               return -ENODEV;
+       i8042_aux_irq_registered = 1;
+       return 0;
 
-       if (i8042_dumbkbd)
-               i8042_kbd_port.write = NULL;
+ err_free_irq:
+       free_irq(I8042_AUX_IRQ, i8042_platform_device);
+ err_free_ports:
+       i8042_free_aux_ports();
+       return error;
+}
 
-       if (!i8042_noaux && !i8042_check_aux(&i8042_aux_values)) {
-               if (!i8042_nomux && !i8042_check_mux(&i8042_aux_values))
-                       for (i = 0; i < 4; i++) {
-                               i8042_init_mux_values(i8042_mux_values + i, i8042_mux_port + i, i);
-                               i8042_port_register(i8042_mux_values + i, i8042_mux_port + i);
-                       }
-               else
-                       i8042_port_register(&i8042_aux_values, &i8042_aux_port);
+static int __devinit i8042_setup_kbd(void)
+{
+       int error;
+
+       error = i8042_create_kbd_port();
+       if (error)
+               return error;
+
+       error = request_irq(I8042_KBD_IRQ, i8042_interrupt, IRQF_SHARED,
+                           "i8042", i8042_platform_device);
+       if (error)
+               goto err_free_port;
+
+       error = i8042_enable_kbd_port();
+       if (error)
+               goto err_free_irq;
+
+       i8042_kbd_irq_registered = 1;
+       return 0;
+
+ err_free_irq:
+       free_irq(I8042_KBD_IRQ, i8042_platform_device);
+ err_free_port:
+       i8042_free_kbd_port();
+       return error;
+}
+
+static int __devinit i8042_probe(struct platform_device *dev)
+{
+       int error;
+
+       error = i8042_controller_selftest();
+       if (error)
+               return error;
+
+       error = i8042_controller_init();
+       if (error)
+               return error;
+
+       if (!i8042_noaux) {
+               error = i8042_setup_aux();
+               if (error && error != -ENODEV && error != -EBUSY)
+                       goto out_fail;
+       }
+
+       if (!i8042_nokbd) {
+               error = i8042_setup_kbd();
+               if (error)
+                       goto out_fail;
        }
 
-       i8042_port_register(&i8042_kbd_values, &i8042_kbd_port);
+/*
+ * Ok, everything is ready, let's register all serio ports
+ */
+       i8042_register_ports();
 
-       mod_timer(&i8042_timer, jiffies + I8042_POLL_PERIOD);
+       return 0;
 
-        if (sysdev_class_register(&kbc_sysclass) == 0) {
-                if (sysdev_register(&device_i8042) == 0)
-                       i8042_sysdev_initialized = 1;
-               else
-                       sysdev_class_unregister(&kbc_sysclass);
-        }
+ out_fail:
+       i8042_free_aux_ports(); /* in case KBD failed but AUX not */
+       i8042_free_irqs();
+       i8042_controller_reset();
 
-       i8042_pm_dev = pm_register(PM_SYS_DEV, PM_SYS_UNKNOWN, i8042_pm_callback);
+       return error;
+}
 
-       register_reboot_notifier(&i8042_notifier);
+static int __devexit i8042_remove(struct platform_device *dev)
+{
+       i8042_unregister_ports();
+       i8042_free_irqs();
+       i8042_controller_reset();
 
        return 0;
 }
 
-void __exit i8042_exit(void)
+static struct platform_driver i8042_driver = {
+       .driver         = {
+               .name   = "i8042",
+               .owner  = THIS_MODULE,
+       },
+       .probe          = i8042_probe,
+       .remove         = __devexit_p(i8042_remove),
+       .suspend        = i8042_suspend,
+       .resume         = i8042_resume,
+       .shutdown       = i8042_shutdown,
+};
+
+static int __init i8042_init(void)
 {
-       int i;
+       int err;
+
+       dbg_init();
 
-       unregister_reboot_notifier(&i8042_notifier);
+       err = i8042_platform_init();
+       if (err)
+               return err;
 
-       if (i8042_pm_dev)
-               pm_unregister(i8042_pm_dev);
+       err = i8042_controller_check();
+       if (err)
+               goto err_platform_exit;
 
-       if (i8042_sysdev_initialized) {
-               sysdev_unregister(&device_i8042);
-               sysdev_class_unregister(&kbc_sysclass);
+       err = platform_driver_register(&i8042_driver);
+       if (err)
+               goto err_platform_exit;
+
+       i8042_platform_device = platform_device_alloc("i8042", -1);
+       if (!i8042_platform_device) {
+               err = -ENOMEM;
+               goto err_unregister_driver;
        }
 
-       i8042_controller_cleanup();
+       err = platform_device_add(i8042_platform_device);
+       if (err)
+               goto err_free_device;
 
-       if (i8042_kbd_values.exists)
-               serio_unregister_port(&i8042_kbd_port);
+       panic_blink = i8042_panic_blink;
 
-       if (i8042_aux_values.exists)
-               serio_unregister_port(&i8042_aux_port);
+       return 0;
 
-       for (i = 0; i < 4; i++)
-               if (i8042_mux_values[i].exists)
-                       serio_unregister_port(i8042_mux_port + i);
-       del_timer_sync(&i8042_timer);
+ err_free_device:
+       platform_device_put(i8042_platform_device);
+ err_unregister_driver:
+       platform_driver_unregister(&i8042_driver);
+ err_platform_exit:
+       i8042_platform_exit();
+
+       return err;
+}
 
+static void __exit i8042_exit(void)
+{
+       platform_device_unregister(i8042_platform_device);
+       platform_driver_unregister(&i8042_driver);
        i8042_platform_exit();
+
+       panic_blink = NULL;
 }
 
 module_init(i8042_init);