* [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-16 7:05 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-16 7:05 UTC (permalink / raw) To: linux-arm-kernel From: Jagan Teki <jagan@amarulasolutions.com> Add missing documentation of max11801-ts dt-binding details. 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 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..4afccbe --- /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 "max11801" +- reg: i2c slave address +- interrupt-parent: the phandle for the interrupt controller +- interrupts: touch controller interrupt + +Example: + +&i2c1 { + max11801: touchscreen at 48 { + compatible = "max11801"; + reg = <0x48>; + interrupt-parent = <&gpio3>; + interrupts = <31 2>; + }; +}; -- 1.9.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-16 7:05 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-16 7:05 UTC (permalink / raw) To: Shawn Guo Cc: linux-arm-kernel, devicetree, linux-kernel, Matteo Lisi, Michael Trimarchi, Jagan Teki, Mark Rutland, Rob Herring From: Jagan Teki <jagan@amarulasolutions.com> Add missing documentation of max11801-ts dt-binding details. 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 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..4afccbe --- /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 "max11801" +- reg: i2c slave address +- interrupt-parent: the phandle for the interrupt controller +- interrupts: touch controller interrupt + +Example: + +&i2c1 { + max11801: touchscreen@48 { + compatible = "max11801"; + reg = <0x48>; + interrupt-parent = <&gpio3>; + interrupts = <31 2>; + }; +}; -- 1.9.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-16 7:05 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-16 7:05 UTC (permalink / raw) To: Shawn Guo Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Matteo Lisi, Michael Trimarchi, Jagan Teki, Mark Rutland, Rob Herring From: Jagan Teki <jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> Add missing documentation of max11801-ts dt-binding details. Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Cc: Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Cc: Michael Trimarchi <michael-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> Signed-off-by: Jagan Teki <jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> --- 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..4afccbe --- /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 "max11801" +- reg: i2c slave address +- interrupt-parent: the phandle for the interrupt controller +- interrupts: touch controller interrupt + +Example: + +&i2c1 { + max11801: touchscreen@48 { + compatible = "max11801"; + reg = <0x48>; + interrupt-parent = <&gpio3>; + interrupts = <31 2>; + }; +}; -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v6 6/8] ARM: dts: imx6q-icore: Add touchscreen node @ 2017-03-16 7:05 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-16 7: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 tc: max11801 at 48 on i2c1. 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 v6: - Replace the lable and name of the node ts: max11801 => max11801: touchscreen at 48 Changes for v5: - %s/arm/ARM/g with subject prefix Changes for v4: - 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..8c1a572 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 = "max11801"; + reg = <0x48>; + interrupt-parent = <&gpio3>; + interrupts = <31 2>; + }; +}; + &ldb { status = "okay"; -- 1.9.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v6 6/8] ARM: dts: imx6q-icore: Add touchscreen node @ 2017-03-16 7:05 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-16 7:05 UTC (permalink / raw) To: Shawn Guo Cc: linux-arm-kernel, devicetree, linux-kernel, Matteo Lisi, Michael Trimarchi, Jagan Teki, Domenico Acri From: Jagan Teki <jagan@amarulasolutions.com> max11801 touchscreen on Engicam iCoreM6 Quad module is connected via i2c1, so add tc: max11801@48 on i2c1. 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 v6: - Replace the lable and name of the node ts: max11801 => max11801: touchscreen@48 Changes for v5: - %s/arm/ARM/g with subject prefix Changes for v4: - 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..8c1a572 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 = "max11801"; + reg = <0x48>; + interrupt-parent = <&gpio3>; + interrupts = <31 2>; + }; +}; + &ldb { status = "okay"; -- 1.9.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v6 6/8] ARM: dts: imx6q-icore: Add touchscreen node @ 2017-03-16 7:05 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-16 7:05 UTC (permalink / raw) To: Shawn Guo Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Matteo Lisi, Michael Trimarchi, Jagan Teki, Domenico Acri From: Jagan Teki <jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> max11801 touchscreen on Engicam iCoreM6 Quad module is connected via i2c1, so add tc: max11801@48 on i2c1. Cc: Domenico Acri <domenico.acri-4s7YQHO/iPVBDgjK7y7TUQ@public.gmane.org> Cc: Matteo Lisi <matteo.lisi-4s7YQHO/iPVBDgjK7y7TUQ@public.gmane.org> Cc: Michael Trimarchi <michael-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> Cc: Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Signed-off-by: Jagan Teki <jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> --- Changes for v6: - Replace the lable and name of the node ts: max11801 => max11801: touchscreen@48 Changes for v5: - %s/arm/ARM/g with subject prefix Changes for v4: - 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..8c1a572 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 = "max11801"; + reg = <0x48>; + interrupt-parent = <&gpio3>; + interrupts = <31 2>; + }; +}; + &ldb { status = "okay"; -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding 2017-03-16 7:05 ` Jagan Teki @ 2017-03-20 2:36 ` Shawn Guo -1 siblings, 0 replies; 14+ messages in thread From: Shawn Guo @ 2017-03-20 2:36 UTC (permalink / raw) To: linux-arm-kernel On Thu, Mar 16, 2017 at 12:35:38PM +0530, Jagan Teki wrote: > From: Jagan Teki <jagan@amarulasolutions.com> > > Add missing documentation of max11801-ts dt-binding details. > > 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 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..4afccbe > --- /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 "max11801" Shouldn't it have a vendor prefix? Also I will need Rob's ACK to get this go via IMX tree. Shawn > +- reg: i2c slave address > +- interrupt-parent: the phandle for the interrupt controller > +- interrupts: touch controller interrupt > + > +Example: > + > +&i2c1 { > + max11801: touchscreen at 48 { > + compatible = "max11801"; > + reg = <0x48>; > + interrupt-parent = <&gpio3>; > + interrupts = <31 2>; > + }; > +}; > -- > 1.9.1 > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-20 2:36 ` Shawn Guo 0 siblings, 0 replies; 14+ messages in thread From: Shawn Guo @ 2017-03-20 2:36 UTC (permalink / raw) To: Jagan Teki Cc: linux-arm-kernel, devicetree, linux-kernel, Matteo Lisi, Michael Trimarchi, Jagan Teki, Mark Rutland, Rob Herring On Thu, Mar 16, 2017 at 12:35:38PM +0530, Jagan Teki wrote: > From: Jagan Teki <jagan@amarulasolutions.com> > > Add missing documentation of max11801-ts dt-binding details. > > 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 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..4afccbe > --- /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 "max11801" Shouldn't it have a vendor prefix? Also I will need Rob's ACK to get this go via IMX tree. Shawn > +- reg: i2c slave address > +- interrupt-parent: the phandle for the interrupt controller > +- interrupts: touch controller interrupt > + > +Example: > + > +&i2c1 { > + max11801: touchscreen@48 { > + compatible = "max11801"; > + reg = <0x48>; > + interrupt-parent = <&gpio3>; > + interrupts = <31 2>; > + }; > +}; > -- > 1.9.1 > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding 2017-03-20 2:36 ` Shawn Guo (?) @ 2017-03-20 6:25 ` Jagan Teki -1 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-20 6:25 UTC (permalink / raw) To: linux-arm-kernel On Mon, Mar 20, 2017 at 8:06 AM, Shawn Guo <shawnguo@kernel.org> wrote: > On Thu, Mar 16, 2017 at 12:35:38PM +0530, Jagan Teki wrote: >> From: Jagan Teki <jagan@amarulasolutions.com> >> >> Add missing documentation of max11801-ts dt-binding details. >> >> 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 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..4afccbe >> --- /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 "max11801" > > Shouldn't it have a vendor prefix? Drivers i2c_device_id.name doesn't have vendor prefix, should I fix that one as well? thanks! -- Jagan Teki Senior Linux Kernel Engineer | Amarula Solutions U-Boot, Linux | Upstream Maintainer Hyderabad, India. ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-20 6:25 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-20 6:25 UTC (permalink / raw) To: Shawn Guo Cc: Jagan Teki, linux-arm-kernel, devicetree, linux-kernel, Matteo Lisi, Michael Trimarchi, Mark Rutland, Rob Herring On Mon, Mar 20, 2017 at 8:06 AM, Shawn Guo <shawnguo@kernel.org> wrote: > On Thu, Mar 16, 2017 at 12:35:38PM +0530, Jagan Teki wrote: >> From: Jagan Teki <jagan@amarulasolutions.com> >> >> Add missing documentation of max11801-ts dt-binding details. >> >> 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 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..4afccbe >> --- /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 "max11801" > > Shouldn't it have a vendor prefix? Drivers i2c_device_id.name doesn't have vendor prefix, should I fix that one as well? thanks! -- Jagan Teki Senior Linux Kernel Engineer | Amarula Solutions U-Boot, Linux | Upstream Maintainer Hyderabad, India. ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-20 6:25 ` Jagan Teki 0 siblings, 0 replies; 14+ messages in thread From: Jagan Teki @ 2017-03-20 6:25 UTC (permalink / raw) To: Shawn Guo Cc: Mark Rutland, devicetree, Matteo Lisi, linux-kernel, Rob Herring, Michael Trimarchi, Jagan Teki, linux-arm-kernel On Mon, Mar 20, 2017 at 8:06 AM, Shawn Guo <shawnguo@kernel.org> wrote: > On Thu, Mar 16, 2017 at 12:35:38PM +0530, Jagan Teki wrote: >> From: Jagan Teki <jagan@amarulasolutions.com> >> >> Add missing documentation of max11801-ts dt-binding details. >> >> 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 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..4afccbe >> --- /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 "max11801" > > Shouldn't it have a vendor prefix? Drivers i2c_device_id.name doesn't have vendor prefix, should I fix that one as well? thanks! -- Jagan Teki Senior Linux Kernel Engineer | Amarula Solutions U-Boot, Linux | Upstream Maintainer Hyderabad, India. ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding 2017-03-20 6:25 ` Jagan Teki (?) @ 2017-03-20 7:43 ` Shawn Guo -1 siblings, 0 replies; 14+ messages in thread From: Shawn Guo @ 2017-03-20 7:43 UTC (permalink / raw) To: linux-arm-kernel On Mon, Mar 20, 2017 at 11:55:27AM +0530, Jagan Teki wrote: > >> @@ -0,0 +1,18 @@ > >> +* MAXI MAX11801 Resistive touch screen controller with i2c interface > >> + > >> +Required properties: > >> +- compatible: must be "max11801" > > > > Shouldn't it have a vendor prefix? > > Drivers i2c_device_id.name doesn't have vendor prefix, should I fix > that one as well? No. The vendor prefix is only needed for device tree compatible. Shawn ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-20 7:43 ` Shawn Guo 0 siblings, 0 replies; 14+ messages in thread From: Shawn Guo @ 2017-03-20 7:43 UTC (permalink / raw) To: Jagan Teki Cc: Jagan Teki, linux-arm-kernel, devicetree, linux-kernel, Matteo Lisi, Michael Trimarchi, Mark Rutland, Rob Herring On Mon, Mar 20, 2017 at 11:55:27AM +0530, Jagan Teki wrote: > >> @@ -0,0 +1,18 @@ > >> +* MAXI MAX11801 Resistive touch screen controller with i2c interface > >> + > >> +Required properties: > >> +- compatible: must be "max11801" > > > > Shouldn't it have a vendor prefix? > > Drivers i2c_device_id.name doesn't have vendor prefix, should I fix > that one as well? No. The vendor prefix is only needed for device tree compatible. Shawn ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding @ 2017-03-20 7:43 ` Shawn Guo 0 siblings, 0 replies; 14+ messages in thread From: Shawn Guo @ 2017-03-20 7:43 UTC (permalink / raw) To: Jagan Teki Cc: Mark Rutland, devicetree, Matteo Lisi, linux-kernel, Rob Herring, Michael Trimarchi, Jagan Teki, linux-arm-kernel On Mon, Mar 20, 2017 at 11:55:27AM +0530, Jagan Teki wrote: > >> @@ -0,0 +1,18 @@ > >> +* MAXI MAX11801 Resistive touch screen controller with i2c interface > >> + > >> +Required properties: > >> +- compatible: must be "max11801" > > > > Shouldn't it have a vendor prefix? > > Drivers i2c_device_id.name doesn't have vendor prefix, should I fix > that one as well? No. The vendor prefix is only needed for device tree compatible. Shawn ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2017-03-20 7:44 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-03-16 7:05 [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding Jagan Teki 2017-03-16 7:05 ` Jagan Teki 2017-03-16 7:05 ` Jagan Teki 2017-03-16 7:05 ` [PATCH v6 6/8] ARM: dts: imx6q-icore: Add touchscreen node Jagan Teki 2017-03-16 7:05 ` Jagan Teki 2017-03-16 7:05 ` Jagan Teki 2017-03-20 2:36 ` [PATCH v6 5/8] dt-bindings: input: touchscreen: Add max11801-ts binding Shawn Guo 2017-03-20 2:36 ` Shawn Guo 2017-03-20 6:25 ` Jagan Teki 2017-03-20 6:25 ` Jagan Teki 2017-03-20 6:25 ` Jagan Teki 2017-03-20 7:43 ` Shawn Guo 2017-03-20 7:43 ` Shawn Guo 2017-03-20 7:43 ` Shawn Guo
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.