Changeset cddd151 in mainline for uspace/drv/bus/usb/usbhid/kbd/kbddev.c

Timestamp:

2011-10-14T14:37:32Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

7b54b99

Parents:

e5024111

Message:

usbhid: kbd store pointer to created function.

Make repeat mutex part of the kbd structure.

File:

• uspace/drv/bus/usb/usbhid/kbd/kbddev.c (modified) (4 diffs)

uspace/drv/bus/usb/usbhid/kbd/kbddev.c

@@ -540 +540 @@
                 usb_log_error("Could not bind DDF function: %s.\n",
                     str_error(rc));
+                fun->driver_data = NULL; /* We need this later */
                 ddf_fun_destroy(fun);
                 return rc;
@@ -554 +555 @@
                     "Could not add DDF function to category %s: %s.\n",
                     HID_KBD_CLASS_NAME, str_error(rc));
+                fun->driver_data = NULL; /* We need this later */
                 ddf_fun_destroy(fun);
                 return rc;
         }
+        kbd_dev->fun = fun;
 
         return EOK;
@@ -687 +690 @@
         kbd_dev->repeat.delay_between = DEFAULT_REPEAT_DELAY;
 
-        kbd_dev->repeat_mtx = (fibril_mutex_t *)(
-            malloc(sizeof(fibril_mutex_t)));
-        if (kbd_dev->repeat_mtx == NULL) {
-                usb_log_fatal("No memory!\n");
-                free(kbd_dev->keys);
-                usb_hid_report_output_free(kbd_dev->output_buffer);
-                free(kbd_dev);
-                return ENOMEM;
-        }
-
-        fibril_mutex_initialize(kbd_dev->repeat_mtx);
+        fibril_mutex_initialize(&kbd_dev->repeat_mtx);
 
         // save the KBD device structure into the HID device structure
@@ -784 +777 @@
         async_hangup(kbd_dev->console_sess);
 
-        if (kbd_dev->repeat_mtx != NULL) {
-                //assert(!fibril_mutex_is_locked((*kbd_dev)->repeat_mtx));
-                // FIXME - the fibril_mutex_is_locked may not cause
-                // fibril scheduling
-                while (fibril_mutex_is_locked(kbd_dev->repeat_mtx)) {}
-                free(kbd_dev->repeat_mtx);
-        }
+        //assert(!fibril_mutex_is_locked((*kbd_dev)->repeat_mtx));
+        // FIXME - the fibril_mutex_is_locked may not cause
+        // fibril scheduling
+        while (fibril_mutex_is_locked(&kbd_dev->repeat_mtx)) {}
 
         // free all buffers
-        if (kbd_dev->keys != NULL) {
-                free(kbd_dev->keys);
-        }
-        if (kbd_dev->keys_old != NULL) {
-                free(kbd_dev->keys_old);
-        }
-        if (kbd_dev->led_data != NULL) {
-                free(kbd_dev->led_data);
-        }
+        free(kbd_dev->keys);
+        free(kbd_dev->keys_old);
+        free(kbd_dev->led_data);
         if (kbd_dev->led_path != NULL) {
                 usb_hid_report_path_free(kbd_dev->led_path);