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

From: Marcel Ziswiler
Date: Wed Feb 03 2016 - 18:17:14 EST


On Wed, 2016-02-03 at 23:55 +0100, Marcel Ziswiler wrote:
> On Mon, 2016-02-01 at 21:29 +0800, Shawn Guo wrote:
>
> [snip]
>
> > + /* STMPE811 touch screen controller */
> > > + 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 <linus.walleij@xxxxxxxxxx>
> 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 <lee.jones@xxxxxxxxxx>
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.