Changes in uspace/srv/hid/input/input.c [d5c1051:7aa94304] in mainline

File:

• uspace/srv/hid/input/input.c (modified) (17 diffs)

uspace/srv/hid/input/input.c

@@ -37 +37 @@
  */
 
+#include <adt/list.h>
+#include <stdbool.h>
+#include <fibril_synch.h>
+#include <ipc/services.h>
+#include <ipc/input.h>
+#include <config.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <ns.h>
+#include <async.h>
+#include <errno.h>
 #include <adt/fifo.h>
-#include <adt/list.h>
-#include <async.h>
-#include <config.h>
-#include <errno.h>
-#include <fibril.h>
-#include <fibril_synch.h>
-#include <io/chardev.h>
 #include <io/console.h>
 #include <io/keycode.h>
-#include <ipc/services.h>
-#include <ipc/input.h>
 #include <loc.h>
-#include <ns.h>
-#include <stdbool.h>
-#include <stdio.h>
-#include <stdlib.h>
 #include <str_error.h>
-
-#include "input.h"
+#include <char_dev_iface.h>
+#include <fibril.h>
+#include "layout.h"
 #include "kbd.h"
 #include "kbd_port.h"
 #include "kbd_ctl.h"
-#include "layout.h"
 #include "mouse.h"
 #include "mouse_proto.h"
 #include "serial.h"
+#include "input.h"
 
 #define NUM_LAYOUTS  4
@@ -365 +364 @@
 }
 
-static void kconsole_event_handler(ipc_call_t *call, void *arg)
+static void kconsole_event_handler(ipc_callid_t callid, ipc_call_t *call,
+    void *arg)
 {
         if (IPC_GET_ARG1(*call)) {
@@ -458 +458 @@
 }
 
+/** Add new legacy mouse device. */
+static void mouse_add_dev(mouse_port_ops_t *port, mouse_proto_ops_t *proto)
+{
+        mouse_dev_t *mdev = mouse_dev_new();
+        if (mdev == NULL)
+                return;
+        
+        mdev->port_ops = port;
+        mdev->proto_ops = proto;
+        mdev->svc_id = 0;
+        
+        /* Initialize port driver. */
+        if ((*mdev->port_ops->init)(mdev) != 0)
+                goto fail;
+        
+        /* Initialize protocol driver. */
+        if ((*mdev->proto_ops->init)(mdev) != 0) {
+                /* XXX Uninit port */
+                goto fail;
+        }
+        
+        list_append(&mdev->link, &mouse_devs);
+        return;
+        
+fail:
+        free(mdev);
+}
+
 /** Add new kbdev device.
  *
@@ -486 +514 @@
         list_append(&kdev->link, &kbd_devs);
         *kdevp = kdev;
-        return 0;
+        return EOK;
         
 fail:
@@ -523 +551 @@
         list_append(&mdev->link, &mouse_devs);
         *mdevp = mdev;
-        return 0;
+        return EOK;
         
 fail:
@@ -536 +564 @@
         while (true) {
                 uint8_t data;
-                size_t nread;
-
-                chardev_read(sdev->chardev, &data, sizeof(data), &nread);
-                /* XXX Handle error */
+
+                char_dev_read(sdev->sess, &data, sizeof(data));
                 kbd_push_data(sdev->kdev, data);
         }
@@ -554 +580 @@
 {
         bool match = false;
-        int rc;
 
         serial_dev_t *sdev = serial_dev_new();
@@ -562 +587 @@
         sdev->kdev->svc_id = service_id;
         
-        rc = loc_service_get_name(service_id, &sdev->kdev->svc_name);
+        int rc = loc_service_get_name(service_id, &sdev->kdev->svc_name);
         if (rc != EOK)
                 goto fail;
@@ -586 +611 @@
                     IPC_FLAG_BLOCKING);
 
-                rc = chardev_open(sdev->sess, &sdev->chardev);
-                if (rc != EOK) {
-                        async_hangup(sdev->sess);
-                        sdev->sess = NULL;
-                        list_remove(&sdev->link);
-                        goto fail;
-                }
-
                 fid_t fid = fibril_create(serial_consumer, sdev);
                 fibril_add_ready(fid);
@@ -599 +616 @@
         
         *sdevp = sdev;
-        return 0;
+        return EOK;
         
 fail:
@@ -626 +643 @@
         kbd_add_dev(&chardev_port, &stty_ctl);
 #endif
+#if defined(UARCH_ppc32)
+        kbd_add_dev(&adb_port, &apple_ctl);
+#endif
 #if defined(UARCH_sparc64) && defined(PROCESSOR_sun4v)
         kbd_add_dev(&chardev_port, &stty_ctl);
@@ -631 +651 @@
         /* Silence warning on abs32le about kbd_add_dev() being unused */
         (void) kbd_add_dev;
+}
+
+/** Add legacy drivers/devices. */
+static void mouse_add_legacy_devs(void)
+{
+        /*
+         * Need to add these drivers based on config unless we can probe
+         * them automatically.
+         */
+#if defined(UARCH_ppc32)
+        mouse_add_dev(&adb_mouse_port, &adb_proto);
+#endif
+        /* Silence warning on abs32le about mouse_add_dev() being unused */
+        (void) mouse_add_dev;
 }
 
@@ -670 +704 @@
                 if (!already_known) {
                         kbd_dev_t *kdev;
-                        if (kbd_add_kbdev(svcs[i], &kdev) == 0) {
+                        if (kbd_add_kbdev(svcs[i], &kdev) == EOK) {
                                 printf("%s: Connected keyboard device '%s'\n",
                                     NAME, kdev->svc_name);
@@ -721 +755 @@
                 if (!already_known) {
                         mouse_dev_t *mdev;
-                        if (mouse_add_mousedev(svcs[i], &mdev) == 0) {
+                        if (mouse_add_mousedev(svcs[i], &mdev) == EOK) {
                                 printf("%s: Connected mouse device '%s'\n",
                                     NAME, mdev->svc_name);
@@ -772 +806 @@
                 if (!already_known) {
                         serial_dev_t *sdev;
-                        if (serial_add_srldev(svcs[i], &sdev) == 0) {
+                        if (serial_add_srldev(svcs[i], &sdev) == EOK) {
                                 printf("%s: Connected serial device '%s'\n",
                                     NAME, sdev->kdev->svc_name);
@@ -827 +861 @@
         int rc = loc_register_cat_change_cb(cat_change_cb);
         if (rc != EOK) {
-                printf("%s: Failed registering callback for device discovery: "
-                    "%s\n", NAME, str_error(rc));
+                printf("%s: Failed registering callback for device discovery. "
+                    "(%d)\n", NAME, rc);
                 return rc;
         }
@@ -860 +894 @@
         /* Add legacy keyboard devices. */
         kbd_add_legacy_devs();
+        
+        /* Add legacy mouse devices. */
+        mouse_add_legacy_devs();
         
         /* Register driver */