From 5b64e07bc32581ee7e8c12125a0d12209e42923c Mon Sep 17 00:00:00 2001 From: Sanchayan Maity Date: Thu, 17 Dec 2015 11:08:46 +0530 Subject: touchscreen: fusion_F0710A: Reset controller if clear interrupt operation fails Add support for resetting the controller using the gpio reset pin if interrupt could not be cleared. If clearing the interrupt fails, the interrupt line stays high and no further interrupts are generated resulting in a non functioning touchscreen until next reboot, when we reset the controller again in probe. Signed-off-by: Sanchayan Maity Signed-off-by: Stefan Agner --- drivers/input/touchscreen/fusion_F0710A.c | 27 +++++++++++++++++---------- drivers/input/touchscreen/fusion_F0710A.h | 1 + 2 files changed, 18 insertions(+), 10 deletions(-) diff --git a/drivers/input/touchscreen/fusion_F0710A.c b/drivers/input/touchscreen/fusion_F0710A.c index edecf98d968e..779c3e83a402 100644 --- a/drivers/input/touchscreen/fusion_F0710A.c +++ b/drivers/input/touchscreen/fusion_F0710A.c @@ -89,6 +89,16 @@ bail1: return ret; } +static void fusion_F0710A_reset(void) +{ + /* Generate a 0 => 1 edge explicitly, and wait for startup... */ + gpio_set_value(fusion_F0710A.gpio_reset, 0); + msleep(10); + gpio_set_value(fusion_F0710A.gpio_reset, 1); + /* Wait for startup (up to 125ms according to datasheet) */ + msleep(125); +} + #define WC_RETRY_COUNT 3 static int fusion_F0710A_write_complete(void) { @@ -99,8 +109,11 @@ static int fusion_F0710A_write_complete(void) ret = fusion_F0710A_write_u8(fusion_F0710A_SCAN_COMPLETE, 0); if(ret == 0) break; - else - dev_err(&fusion_F0710A.client->dev, "Write complete failed(%d): %d\n", i, ret); + else { + dev_warn(&fusion_F0710A.client->dev, + "Write complete failed(%d): %d. Resetting controller...\n", i, ret); + fusion_F0710A_reset(); + } } return ret; @@ -285,14 +298,8 @@ static int fusion_F0710A_probe(struct i2c_client *i2c, const struct i2c_device_i if ((gpio_request(pdata->gpio_reset, "Fusion reset") == 0) && (gpio_direction_output(pdata->gpio_reset, 1) == 0)) { - - /* Generate a 0 => 1 edge explicitly, and wait for startup... */ - gpio_set_value(pdata->gpio_reset, 0); - msleep(10); - gpio_set_value(pdata->gpio_reset, 1); - /* Wait for startup (up to 125ms according to datasheet) */ - msleep(125); - + fusion_F0710A.gpio_reset = pdata->gpio_reset; + fusion_F0710A_reset(); gpio_export(pdata->gpio_reset, 0); } else { dev_warn(&i2c->dev, "Could not obtain GPIO for Fusion reset\n"); diff --git a/drivers/input/touchscreen/fusion_F0710A.h b/drivers/input/touchscreen/fusion_F0710A.h index 85f8210345a9..3e5af9e72b78 100644 --- a/drivers/input/touchscreen/fusion_F0710A.h +++ b/drivers/input/touchscreen/fusion_F0710A.h @@ -71,6 +71,7 @@ struct fusion_F0710A_data { struct i2c_client *client; struct workqueue_struct *workq; struct input_dev *input; + int gpio_reset; u16 x1; u16 y1; u8 z1; -- cgit v1.2.3