If we don't set proper debouce time for ads7846, then there are
flooded interrupt counters of ads7846 responding to one time
touch on screen, so the driver couldn't work well.

And since most OMAP3 series boards pass NULL pointer of board_pdata
to omap_ads7846_init, so it's more proper to set it in driver level
after having gpio_request done.

This patch has been validated on 3530evm.

Signed-off-by: Zumeng Chen <zumeng.c...@gmail.com>
Signed-off-by: Syed Mohammed Khasim <kha...@ti.com>
Signed-off-by: Tony Lindgren <t...@atomide.com>
---
 drivers/input/touchscreen/ads7846.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/input/touchscreen/ads7846.c 
b/drivers/input/touchscreen/ads7846.c
index f02028e..a82a5fb 100644
--- a/drivers/input/touchscreen/ads7846.c
+++ b/drivers/input/touchscreen/ads7846.c
@@ -61,6 +61,7 @@
 
 /* this driver doesn't aim at the peak continuous sample rate */
 #define        SAMPLE_BITS     (8 /*cmd*/ + 16 /*sample*/ + 2 /* before, after 
*/)
+#define        DEBOUNCE_TIME   310 /* About 10 ms */
 
 struct ts_event {
        /*
@@ -980,6 +981,7 @@ static int __devinit ads7846_setup_pendown(struct 
spi_device *spi, struct ads784
                }
 
                ts->gpio_pendown = pdata->gpio_pendown;
+               gpio_set_debounce(pdata->gpio_pendown, DEBOUNCE_TIME);
 
        } else {
                dev_err(&spi->dev, "no get_pendown_state nor gpio_pendown?\n");
-- 
1.7.5.4

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to