Re: [PATCH v4 1/2] ARM: dts: imx6: Add support for Toradex Apalis iMX6Q/D SoM

On Thu, Feb 4, 2016 at 12:02 AM, Marcel Ziswiler
[off-list ref] wrote:

On Wed, 2016-02-03 at 23:55 +0100, Marcel Ziswiler wrote:

quoted

On Mon, 2016-02-01 at 21:29 +0800, Shawn Guo wrote:

[snip]

quoted

+   /* STMPE811 touch screen controller */

quoted

+ stmpe811@41 {
+         compatible = "st,stmpe811";

The compatible seems to be used by kernel without a binding
document.

Indeed and looking at git logs Linus himself introduced it (;-p):

commit 5a826feedc331a2d5ced2afd832199a70b3af891
Author: Linus Walleij [off-list ref]
Date:   Wed Apr 23 23:35:58 2014 +0200
Subject: mfd: stmpe: Probe properly from the Device Tree

Looking more closely there is actually a proper binding document which
got added by Lee Jones:

commit 84e6de813b2d1bdb127943d3c8edf1c1afaa90da
Author: Lee Jones [off-list ref]
Date:   Mon Nov 5 16:10:34 2012 +0100
Subject: Documentation: Describe bindings for STMPE Multi-Functional
Device driver

Documentation/devicetree/bindings/mfd/stmpe.txt

Required properties:
 - compatible                   : "st,stmpe[811|1601|2401|2403]"

Sorry for the noise.

Yeah the STMPE driver was a common work between Rabin and
Luotao Fu who pitched in the 811 with touchscreen support.

Then Lee added the device tree stuff.

I know it's a bit confusing that the DT bindings for the whole shebang
is in bindings/mfd but it is also confusing to split them up, don't know
what is best really :(

Yours,
Linus Walleij