Re: [PATCH] dt: add Atmel Captouch bindings
On Sat, May 14, 2016 at 7:52 AM, Rob Herring wrote: > On Tue, May 10, 2016 at 07:07:48PM -0700, Grant Grundler wrote: >> From: Daniel Hung-yu Wu >> >> Add binding for Atmel Capacitive Touch Button device. >> >> Signed-off-by: Daniel Hung-yu Wu >> Signed-off-by: Grant Grundler >> --- >> .../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. > >> >> 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 000..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 > >> + compatible = "atmel,captouch"; >> + reg = <0x51>; >> + interrupt-parent = <&tlmm>; >> + interrupts = <67 IRQ_TYPE_EDGE_FALLING>; >> + linux,keycodes = , , >> + , , >> + , , >> + , ; >> + autorepeat; >> + }; >> -- >> 2.1.2 >>
Re: [PATCH] dt: add Atmel Captouch bindings
On Tue, May 10, 2016 at 07:07:48PM -0700, Grant Grundler wrote: > From: Daniel Hung-yu Wu > > Add binding for Atmel Capacitive Touch Button device. > > Signed-off-by: Daniel Hung-yu Wu > Signed-off-by: Grant Grundler > --- > .../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? > > 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 000..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. > + compatible = "atmel,captouch"; > + reg = <0x51>; > + interrupt-parent = <&tlmm>; > + interrupts = <67 IRQ_TYPE_EDGE_FALLING>; > + linux,keycodes = , , > + , , > + , , > + , ; > + autorepeat; > + }; > -- > 2.1.2 >
Re: [PATCH] dt: add Atmel Captouch bindings
On Tue, May 10, 2016 at 7:07 PM, Grant Grundler wrote: > From: Daniel Hung-yu Wu > > Add binding for Atmel Capacitive Touch Button device. > > Signed-off-by: Daniel Hung-yu Wu > Signed-off-by: Grant Grundler > --- > .../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. Rob, are you OK with the new bindings? I'd like to merge the driver if there are no objections from DT folks... > > 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 000..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 { > + compatible = "atmel,captouch"; > + reg = <0x51>; > + interrupt-parent = <&tlmm>; > + interrupts = <67 IRQ_TYPE_EDGE_FALLING>; > + linux,keycodes = , , > + , , > + , , > + , ; > + autorepeat; > + }; > -- > 2.1.2 > -- Dmitry