Messages in this thread Patch in this message | | | Subject | [PATCH] Input: cy8ctmg110_ts - set reset_pin and irq_pin from platform data | From | Axel Lin <> | Date | Wed, 13 Jul 2011 21:06:59 +0800 |
| |
The implementation in cy8ctmg110_probe() does not properly set reset_pin and irq_pin from platform data. Let's fix it.
Signed-off-by: Axel Lin <axel.lin@gmail.com> --- Current implementation returns -ENODEV if no pdata available. But it does not use the information from pdata. I think currently the driver does not work because both ts->reset_pin and ts->irq_pin are 0, then gpio_request with ts->irq_pin will fail.
Just found this bug while reading the code. I don't have this hardware. I appreciate if someone can test it.
Axel
drivers/input/touchscreen/cy8ctmg110_ts.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/input/touchscreen/cy8ctmg110_ts.c b/drivers/input/touchscreen/cy8ctmg110_ts.c index d7afa20..cd60b3c 100644 --- a/drivers/input/touchscreen/cy8ctmg110_ts.c +++ b/drivers/input/touchscreen/cy8ctmg110_ts.c @@ -193,6 +193,8 @@ static int __devinit cy8ctmg110_probe(struct i2c_client *client, ts->client = client; ts->input = input_dev; + ts->reset_pin = pdata->reset_pin; + ts->irq_pin = pdata->irq_pin; snprintf(ts->phys, sizeof(ts->phys), "%s/input0", dev_name(&client->dev)); -- 1.7.4.1
| |