summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndreas Pretzsch <apr@cn-eng.de>2012-11-25 08:31:38 +0100
committerDmitry Torokhov <dmitry.torokhov@gmail.com>2012-11-25 09:10:18 +0100
commit92aab96034c2cdf11eb61a6b14409c2da8e5158d (patch)
tree95c67864b67db9eb9a081b371e12c0a36a053313
parentInput: HIL - do not call tasklet_disable right before tasklet_kill (diff)
downloadlinux-92aab96034c2cdf11eb61a6b14409c2da8e5158d.tar.xz
linux-92aab96034c2cdf11eb61a6b14409c2da8e5158d.zip
Input: imx_keypad - only set enabled columns to open-drain
In imx_keypad_inhibit(), all 8 columns were set to open-drain, in contrast to the rest of the driver, where only the enabled columns are modified/used. Contrary to the normal expectation, this also affects column I/Os not even mapped via IOMUX to the KPP hardware module but used as a GPIO. Therefore only init enabled columns to open-drain and leave all others with their default reset value of 0, i.e. totem-pole. Signed-off-by: Andreas Pretzsch <apr@cn-eng.de> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
-rw-r--r--drivers/input/keyboard/imx_keypad.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/input/keyboard/imx_keypad.c b/drivers/input/keyboard/imx_keypad.c
index 7ad74517654b..6d150e3e1f55 100644
--- a/drivers/input/keyboard/imx_keypad.c
+++ b/drivers/input/keyboard/imx_keypad.c
@@ -362,7 +362,8 @@ static void imx_keypad_inhibit(struct imx_keypad *keypad)
writew(reg_val, keypad->mmio_base + KPSR);
/* Colums as open drain and disable all rows */
- writew(0xff00, keypad->mmio_base + KPCR);
+ reg_val = (keypad->cols_en_mask & 0xff) << 8;
+ writew(reg_val, keypad->mmio_base + KPCR);
}
static void imx_keypad_close(struct input_dev *dev)