[PATCH] msm: trout: add keypad support
dwalker at codeaurora.org
Thu Dec 9 17:03:36 EST 2010
On Thu, 2010-12-09 at 14:01 -0800, Arve Hjønnevåg wrote:
> On Thu, Dec 9, 2010 at 9:45 AM, Daniel Walker <dwalker at codeaurora.org> wrote:
> > On Thu, 2010-12-09 at 12:36 +0530, Trilok Soni wrote:
> >> > +static char *keycaps = "--qwerty";
> >> > +#undef MODULE_PARAM_PREFIX
> >> > +#define MODULE_PARAM_PREFIX "board_trout."
> >> > +module_param_named(keycaps, keycaps, charp, 0);
> >> I would like to know what is the use of this param?
> > No idea .. It was part of Google's implementation ..
> You removed the code that uses it. If you look at the original file
> you can see how it is used:
> It is set by the bootloader according to the hardware version.
> You can also see that the "/* userspace needs to know about these
> changes as well */" comments refer to the code that changes the name
> of the input device.
> I don't think this change will work correctly with our existing
> user-space code though since you are now reporting the power-key,
> camera-key and slide-switch from a separate input device.
Yeah, I did remove those parts.. I'm not sure where to put that stuff in
this new implementation. I can try to find an appropriate place , unless
you have some suggestions ?
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
More information about the linux-arm-kernel