On Sat, May 14, 2016 at 7:52 AM, Rob Herring [off-list ref] wrote:

On Tue, May 10, 2016 at 07:07:48PM -0700, Grant Grundler wrote:

quoted

From: Daniel Hung-yu Wu <redacted>

Add binding for Atmel Capacitive Touch Button device.

Signed-off-by: Daniel Hung-yu Wu <redacted>
Signed-off-by: Grant Grundler <redacted>
---
 .../devicetree/bindings/input/atmel,captouch.txt   | 34 ++++++++++++++++++++++
 1 file changed, 34 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/input/atmel,captouch.txt

I haven't posted the corresponding device driver and dtsi changes yet but will shortly.

V2: split Documentation/devicetree/bindings from driver patch
    rename "atmel,atmegaxx_captouch" to "atmel,captouch" since it's irrelevant
           which controller implements the register set/controller.

Is there any notion of firmware versions though?

There currently is not. This is something we need to add - future patch.

quoted

V1: posted https://lkml.org/lkml/2016/5/3/613

diff --git a/Documentation/devicetree/bindings/input/atmel,captouch.txt b/Documentation/devicetree/bindings/input/atmel,captouch.txt
new file mode 100644
index 0000000..405d287
--- /dev/null
+++ b/Documentation/devicetree/bindings/input/atmel,captouch.txt

@@ -0,0 +1,34 @@
+Device tree bindings for Atmel AtmegaXX capacitive touch sensor
+
+The node for this device must be a child of a I2C controller node, as the
+device communicates via I2C.
+
+Required properties:
+
+     compatible:     Must be "atmel,captouch".
+     reg:            The I2C slave address of the device.
+     interrupts:     Property describing the interrupt line the device
+                     is connected to. The device only has one interrupt
+                     source.
+     linux,keycodes: Specifies an array of numeric keycode values to
+                     be used for reporting button presses. The array can
+                     contain up to 8 entries.
+
+Optional properties:
+
+     autorepeat:     Enables the Linux input system's autorepeat
+                     feature on the input device.
+
+Example:
+
+     atmel_captouch@51 {

No underscores in node names please.

Aaahh! Sorry! you told me that before. Will post V3 with dash in node name.

cheers,
grant

quoted

+             compatible = "atmel,captouch";
+             reg = <0x51>;
+             interrupt-parent = <&tlmm>;
+             interrupts = <67 IRQ_TYPE_EDGE_FALLING>;
+             linux,keycodes = <BTN_0>, <BTN_1>,
+                     <BTN_2>, <BTN_3>,
+                     <BTN_4>, <BTN_5>,
+                     <BTN_6>, <BTN_7>;
+             autorepeat;
+     };
--
2.1.2