Messages in this thread | | | From | Eric Miao <> | Date | Wed, 9 Feb 2011 16:02:27 +0800 | Subject | Re: [RFC v2 PATCH 1/7] matrix_keypad: Increase the max limit of rows and columns |
| |
On Tue, Feb 1, 2011 at 9:47 PM, Anirudh Ghayal <aghayal@codeaurora.org> wrote: > From: Trilok Soni <tsoni@codeaurora.org> > > Some keyboard controller have support for more than > 16 columns and rows. Moving this value to 32. > > Cc: Eric Miao <eric.y.miao@gmail.com> > Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com> > Signed-off-by: Trilok Soni <tsoni@codeaurora.org>
This looks good to me, as long as the bits don't exceed 256, which is used in the #KEY() macro. So far, 32 looks to be a sane number to me, unless we have further requirement to get this larger.
Acked-by: Eric Miao <eric.y.miao@gmail.com>
> --- > Changes from v1: > Moved the max columns and rows to 32 > > include/linux/input/matrix_keypad.h | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/linux/input/matrix_keypad.h b/include/linux/input/matrix_keypad.h > index 6974746..fe7c4b9 100644 > --- a/include/linux/input/matrix_keypad.h > +++ b/include/linux/input/matrix_keypad.h > @@ -4,8 +4,8 @@ > #include <linux/types.h> > #include <linux/input.h> > > -#define MATRIX_MAX_ROWS 16 > -#define MATRIX_MAX_COLS 16 > +#define MATRIX_MAX_ROWS 32 > +#define MATRIX_MAX_COLS 32 > > #define KEY(row, col, val) ((((row) & (MATRIX_MAX_ROWS - 1)) << 24) |\ > (((col) & (MATRIX_MAX_COLS - 1)) << 16) |\ > -- > 1.7.3.5 > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |