From afc00cad1fc7de678afbf0003b371f37876d8c09 Mon Sep 17 00:00:00 2001
From: notaz <notasas@gmail.com>
Date: Thu, 9 Dec 2010 22:21:07 +0000
Subject: [PATCH] in_evdev: revert multi event handling from r901 to avoid
 unwanted blocking

git-svn-id: file:///home/notaz/opt/svn/PicoDrive@910 be3aeb3a-fb24-0410-a615-afba39da0efa
---
 platform/linux/in_evdev.c | 95 +++++++++++++++++++--------------------
 1 file changed, 47 insertions(+), 48 deletions(-)

diff --git a/platform/linux/in_evdev.c b/platform/linux/in_evdev.c
index 0b73302c..a51fe2de 100644
--- a/platform/linux/in_evdev.c
+++ b/platform/linux/in_evdev.c
@@ -402,60 +402,59 @@ static int in_evdev_update_keycode(void *data, int *is_down)
 	struct input_event ev;
 	int rd;
 
-	while (1)
-	{
-		rd = read(dev->fd, &ev, sizeof(ev));
-		if (rd < (int) sizeof(ev)) {
-			if (errno != EAGAIN) {
-				perror("in_evdev: error reading");
-				sleep(1);
-			}
-			goto out;
+	/* do single event, the caller sometimes wants
+	 * to do select() in blocking mode */
+	rd = read(dev->fd, &ev, sizeof(ev));
+	if (rd < (int) sizeof(ev)) {
+		if (errno != EAGAIN) {
+			perror("in_evdev: error reading");
+			sleep(1);
 		}
+		goto out;
+	}
 
-		if (ev.type == EV_KEY) {
-			if (ev.value < 0 || ev.value > 1)
-				continue;
-			ret_kc = ev.code;
-			ret_down = ev.value;
+	if (ev.type == EV_KEY) {
+		if (ev.value < 0 || ev.value > 1)
 			goto out;
+		ret_kc = ev.code;
+		ret_down = ev.value;
+		goto out;
+	}
+	else if (ev.type == EV_ABS)
+	{
+		int lzone = dev->abs_lzone, down = 0, *last;
+
+		// map absolute to up/down/left/right
+		if (lzone != 0 && ev.code == ABS_X) {
+			if (ev.value < dev->abs_min_x + lzone)
+				down = KEY_LEFT;
+			else if (ev.value > dev->abs_max_x - lzone)
+				down = KEY_RIGHT;
+			last = &dev->abs_lastx;
 		}
-		else if (ev.type == EV_ABS)
-		{
-			int lzone = dev->abs_lzone, down = 0, *last;
-
-			// map absolute to up/down/left/right
-			if (lzone != 0 && ev.code == ABS_X) {
-				if (ev.value < dev->abs_min_x + lzone)
-					down = KEY_LEFT;
-				else if (ev.value > dev->abs_max_x - lzone)
-					down = KEY_RIGHT;
-				last = &dev->abs_lastx;
-			}
-			else if (lzone != 0 && ev.code == ABS_Y) {
-				if (ev.value < dev->abs_min_y + lzone)
-					down = KEY_UP;
-				else if (ev.value > dev->abs_max_y - lzone)
-					down = KEY_DOWN;
-				last = &dev->abs_lasty;
-			}
-			else
-				continue;
-
-			if (down == *last)
-				continue;
-
-			if (down == 0 || *last != 0) {
-				/* key up or direction change, return up event for old key */
-				ret_kc = *last;
-				ret_down = 0;
-				*last = 0;
-				goto out;
-			}
-			ret_kc = *last = down;
-			ret_down = 1;
+		else if (lzone != 0 && ev.code == ABS_Y) {
+			if (ev.value < dev->abs_min_y + lzone)
+				down = KEY_UP;
+			else if (ev.value > dev->abs_max_y - lzone)
+				down = KEY_DOWN;
+			last = &dev->abs_lasty;
+		}
+		else
+			goto out;
+
+		if (down == *last)
+			goto out;
+
+		if (down == 0 || *last != 0) {
+			/* key up or direction change, return up event for old key */
+			ret_kc = *last;
+			ret_down = 0;
+			*last = 0;
 			goto out;
 		}
+		ret_kc = *last = down;
+		ret_down = 1;
+		goto out;
 	}
 
 out:
-- 
2.39.5