Re: [PATCH] input: bitmap update for sh_keysc V2
From: Magnus Damm <magnus.damm@gmail.com>
Date: 2010-02-10 04:20:59
Also in:
linux-sh
From: Magnus Damm <magnus.damm@gmail.com>
Date: 2010-02-10 04:20:59
Also in:
linux-sh
On Tue, Feb 9, 2010 at 6:39 PM, Dmitry Torokhov [off-list ref] wrote:
On Tue, Feb 09, 2010 at 06:16:09PM +0900, Magnus Damm wrote:quoted
From: Magnus Damm <redacted> Update the sh_keysc driver with proper bitmap support V2. This instead of using a fixed 32-bit integer to keep track for the key states. With this change in place the driver supports key pads with more than 32 keys. Signed-off-by: Magnus Damm <redacted> --- Changes since V1: - ditched the wrapping macros - use __set_bit() and __clear_bit()Thank you for making the change. It indeed is easier to read than the original since one does not have to go and look up what wrappers are doing.
Yeah, I initially disliked the SH_KEYSC_MAXKEYS that now are sprinkled all over the place, but I have to admit that not using wrappers made me realize that I should use __set_bit() and __clear_bit() instead of the atomic ones. So all good. Thanks! / magnus -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html