Merge branch 'sched-fixes-for-linus' of git://git.kernel.org/pub/scm/linux/kernel...
[pandora-kernel.git] / drivers / input / keyboard / pxa27x_keypad.c
index 43fb63d..6f1516f 100644 (file)
 #include <asm/mach/map.h>
 
 #include <asm/arch/hardware.h>
-#include <asm/arch/pxa-regs.h>
-#include <asm/arch/irqs.h>
 #include <asm/arch/pxa27x_keypad.h>
 
-#define DRIVER_NAME            "pxa27x-keypad"
+/*
+ * Keypad Controller registers
+ */
+#define KPC             0x0000 /* Keypad Control register */
+#define KPDK            0x0008 /* Keypad Direct Key register */
+#define KPREC           0x0010 /* Keypad Rotary Encoder register */
+#define KPMK            0x0018 /* Keypad Matrix Key register */
+#define KPAS            0x0020 /* Keypad Automatic Scan register */
+
+/* Keypad Automatic Scan Multiple Key Presser register 0-3 */
+#define KPASMKP0        0x0028
+#define KPASMKP1        0x0030
+#define KPASMKP2        0x0038
+#define KPASMKP3        0x0040
+#define KPKDI           0x0048
+
+/* bit definitions */
+#define KPC_MKRN(n)    ((((n) - 1) & 0x7) << 26) /* matrix key row number */
+#define KPC_MKCN(n)    ((((n) - 1) & 0x7) << 23) /* matrix key column number */
+#define KPC_DKN(n)     ((((n) - 1) & 0x7) << 6)  /* direct key number */
+
+#define KPC_AS          (0x1 << 30)  /* Automatic Scan bit */
+#define KPC_ASACT       (0x1 << 29)  /* Automatic Scan on Activity */
+#define KPC_MI          (0x1 << 22)  /* Matrix interrupt bit */
+#define KPC_IMKP        (0x1 << 21)  /* Ignore Multiple Key Press */
+
+#define KPC_MS(n)      (0x1 << (13 + (n)))     /* Matrix scan line 'n' */
+#define KPC_MS_ALL      (0xff << 13)
+
+#define KPC_ME          (0x1 << 12)  /* Matrix Keypad Enable */
+#define KPC_MIE         (0x1 << 11)  /* Matrix Interrupt Enable */
+#define KPC_DK_DEB_SEL (0x1 <<  9)  /* Direct Keypad Debounce Select */
+#define KPC_DI          (0x1 <<  5)  /* Direct key interrupt bit */
+#define KPC_RE_ZERO_DEB (0x1 <<  4)  /* Rotary Encoder Zero Debounce */
+#define KPC_REE1        (0x1 <<  3)  /* Rotary Encoder1 Enable */
+#define KPC_REE0        (0x1 <<  2)  /* Rotary Encoder0 Enable */
+#define KPC_DE          (0x1 <<  1)  /* Direct Keypad Enable */
+#define KPC_DIE         (0x1 <<  0)  /* Direct Keypad interrupt Enable */
+
+#define KPDK_DKP        (0x1 << 31)
+#define KPDK_DK(n)     ((n) & 0xff)
+
+#define KPREC_OF1       (0x1 << 31)
+#define kPREC_UF1       (0x1 << 30)
+#define KPREC_OF0       (0x1 << 15)
+#define KPREC_UF0       (0x1 << 14)
+
+#define KPREC_RECOUNT0(n)      ((n) & 0xff)
+#define KPREC_RECOUNT1(n)      (((n) >> 16) & 0xff)
+
+#define KPMK_MKP        (0x1 << 31)
+#define KPAS_SO         (0x1 << 31)
+#define KPASMKPx_SO     (0x1 << 31)
+
+#define KPAS_MUKP(n)   (((n) >> 26) & 0x1f)
+#define KPAS_RP(n)     (((n) >> 4) & 0xf)
+#define KPAS_CP(n)     ((n) & 0xf)
+
+#define KPASMKP_MKC_MASK       (0xff)
+
+#define keypad_readl(off)      __raw_readl(keypad->mmio_base + (off))
+#define keypad_writel(off, v)  __raw_writel((v), keypad->mmio_base + (off))
+
+#define MAX_MATRIX_KEY_NUM     (8 * 8)
+
+struct pxa27x_keypad {
+       struct pxa27x_keypad_platform_data *pdata;
+
+       struct clk *clk;
+       struct input_dev *input_dev;
+       void __iomem *mmio_base;
 
-#define KPASMKP(col)           (col/2 == 0 ? KPASMKP0 : \
-                                col/2 == 1 ? KPASMKP1 : \
-                                col/2 == 2 ? KPASMKP2 : KPASMKP3)
-#define KPASMKPx_MKC(row, col) (1 << (row + 16 * (col % 2)))
+       int irq;
 
-static struct clk *pxa27x_keypad_clk;
+       /* matrix key code map */
+       unsigned int matrix_keycodes[MAX_MATRIX_KEY_NUM];
 
-static irqreturn_t pxa27x_keypad_irq_handler(int irq, void *dev_id)
+       /* state row bits of each column scan */
+       uint32_t matrix_key_state[MAX_MATRIX_KEY_COLS];
+       uint32_t direct_key_state;
+
+       unsigned int direct_key_mask;
+
+       int rotary_rel_code[2];
+       int rotary_up_key[2];
+       int rotary_down_key[2];
+};
+
+static void pxa27x_keypad_build_keycode(struct pxa27x_keypad *keypad)
 {
-       struct platform_device *pdev = dev_id;
-       struct pxa27x_keypad_platform_data *pdata = pdev->dev.platform_data;
-       struct input_dev *input_dev = platform_get_drvdata(pdev);
-       unsigned long kpc = KPC;
-       int p, row, col, rel;
-
-       if (kpc & KPC_DI) {
-               unsigned long kpdk = KPDK;
-
-               if (!(kpdk & KPDK_DKP)) {
-                       /* better luck next time */
-               } else if (kpc & KPC_REE0) {
-                       unsigned long kprec = KPREC;
-                       KPREC = 0x7f;
-
-                       if (kprec & KPREC_OF0)
-                               rel = (kprec & 0xff) + 0x7f;
-                       else if (kprec & KPREC_UF0)
-                               rel = (kprec & 0xff) - 0x7f - 0xff;
-                       else
-                               rel = (kprec & 0xff) - 0x7f;
-
-                       if (rel) {
-                               input_report_rel(input_dev, REL_WHEEL, rel);
-                               input_sync(input_dev);
-                       }
-               }
+       struct pxa27x_keypad_platform_data *pdata = keypad->pdata;
+       struct input_dev *input_dev = keypad->input_dev;
+       unsigned int *key;
+       int i;
+
+       key = &pdata->matrix_key_map[0];
+       for (i = 0; i < pdata->matrix_key_map_size; i++, key++) {
+               int row = ((*key) >> 28) & 0xf;
+               int col = ((*key) >> 24) & 0xf;
+               int code = (*key) & 0xffffff;
+
+               keypad->matrix_keycodes[(row << 3) + col] = code;
+               set_bit(code, input_dev->keybit);
+       }
+
+       for (i = 0; i < pdata->direct_key_num; i++)
+               set_bit(pdata->direct_key_map[i], input_dev->keybit);
+
+       keypad->rotary_up_key[0] = pdata->rotary0_up_key;
+       keypad->rotary_up_key[1] = pdata->rotary1_up_key;
+       keypad->rotary_down_key[0] = pdata->rotary0_down_key;
+       keypad->rotary_down_key[1] = pdata->rotary1_down_key;
+       keypad->rotary_rel_code[0] = pdata->rotary0_rel_code;
+       keypad->rotary_rel_code[1] = pdata->rotary1_rel_code;
+
+       if (pdata->enable_rotary0) {
+               if (pdata->rotary0_up_key && pdata->rotary0_down_key) {
+                       set_bit(pdata->rotary0_up_key, input_dev->keybit);
+                       set_bit(pdata->rotary0_down_key, input_dev->keybit);
+               } else
+                       set_bit(pdata->rotary0_rel_code, input_dev->relbit);
+       }
+
+       if (pdata->enable_rotary1) {
+               if (pdata->rotary1_up_key && pdata->rotary1_down_key) {
+                       set_bit(pdata->rotary1_up_key, input_dev->keybit);
+                       set_bit(pdata->rotary1_down_key, input_dev->keybit);
+               } else
+                       set_bit(pdata->rotary1_rel_code, input_dev->relbit);
+       }
+}
+
+static inline unsigned int lookup_matrix_keycode(
+               struct pxa27x_keypad *keypad, int row, int col)
+{
+       return keypad->matrix_keycodes[(row << 3) + col];
+}
+
+static void pxa27x_keypad_scan_matrix(struct pxa27x_keypad *keypad)
+{
+       struct pxa27x_keypad_platform_data *pdata = keypad->pdata;
+       int row, col, num_keys_pressed = 0;
+       uint32_t new_state[MAX_MATRIX_KEY_COLS];
+       uint32_t kpas = keypad_readl(KPAS);
+
+       num_keys_pressed = KPAS_MUKP(kpas);
+
+       memset(new_state, 0, sizeof(new_state));
+
+       if (num_keys_pressed == 0)
+               goto scan;
+
+       if (num_keys_pressed == 1) {
+               col = KPAS_CP(kpas);
+               row = KPAS_RP(kpas);
+
+               /* if invalid row/col, treat as no key pressed */
+               if (col >= pdata->matrix_key_cols ||
+                   row >= pdata->matrix_key_rows)
+                       goto scan;
+
+               new_state[col] = (1 << row);
+               goto scan;
        }
 
-       if (kpc & KPC_MI) {
-               /* report the status of every button */
-               for (row = 0; row < pdata->nr_rows; row++) {
-                       for (col = 0; col < pdata->nr_cols; col++) {
-                               p = KPASMKP(col) & KPASMKPx_MKC(row, col) ?
-                                       1 : 0;
-                               pr_debug("keycode %x - pressed %x\n",
-                                               pdata->keycodes[row][col], p);
-                               input_report_key(input_dev,
-                                               pdata->keycodes[row][col], p);
-                       }
+       if (num_keys_pressed > 1) {
+               uint32_t kpasmkp0 = keypad_readl(KPASMKP0);
+               uint32_t kpasmkp1 = keypad_readl(KPASMKP1);
+               uint32_t kpasmkp2 = keypad_readl(KPASMKP2);
+               uint32_t kpasmkp3 = keypad_readl(KPASMKP3);
+
+               new_state[0] = kpasmkp0 & KPASMKP_MKC_MASK;
+               new_state[1] = (kpasmkp0 >> 16) & KPASMKP_MKC_MASK;
+               new_state[2] = kpasmkp1 & KPASMKP_MKC_MASK;
+               new_state[3] = (kpasmkp1 >> 16) & KPASMKP_MKC_MASK;
+               new_state[4] = kpasmkp2 & KPASMKP_MKC_MASK;
+               new_state[5] = (kpasmkp2 >> 16) & KPASMKP_MKC_MASK;
+               new_state[6] = kpasmkp3 & KPASMKP_MKC_MASK;
+               new_state[7] = (kpasmkp3 >> 16) & KPASMKP_MKC_MASK;
+       }
+scan:
+       for (col = 0; col < pdata->matrix_key_cols; col++) {
+               uint32_t bits_changed;
+
+               bits_changed = keypad->matrix_key_state[col] ^ new_state[col];
+               if (bits_changed == 0)
+                       continue;
+
+               for (row = 0; row < pdata->matrix_key_rows; row++) {
+                       if ((bits_changed & (1 << row)) == 0)
+                               continue;
+
+                       input_report_key(keypad->input_dev,
+                               lookup_matrix_keycode(keypad, row, col),
+                               new_state[col] & (1 << row));
                }
-               input_sync(input_dev);
        }
+       input_sync(keypad->input_dev);
+       memcpy(keypad->matrix_key_state, new_state, sizeof(new_state));
+}
+
+#define DEFAULT_KPREC  (0x007f007f)
+
+static inline int rotary_delta(uint32_t kprec)
+{
+       if (kprec & KPREC_OF0)
+               return (kprec & 0xff) + 0x7f;
+       else if (kprec & KPREC_UF0)
+               return (kprec & 0xff) - 0x7f - 0xff;
+       else
+               return (kprec & 0xff) - 0x7f;
+}
+
+static void report_rotary_event(struct pxa27x_keypad *keypad, int r, int delta)
+{
+       struct input_dev *dev = keypad->input_dev;
+
+       if (delta == 0)
+               return;
+
+       if (keypad->rotary_up_key[r] && keypad->rotary_down_key[r]) {
+               int keycode = (delta > 0) ? keypad->rotary_up_key[r] :
+                                           keypad->rotary_down_key[r];
+
+               /* simulate a press-n-release */
+               input_report_key(dev, keycode, 1);
+               input_sync(dev);
+               input_report_key(dev, keycode, 0);
+               input_sync(dev);
+       } else {
+               input_report_rel(dev, keypad->rotary_rel_code[r], delta);
+               input_sync(dev);
+       }
+}
+
+static void pxa27x_keypad_scan_rotary(struct pxa27x_keypad *keypad)
+{
+       struct pxa27x_keypad_platform_data *pdata = keypad->pdata;
+       uint32_t kprec;
+
+       /* read and reset to default count value */
+       kprec = keypad_readl(KPREC);
+       keypad_writel(KPREC, DEFAULT_KPREC);
+
+       if (pdata->enable_rotary0)
+               report_rotary_event(keypad, 0, rotary_delta(kprec));
+
+       if (pdata->enable_rotary1)
+               report_rotary_event(keypad, 1, rotary_delta(kprec >> 16));
+}
+
+static void pxa27x_keypad_scan_direct(struct pxa27x_keypad *keypad)
+{
+       struct pxa27x_keypad_platform_data *pdata = keypad->pdata;
+       unsigned int new_state;
+       uint32_t kpdk, bits_changed;
+       int i;
+
+       kpdk = keypad_readl(KPDK);
+
+       if (pdata->enable_rotary0 || pdata->enable_rotary1)
+               pxa27x_keypad_scan_rotary(keypad);
+
+       if (pdata->direct_key_map == NULL)
+               return;
+
+       new_state = KPDK_DK(kpdk) & keypad->direct_key_mask;
+       bits_changed = keypad->direct_key_state ^ new_state;
+
+       if (bits_changed == 0)
+               return;
+
+       for (i = 0; i < pdata->direct_key_num; i++) {
+               if (bits_changed & (1 << i))
+                       input_report_key(keypad->input_dev,
+                                       pdata->direct_key_map[i],
+                                       (new_state & (1 << i)));
+       }
+       input_sync(keypad->input_dev);
+       keypad->direct_key_state = new_state;
+}
+
+static irqreturn_t pxa27x_keypad_irq_handler(int irq, void *dev_id)
+{
+       struct pxa27x_keypad *keypad = dev_id;
+       unsigned long kpc = keypad_readl(KPC);
+
+       if (kpc & KPC_DI)
+               pxa27x_keypad_scan_direct(keypad);
+
+       if (kpc & KPC_MI)
+               pxa27x_keypad_scan_matrix(keypad);
 
        return IRQ_HANDLED;
 }
 
-static int pxa27x_keypad_open(struct input_dev *dev)
+static void pxa27x_keypad_config(struct pxa27x_keypad *keypad)
 {
-       /* Set keypad control register */
-       KPC |= (KPC_ASACT |
-               KPC_MS_ALL |
-               (2 << 6) | KPC_REE0 | KPC_DK_DEB_SEL |
-               KPC_ME | KPC_MIE | KPC_DE | KPC_DIE);
+       struct pxa27x_keypad_platform_data *pdata = keypad->pdata;
+       unsigned int mask = 0, direct_key_num = 0;
+       unsigned long kpc = 0;
+
+       /* enable matrix keys with automatic scan */
+       if (pdata->matrix_key_rows && pdata->matrix_key_cols) {
+               kpc |= KPC_ASACT | KPC_MIE | KPC_ME | KPC_MS_ALL;
+               kpc |= KPC_MKRN(pdata->matrix_key_rows) |
+                      KPC_MKCN(pdata->matrix_key_cols);
+       }
+
+       /* enable rotary key, debounce interval same as direct keys */
+       if (pdata->enable_rotary0) {
+               mask |= 0x03;
+               direct_key_num = 2;
+               kpc |= KPC_REE0;
+       }
+
+       if (pdata->enable_rotary1) {
+               mask |= 0x0c;
+               direct_key_num = 4;
+               kpc |= KPC_REE1;
+       }
+
+       if (pdata->direct_key_num > direct_key_num)
+               direct_key_num = pdata->direct_key_num;
 
-       KPC &= ~KPC_AS;         /* disable automatic scan */
-       KPC &= ~KPC_IMKP;       /* do not ignore multiple keypresses */
+       keypad->direct_key_mask = ((2 << direct_key_num) - 1) & ~mask;
 
-       /* Set rotary count to mid-point value */
-       KPREC = 0x7F;
+       /* enable direct key */
+       if (direct_key_num)
+               kpc |= KPC_DE | KPC_DIE | KPC_DKN(direct_key_num);
+
+       keypad_writel(KPC, kpc | KPC_RE_ZERO_DEB);
+       keypad_writel(KPREC, DEFAULT_KPREC);
+       keypad_writel(KPKDI, pdata->debounce_interval);
+}
+
+static int pxa27x_keypad_open(struct input_dev *dev)
+{
+       struct pxa27x_keypad *keypad = input_get_drvdata(dev);
 
        /* Enable unit clock */
-       clk_enable(pxa27x_keypad_clk);
+       clk_enable(keypad->clk);
+       pxa27x_keypad_config(keypad);
 
        return 0;
 }
 
 static void pxa27x_keypad_close(struct input_dev *dev)
 {
+       struct pxa27x_keypad *keypad = input_get_drvdata(dev);
+
        /* Disable clock unit */
-       clk_disable(pxa27x_keypad_clk);
+       clk_disable(keypad->clk);
 }
 
 #ifdef CONFIG_PM
 static int pxa27x_keypad_suspend(struct platform_device *pdev, pm_message_t state)
 {
-       struct pxa27x_keypad_platform_data *pdata = pdev->dev.platform_data;
+       struct pxa27x_keypad *keypad = platform_get_drvdata(pdev);
+
+       clk_disable(keypad->clk);
 
-       /* Save controller status */
-       pdata->reg_kpc = KPC;
-       pdata->reg_kprec = KPREC;
+       if (device_may_wakeup(&pdev->dev))
+               enable_irq_wake(keypad->irq);
 
        return 0;
 }
 
 static int pxa27x_keypad_resume(struct platform_device *pdev)
 {
-       struct pxa27x_keypad_platform_data *pdata = pdev->dev.platform_data;
-       struct input_dev *input_dev = platform_get_drvdata(pdev);
+       struct pxa27x_keypad *keypad = platform_get_drvdata(pdev);
+       struct input_dev *input_dev = keypad->input_dev;
+
+       if (device_may_wakeup(&pdev->dev))
+               disable_irq_wake(keypad->irq);
 
        mutex_lock(&input_dev->mutex);
 
        if (input_dev->users) {
-               /* Restore controller status */
-               KPC = pdata->reg_kpc;
-               KPREC = pdata->reg_kprec;
-
                /* Enable unit clock */
-               clk_disable(pxa27x_keypad_clk);
-               clk_enable(pxa27x_keypad_clk);
+               clk_enable(keypad->clk);
+               pxa27x_keypad_config(keypad);
        }
 
        mutex_unlock(&input_dev->mutex);
@@ -157,99 +426,161 @@ static int pxa27x_keypad_resume(struct platform_device *pdev)
 #define pxa27x_keypad_resume   NULL
 #endif
 
+#define res_size(res)  ((res)->end - (res)->start + 1)
+
 static int __devinit pxa27x_keypad_probe(struct platform_device *pdev)
 {
-       struct pxa27x_keypad_platform_data *pdata = pdev->dev.platform_data;
+       struct pxa27x_keypad *keypad;
        struct input_dev *input_dev;
-       int i, row, col, error;
+       struct resource *res;
+       int irq, error;
 
-       pxa27x_keypad_clk = clk_get(&pdev->dev, "KBDCLK");
-       if (IS_ERR(pxa27x_keypad_clk)) {
-               error = PTR_ERR(pxa27x_keypad_clk);
-               goto err_clk;
+       keypad = kzalloc(sizeof(struct pxa27x_keypad), GFP_KERNEL);
+       if (keypad == NULL) {
+               dev_err(&pdev->dev, "failed to allocate driver data\n");
+               return -ENOMEM;
+       }
+
+       keypad->pdata = pdev->dev.platform_data;
+       if (keypad->pdata == NULL) {
+               dev_err(&pdev->dev, "no platform data defined\n");
+               error = -EINVAL;
+               goto failed_free;
+       }
+
+       irq = platform_get_irq(pdev, 0);
+       if (irq < 0) {
+               dev_err(&pdev->dev, "failed to get keypad irq\n");
+               error = -ENXIO;
+               goto failed_free;
+       }
+
+       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+       if (res == NULL) {
+               dev_err(&pdev->dev, "failed to get I/O memory\n");
+               error = -ENXIO;
+               goto failed_free;
+       }
+
+       res = request_mem_region(res->start, res_size(res), pdev->name);
+       if (res == NULL) {
+               dev_err(&pdev->dev, "failed to request I/O memory\n");
+               error = -EBUSY;
+               goto failed_free;
+       }
+
+       keypad->mmio_base = ioremap(res->start, res_size(res));
+       if (keypad->mmio_base == NULL) {
+               dev_err(&pdev->dev, "failed to remap I/O memory\n");
+               error = -ENXIO;
+               goto failed_free_mem;
+       }
+
+       keypad->clk = clk_get(&pdev->dev, "KBDCLK");
+       if (IS_ERR(keypad->clk)) {
+               dev_err(&pdev->dev, "failed to get keypad clock\n");
+               error = PTR_ERR(keypad->clk);
+               goto failed_free_io;
        }
 
        /* Create and register the input driver. */
        input_dev = input_allocate_device();
        if (!input_dev) {
-               printk(KERN_ERR "Cannot request keypad device\n");
+               dev_err(&pdev->dev, "failed to allocate input device\n");
                error = -ENOMEM;
-               goto err_alloc;
+               goto failed_put_clk;
        }
 
-       input_dev->name = DRIVER_NAME;
+       input_dev->name = pdev->name;
        input_dev->id.bustype = BUS_HOST;
        input_dev->open = pxa27x_keypad_open;
        input_dev->close = pxa27x_keypad_close;
        input_dev->dev.parent = &pdev->dev;
 
-       input_dev->evbit[0] = BIT_MASK(EV_KEY) | BIT_MASK(EV_REP) |
-               BIT_MASK(EV_REL);
-       input_dev->relbit[BIT_WORD(REL_WHEEL)] = BIT_MASK(REL_WHEEL);
-       for (row = 0; row < pdata->nr_rows; row++) {
-               for (col = 0; col < pdata->nr_cols; col++) {
-                       int code = pdata->keycodes[row][col];
-                       if (code > 0)
-                               set_bit(code, input_dev->keybit);
-               }
+       keypad->input_dev = input_dev;
+       input_set_drvdata(input_dev, keypad);
+
+       input_dev->evbit[0] = BIT_MASK(EV_KEY) | BIT_MASK(EV_REP);
+       if ((keypad->pdata->enable_rotary0 &&
+                       keypad->pdata->rotary0_rel_code) ||
+           (keypad->pdata->enable_rotary1 &&
+                       keypad->pdata->rotary1_rel_code)) {
+               input_dev->evbit[0] |= BIT_MASK(EV_REL);
        }
 
-       error = request_irq(IRQ_KEYPAD, pxa27x_keypad_irq_handler, IRQF_DISABLED,
-                           DRIVER_NAME, pdev);
+       pxa27x_keypad_build_keycode(keypad);
+       platform_set_drvdata(pdev, keypad);
+
+       error = request_irq(irq, pxa27x_keypad_irq_handler, IRQF_DISABLED,
+                           pdev->name, keypad);
        if (error) {
-               printk(KERN_ERR "Cannot request keypad IRQ\n");
-               goto err_free_dev;
+               dev_err(&pdev->dev, "failed to request IRQ\n");
+               goto failed_free_dev;
        }
 
-       platform_set_drvdata(pdev, input_dev);
+       keypad->irq = irq;
 
        /* Register the input device */
        error = input_register_device(input_dev);
-       if (error)
-               goto err_free_irq;
-
-       /*
-        * Store rows/cols info into keyboard registers.
-        */
-
-       KPC |= (pdata->nr_rows - 1) << 26;
-       KPC |= (pdata->nr_cols - 1) << 23;
+       if (error) {
+               dev_err(&pdev->dev, "failed to register input device\n");
+               goto failed_free_irq;
+       }
 
-       for (col = 0; col < pdata->nr_cols; col++)
-               KPC |= KPC_MS0 << col;
+       device_init_wakeup(&pdev->dev, 1);
 
        return 0;
 
- err_free_irq:
+failed_free_irq:
+       free_irq(irq, pdev);
        platform_set_drvdata(pdev, NULL);
-       free_irq(IRQ_KEYPAD, pdev);
- err_free_dev:
+failed_free_dev:
        input_free_device(input_dev);
- err_alloc:
-       clk_put(pxa27x_keypad_clk);
- err_clk:
+failed_put_clk:
+       clk_put(keypad->clk);
+failed_free_io:
+       iounmap(keypad->mmio_base);
+failed_free_mem:
+       release_mem_region(res->start, res_size(res));
+failed_free:
+       kfree(keypad);
        return error;
 }
 
 static int __devexit pxa27x_keypad_remove(struct platform_device *pdev)
 {
-       struct input_dev *input_dev = platform_get_drvdata(pdev);
+       struct pxa27x_keypad *keypad = platform_get_drvdata(pdev);
+       struct resource *res;
 
-       input_unregister_device(input_dev);
-       free_irq(IRQ_KEYPAD, pdev);
-       clk_put(pxa27x_keypad_clk);
-       platform_set_drvdata(pdev, NULL);
+       free_irq(keypad->irq, pdev);
 
+       clk_disable(keypad->clk);
+       clk_put(keypad->clk);
+
+       input_unregister_device(keypad->input_dev);
+       input_free_device(keypad->input_dev);
+
+       iounmap(keypad->mmio_base);
+
+       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+       release_mem_region(res->start, res_size(res));
+
+       platform_set_drvdata(pdev, NULL);
+       kfree(keypad);
        return 0;
 }
 
+/* work with hotplug and coldplug */
+MODULE_ALIAS("platform:pxa27x-keypad");
+
 static struct platform_driver pxa27x_keypad_driver = {
        .probe          = pxa27x_keypad_probe,
        .remove         = __devexit_p(pxa27x_keypad_remove),
        .suspend        = pxa27x_keypad_suspend,
        .resume         = pxa27x_keypad_resume,
        .driver         = {
-               .name   = DRIVER_NAME,
+               .name   = "pxa27x-keypad",
+               .owner  = THIS_MODULE,
        },
 };