[PATCH v2 2/3] Input: omap-keypad: errata i689: Correct debounce time

From: Illia Smyrnov
Date: Fri Jul 26 2013 - 13:09:44 EST

From: Axel Haslam <axelhaslam@xxxxxx>

Set debounce time value to 12ms to workaround the errata.

When a key is released for a time shorter than the debounce time,
in-between 2 key press (KP1 and KP2), the keyboard state machine will go to
idle mode and will never detect the key release (after KP1,and also after KP2),
and thus will never generate a new IRQ indicating the key release.
>From the operating system standpoint, only a key press as been detected,
and the key will keep on being printed on the screen until another or
the same key is pressed again.When the failing state has been reached,
the KBD_STATEMACHINE register will show "idle state" and the KBD_FULLCODE
register won't be empty, this is the signature of the bug.There is no impact
on the power consumption of the system as the state machine goes to IDLE state.

First thing is to program the debounce time correctly:
If X (us) is the maximum time of bounces when a key is pressed or released,
and Y (us) is the minimum time of a key release that is expected to be detected,
then the debounce time should be set to a value inbetween X and Y.

In case it is still possible to get shorter than debounce time key-release
events, then the only solution is to implement a software workaround:

Before printing a second character on the screen, the software must check if
the keyboard has hit the failing condition (cf signature of the bug above)
or if the key is still really pressed and then take the appropriate actions.

Silicon revisions impacted: OMAP4430 ES 1.0,2.0,2.1,2.2,2.3
OMAP4460 ES 1.0,1.1
OMAP5430 ES 1.0

Signed-off-by: Axel Haslam <axelhaslam@xxxxxx>
Signed-off-by: Illia Smyrnov <illia.smyrnov@xxxxxx>
Based on top of v3.11-rc2

Depends on:
Input: omap-keypad: Cleanup - use bitfiled instead of hardcoded values

drivers/input/keyboard/omap4-keypad.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/input/keyboard/omap4-keypad.c b/drivers/input/keyboard/omap4-keypad.c
index feab00f..e8bdc76 100644
--- a/drivers/input/keyboard/omap4-keypad.c
+++ b/drivers/input/keyboard/omap4-keypad.c
@@ -62,8 +62,10 @@

/* OMAP4 values */
-#define OMAP4_VAL_PVT 0x7
+/* Errata i689: set debounce time value to 12ms */
+#define OMAP4_VAL_PVT 0x6

enum {

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/