Thread (10 messages) 10 messages, 4 authors, 2012-04-26

Re: [PATCHv4 1/2] drivers: input: keypad: Add device tree support

From: Poddar, Sourav <hidden>
Date: 2012-04-25 13:00:43
Also in: linux-arm-kernel, linux-devicetree, lkml

Hi Roland,

On Wed, Apr 25, 2012 at 5:45 PM, Roland Stigge [off-list ref] wrote:
Hi!

On 04/25/2012 01:56 PM, Poddar, Sourav wrote:
quoted
Even I was using the bindings as used by samsung-keypad driver. But according
to the comments and logically too, we should use the common matrix keyboard
bindings already descibed in
Documentation/devicetree/bindings/input/matrix-keymap.txt.
Fine with me. Can switch over to
Documentation/devicetree/bindings/input/matrix-keymap.txt

Maybe samsung-keypad should also be converted because it's subnodes
structure duplicates exactly the same thing (row,col,code) in a
different way, and we both were distracted by it, ignoring
matrix-keymap.txt? Or is it already too late (both samsung-keypad and
Not sure about the standards, but surely have seen few others like tegra
keyboards using common matrix-keyboard bindings.

One more comment which I saw in some other mail thread about
using the following
              key_1 {
                        keypad,row = <0>;
                        keypad,column = <3>;
                        linux,code = <2>;
                };
is that when there are too many keys, the whole data become quilte huge.
These problem get sorted out on using matrix-keymap bindings, since there
is a single value computed in order to describe a particular key.

~Sourav
matrix-keypad both being standards in use) and we should put a
recommendation into one of them to use the other one for new designs?
Thanks,

Roland
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help