Changeset 47a350f in mainline for uspace/srv

Timestamp:

2009-12-16T01:49:16Z (15 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

df747bd8

Parents:

4491338

Message:

keyboard is not a single possible human input device, thus don't register the kbd driver as a singleton service to the Naming service, but to the Device mapper

Location:

uspace/srv

Files:

• console/console.c (modified) (4 diffs)
• kbd/generic/kbd.c (modified) (6 diffs)

uspace/srv/console/console.c

@@ -50 +50 @@
 #include <event.h>
 #include <devmap.h>
+#include <fcntl.h>
+#include <vfs/vfs.h>
 #include <fibril_synch.h>
 
@@ -665 +667 @@
 }
 
-static bool console_init(void)
-{
-        ipcarg_t color_cap;
-
-        /* Connect to keyboard driver */
-        kbd_phone = ipc_connect_me_to_blocking(PHONE_NS, SERVICE_KEYBOARD, 0, 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 keyboard service\n");
+                printf(NAME ": Failed to connect to input device\n");
                 return false;
         }
         
+        /* NB: The callback connection is slotted for removal */
         ipcarg_t phonehash;
         if (ipc_connect_to_me(kbd_phone, SERVICE_CONSOLE, 0, 0, &phonehash) != 0) {
-                printf(NAME ": Failed to create callback from keyboard service\n");
+                printf(NAME ": Failed to create callback from input device\n");
                 return false;
         }
@@ -702 +709 @@
         
         /* Synchronize, the gcons could put something in queue */
+        ipcarg_t color_cap;
         async_req_0_0(fb_info.phone, FB_FLUSH);
         async_req_0_2(fb_info.phone, FB_GET_CSIZE, &fb_info.cols, &fb_info.rows);
@@ -771 +779 @@
 }
 
+static void usage(void)
+{
+        printf("Usage: console <input>\n");
+}
+
 int main(int argc, char *argv[])
 {
+        if (argc < 2) {
+                usage();
+                return -1;
+        }
+        
         printf(NAME ": HelenOS Console service\n");
         
-        if (!console_init())
+        if (!console_init(argv[1]))
                 return -1;
         

uspace/srv/kbd/generic/kbd.c

@@ -50 +50 @@
 #include <io/console.h>
 #include <io/keycode.h>
+#include <devmap.h>
 
 #include <kbd.h>
@@ -56 +57 @@
 #include <layout.h>
 
-#define NAME "kbd"
-
-int cons_connected = 0;
+#define NAME       "kbd"
+#define NAMESPACE  "hid_in"
+
 int phone2cons = -1;
 
@@ -172 +173 @@
         int retval;
 
-        if (cons_connected) {
-                ipc_answer_0(iid, ELIMIT);
-                return;
-        }
-        cons_connected = 1;
         ipc_answer_0(iid, EOK);
 
@@ -183 +179 @@
                 switch (IPC_GET_METHOD(call)) {
                 case IPC_M_PHONE_HUNGUP:
-                        cons_connected = 0;
-                        ipc_hangup(phone2cons);
-                        phone2cons = -1;
+                        if (phone2cons != -1) {
+                                ipc_hangup(phone2cons);
+                                phone2cons = -1;
+                        }
+                        
                         ipc_answer_0(callid, EOK);
                         return;
@@ -216 +214 @@
         printf(NAME ": HelenOS Keyboard service\n");
         
-        ipcarg_t phonead;
-        
         if (sysinfo_value("kbd.cir.fhc") == 1)
                 cir_service = SERVICE_FHC;
@@ -241 +237 @@
         layout[active_layout]->reset();
         
-        async_set_client_connection(console_connection);
-
-        /* Register service at nameserver. */
-        if (ipc_connect_to_me(PHONE_NS, SERVICE_KEYBOARD, 0, 0, &phonead) != 0)
-                return -1;
-        
+        /* Register driver */
+        int rc = devmap_driver_register(NAME, console_connection);
+        if (rc < 0) {
+                printf(NAME ": Unable to register driver (%d)\n", rc);
+                return -1;
+        }
+        
+        char kbd[DEVMAP_NAME_MAXLEN + 1];
+        snprintf(kbd, DEVMAP_NAME_MAXLEN, "%s/%s", NAMESPACE, NAME);
+        
+        dev_handle_t dev_handle;
+        if (devmap_device_register(kbd, &dev_handle) != EOK) {
+                printf(NAME ": Unable to register device %s\n", kbd);
+                return -1;
+        }
+
         printf(NAME ": Accepting connections\n");
         async_manager();