Thread (6 messages) 6 messages, 2 authors, 2010-02-10

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

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help