* [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table
@ 2017-03-22 19:05 Jagan Teki
2017-03-22 19:05 ` [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding Jagan Teki
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Jagan Teki @ 2017-03-22 19:05 UTC (permalink / raw)
To: linux-arm-kernel
From: Jagan Teki <jagan@amarulasolutions.com>
Added missing of_match_table for max11801_ts driver with
compatible as "maxim,max11801_ts"
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Matteo Lisi <matteo.lisi@engicam.com>
Cc: Michael Trimarchi <michael@amarulasolutions.com>
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
Changes for v8:
- use the compatible as maxim,max11801 with remove suffix _ts since
i2c node follow the same.
Changes for v7:
- newly added patch
drivers/input/touchscreen/max11801_ts.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/input/touchscreen/max11801_ts.c b/drivers/input/touchscreen/max11801_ts.c
index a595ae5..d3c7fc0 100644
--- a/drivers/input/touchscreen/max11801_ts.c
+++ b/drivers/input/touchscreen/max11801_ts.c
@@ -224,9 +224,16 @@ static int max11801_ts_probe(struct i2c_client *client,
};
MODULE_DEVICE_TABLE(i2c, max11801_ts_id);
+static const struct of_device_id max11801_ts_dt_ids[] = {
+ { .compatible = "maxim,max11801" },
+ { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, max11801_ts_dt_ids);
+
static struct i2c_driver max11801_ts_driver = {
.driver = {
.name = "max11801_ts",
+ .of_match_table = max11801_ts_dt_ids,
},
.id_table = max11801_ts_id,
.probe = max11801_ts_probe,
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding
2017-03-22 19:05 [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table Jagan Teki
@ 2017-03-22 19:05 ` Jagan Teki
2017-03-29 1:57 ` Rob Herring
2017-03-22 19:05 ` [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node Jagan Teki
2017-03-29 7:26 ` [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table Dmitry Torokhov
2 siblings, 1 reply; 7+ messages in thread
From: Jagan Teki @ 2017-03-22 19:05 UTC (permalink / raw)
To: linux-arm-kernel
From: Jagan Teki <jagan@amarulasolutions.com>
Add missing documentation of max11801-ts dt-binding details.
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Michael Trimarchi <michael@amarulasolutions.com>
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
Changes for v8:
- Use IRQ_TYPE_EDGE_FALLING for interrupt trigger type.
Changes for v7:
- add vendor prefix as maxim
Changes for v6:
- Replace the lable and name of the node
ts: max11801 => max11801: touchscreen at 48
Changes for v5:
- Newly added patch
.../bindings/input/touchscreen/max11801-ts.txt | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
create mode 100644 Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt
diff --git a/Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt b/Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt
new file mode 100644
index 0000000..40ac0fe
--- /dev/null
+++ b/Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt
@@ -0,0 +1,18 @@
+* MAXI MAX11801 Resistive touch screen controller with i2c interface
+
+Required properties:
+- compatible: must be "maxim,max11801"
+- reg: i2c slave address
+- interrupt-parent: the phandle for the interrupt controller
+- interrupts: touch controller interrupt
+
+Example:
+
+&i2c1 {
+ max11801: touchscreen at 48 {
+ compatible = "maxim,max11801";
+ reg = <0x48>;
+ interrupt-parent = <&gpio3>;
+ interrupts = <31 IRQ_TYPE_EDGE_FALLING>;
+ };
+};
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding
2017-03-22 19:05 ` [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding Jagan Teki
@ 2017-03-29 1:57 ` Rob Herring
2017-03-29 7:26 ` Dmitry Torokhov
0 siblings, 1 reply; 7+ messages in thread
From: Rob Herring @ 2017-03-29 1:57 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Mar 23, 2017 at 12:35:17AM +0530, Jagan Teki wrote:
> From: Jagan Teki <jagan@amarulasolutions.com>
>
> Add missing documentation of max11801-ts dt-binding details.
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Michael Trimarchi <michael@amarulasolutions.com>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> ---
> Changes for v8:
> - Use IRQ_TYPE_EDGE_FALLING for interrupt trigger type.
> Changes for v7:
> - add vendor prefix as maxim
> Changes for v6:
> - Replace the lable and name of the node
> ts: max11801 => max11801: touchscreen at 48
> Changes for v5:
> - Newly added patch
>
> .../bindings/input/touchscreen/max11801-ts.txt | 18 ++++++++++++++++++
> 1 file changed, 18 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding
2017-03-29 1:57 ` Rob Herring
@ 2017-03-29 7:26 ` Dmitry Torokhov
0 siblings, 0 replies; 7+ messages in thread
From: Dmitry Torokhov @ 2017-03-29 7:26 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Mar 28, 2017 at 08:57:55PM -0500, Rob Herring wrote:
> On Thu, Mar 23, 2017 at 12:35:17AM +0530, Jagan Teki wrote:
> > From: Jagan Teki <jagan@amarulasolutions.com>
> >
> > Add missing documentation of max11801-ts dt-binding details.
> >
> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> > Cc: Mark Rutland <mark.rutland@arm.com>
> > Cc: Rob Herring <robh+dt@kernel.org>
> > Cc: Shawn Guo <shawnguo@kernel.org>
> > Cc: Michael Trimarchi <michael@amarulasolutions.com>
> > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> > ---
> > Changes for v8:
> > - Use IRQ_TYPE_EDGE_FALLING for interrupt trigger type.
> > Changes for v7:
> > - add vendor prefix as maxim
> > Changes for v6:
> > - Replace the lable and name of the node
> > ts: max11801 => max11801: touchscreen at 48
> > Changes for v5:
> > - Newly added patch
> >
> > .../bindings/input/touchscreen/max11801-ts.txt | 18 ++++++++++++++++++
> > 1 file changed, 18 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt
>
> Acked-by: Rob Herring <robh@kernel.org>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node
2017-03-22 19:05 [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table Jagan Teki
2017-03-22 19:05 ` [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding Jagan Teki
@ 2017-03-22 19:05 ` Jagan Teki
2017-03-29 8:22 ` Shawn Guo
2017-03-29 7:26 ` [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table Dmitry Torokhov
2 siblings, 1 reply; 7+ messages in thread
From: Jagan Teki @ 2017-03-22 19:05 UTC (permalink / raw)
To: linux-arm-kernel
From: Jagan Teki <jagan@amarulasolutions.com>
max11801 touchscreen on Engicam iCoreM6 Quad module is
connected via i2c1, so add max11801: touchscreen at 48 on i2c1.
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Domenico Acri <domenico.acri@engicam.com>
Cc: Matteo Lisi <matteo.lisi@engicam.com>
Cc: Michael Trimarchi <michael@amarulasolutions.com>
Cc: Shawn Guo <shawnguo@kernel.org>
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
Changes for v8:
- Use IRQ_TYPE_EDGE_FALLING for interrupt trigger type.
Changes for v7:
- add vendor prefix as maxim
Changes for v6:
- Replace the lable and name of the node
ts: max11801 => max11801: touchscreen at 48
Changes for v5:
- Newly added patch
arch/arm/boot/dts/imx6q-icore.dts | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/boot/dts/imx6q-icore.dts b/arch/arm/boot/dts/imx6q-icore.dts
index 73f34d1..5613dd9 100644
--- a/arch/arm/boot/dts/imx6q-icore.dts
+++ b/arch/arm/boot/dts/imx6q-icore.dts
@@ -58,6 +58,15 @@
status = "okay";
};
+&i2c1 {
+ max11801: touchscreen at 48 {
+ compatible = "maxim,max11801";
+ reg = <0x48>;
+ interrupt-parent = <&gpio3>;
+ interrupts = <31 IRQ_TYPE_EDGE_FALLING>;
+ };
+};
+
&ldb {
status = "okay";
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node
2017-03-22 19:05 ` [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node Jagan Teki
@ 2017-03-29 8:22 ` Shawn Guo
0 siblings, 0 replies; 7+ messages in thread
From: Shawn Guo @ 2017-03-29 8:22 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Mar 23, 2017 at 12:35:18AM +0530, Jagan Teki wrote:
> From: Jagan Teki <jagan@amarulasolutions.com>
>
> max11801 touchscreen on Engicam iCoreM6 Quad module is
> connected via i2c1, so add max11801: touchscreen at 48 on i2c1.
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Domenico Acri <domenico.acri@engicam.com>
> Cc: Matteo Lisi <matteo.lisi@engicam.com>
> Cc: Michael Trimarchi <michael@amarulasolutions.com>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table
2017-03-22 19:05 [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table Jagan Teki
2017-03-22 19:05 ` [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding Jagan Teki
2017-03-22 19:05 ` [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node Jagan Teki
@ 2017-03-29 7:26 ` Dmitry Torokhov
2 siblings, 0 replies; 7+ messages in thread
From: Dmitry Torokhov @ 2017-03-29 7:26 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Mar 23, 2017 at 12:35:16AM +0530, Jagan Teki wrote:
> From: Jagan Teki <jagan@amarulasolutions.com>
>
> Added missing of_match_table for max11801_ts driver with
> compatible as "maxim,max11801_ts"
>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Matteo Lisi <matteo.lisi@engicam.com>
> Cc: Michael Trimarchi <michael@amarulasolutions.com>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Applied, thank you.
> ---
> Changes for v8:
> - use the compatible as maxim,max11801 with remove suffix _ts since
> i2c node follow the same.
> Changes for v7:
> - newly added patch
>
> drivers/input/touchscreen/max11801_ts.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/input/touchscreen/max11801_ts.c b/drivers/input/touchscreen/max11801_ts.c
> index a595ae5..d3c7fc0 100644
> --- a/drivers/input/touchscreen/max11801_ts.c
> +++ b/drivers/input/touchscreen/max11801_ts.c
> @@ -224,9 +224,16 @@ static int max11801_ts_probe(struct i2c_client *client,
> };
> MODULE_DEVICE_TABLE(i2c, max11801_ts_id);
>
> +static const struct of_device_id max11801_ts_dt_ids[] = {
> + { .compatible = "maxim,max11801" },
> + { /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, max11801_ts_dt_ids);
> +
> static struct i2c_driver max11801_ts_driver = {
> .driver = {
> .name = "max11801_ts",
> + .of_match_table = max11801_ts_dt_ids,
> },
> .id_table = max11801_ts_id,
> .probe = max11801_ts_probe,
> --
> 1.9.1
>
--
Dmitry
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-03-29 8:22 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-22 19:05 [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table Jagan Teki
2017-03-22 19:05 ` [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding Jagan Teki
2017-03-29 1:57 ` Rob Herring
2017-03-29 7:26 ` Dmitry Torokhov
2017-03-22 19:05 ` [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node Jagan Teki
2017-03-29 8:22 ` Shawn Guo
2017-03-29 7:26 ` [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table Dmitry Torokhov
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).