Skip to contentSkip to conversation

Input: max11801_ts: Add missing of_match_table

PATCH Applied Jagan Teki 4 participants 7 messages Mar 22, 2017 – Mar 29, 2017

Subsystems (4)
lore
Share & tools
lore

Jagan Teki

author
#0 lore raw

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]>

---
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(+)
drivers/input/touchscreen/max11801_ts.c +7 -0
index a595ae5..d3c7fc0 100644
@@ -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,
Signature · 1 line
1.9.1
author
#1 lore raw

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]>

---
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
Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt new file +18 -0
new file mode 100644
index 0000000..40ac0fe
@@ -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>;
+	};
+};
Signature · 1 line
1.9.1
maintainer Acked-by on v8 2/3
#2 lore raw

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

Quoted 21 lines · Jagan Teki

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]>

---
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

Quoted 3 lines
 .../bindings/input/touchscreen/max11801-ts.txt         | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt
maintainer Applied
#3 lore raw

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

Quoted 22 lines · Rob Herring
Quoted 21 lines · Jagan Teki

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]>

---
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

Quoted 5 lines
Quoted 3 lines
 .../bindings/input/touchscreen/max11801-ts.txt         | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/input/touchscreen/max11801-ts.txt

Applied, thank you.

Signature · 1 line
Dmitry
author
#4 lore raw

PATCH v8 3/3 ARM: dts: imx6q-icore: Add touchscreen node

In reply to Jagan Teki

From: Jagan Teki <[email protected]>

max11801 touchscreen on Engicam iCoreM6 Quad module is connected via i2c1, so add max11801: touchscreen at 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]>

---
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(+)
arch/arm/boot/dts/imx6q-icore.dts +9 -0
index 73f34d1..5613dd9 100644
@@ -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";
 
Signature · 1 line
1.9.1
Applied
#5 lore raw

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

Quoted 11 lines · Jagan Teki

From: Jagan Teki <[email protected]>

max11801 touchscreen on Engicam iCoreM6 Quad module is connected via i2c1, so add max11801: touchscreen at 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]>

Applied, thanks.

maintainer Applied
#6 lore raw
In reply to Jagan Teki
Quoted 10 lines · Jagan Teki

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]>

Applied, thank you.

Quoted 34 lines

---
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(+)
drivers/input/touchscreen/max11801_ts.c +7 -0
index a595ae5..d3c7fc0 100644
@@ -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,
Signature · 1 line
1.9.1
Signature · 1 line
Dmitry