public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 3/3] fakehid: Use the same constant as declared
@ 2012-06-16 23:32 Bastien Nocera
  0 siblings, 0 replies; 2+ messages in thread
From: Bastien Nocera @ 2012-06-16 23:32 UTC (permalink / raw)
  To: linux-bluetooth

ps3remote_keymap[] uses 0xff as the max value, so should we.
---
 input/fakehid.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/input/fakehid.c b/input/fakehid.c
index a125356..a758413 100644
--- a/input/fakehid.c
+++ b/input/fakehid.c
@@ -335,7 +335,7 @@ static int ps3remote_setup_uinput(struct fake_input *fake,
 	}
 
 	/* enabling keys */
-	for (i = 0; i < 256; i++)
+	for (i = 0; i < 0xff; i++)
 		if (ps3remote_keymap[i] != KEY_RESERVED)
 			if (ioctl(fake->uinput, UI_SET_KEYBIT,
 						ps3remote_keymap[i]) < 0) {
-- 
1.7.10



^ permalink raw reply related	[flat|nested] 2+ messages in thread
* [PATCH 3/3] fakehid: Use the same constant as declared
@ 2012-09-07 11:01 Bastien Nocera
  0 siblings, 0 replies; 2+ messages in thread
From: Bastien Nocera @ 2012-09-07 11:01 UTC (permalink / raw)
  To: linux-bluetooth


ps3remote_keymap[] uses 0xff as the max value, so should we.
---
 profiles/input/fakehid.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/profiles/input/fakehid.c b/profiles/input/fakehid.c
index b12c526..e665869 100644
--- a/profiles/input/fakehid.c
+++ b/profiles/input/fakehid.c
@@ -328,7 +328,7 @@ static int ps3remote_setup_uinput(struct fake_input *fake,
 	}
 
 	/* enabling keys */
-	for (i = 0; i < 256; i++)
+	for (i = 0; i < 0xff; i++)
 		if (ps3remote_keymap[i] != KEY_RESERVED)
 			if (ioctl(fake->uinput, UI_SET_KEYBIT,
 						ps3remote_keymap[i]) < 0) {
-- 
1.7.12



^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-09-07 11:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-06-16 23:32 [PATCH 3/3] fakehid: Use the same constant as declared Bastien Nocera
  -- strict thread matches above, loose matches on Subject: below --
2012-09-07 11:01 Bastien Nocera

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox