* [PATCH V2 01/10] input: touchscreen: ili210x: Add DT binding document [not found] <20181220204305.28807-1-marex@denx.de> @ 2018-12-20 20:42 ` Marek Vasut 2018-12-27 21:41 ` Rob Herring 0 siblings, 1 reply; 3+ messages in thread From: Marek Vasut @ 2018-12-20 20:42 UTC (permalink / raw) To: linux-input Cc: Marek Vasut, Dmitry Torokhov, Henrik Rydberg, Olivier Sobrie, Philipp Puschmann, Rob Herring, devicetree Add DT binding document for the Ilitek ILI210x and ILI251x touchscreen controllers. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> Cc: Henrik Rydberg <rydberg@bitmath.org> Cc: Olivier Sobrie <olivier@sobrie.be> Cc: Philipp Puschmann <pp@emlix.com> Cc: Rob Herring <robh+dt@kernel.org> Cc: devicetree@vger.kernel.org To: linux-input@vger.kernel.org --- V2: New patch --- .../bindings/input/ilitek,ili2xxx.txt | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+) create mode 100644 Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt diff --git a/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt b/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt new file mode 100644 index 000000000000..bc256557ae8c --- /dev/null +++ b/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt @@ -0,0 +1,25 @@ +Ilitek ILI210x/ILI251x touchscreen controller + +Required properties: +- compatible: + ilitek,ili210x for ILI210x + ilitek,ili251x for ILI251x + +- reg: The I2C address of the device + +- interrupts: The sink for the touchscreen's IRQ output + See ../interrupt-controller/interrupts.txt + +Optional properties for main touchpad device: + +- reset-gpios: GPIO specifier for the touchscreen's reset pin (active low) + +Example: + + touch@41 { + compatible = "ilitek,ili251x"; + reg = <0x41>; + interrupt-parent = <&gpio4>; + interrupts = <7 IRQ_TYPE_EDGE_FALLING>; + reset-gpios = <&gpio5 21 GPIO_ACTIVE_LOW>; + }; -- 2.19.2 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH V2 01/10] input: touchscreen: ili210x: Add DT binding document 2018-12-20 20:42 ` [PATCH V2 01/10] input: touchscreen: ili210x: Add DT binding document Marek Vasut @ 2018-12-27 21:41 ` Rob Herring 2018-12-30 16:01 ` Marek Vasut 0 siblings, 1 reply; 3+ messages in thread From: Rob Herring @ 2018-12-27 21:41 UTC (permalink / raw) To: Marek Vasut Cc: linux-input, Dmitry Torokhov, Henrik Rydberg, Olivier Sobrie, Philipp Puschmann, devicetree On Thu, Dec 20, 2018 at 09:42:56PM +0100, Marek Vasut wrote: > Add DT binding document for the Ilitek ILI210x and ILI251x > touchscreen controllers. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> > Cc: Henrik Rydberg <rydberg@bitmath.org> > Cc: Olivier Sobrie <olivier@sobrie.be> > Cc: Philipp Puschmann <pp@emlix.com> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: devicetree@vger.kernel.org > To: linux-input@vger.kernel.org > --- > V2: New patch > --- > .../bindings/input/ilitek,ili2xxx.txt | 25 +++++++++++++++++++ > 1 file changed, 25 insertions(+) > create mode 100644 Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt > > diff --git a/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt b/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt > new file mode 100644 > index 000000000000..bc256557ae8c > --- /dev/null > +++ b/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt > @@ -0,0 +1,25 @@ > +Ilitek ILI210x/ILI251x touchscreen controller > + > +Required properties: > +- compatible: > + ilitek,ili210x for ILI210x > + ilitek,ili251x for ILI251x Generally, we don't have wildcards in compatible strings. What's the 'x' signify? I couldn't find much info on these parts. > + > +- reg: The I2C address of the device > + > +- interrupts: The sink for the touchscreen's IRQ output > + See ../interrupt-controller/interrupts.txt > + > +Optional properties for main touchpad device: > + > +- reset-gpios: GPIO specifier for the touchscreen's reset pin (active low) > + > +Example: > + > + touch@41 { touchscreen@41 > + compatible = "ilitek,ili251x"; > + reg = <0x41>; > + interrupt-parent = <&gpio4>; > + interrupts = <7 IRQ_TYPE_EDGE_FALLING>; > + reset-gpios = <&gpio5 21 GPIO_ACTIVE_LOW>; > + }; > -- > 2.19.2 > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V2 01/10] input: touchscreen: ili210x: Add DT binding document 2018-12-27 21:41 ` Rob Herring @ 2018-12-30 16:01 ` Marek Vasut 0 siblings, 0 replies; 3+ messages in thread From: Marek Vasut @ 2018-12-30 16:01 UTC (permalink / raw) To: Rob Herring Cc: linux-input, Dmitry Torokhov, Henrik Rydberg, Olivier Sobrie, Philipp Puschmann, devicetree On 12/27/18 10:41 PM, Rob Herring wrote: > On Thu, Dec 20, 2018 at 09:42:56PM +0100, Marek Vasut wrote: >> Add DT binding document for the Ilitek ILI210x and ILI251x >> touchscreen controllers. >> >> Signed-off-by: Marek Vasut <marex@denx.de> >> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> >> Cc: Henrik Rydberg <rydberg@bitmath.org> >> Cc: Olivier Sobrie <olivier@sobrie.be> >> Cc: Philipp Puschmann <pp@emlix.com> >> Cc: Rob Herring <robh+dt@kernel.org> >> Cc: devicetree@vger.kernel.org >> To: linux-input@vger.kernel.org >> --- >> V2: New patch >> --- >> .../bindings/input/ilitek,ili2xxx.txt | 25 +++++++++++++++++++ >> 1 file changed, 25 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt >> >> diff --git a/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt b/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt >> new file mode 100644 >> index 000000000000..bc256557ae8c >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/input/ilitek,ili2xxx.txt >> @@ -0,0 +1,25 @@ >> +Ilitek ILI210x/ILI251x touchscreen controller >> + >> +Required properties: >> +- compatible: >> + ilitek,ili210x for ILI210x >> + ilitek,ili251x for ILI251x > > Generally, we don't have wildcards in compatible strings. What's the 'x' > signify? I couldn't find much info on these parts. Presumably there are other parts in that family of chips. I don't have the datasheet for any of the chips either, I extracted the necessary driver adjustments from the ilitek kernel driver (6500 kLoC , where C probably doesn't stand for Code, but something else ...). In my case, the chip itself is attached to the panel assembly, which is covered in thick aluminum casing, so I'm unable to visually check the chip markings ; if there are any on it. As for the older ili210x, that's at least what some DTs might've matched on as an i2c simple device. However, there's no in-tree DT that uses it. >> + >> +- reg: The I2C address of the device >> + >> +- interrupts: The sink for the touchscreen's IRQ output >> + See ../interrupt-controller/interrupts.txt >> + >> +Optional properties for main touchpad device: >> + >> +- reset-gpios: GPIO specifier for the touchscreen's reset pin (active low) >> + >> +Example: >> + >> + touch@41 { > > touchscreen@41 Fixed >> + compatible = "ilitek,ili251x"; >> + reg = <0x41>; >> + interrupt-parent = <&gpio4>; >> + interrupts = <7 IRQ_TYPE_EDGE_FALLING>; >> + reset-gpios = <&gpio5 21 GPIO_ACTIVE_LOW>; >> + }; >> -- >> 2.19.2 >> -- Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-12-30 16:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20181220204305.28807-1-marex@denx.de>
2018-12-20 20:42 ` [PATCH V2 01/10] input: touchscreen: ili210x: Add DT binding document Marek Vasut
2018-12-27 21:41 ` Rob Herring
2018-12-30 16:01 ` Marek Vasut
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).