* [PATCH 1/2] OMAP: Add keypad driver #3
@ 2006-07-27 7:09 Komal Shah
2006-07-27 8:53 ` Russell King
0 siblings, 1 reply; 3+ messages in thread
From: Komal Shah @ 2006-07-27 7:09 UTC (permalink / raw)
To: akpm, juha.yrjola, tony, ext-timo.teras
Cc: r-woodruff2, linux-input, linux-kernel, dbrownell, kjh
[-- Attachment #1: Type: text/plain, Size: 329 bytes --]
Andrew/Tony/Richard/Dmitry,
This is a revised patch as per the review comments from the Dmitry
on thread:
http://lkml.org/lkml/2006/7/26/136
Please review it and give me the Ack if looks ok.
---Komal Shah
http://komalshah.blogspot.com
--
http://www.fastmail.fm - And now for something completely different
[-- Attachment #2: 0001-OMAP-Add-keypad-driver.patch --]
[-- Type: application/octet-stream, Size: 15242 bytes --]
From nobody Mon Sep 17 00:00:00 2001
From: Komal Shah <komal_shah802003@yahoo.com>
Date: Thu, 27 Jul 2006 18:03:23 +0530
Subject: [PATCH 1/2] OMAP: Add keypad driver
This patch adds support for keypad driver running on different
TI OMAP(http://www.ti.com/omap) processor based boards like
OSK, H2, H3, H4, Persuas and Nokia 770.
Signed-off-by: Komal Shah <komal_shah802003@yahoo.com>
---
drivers/input/keyboard/Kconfig | 9 +
drivers/input/keyboard/Makefile | 1
drivers/input/keyboard/omap-keypad.c | 491 ++++++++++++++++++++++++++++++++++
include/asm-arm/arch-omap/keypad.h | 1
4 files changed, 502 insertions(+), 0 deletions(-)
create mode 100644 drivers/input/keyboard/omap-keypad.c
57aadad2a2482f971374563778d00d071e9a0963
diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
index a9dda56..83eac3a 100644
--- a/drivers/input/keyboard/Kconfig
+++ b/drivers/input/keyboard/Kconfig
@@ -183,4 +183,13 @@ config KEYBOARD_HIL
This driver implements support for HIL-keyboards attached
to your machine, so normally you should say Y here.
+config KEYBOARD_OMAP
+ tristate "TI OMAP keypad support"
+ depends on (ARCH_OMAP1 || ARCH_OMAP2)
+ help
+ Say Y here if you want to use the OMAP keypad.
+
+ To compile this driver as a module, choose M here: the
+ module will be called omap-keypad.
+
endif
diff --git a/drivers/input/keyboard/Makefile b/drivers/input/keyboard/Makefile
index 2708167..b265391 100644
--- a/drivers/input/keyboard/Makefile
+++ b/drivers/input/keyboard/Makefile
@@ -15,4 +15,5 @@ obj-$(CONFIG_KEYBOARD_CORGI) += corgikb
obj-$(CONFIG_KEYBOARD_SPITZ) += spitzkbd.o
obj-$(CONFIG_KEYBOARD_HIL) += hil_kbd.o
obj-$(CONFIG_KEYBOARD_HIL_OLD) += hilkbd.o
+obj-$(CONFIG_KEYBOARD_OMAP) += omap-keypad.o
diff --git a/drivers/input/keyboard/omap-keypad.c b/drivers/input/keyboard/omap-keypad.c
new file mode 100644
index 0000000..3f880c6
--- /dev/null
+++ b/drivers/input/keyboard/omap-keypad.c
@@ -0,0 +1,491 @@
+/*
+ * linux/drivers/input/keyboard/omap-keypad.c
+ *
+ * OMAP Keypad Driver
+ *
+ * Copyright (C) 2003 Nokia Corporation
+ * Written by Timo Teräs <ext-timo.teras@nokia.com>
+ *
+ * Added support for H2 & H3 Keypad
+ * Copyright (C) 2004 Texas Instruments
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#include <linux/module.h>
+#include <linux/init.h>
+#include <linux/interrupt.h>
+#include <linux/types.h>
+#include <linux/input.h>
+#include <linux/kernel.h>
+#include <linux/delay.h>
+#include <linux/platform_device.h>
+#include <linux/mutex.h>
+#include <asm/arch/irqs.h>
+#include <asm/arch/gpio.h>
+#include <asm/arch/hardware.h>
+#include <asm/arch/keypad.h>
+#include <asm/arch/menelaus.h>
+#include <asm/io.h>
+#include <asm/errno.h>
+#include <asm/mach-types.h>
+#include <asm/arch/mux.h>
+
+#define NEW_BOARD_LEARNING_MODE
+
+static void omap_kp_tasklet(unsigned long);
+static void omap_kp_timer(unsigned long);
+
+static unsigned char keypad_state[8];
+static DEFINE_MUTEX(kp_enable_mutex);
+static int kp_enable = 1;
+static int kp_cur_group = -1;
+
+struct omap_kp {
+ struct input_dev *input;
+ struct timer_list timer;
+ int irq;
+ unsigned int rows;
+ unsigned int cols;
+};
+
+DECLARE_TASKLET_DISABLED(kp_tasklet, omap_kp_tasklet, 0);
+
+static int *keymap;
+static unsigned int *row_gpios;
+static unsigned int *col_gpios;
+
+#ifdef CONFIG_ARCH_OMAP2
+static void set_col_gpio_val(struct omap_kp *omap_kp, u8 value)
+{
+ int col;
+ for (col = 0; col < omap_kp->cols; col++) {
+ if (value & (1 << col))
+ omap_set_gpio_dataout(col_gpios[col], 1);
+ else
+ omap_set_gpio_dataout(col_gpios[col], 0);
+ }
+}
+
+static u8 get_row_gpio_val(struct omap_kp *omap_kp)
+{
+ int row;
+ u8 value = 0;
+
+ for (row = 0; row < omap_kp->rows; row++) {
+ if (omap_get_gpio_datain(row_gpios[row]))
+ value |= (1 << row);
+ }
+ return value;
+}
+#else
+#define set_col_gpio_val(x, y) do {} while (0)
+#define get_row_gpio_val(x) 0
+#endif
+
+static irqreturn_t omap_kp_interrupt(int irq, void *dev_id,
+ struct pt_regs *regs)
+{
+ struct omap_kp *omap_kp = dev_id;
+
+ /* disable keyboard interrupt and schedule for handling */
+ if (cpu_is_omap24xx()) {
+ int i;
+ for (i = 0; i < omap_kp->rows; i++)
+ disable_irq(OMAP_GPIO_IRQ(row_gpios[i]));
+ } else
+ /* disable keyboard interrupt and schedule for handling */
+ omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
+
+ tasklet_schedule(&kp_tasklet);
+
+ return IRQ_HANDLED;
+}
+
+static void omap_kp_timer(unsigned long data)
+{
+ tasklet_schedule(&kp_tasklet);
+}
+
+static void omap_kp_scan_keypad(struct omap_kp *omap_kp, unsigned char *state)
+{
+ int col = 0;
+
+ /* read the keypad status */
+ if (cpu_is_omap24xx()) {
+ int i;
+ for (i = 0; i < omap_kp->rows; i++)
+ disable_irq(OMAP_GPIO_IRQ(row_gpios[i]));
+ } else
+ /* disable keyboard interrupt and schedule for handling */
+ omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
+
+ if (!cpu_is_omap24xx()) {
+ /* read the keypad status */
+ omap_writew(0xff, OMAP_MPUIO_BASE + OMAP_MPUIO_KBC);
+ for (col = 0; col < omap_kp->cols; col++) {
+ omap_writew(~(1 << col) & 0xff,
+ OMAP_MPUIO_BASE + OMAP_MPUIO_KBC);
+
+ if (machine_is_omap_osk() || machine_is_omap_h2()
+ || machine_is_omap_h3())
+ udelay(9);
+ else
+ udelay(4);
+ state[col] = ~omap_readw(OMAP_MPUIO_BASE +
+ OMAP_MPUIO_KBR_LATCH) & 0xff;
+ }
+ omap_writew(0x00, OMAP_MPUIO_BASE + OMAP_MPUIO_KBC);
+ udelay(2);
+ } else {
+ /* read the keypad status */
+ for (col = 0; col < omap_kp->cols; col++) {
+ set_col_gpio_val(omap_kp, ~(1 << col));
+ state[col] = ~(get_row_gpio_val(omap_kp)) & 0x3f;
+ }
+ set_col_gpio_val(omap_kp, 0);
+ }
+}
+
+static inline int omap_kp_find_key(int col, int row)
+{
+ int i, key;
+
+ key = KEY(col, row, 0);
+ for (i = 0; keymap[i] != 0; i++)
+ if ((keymap[i] & 0xff000000) == key)
+ return keymap[i] & 0x00ffffff;
+ return -1;
+}
+
+static void omap_kp_tasklet(unsigned long data)
+{
+ struct omap_kp *omap_kp_data = (struct omap_kp *) data;
+ unsigned char new_state[8], changed, key_down = 0;
+ int col, row;
+ int spurious = 0;
+
+ /* check for any changes */
+ omap_kp_scan_keypad(omap_kp_data, new_state);
+
+ /* check for changes and print those */
+ for (col = 0; col < omap_kp_data->cols; col++) {
+ changed = new_state[col] ^ keypad_state[col];
+ key_down |= new_state[col];
+ if (changed == 0)
+ continue;
+
+ for (row = 0; row < omap_kp_data->rows; row++) {
+ int key;
+ if (!(changed & (1 << row)))
+ continue;
+#ifdef NEW_BOARD_LEARNING_MODE
+ printk(KERN_INFO "omap-keypad: key %d-%d %s\n", col,
+ row, (new_state[col] & (1 << row)) ?
+ "pressed" : "released");
+#else
+ key = omap_kp_find_key(col, row);
+ if (key < 0) {
+ printk(KERN_WARNING
+ "omap-keypad: Spurious key event %d-%d\n",
+ col, row);
+ /* We scan again after a couple of seconds */
+ spurious = 1;
+ continue;
+ }
+
+ if (!(kp_cur_group == (key & GROUP_MASK) ||
+ kp_cur_group == -1))
+ continue;
+
+ kp_cur_group = key & GROUP_MASK;
+ input_report_key(omap_kp_data->input, key & ~GROUP_MASK,
+ new_state[col] & (1 << row));
+#endif
+ }
+ }
+ memcpy(keypad_state, new_state, sizeof(keypad_state));
+
+ if (key_down) {
+ int delay = HZ / 20;
+ /* some key is pressed - keep irq disabled and use timer
+ * to poll the keypad */
+ if (spurious)
+ delay = 2 * HZ;
+ mod_timer(&omap_kp_data->timer, jiffies + delay);
+ } else {
+ /* enable interrupts */
+ if (cpu_is_omap24xx()) {
+ int i;
+ for (i = 0; i < omap_kp_data->rows; i++)
+ enable_irq(OMAP_GPIO_IRQ(row_gpios[i]));
+ } else {
+ omap_writew(0, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
+ kp_cur_group = -1;
+ }
+ }
+}
+
+static ssize_t omap_kp_enable_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
+{
+ return sprintf(buf, "%u\n", kp_enable);
+}
+
+static ssize_t omap_kp_enable_store(struct device *dev, struct device_attribute *attr,
+ const char *buf, size_t count)
+{
+ int state;
+
+ if (sscanf(buf, "%u", &state) != 1)
+ return -EINVAL;
+
+ if ((state != 1) && (state != 0))
+ return -EINVAL;
+
+ mutex_lock(&kp_enable_mutex);
+ if (state != kp_enable) {
+ if (state)
+ enable_irq(INT_KEYBOARD);
+ else
+ disable_irq(INT_KEYBOARD);
+ kp_enable = state;
+ }
+ mutex_unlock(&kp_enable_mutex);
+
+ return strnlen(buf, count);
+}
+
+static DEVICE_ATTR(enable, S_IRUGO | S_IWUSR, omap_kp_enable_show, omap_kp_enable_store);
+
+#ifdef CONFIG_PM
+static int omap_kp_suspend(struct platform_device *dev, pm_message_t state)
+{
+ /* Nothing yet */
+
+ return 0;
+}
+
+static int omap_kp_resume(struct platform_device *dev)
+{
+ /* Nothing yet */
+
+ return 0;
+}
+#else
+#define omap_kp_suspend NULL
+#define omap_kp_resume NULL
+#endif
+
+static int __init omap_kp_probe(struct platform_device *pdev)
+{
+ struct omap_kp *omap_kp;
+ struct input_dev *input_dev;
+ struct omap_kp_platform_data *pdata = pdev->dev.platform_data;
+ int i, col_idx, row_idx, irq_idx, ret;
+
+ if (!pdata->rows || !pdata->cols || !pdata->keymap) {
+ printk(KERN_ERR "No rows, cols or keymap from pdata\n");
+ return -EINVAL;
+ }
+
+ omap_kp = kzalloc(sizeof(struct omap_kp), GFP_KERNEL);
+ input_dev = input_allocate_device();
+ if (!omap_kp || !input_dev) {
+ kfree(omap_kp);
+ input_free_device(input_dev);
+ return -ENOMEM;
+ }
+
+ platform_set_drvdata(pdev, omap_kp);
+
+ omap_kp->input = input_dev;
+
+ /* Disable the interrupt for the MPUIO keyboard */
+ if (!cpu_is_omap24xx())
+ omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
+
+ keymap = pdata->keymap;
+
+ if (pdata->rep)
+ set_bit(EV_REP, input_dev->evbit);
+
+ if (pdata->row_gpios && pdata->col_gpios) {
+ row_gpios = pdata->row_gpios;
+ col_gpios = pdata->col_gpios;
+ }
+
+ omap_kp->rows = pdata->rows;
+ omap_kp->cols = pdata->cols;
+
+ if (cpu_is_omap24xx()) {
+ /* Cols: outputs */
+ for (col_idx = 0; col_idx < omap_kp->cols; col_idx++) {
+ if (omap_request_gpio(col_gpios[col_idx]) < 0) {
+ printk(KERN_ERR "Failed to request"
+ "GPIO%d for keypad\n",
+ col_gpios[col_idx]);
+ goto err1;
+ }
+ omap_set_gpio_direction(col_gpios[col_idx], 0);
+ }
+ /* Rows: inputs */
+ for (row_idx = 0; row_idx < omap_kp->rows; row_idx++) {
+ if (omap_request_gpio(row_gpios[row_idx]) < 0) {
+ printk(KERN_ERR "Failed to request"
+ "GPIO%d for keypad\n",
+ row_gpios[row_idx]);
+ goto err2;
+ }
+ omap_set_gpio_direction(row_gpios[row_idx], 1);
+ }
+ }
+
+ setup_timer(&omap_kp->timer, omap_kp_timer, (unsigned long)omap_kp);
+
+ /* get the irq and init timer*/
+ tasklet_enable(&kp_tasklet);
+ kp_tasklet.data = (unsigned long) omap_kp;
+
+ ret = device_create_file(&pdev->dev, &dev_attr_enable);
+ if (ret < 0)
+ goto err2;
+
+ /* setup input device */
+ set_bit(EV_KEY, input_dev->evbit);
+ for (i = 0; keymap[i] != 0; i++)
+ set_bit(keymap[i] & KEY_MAX, input_dev->keybit);
+ input_dev->name = "omap-keypad";
+ input_dev->phys = "omap-keypad/input0";
+ input_dev->cdev.dev = &pdev->dev;
+ input_dev->private = omap_kp;
+
+ input_dev->id.bustype = BUS_HOST;
+ input_dev->id.vendor = 0x0001;
+ input_dev->id.product = 0x0001;
+ input_dev->id.version = 0x0100;
+
+ input_dev->keycode = keymap;
+ input_dev->keycodesize = sizeof(unsigned int);
+ input_dev->keycodemax = pdata->keymapsize;
+
+ ret = input_register_device(omap_kp->input);
+ if (ret < 0) {
+ printk(KERN_ERR "Unable to register omap-keypad input device\n");
+ goto err3;
+ }
+
+ if (machine_is_omap_h2() || machine_is_omap_h3() ||
+ machine_is_omap_perseus2()) {
+ omap_writew(0xff, OMAP_MPUIO_BASE + OMAP_MPUIO_GPIO_DEBOUNCING);
+ }
+ /* scan current status and enable interrupt */
+ omap_kp_scan_keypad(omap_kp, keypad_state);
+ if (!cpu_is_omap24xx()) {
+ omap_kp->irq = platform_get_irq(pdev, 0);
+ if (omap_kp->irq >= 0) {
+ if (request_irq(omap_kp->irq, omap_kp_interrupt, 0,
+ "omap-keypad", omap_kp) < 0)
+ goto err4;
+ }
+ omap_writew(0, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
+ } else {
+ for (irq_idx = 0; irq_idx < omap_kp->rows; irq_idx++) {
+ if (request_irq(OMAP_GPIO_IRQ(row_gpios[irq_idx]),
+ omap_kp_interrupt,
+ IRQF_TRIGGER_FALLING,
+ "omap-keypad", omap_kp) < 0)
+ goto err5;
+ }
+ }
+ return 0;
+err5:
+ for (i = irq_idx-1; i >=0; i--)
+ free_irq(row_gpios[i], 0);
+err4:
+ input_unregister_device(omap_kp->input);
+ input_dev = NULL;
+err3:
+ device_remove_file(&pdev->dev, &dev_attr_enable);
+err2:
+ for (i = row_idx-1; i >=0; i--)
+ omap_free_gpio(row_gpios[i]);
+err1:
+ for (i = col_idx-1; i >=0; i--)
+ omap_free_gpio(col_gpios[i]);
+
+ kfree(omap_kp);
+ input_free_device(input_dev);
+
+ return -EINVAL;
+}
+
+static int omap_kp_remove(struct platform_device *pdev)
+{
+ struct omap_kp *omap_kp = platform_get_drvdata(pdev);
+
+ /* disable keypad interrupt handling */
+ tasklet_disable(&kp_tasklet);
+ if (cpu_is_omap24xx()) {
+ int i;
+ for (i = 0; i < omap_kp->cols; i++)
+ omap_free_gpio(col_gpios[i]);
+ for (i = 0; i < omap_kp->rows; i++) {
+ omap_free_gpio(row_gpios[i]);
+ free_irq(OMAP_GPIO_IRQ(row_gpios[i]), 0);
+ }
+ } else {
+ omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
+ free_irq(omap_kp->irq, 0);
+ }
+
+ del_timer_sync(&omap_kp->timer);
+ tasklet_kill(&kp_tasklet);
+
+ /* unregister everything */
+ input_unregister_device(omap_kp->input);
+
+ kfree(omap_kp);
+
+ return 0;
+}
+
+static struct platform_driver omap_kp_driver = {
+ .probe = omap_kp_probe,
+ .remove = omap_kp_remove,
+ .suspend = omap_kp_suspend,
+ .resume = omap_kp_resume,
+ .driver = {
+ .name = "omap-keypad",
+ },
+};
+
+static int __devinit omap_kp_init(void)
+{
+ printk(KERN_INFO "OMAP Keypad Driver\n");
+ return platform_driver_register(&omap_kp_driver);
+}
+
+static void __exit omap_kp_exit(void)
+{
+ platform_driver_unregister(&omap_kp_driver);
+}
+
+module_init(omap_kp_init);
+module_exit(omap_kp_exit);
+
+MODULE_AUTHOR("Timo Teräs");
+MODULE_DESCRIPTION("OMAP Keypad Driver");
+MODULE_LICENSE("GPL");
diff --git a/include/asm-arm/arch-omap/keypad.h b/include/asm-arm/arch-omap/keypad.h
index 8a023a9..800fb19 100644
--- a/include/asm-arm/arch-omap/keypad.h
+++ b/include/asm-arm/arch-omap/keypad.h
@@ -14,6 +14,7 @@ struct omap_kp_platform_data {
int rows;
int cols;
int *keymap;
+ unsigned int keymapsize;
unsigned int rep:1;
/* specific to OMAP242x*/
unsigned int *row_gpios;
--
1.3.3
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH 1/2] OMAP: Add keypad driver #3
2006-07-27 7:09 [PATCH 1/2] OMAP: Add keypad driver #3 Komal Shah
@ 2006-07-27 8:53 ` Russell King
2006-07-27 9:26 ` Komal Shah
0 siblings, 1 reply; 3+ messages in thread
From: Russell King @ 2006-07-27 8:53 UTC (permalink / raw)
To: Komal Shah
Cc: akpm, juha.yrjola, tony, ext-timo.teras, r-woodruff2, linux-input,
linux-kernel, dbrownell, kjh
On Thu, Jul 27, 2006 at 12:09:04AM -0700, Komal Shah wrote:
> diff --git a/drivers/input/keyboard/omap-keypad.c b/drivers/input/keyboard/omap-keypad.c
> new file mode 100644
> index 0000000..3f880c6
> --- /dev/null
> +++ b/drivers/input/keyboard/omap-keypad.c
> @@ -0,0 +1,491 @@
> +/*
> + * linux/drivers/input/keyboard/omap-keypad.c
> + *
> + * OMAP Keypad Driver
> + *
> + * Copyright (C) 2003 Nokia Corporation
> + * Written by Timo Ter?s <ext-timo.teras@nokia.com>
> + *
> + * Added support for H2 & H3 Keypad
> + * Copyright (C) 2004 Texas Instruments
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
> + */
> +
> +#include <linux/module.h>
> +#include <linux/init.h>
> +#include <linux/interrupt.h>
> +#include <linux/types.h>
> +#include <linux/input.h>
> +#include <linux/kernel.h>
> +#include <linux/delay.h>
> +#include <linux/platform_device.h>
> +#include <linux/mutex.h>
> +#include <asm/arch/irqs.h>
Please don't include asm/arch/irqs.h directly - use asm/irq.h instead.
> +#include <asm/arch/gpio.h>
> +#include <asm/arch/hardware.h>
Please use asm/hardware.h instead.
> +#include <asm/arch/keypad.h>
> +#include <asm/arch/menelaus.h>
> +#include <asm/io.h>
> +#include <asm/errno.h>
Should be linux/errno.h.
> +#include <asm/mach-types.h>
> +#include <asm/arch/mux.h>
> +
> +#define NEW_BOARD_LEARNING_MODE
> +
> +static void omap_kp_tasklet(unsigned long);
> +static void omap_kp_timer(unsigned long);
> +
> +static unsigned char keypad_state[8];
> +static DEFINE_MUTEX(kp_enable_mutex);
> +static int kp_enable = 1;
> +static int kp_cur_group = -1;
> +
> +struct omap_kp {
> + struct input_dev *input;
> + struct timer_list timer;
> + int irq;
> + unsigned int rows;
> + unsigned int cols;
> +};
> +
> +DECLARE_TASKLET_DISABLED(kp_tasklet, omap_kp_tasklet, 0);
> +
> +static int *keymap;
> +static unsigned int *row_gpios;
> +static unsigned int *col_gpios;
> +
> +#ifdef CONFIG_ARCH_OMAP2
> +static void set_col_gpio_val(struct omap_kp *omap_kp, u8 value)
> +{
> + int col;
> + for (col = 0; col < omap_kp->cols; col++) {
> + if (value & (1 << col))
> + omap_set_gpio_dataout(col_gpios[col], 1);
> + else
> + omap_set_gpio_dataout(col_gpios[col], 0);
> + }
> +}
> +
> +static u8 get_row_gpio_val(struct omap_kp *omap_kp)
> +{
> + int row;
> + u8 value = 0;
> +
> + for (row = 0; row < omap_kp->rows; row++) {
> + if (omap_get_gpio_datain(row_gpios[row]))
> + value |= (1 << row);
> + }
> + return value;
> +}
> +#else
> +#define set_col_gpio_val(x, y) do {} while (0)
> +#define get_row_gpio_val(x) 0
> +#endif
> +
> +static irqreturn_t omap_kp_interrupt(int irq, void *dev_id,
> + struct pt_regs *regs)
> +{
> + struct omap_kp *omap_kp = dev_id;
> +
> + /* disable keyboard interrupt and schedule for handling */
> + if (cpu_is_omap24xx()) {
> + int i;
> + for (i = 0; i < omap_kp->rows; i++)
> + disable_irq(OMAP_GPIO_IRQ(row_gpios[i]));
> + } else
> + /* disable keyboard interrupt and schedule for handling */
> + omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
> +
> + tasklet_schedule(&kp_tasklet);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static void omap_kp_timer(unsigned long data)
> +{
> + tasklet_schedule(&kp_tasklet);
> +}
> +
> +static void omap_kp_scan_keypad(struct omap_kp *omap_kp, unsigned char *state)
> +{
> + int col = 0;
> +
> + /* read the keypad status */
> + if (cpu_is_omap24xx()) {
> + int i;
> + for (i = 0; i < omap_kp->rows; i++)
> + disable_irq(OMAP_GPIO_IRQ(row_gpios[i]));
> + } else
> + /* disable keyboard interrupt and schedule for handling */
> + omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
> +
> + if (!cpu_is_omap24xx()) {
This seems obfuscated. It would be trivial to combine these two if()
clauses.
And a general note about the omap24xx vs !omap24xx differences in this
file - would it make more sense for code readability to have two
completely separate drivers?
> + /* read the keypad status */
> + omap_writew(0xff, OMAP_MPUIO_BASE + OMAP_MPUIO_KBC);
> + for (col = 0; col < omap_kp->cols; col++) {
> + omap_writew(~(1 << col) & 0xff,
> + OMAP_MPUIO_BASE + OMAP_MPUIO_KBC);
> +
> + if (machine_is_omap_osk() || machine_is_omap_h2()
> + || machine_is_omap_h3())
> + udelay(9);
> + else
> + udelay(4);
Wouldn't it be better to pass this via the platform device driver? It
seems likely that other delays may be required with differing hardware.
> + state[col] = ~omap_readw(OMAP_MPUIO_BASE +
> + OMAP_MPUIO_KBR_LATCH) & 0xff;
> + }
> + omap_writew(0x00, OMAP_MPUIO_BASE + OMAP_MPUIO_KBC);
> + udelay(2);
> + } else {
> + /* read the keypad status */
> + for (col = 0; col < omap_kp->cols; col++) {
> + set_col_gpio_val(omap_kp, ~(1 << col));
> + state[col] = ~(get_row_gpio_val(omap_kp)) & 0x3f;
> + }
> + set_col_gpio_val(omap_kp, 0);
> + }
> +}
> +
> +static inline int omap_kp_find_key(int col, int row)
> +{
> + int i, key;
> +
> + key = KEY(col, row, 0);
> + for (i = 0; keymap[i] != 0; i++)
> + if ((keymap[i] & 0xff000000) == key)
> + return keymap[i] & 0x00ffffff;
> + return -1;
> +}
> +
> +static void omap_kp_tasklet(unsigned long data)
> +{
> + struct omap_kp *omap_kp_data = (struct omap_kp *) data;
> + unsigned char new_state[8], changed, key_down = 0;
> + int col, row;
> + int spurious = 0;
> +
> + /* check for any changes */
> + omap_kp_scan_keypad(omap_kp_data, new_state);
> +
> + /* check for changes and print those */
> + for (col = 0; col < omap_kp_data->cols; col++) {
> + changed = new_state[col] ^ keypad_state[col];
> + key_down |= new_state[col];
> + if (changed == 0)
> + continue;
> +
> + for (row = 0; row < omap_kp_data->rows; row++) {
> + int key;
> + if (!(changed & (1 << row)))
> + continue;
> +#ifdef NEW_BOARD_LEARNING_MODE
> + printk(KERN_INFO "omap-keypad: key %d-%d %s\n", col,
> + row, (new_state[col] & (1 << row)) ?
> + "pressed" : "released");
> +#else
> + key = omap_kp_find_key(col, row);
> + if (key < 0) {
> + printk(KERN_WARNING
> + "omap-keypad: Spurious key event %d-%d\n",
> + col, row);
> + /* We scan again after a couple of seconds */
> + spurious = 1;
> + continue;
> + }
> +
> + if (!(kp_cur_group == (key & GROUP_MASK) ||
> + kp_cur_group == -1))
> + continue;
> +
> + kp_cur_group = key & GROUP_MASK;
> + input_report_key(omap_kp_data->input, key & ~GROUP_MASK,
> + new_state[col] & (1 << row));
> +#endif
> + }
> + }
> + memcpy(keypad_state, new_state, sizeof(keypad_state));
> +
> + if (key_down) {
> + int delay = HZ / 20;
> + /* some key is pressed - keep irq disabled and use timer
> + * to poll the keypad */
> + if (spurious)
> + delay = 2 * HZ;
> + mod_timer(&omap_kp_data->timer, jiffies + delay);
> + } else {
> + /* enable interrupts */
> + if (cpu_is_omap24xx()) {
> + int i;
> + for (i = 0; i < omap_kp_data->rows; i++)
> + enable_irq(OMAP_GPIO_IRQ(row_gpios[i]));
> + } else {
> + omap_writew(0, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
> + kp_cur_group = -1;
> + }
> + }
> +}
> +
> +static ssize_t omap_kp_enable_show(struct device *dev,
> + struct device_attribute *attr, char *buf)
> +{
> + return sprintf(buf, "%u\n", kp_enable);
> +}
> +
> +static ssize_t omap_kp_enable_store(struct device *dev, struct device_attribute *attr,
> + const char *buf, size_t count)
> +{
> + int state;
> +
> + if (sscanf(buf, "%u", &state) != 1)
> + return -EINVAL;
> +
> + if ((state != 1) && (state != 0))
> + return -EINVAL;
> +
> + mutex_lock(&kp_enable_mutex);
> + if (state != kp_enable) {
> + if (state)
> + enable_irq(INT_KEYBOARD);
> + else
> + disable_irq(INT_KEYBOARD);
> + kp_enable = state;
> + }
> + mutex_unlock(&kp_enable_mutex);
> +
> + return strnlen(buf, count);
> +}
> +
> +static DEVICE_ATTR(enable, S_IRUGO | S_IWUSR, omap_kp_enable_show, omap_kp_enable_store);
> +
> +#ifdef CONFIG_PM
> +static int omap_kp_suspend(struct platform_device *dev, pm_message_t state)
> +{
> + /* Nothing yet */
> +
> + return 0;
> +}
> +
> +static int omap_kp_resume(struct platform_device *dev)
> +{
> + /* Nothing yet */
> +
> + return 0;
> +}
> +#else
> +#define omap_kp_suspend NULL
> +#define omap_kp_resume NULL
> +#endif
> +
> +static int __init omap_kp_probe(struct platform_device *pdev)
> +{
> + struct omap_kp *omap_kp;
> + struct input_dev *input_dev;
> + struct omap_kp_platform_data *pdata = pdev->dev.platform_data;
> + int i, col_idx, row_idx, irq_idx, ret;
> +
> + if (!pdata->rows || !pdata->cols || !pdata->keymap) {
> + printk(KERN_ERR "No rows, cols or keymap from pdata\n");
> + return -EINVAL;
> + }
> +
> + omap_kp = kzalloc(sizeof(struct omap_kp), GFP_KERNEL);
> + input_dev = input_allocate_device();
> + if (!omap_kp || !input_dev) {
> + kfree(omap_kp);
> + input_free_device(input_dev);
> + return -ENOMEM;
> + }
> +
> + platform_set_drvdata(pdev, omap_kp);
> +
> + omap_kp->input = input_dev;
> +
> + /* Disable the interrupt for the MPUIO keyboard */
> + if (!cpu_is_omap24xx())
> + omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
> +
> + keymap = pdata->keymap;
> +
> + if (pdata->rep)
> + set_bit(EV_REP, input_dev->evbit);
> +
> + if (pdata->row_gpios && pdata->col_gpios) {
> + row_gpios = pdata->row_gpios;
> + col_gpios = pdata->col_gpios;
> + }
> +
> + omap_kp->rows = pdata->rows;
> + omap_kp->cols = pdata->cols;
> +
> + if (cpu_is_omap24xx()) {
> + /* Cols: outputs */
> + for (col_idx = 0; col_idx < omap_kp->cols; col_idx++) {
> + if (omap_request_gpio(col_gpios[col_idx]) < 0) {
> + printk(KERN_ERR "Failed to request"
> + "GPIO%d for keypad\n",
> + col_gpios[col_idx]);
> + goto err1;
> + }
> + omap_set_gpio_direction(col_gpios[col_idx], 0);
> + }
> + /* Rows: inputs */
> + for (row_idx = 0; row_idx < omap_kp->rows; row_idx++) {
> + if (omap_request_gpio(row_gpios[row_idx]) < 0) {
> + printk(KERN_ERR "Failed to request"
> + "GPIO%d for keypad\n",
> + row_gpios[row_idx]);
> + goto err2;
> + }
> + omap_set_gpio_direction(row_gpios[row_idx], 1);
> + }
> + }
> +
> + setup_timer(&omap_kp->timer, omap_kp_timer, (unsigned long)omap_kp);
> +
> + /* get the irq and init timer*/
> + tasklet_enable(&kp_tasklet);
> + kp_tasklet.data = (unsigned long) omap_kp;
> +
> + ret = device_create_file(&pdev->dev, &dev_attr_enable);
> + if (ret < 0)
> + goto err2;
> +
> + /* setup input device */
> + set_bit(EV_KEY, input_dev->evbit);
> + for (i = 0; keymap[i] != 0; i++)
> + set_bit(keymap[i] & KEY_MAX, input_dev->keybit);
> + input_dev->name = "omap-keypad";
> + input_dev->phys = "omap-keypad/input0";
> + input_dev->cdev.dev = &pdev->dev;
> + input_dev->private = omap_kp;
> +
> + input_dev->id.bustype = BUS_HOST;
> + input_dev->id.vendor = 0x0001;
> + input_dev->id.product = 0x0001;
> + input_dev->id.version = 0x0100;
> +
> + input_dev->keycode = keymap;
> + input_dev->keycodesize = sizeof(unsigned int);
> + input_dev->keycodemax = pdata->keymapsize;
> +
> + ret = input_register_device(omap_kp->input);
> + if (ret < 0) {
> + printk(KERN_ERR "Unable to register omap-keypad input device\n");
> + goto err3;
> + }
> +
> + if (machine_is_omap_h2() || machine_is_omap_h3() ||
> + machine_is_omap_perseus2()) {
> + omap_writew(0xff, OMAP_MPUIO_BASE + OMAP_MPUIO_GPIO_DEBOUNCING);
> + }
Maybe this should be a flag or something? Why does h2, h3 and perseus2
require this and not others?
> + /* scan current status and enable interrupt */
> + omap_kp_scan_keypad(omap_kp, keypad_state);
> + if (!cpu_is_omap24xx()) {
> + omap_kp->irq = platform_get_irq(pdev, 0);
> + if (omap_kp->irq >= 0) {
> + if (request_irq(omap_kp->irq, omap_kp_interrupt, 0,
> + "omap-keypad", omap_kp) < 0)
> + goto err4;
> + }
> + omap_writew(0, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
> + } else {
> + for (irq_idx = 0; irq_idx < omap_kp->rows; irq_idx++) {
> + if (request_irq(OMAP_GPIO_IRQ(row_gpios[irq_idx]),
> + omap_kp_interrupt,
> + IRQF_TRIGGER_FALLING,
> + "omap-keypad", omap_kp) < 0)
> + goto err5;
> + }
> + }
> + return 0;
> +err5:
> + for (i = irq_idx-1; i >=0; i--)
> + free_irq(row_gpios[i], 0);
> +err4:
> + input_unregister_device(omap_kp->input);
> + input_dev = NULL;
> +err3:
> + device_remove_file(&pdev->dev, &dev_attr_enable);
> +err2:
> + for (i = row_idx-1; i >=0; i--)
> + omap_free_gpio(row_gpios[i]);
> +err1:
> + for (i = col_idx-1; i >=0; i--)
> + omap_free_gpio(col_gpios[i]);
> +
> + kfree(omap_kp);
> + input_free_device(input_dev);
> +
> + return -EINVAL;
> +}
> +
> +static int omap_kp_remove(struct platform_device *pdev)
> +{
> + struct omap_kp *omap_kp = platform_get_drvdata(pdev);
> +
> + /* disable keypad interrupt handling */
> + tasklet_disable(&kp_tasklet);
> + if (cpu_is_omap24xx()) {
> + int i;
> + for (i = 0; i < omap_kp->cols; i++)
> + omap_free_gpio(col_gpios[i]);
> + for (i = 0; i < omap_kp->rows; i++) {
> + omap_free_gpio(row_gpios[i]);
> + free_irq(OMAP_GPIO_IRQ(row_gpios[i]), 0);
> + }
> + } else {
> + omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
> + free_irq(omap_kp->irq, 0);
> + }
> +
> + del_timer_sync(&omap_kp->timer);
> + tasklet_kill(&kp_tasklet);
> +
> + /* unregister everything */
> + input_unregister_device(omap_kp->input);
> +
> + kfree(omap_kp);
> +
> + return 0;
> +}
> +
> +static struct platform_driver omap_kp_driver = {
> + .probe = omap_kp_probe,
> + .remove = omap_kp_remove,
> + .suspend = omap_kp_suspend,
> + .resume = omap_kp_resume,
> + .driver = {
> + .name = "omap-keypad",
> + },
> +};
> +
> +static int __devinit omap_kp_init(void)
> +{
> + printk(KERN_INFO "OMAP Keypad Driver\n");
> + return platform_driver_register(&omap_kp_driver);
> +}
> +
> +static void __exit omap_kp_exit(void)
> +{
> + platform_driver_unregister(&omap_kp_driver);
> +}
> +
> +module_init(omap_kp_init);
> +module_exit(omap_kp_exit);
> +
> +MODULE_AUTHOR("Timo Ter?s");
> +MODULE_DESCRIPTION("OMAP Keypad Driver");
> +MODULE_LICENSE("GPL");
--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH 1/2] OMAP: Add keypad driver #3
2006-07-27 8:53 ` Russell King
@ 2006-07-27 9:26 ` Komal Shah
0 siblings, 0 replies; 3+ messages in thread
From: Komal Shah @ 2006-07-27 9:26 UTC (permalink / raw)
To: Russell King
Cc: akpm, juha.yrjola, tony, ext-timo.teras, r-woodruff2, linux-input,
linux-kernel, dbrownell, kjh
--- Russell King <rmk+lkml@arm.linux.org.uk> wrote:
> > +
> > + /* read the keypad status */
> > + if (cpu_is_omap24xx()) {
> > + int i;
> > + for (i = 0; i < omap_kp->rows; i++)
> > + disable_irq(OMAP_GPIO_IRQ(row_gpios[i]));
> > + } else
> > + /* disable keyboard interrupt and schedule for handling */
> > + omap_writew(1, OMAP_MPUIO_BASE + OMAP_MPUIO_KBD_MASKIT);
> > +
> > + if (!cpu_is_omap24xx()) {
>
> This seems obfuscated. It would be trivial to combine these two if()
> clauses.
Oops. I will update this in the next patch.
>
> And a general note about the omap24xx vs !omap24xx differences in
> this
> file - would it make more sense for code readability to have two
> completely separate drivers?
Yeah, I had same thought when I did the integration of omap24xx H4 gpio
based keypad driver from TI OMAP tree to omap-git. But, if Tony, Juha
and Richard agrees, then I can roll-out new omap2-gpio-keypad driver
patch along with changes into existing omap-keypad.c(will become
omap1-keypad.c then). It will be also easy for me to maintain omap2
keypad driver, as I don't have access to OMAP1 based boards.
> > +
> > + if (machine_is_omap_osk() || machine_is_omap_h2()
> > + || machine_is_omap_h3())
> > + udelay(9);
> > + else
> > + udelay(4);
>
> Wouldn't it be better to pass this via the platform device driver?
> It
> seems likely that other delays may be required with differing
> hardware.
Yes, we can. I will make that change.
> > +
> > + if (machine_is_omap_h2() || machine_is_omap_h3() ||
> > + machine_is_omap_perseus2()) {
> > + omap_writew(0xff, OMAP_MPUIO_BASE + OMAP_MPUIO_GPIO_DEBOUNCING);
> > + }
>
> Maybe this should be a flag or something? Why does h2, h3 and
> perseus2
> require this and not others?
Yes we can put the flag there through platform data, but OMAP1
(h2/h3/perseus) owners should comment on that. Kevin/Tony?
Thanx for the detailed review.
---Komal Shah
http://komalshah.blogspot.com/
__________________________________________________
Do You Yahoo!?
Tired of spam? Yahoo! Mail has the best spam protection around
http://mail.yahoo.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-07-27 9:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-07-27 7:09 [PATCH 1/2] OMAP: Add keypad driver #3 Komal Shah
2006-07-27 8:53 ` Russell King
2006-07-27 9:26 ` Komal Shah
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox