Changes in uspace/drv/usbhid/kbd/kbddev.c [1b7dc5e9:323b0ec] in mainline

File:

• uspace/drv/usbhid/kbd/kbddev.c (modified) (7 diffs)

uspace/drv/usbhid/kbd/kbddev.c

@@ -255 +255 @@
         
         if (hid_dev == NULL || hid_dev->data == NULL) {
-                usb_log_debug("default_connection_handler: "
-                    "Missing parameter.\n");
                 async_answer_0(icallid, EINVAL);
                 return;
@@ -269 +267 @@
 
                 if (kbd_dev->console_phone != -1) {
-                        usb_log_debug("default_connection_handler: "
-                            "console phone already set\n");
                         async_answer_0(icallid, ELIMIT);
                         return;
@@ -276 +272 @@
 
                 kbd_dev->console_phone = callback;
-                
-                usb_log_debug("default_connection_handler: OK\n");
                 async_answer_0(icallid, EOK);
                 return;
         }
         
-        usb_log_debug("default_connection_handler: Wrong function.\n");
         async_answer_0(icallid, EINVAL);
 }
@@ -562 +555 @@
                         usb_log_debug2("Key pressed: %d (keycode: %d)\n", key,
                             kbd_dev->keys[i]);
+                        usb_kbd_push_ev(hid_dev, kbd_dev, KEY_PRESS, key);
                         if (!usb_kbd_is_lock(key)) {
                                 usb_kbd_repeat_start(kbd_dev, key);
                         }
-                        usb_kbd_push_ev(hid_dev, kbd_dev, KEY_PRESS, key);
                 } else {
                         // found, nothing happens
@@ -773 +766 @@
 
 /*----------------------------------------------------------------------------*/
-
-static int usb_kbd_create_function(usb_hid_dev_t *hid_dev)
-{
-        assert(hid_dev != NULL);
-        assert(hid_dev->usb_dev != NULL);
-        
-        /* Create the function exposed under /dev/devices. */
-        usb_log_debug("Creating DDF function %s...\n", HID_KBD_FUN_NAME);
-        ddf_fun_t *fun = ddf_fun_create(hid_dev->usb_dev->ddf_dev, fun_exposed, 
-            HID_KBD_FUN_NAME);
-        if (fun == NULL) {
-                usb_log_error("Could not create DDF function node.\n");
-                return ENOMEM;
-        }
-        
-        /*
-         * Store the initialized HID device and HID ops
-         * to the DDF function.
-         */
-        fun->ops = &hid_dev->ops;
-        fun->driver_data = hid_dev;   // TODO: maybe change to hid_dev->data
-
-        int rc = ddf_fun_bind(fun);
-        if (rc != EOK) {
-                usb_log_error("Could not bind DDF function: %s.\n",
-                    str_error(rc));
-                ddf_fun_destroy(fun);
-                return rc;
-        }
-        
-        usb_log_debug("Adding DDF function to class %s...\n", 
-            HID_KBD_CLASS_NAME);
-        rc = ddf_fun_add_to_class(fun, HID_KBD_CLASS_NAME);
-        if (rc != EOK) {
-                usb_log_error(
-                    "Could not add DDF function to class %s: %s.\n",
-                    HID_KBD_CLASS_NAME, str_error(rc));
-                ddf_fun_destroy(fun);
-                return rc;
-        }
-        
-        return EOK;
-}
-
-/*----------------------------------------------------------------------------*/
 /* API functions                                                              */
 /*----------------------------------------------------------------------------*/
@@ -982 +930 @@
         usb_log_debug("HID/KBD device structure initialized.\n");
         
-        usb_log_debug("Creating KBD function...\n");
-        int rc = usb_kbd_create_function(hid_dev);
-        if (rc != EOK) {
-                usb_kbd_free(&kbd_dev);
-                return rc;
-        }
-        
         return EOK;
 }
@@ -1052 +993 @@
         if ((*kbd_dev)->led_data != NULL) {
                 free((*kbd_dev)->led_data);
+        }
+        if ((*kbd_dev)->output_buffer != NULL) {
+                free((*kbd_dev)->output_buffer);
         }
         if ((*kbd_dev)->led_path != NULL) {