Changes in uspace/srv/hid/console/console.c [30db06c:ffa2c8ef] in mainline

File:

• uspace/srv/hid/console/console.c (modified) (7 diffs)

uspace/srv/hid/console/console.c

@@ -41 +41 @@
 #include <ipc/ns.h>
 #include <errno.h>
-#include <str_error.h>
 #include <ipc/console.h>
 #include <unistd.h>
@@ -65 +64 @@
 #define NAME       "console"
 #define NAMESPACE  "term"
-/** Interval for checking for new keyboard (1/4s). */
-#define HOTPLUG_WATCH_INTERVAL (1000 * 250)
 
 /** Phone to the keyboard driver. */
@@ -320 +317 @@
 static void change_console(console_t *cons)
 {
-        if (cons == active_console) {
+        if (cons == active_console)
                 return;
-        }
         
         fb_pending_flush();
@@ -462 +458 @@
                         if (IPC_GET_ARG1(call) == 1) {
                                 int newcon = gcons_mouse_btn((bool) IPC_GET_ARG2(call));
-                                if (newcon != -1) {
+                                if (newcon != -1)
                                         change_console(&consoles[newcon]);
-                                }
                         }
                         retval = 0;
@@ -715 +710 @@
 }
 
-static int connect_keyboard(char *path)
-{
-        int fd = open(path, O_RDONLY);
-        if (fd < 0) {
-                return fd;
-        }
-        
-        int phone = fd_phone(fd);
-        if (phone < 0) {
+static bool console_init(char *input)
+{
+        /* Connect to input device */
+        int input_fd = open(input, O_RDONLY);
+        if (input_fd < 0) {
+                printf(NAME ": Failed opening %s\n", input);
+                return false;
+        }
+        
+        kbd_phone = fd_phone(input_fd);
+        if (kbd_phone < 0) {
                 printf(NAME ": Failed to connect to input device\n");
-                return phone;
-        }
-        
-        int rc = async_connect_to_me(phone, SERVICE_CONSOLE, 0, 0,
-            keyboard_events);
-        if (rc != EOK) {
-                printf(NAME ": " \
-                    "Failed to create callback from input device: %s.\n",
-                    str_error(rc));
-                return rc;
-        }
-        
-        printf(NAME ": found keyboard \"%s\".\n", path);
-
-        return phone;
-}
-
-
-/** Periodically check for new keyboards in /dev/class/.
- *
- * @param arg Class name.
- * @return This function should never exit.
- */
-static int check_new_keyboards(void *arg)
-{
-        char *class_name = (char *) arg;
-
-        size_t index = 1;
-
-        while (true) {
-                async_usleep(HOTPLUG_WATCH_INTERVAL);
-                char *path;
-                int rc = asprintf(&path, "/dev/class/%s\\%zu",
-                    class_name, index);
-                if (rc < 0) {
-                        continue;
-                }
-                rc = 0;
-                rc = connect_keyboard(path);
-                if (rc > 0) {
-                        /* We do not allow unplug. */
-                        index++;
-                }
-
-                free(path);
-        }
-
-        return EOK;
-}
-
-
-/** Start a fibril monitoring hot-plugged keyboards.
- */
-static void check_new_keyboards_in_background()
-{
-        fid_t fid = fibril_create(check_new_keyboards, (void *)"keyboard");
-        if (!fid) {
-                printf(NAME ": failed to create hot-plug-watch fibril.\n");
-                return;
-        }
-        fibril_add_ready(fid);
-}
-
-static bool console_init(char *input)
-{
-        /* Connect to input device */
-        kbd_phone = connect_keyboard(input);
-        if (kbd_phone < 0) {
                 return false;
         }
-
+        
+        /* NB: The callback connection is slotted for removal */
+        if (async_connect_to_me(kbd_phone, SERVICE_CONSOLE, 0, 0, keyboard_events)
+            != 0) {
+                printf(NAME ": Failed to create callback from input device\n");
+                return false;
+        }
+        
         /* Connect to mouse device */
         mouse_phone = -1;
@@ -901 +837 @@
                 printf(NAME ": Error registering kconsole notifications\n");
         
-        /* Start fibril for checking on hot-plugged keyboards. */
-        check_new_keyboards_in_background();
-
         return true;
 }
@@ -923 +856 @@
         if (!console_init(argv[1]))
                 return -1;
-
+        
         printf(NAME ": Accepting connections\n");
         async_manager();