2017-03-22 19:07:53

by Jagan Teki

[permalink] [raw]
Subject: [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table

From: Jagan Teki <[email protected]>

Added missing of_match_table for max11801_ts driver with
compatible as "maxim,max11801_ts"

Cc: Dmitry Torokhov <[email protected]>
Cc: Shawn Guo <[email protected]>
Cc: Matteo Lisi <[email protected]>
Cc: Michael Trimarchi <[email protected]>
Signed-off-by: Jagan Teki <[email protected]>
---
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


2017-03-22 19:08:06

by Jagan Teki

[permalink] [raw]
Subject: [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding

From: Jagan Teki <[email protected]>

Add missing documentation of max11801-ts dt-binding details.

Cc: Dmitry Torokhov <[email protected]>
Cc: Mark Rutland <[email protected]>
Cc: Rob Herring <[email protected]>
Cc: Shawn Guo <[email protected]>
Cc: Michael Trimarchi <[email protected]>
Signed-off-by: Jagan Teki <[email protected]>
---
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@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@48 {
+ compatible = "maxim,max11801";
+ reg = <0x48>;
+ interrupt-parent = <&gpio3>;
+ interrupts = <31 IRQ_TYPE_EDGE_FALLING>;
+ };
+};
--
1.9.1

2017-03-22 19:08:21

by Jagan Teki

[permalink] [raw]
Subject: [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node

From: Jagan Teki <[email protected]>

max11801 touchscreen on Engicam iCoreM6 Quad module is
connected via i2c1, so add max11801: touchscreen@48 on i2c1.

Cc: Dmitry Torokhov <[email protected]>
Cc: Domenico Acri <[email protected]>
Cc: Matteo Lisi <[email protected]>
Cc: Michael Trimarchi <[email protected]>
Cc: Shawn Guo <[email protected]>
Signed-off-by: Jagan Teki <[email protected]>
---
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@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@48 {
+ compatible = "maxim,max11801";
+ reg = <0x48>;
+ interrupt-parent = <&gpio3>;
+ interrupts = <31 IRQ_TYPE_EDGE_FALLING>;
+ };
+};
+
&ldb {
status = "okay";

--
1.9.1

2017-03-29 01:58:00

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding

On Thu, Mar 23, 2017 at 12:35:17AM +0530, Jagan Teki wrote:
> From: Jagan Teki <[email protected]>
>
> Add missing documentation of max11801-ts dt-binding details.
>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Mark Rutland <[email protected]>
> Cc: Rob Herring <[email protected]>
> Cc: Shawn Guo <[email protected]>
> Cc: Michael Trimarchi <[email protected]>
> Signed-off-by: Jagan Teki <[email protected]>
> ---
> 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@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 <[email protected]>

2017-03-29 07:26:22

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH v8 2/3] dt-bindings: input: touchscreen: Add max11801-ts binding

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 <[email protected]>
> >
> > Add missing documentation of max11801-ts dt-binding details.
> >
> > Cc: Dmitry Torokhov <[email protected]>
> > Cc: Mark Rutland <[email protected]>
> > Cc: Rob Herring <[email protected]>
> > Cc: Shawn Guo <[email protected]>
> > Cc: Michael Trimarchi <[email protected]>
> > Signed-off-by: Jagan Teki <[email protected]>
> > ---
> > 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@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 <[email protected]>

Applied, thank you.

--
Dmitry

2017-03-29 07:27:05

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH v8 1/3] Input: max11801_ts: Add missing of_match_table

On Thu, Mar 23, 2017 at 12:35:16AM +0530, Jagan Teki wrote:
> From: Jagan Teki <[email protected]>
>
> Added missing of_match_table for max11801_ts driver with
> compatible as "maxim,max11801_ts"
>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Shawn Guo <[email protected]>
> Cc: Matteo Lisi <[email protected]>
> Cc: Michael Trimarchi <[email protected]>
> Signed-off-by: Jagan Teki <[email protected]>

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

2017-03-29 08:23:41

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v8 3/3] ARM: dts: imx6q-icore: Add touchscreen node

On Thu, Mar 23, 2017 at 12:35:18AM +0530, Jagan Teki wrote:
> From: Jagan Teki <[email protected]>
>
> max11801 touchscreen on Engicam iCoreM6 Quad module is
> connected via i2c1, so add max11801: touchscreen@48 on i2c1.
>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Domenico Acri <[email protected]>
> Cc: Matteo Lisi <[email protected]>
> Cc: Michael Trimarchi <[email protected]>
> Cc: Shawn Guo <[email protected]>
> Signed-off-by: Jagan Teki <[email protected]>

Applied, thanks.