On Mon, Oct 28, 2013 at 01:42:47AM -0500, Kumar Gala wrote:

quoted

+This binding is based on the matrix-keymap binding with the following
+changes:

Maybe be a bit more specific and say 'based on the input/matrix-keymap.txt binding'..

OK.

quoted

+ * keypad,num-rows and keypad,num-columns are required.

Is linux,keymap required from matrix-keymap.txt?

Yes, matrix-keymap.txt contains descriptions for the following:

required:
 - linux,keymap
optional:
 - keypad,num-rows
 - keypad,num-columns

quoted

+Optional Properties specific to linux:
+- linux,keypad-no-autorepeat: do no enable autorepeat feature.

Does it make sense to update the matrix-keymap.txt binding to add
'linux,keypad-no-autorepeat' there?

At least according to devicetree documentation there are
keymap-matrix.txt based drivers, which do not support
"linux,keypad-no-autorepeat".

-- Sebastian