* [PATCH] input: qt1070: add device tree support
@ 2012-08-17 10:03 Bo Shen
2012-08-17 18:26 ` Shubhrajyoti
0 siblings, 1 reply; 3+ messages in thread
From: Bo Shen @ 2012-08-17 10:03 UTC (permalink / raw)
To: dmitry.torokhov; +Cc: nicolas.ferre, jm.lin, linux-input, Bo Shen
This patch enable the qt1070 device tree support
If using device tree, the node of qt1070 looks like
i2c@0 { // this is an example
status = "okay"; // Enable i2c master
qt1070@1b {
compatible = "atmel,qt1070"; // FDT device id
reg = <0x1b>; // qt1070 i2c address
change-gpio = <&pioA 7 1>; // gpio used as interrupt
};
};
Signed-off-by: Bo Shen <voice.shen@atmel.com>
---
drivers/input/keyboard/qt1070.c | 19 ++++++++++++++++---
1 file changed, 16 insertions(+), 3 deletions(-)
diff --git a/drivers/input/keyboard/qt1070.c b/drivers/input/keyboard/qt1070.c
index ca68f29..7d656b5 100644
--- a/drivers/input/keyboard/qt1070.c
+++ b/drivers/input/keyboard/qt1070.c
@@ -33,6 +33,8 @@
#include <linux/interrupt.h>
#include <linux/jiffies.h>
#include <linux/delay.h>
+#include <linux/of.h>
+#include <linux/of_gpio.h>
/* Address for each register */
#define CHIP_ID 0x00
@@ -155,6 +157,10 @@ static int __devinit qt1070_probe(struct i2c_client *client,
return -ENODEV;
}
+ if (client->dev.of_node)
+ client->irq = of_get_named_gpio(client->dev.of_node,
+ "change-gpio", 0);
+
if (!client->irq) {
dev_err(&client->dev, "please assign the irq to this device\n");
return -EINVAL;
@@ -200,9 +206,9 @@ static int __devinit qt1070_probe(struct i2c_client *client,
qt1070_write(client, RESET, 1);
msleep(QT1070_RESET_TIME);
- err = request_threaded_irq(client->irq, NULL, qt1070_interrupt,
- IRQF_TRIGGER_NONE | IRQF_ONESHOT,
- client->dev.driver->name, data);
+ err = request_threaded_irq(gpio_to_irq(client->irq), NULL,
+ qt1070_interrupt, IRQF_TRIGGER_NONE | IRQF_ONESHOT,
+ client->dev.driver->name, data);
if (err) {
dev_err(&client->dev, "fail to request irq\n");
goto err_free_mem;
@@ -243,6 +249,12 @@ static int __devexit qt1070_remove(struct i2c_client *client)
return 0;
}
+static const struct of_device_id atmel_qt1070_of_match[] = {
+ { .compatible = "atmel,qt1070", },
+ { },
+};
+MODULE_DEVICE_TABLE(of, atmel_qt1070_of_match);
+
static const struct i2c_device_id qt1070_id[] = {
{ "qt1070", 0 },
{ },
@@ -253,6 +265,7 @@ static struct i2c_driver qt1070_driver = {
.driver = {
.name = "qt1070",
.owner = THIS_MODULE,
+ .of_match_table = atmel_qt1070_of_match,
},
.id_table = qt1070_id,
.probe = qt1070_probe,
--
1.7.10
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] input: qt1070: add device tree support
2012-08-17 10:03 [PATCH] input: qt1070: add device tree support Bo Shen
@ 2012-08-17 18:26 ` Shubhrajyoti
2012-08-20 2:08 ` Bo Shen
0 siblings, 1 reply; 3+ messages in thread
From: Shubhrajyoti @ 2012-08-17 18:26 UTC (permalink / raw)
To: Bo Shen; +Cc: dmitry.torokhov, nicolas.ferre, jm.lin, linux-input
Hi Bo,
On Friday 17 August 2012 03:33 PM, Bo Shen wrote:
> This patch enable the qt1070 device tree support
>
> If using device tree, the node of qt1070 looks like
> i2c@0 { // this is an example
> status = "okay"; // Enable i2c master
> qt1070@1b {
> compatible = "atmel,qt1070"; // FDT device id
> reg = <0x1b>; // qt1070 i2c address
> change-gpio = <&pioA 7 1>; // gpio used as interrupt
> };
> };
>
> Signed-off-by: Bo Shen <voice.shen@atmel.com>
> ---
> drivers/input/keyboard/qt1070.c | 19 ++++++++++++++++---
> 1 file changed, 16 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/input/keyboard/qt1070.c b/drivers/input/keyboard/qt1070.c
> index ca68f29..7d656b5 100644
> --- a/drivers/input/keyboard/qt1070.c
> +++ b/drivers/input/keyboard/qt1070.c
> @@ -33,6 +33,8 @@
> #include <linux/interrupt.h>
> #include <linux/jiffies.h>
> #include <linux/delay.h>
> +#include <linux/of.h>
> +#include <linux/of_gpio.h>
>
> /* Address for each register */
> #define CHIP_ID 0x00
> @@ -155,6 +157,10 @@ static int __devinit qt1070_probe(struct i2c_client *client,
> return -ENODEV;
> }
>
> + if (client->dev.of_node)
> + client->irq = of_get_named_gpio(client->dev.of_node,
> + "change-gpio", 0);
> +
> if (!client->irq) {
> dev_err(&client->dev, "please assign the irq to this device\n");
> return -EINVAL;
> @@ -200,9 +206,9 @@ static int __devinit qt1070_probe(struct i2c_client *client,
> qt1070_write(client, RESET, 1);
> msleep(QT1070_RESET_TIME);
>
> - err = request_threaded_irq(client->irq, NULL, qt1070_interrupt,
> - IRQF_TRIGGER_NONE | IRQF_ONESHOT,
> - client->dev.driver->name, data);
where is the gpio request done?
Am I missing something?
> + err = request_threaded_irq(gpio_to_irq(client->irq), NULL,
> + qt1070_interrupt, IRQF_TRIGGER_NONE | IRQF_ONESHOT,
> + client->dev.driver->name, data);
> if (err) {
> dev_err(&client->dev, "fail to request irq\n");
> goto err_free_mem;
> @@ -243,6 +249,12 @@ static int __devexit qt1070_remove(struct i2c_client *client)
> return 0;
> }
>
> +static const struct of_device_id atmel_qt1070_of_match[] = {
> + { .compatible = "atmel,qt1070", },
> + { },
> +};
> +MODULE_DEVICE_TABLE(of, atmel_qt1070_of_match);
> +
> static const struct i2c_device_id qt1070_id[] = {
> { "qt1070", 0 },
> { },
> @@ -253,6 +265,7 @@ static struct i2c_driver qt1070_driver = {
> .driver = {
> .name = "qt1070",
> .owner = THIS_MODULE,
> + .of_match_table = atmel_qt1070_of_match,
> },
> .id_table = qt1070_id,
> .probe = qt1070_probe,
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] input: qt1070: add device tree support
2012-08-17 18:26 ` Shubhrajyoti
@ 2012-08-20 2:08 ` Bo Shen
0 siblings, 0 replies; 3+ messages in thread
From: Bo Shen @ 2012-08-20 2:08 UTC (permalink / raw)
To: Shubhrajyoti; +Cc: dmitry.torokhov, nicolas.ferre, jm.lin, linux-input
Hi Sir,
On 8/18/2012 2:26, Shubhrajyoti wrote:
> Hi Bo,
> On Friday 17 August 2012 03:33 PM, Bo Shen wrote:
>> This patch enable the qt1070 device tree support
>>
>> If using device tree, the node of qt1070 looks like
>> i2c@0 { // this is an example
>> status = "okay"; // Enable i2c master
>> qt1070@1b {
>> compatible = "atmel,qt1070"; // FDT device id
>> reg = <0x1b>; // qt1070 i2c address
>> change-gpio = <&pioA 7 1>; // gpio used as interrupt
>> };
>> };
>>
>> Signed-off-by: Bo Shen <voice.shen@atmel.com>
>> ---
>> drivers/input/keyboard/qt1070.c | 19 ++++++++++++++++---
>> 1 file changed, 16 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/input/keyboard/qt1070.c b/drivers/input/keyboard/qt1070.c
>> index ca68f29..7d656b5 100644
>> --- a/drivers/input/keyboard/qt1070.c
>> +++ b/drivers/input/keyboard/qt1070.c
>> @@ -33,6 +33,8 @@
>> #include <linux/interrupt.h>
>> #include <linux/jiffies.h>
>> #include <linux/delay.h>
>> +#include <linux/of.h>
>> +#include <linux/of_gpio.h>
>>
>> /* Address for each register */
>> #define CHIP_ID 0x00
>> @@ -155,6 +157,10 @@ static int __devinit qt1070_probe(struct i2c_client *client,
>> return -ENODEV;
>> }
>>
>> + if (client->dev.of_node)
>> + client->irq = of_get_named_gpio(client->dev.of_node,
>> + "change-gpio", 0);
>> +
>> if (!client->irq) {
>> dev_err(&client->dev, "please assign the irq to this device\n");
>> return -EINVAL;
>> @@ -200,9 +206,9 @@ static int __devinit qt1070_probe(struct i2c_client *client,
>> qt1070_write(client, RESET, 1);
>> msleep(QT1070_RESET_TIME);
>>
>> - err = request_threaded_irq(client->irq, NULL, qt1070_interrupt,
>> - IRQF_TRIGGER_NONE | IRQF_ONESHOT,
>> - client->dev.driver->name, data);
> where is the gpio request done?
> Am I missing something?
OK. I will add gpio request.
Thanks.
>> + err = request_threaded_irq(gpio_to_irq(client->irq), NULL,
>> + qt1070_interrupt, IRQF_TRIGGER_NONE | IRQF_ONESHOT,
>> + client->dev.driver->name, data);
>> if (err) {
>> dev_err(&client->dev, "fail to request irq\n");
>> goto err_free_mem;
>> @@ -243,6 +249,12 @@ static int __devexit qt1070_remove(struct i2c_client *client)
>> return 0;
>> }
>>
>> +static const struct of_device_id atmel_qt1070_of_match[] = {
>> + { .compatible = "atmel,qt1070", },
>> + { },
>> +};
>> +MODULE_DEVICE_TABLE(of, atmel_qt1070_of_match);
>> +
>> static const struct i2c_device_id qt1070_id[] = {
>> { "qt1070", 0 },
>> { },
>> @@ -253,6 +265,7 @@ static struct i2c_driver qt1070_driver = {
>> .driver = {
>> .name = "qt1070",
>> .owner = THIS_MODULE,
>> + .of_match_table = atmel_qt1070_of_match,
>> },
>> .id_table = qt1070_id,
>> .probe = qt1070_probe,
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-08-20 2:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-17 10:03 [PATCH] input: qt1070: add device tree support Bo Shen
2012-08-17 18:26 ` Shubhrajyoti
2012-08-20 2:08 ` Bo Shen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).