From: Rob Herring <robh@kernel.org>
To: Markus Pargmann <mpa@pengutronix.de>
Cc: "Shawn Guo" <shawnguo@kernel.org>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"Jonathan Cameron" <jic23@kernel.org>,
"Lee Jones" <lee.jones@linaro.org>,
"Denis Carikli" <denis@eukrea.com>,
"Eric Bénard" <eric@eukrea.com>,
"Sascha Hauer" <kernel@pengutronix.de>,
devicetree@vger.kernel.org, linux-input@vger.kernel.org,
linux-iio@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
"Hartmut Knaack" <knaack.h@gmx.de>,
"Fabio Estevam" <festevam@gmail.com>
Subject: Re: [PATCH v9 3/8] ARM: dt: Binding documentation for imx25 touchscreen controller
Date: Mon, 7 Dec 2015 14:45:46 -0600 [thread overview]
Message-ID: <20151207204546.GA30936@rob-hp-laptop> (raw)
In-Reply-To: <1449431542-1892-4-git-send-email-mpa@pengutronix.de>
On Sun, Dec 06, 2015 at 08:52:16PM +0100, Markus Pargmann wrote:
> This is the touchscreen conversion queue binding documentation. It uses
> the shared imx25 ADC.
>
> Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
> ---
>
> Notes:
> Changes in v9:
> - Updated binding descriptions
>
> Changes in v5:
> - Fix signed/unsigned comparison
> - Fix unused variable settling_time by putting it in the correct argument list
> - Use continous conversion queue with the repeat feature and a proper
> repeat-wait. Previously the touchscreen caused massive number of interrupts.
>
> .../bindings/input/touchscreen/fsl-mx25-tcq.txt | 35 ++++++++++++++++++++++
> 1 file changed, 35 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/input/touchscreen/fsl-mx25-tcq.txt
>
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/fsl-mx25-tcq.txt b/Documentation/devicetree/bindings/input/touchscreen/fsl-mx25-tcq.txt
> new file mode 100644
> index 000000000000..d45e50872114
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/input/touchscreen/fsl-mx25-tcq.txt
> @@ -0,0 +1,35 @@
> +Freescale mx25 TS conversion queue module
> +
> +mx25 touchscreen conversion queue module which controls the ADC unit of the
> +mx25 for attached touchscreens.
> +
> +Required properties:
> + - compatible: Should be "fsl,imx25-tcq".
> + - reg: Memory range of the device.
> + - interrupts: Should be the interrupt number associated with this module within
> + the tscadc unit (<0>).
> + - interrupt-parent: Should be a phandle to the tscadc unit.
> + - fsl,wires: Should be '<4>' or '<5>'
> +
> +Optional properties:
> + - fsl,pen-debounce: Pen debounce time in nanoseconds.
This and ...
> + - fsl,pen-threshold: Pen-down threshold for the touchscreen. This is a value
> + between 1 and 4096. It is the ratio between the internal reference voltage
> + and the measured voltage after the plate was precharged. Resistence between
> + plates and therefore the voltage decreases with pressure so that a smaller
> + value is equivalent to a higher pressure.
> + - fsl,settling-time: Settling time in nanoseconds. The settling time is before
> + the actual touch detection to wait for an even charge distribution in the
> + plate.
... this still need units (-ns) appended to the property name.
> +
> +This device includes two conversion queues which can be added as subnodes.
> +The first queue is for the touchscreen, the second for general purpose ADC.
> +
> +Example:
> + tsc: tcq@50030400 {
> + compatible = "fsl,imx25-tcq";
> + reg = <0x50030400 0x60>;
> + interrupt-parent = <&tscadc>;
> + interrupts = <0>;
> + fsl,wires = <4>;
> + };
> --
> 2.6.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-12-07 20:45 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-06 19:52 [PATCH v9 0/8] imx25 adc and touchscreen driver Markus Pargmann
2015-12-06 19:52 ` [PATCH v9 1/8] ARM: dt: Binding documentation for imx25 ADC/TSC Markus Pargmann
2015-12-06 19:52 ` [PATCH v9 2/8] ARM: dt: Binding documentation for imx25 GCQ Markus Pargmann
2015-12-07 17:39 ` Rob Herring
2015-12-12 17:03 ` Jonathan Cameron
[not found] ` <1449431542-1892-1-git-send-email-mpa-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2015-12-06 19:52 ` [PATCH v9 3/8] ARM: dt: Binding documentation for imx25 touchscreen controller Markus Pargmann
2015-12-07 20:45 ` Rob Herring [this message]
2015-12-08 10:33 ` Markus Pargmann
2015-12-06 19:52 ` [PATCH v9 4/8] mfd: fsl imx25 Touchscreen ADC driver Markus Pargmann
2015-12-06 19:52 ` [PATCH v9 6/8] input: touchscreen: imx25 tcq driver Markus Pargmann
2015-12-06 19:52 ` [PATCH v9 5/8] iio: adc: fsl,imx25-gcq driver Markus Pargmann
2015-12-12 17:14 ` Jonathan Cameron
[not found] ` <566C55F0.6080901-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2015-12-14 11:26 ` Markus Pargmann
2015-12-06 19:52 ` [PATCH v9 7/8] ARM: dts: imx25: Add TSC and ADC support Markus Pargmann
2015-12-06 19:52 ` [PATCH v9 8/8] ARM: imx_v4_v5_defconfig: Add I.MX25 Touchscreen controller " Markus Pargmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151207204546.GA30936@rob-hp-laptop \
--to=robh@kernel.org \
--cc=denis@eukrea.com \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=eric@eukrea.com \
--cc=festevam@gmail.com \
--cc=jic23@kernel.org \
--cc=kernel@pengutronix.de \
--cc=knaack.h@gmx.de \
--cc=lee.jones@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=mpa@pengutronix.de \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox