Changeset 4d3c13e in mainline for uspace/drv/usbhid/kbd/kbddev.c

Timestamp:

2011-06-16T19:19:10Z (13 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

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

Children:

9f7276d

Parents:

98dc29d

Message:

Unused code removal

File:

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

uspace/drv/usbhid/kbd/kbddev.c

@@ -434 +434 @@
  *     usb_hid_parse_report().
  */
-static void usb_kbd_process_data(usb_hid_dev_t *hid_dev, usb_kbd_t *kbd_dev/*,
-                                 uint8_t *buffer, size_t actual_size*/)
+static void usb_kbd_process_data(usb_hid_dev_t *hid_dev, usb_kbd_t *kbd_dev)
 {
         assert(hid_dev->report != NULL);
         assert(hid_dev != NULL);
         assert(kbd_dev != NULL);
-
-//      usb_log_debug("Calling usb_hid_parse_report() with "
-//          "buffer %s\n", usb_debug_str_buffer(buffer, actual_size, 0));
         
         usb_hid_report_path_t *path = usb_hid_report_path();
         usb_hid_report_path_append_item(path, USB_HIDUT_PAGE_KEYBOARD, 0);
 
-//      uint8_t report_id;
-//      int rc = usb_hid_parse_report(hid_dev->report, buffer, actual_size, 
-//          &report_id);
-        
-//      if (rc != EOK) {
-//              usb_log_warning("Error in usb_hid_parse_report():"
-//                  "%s\n", str_error(rc));
-//      }
-        
         usb_hid_report_path_set_report_id (path, hid_dev->report_id);
         
@@ -756 +743 @@
 /*----------------------------------------------------------------------------*/
 
-bool usb_kbd_polling_callback(usb_hid_dev_t *hid_dev, void *data/*, 
-     uint8_t *buffer, size_t buffer_size*/)
+bool usb_kbd_polling_callback(usb_hid_dev_t *hid_dev, void *data)
 {
         if (hid_dev == NULL/* || buffer == NULL*/ || data == NULL) {
@@ -768 +754 @@
         
         // TODO: add return value from this function
-        usb_kbd_process_data(hid_dev, kbd_dev/*, buffer, buffer_size*/);
+        usb_kbd_process_data(hid_dev, kbd_dev);
         
         return true;
@@ -804 +790 @@
         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);