Changeset 65b458c4 in mainline for uspace/drv/usbhid/kbd/kbddev.c

Timestamp:

2011-05-13T12:44:37Z (14 years ago)

Author:

Lubos Slovak <lubos.slovak@…>

Branches:

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

Children:

37f87fa

Parents:

97cb542

Message:

Subdriver-specific data moved to the subdriver structure.

+ DDF ops removed from hid_dev_t.

Should fix #225

File:

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

uspace/drv/usbhid/kbd/kbddev.c

@@ -252 +252 @@
         sysarg_t method = IPC_GET_IMETHOD(*icall);
         
-        usb_hid_dev_t *hid_dev = (usb_hid_dev_t *)fun->driver_data;
-        
-        if (hid_dev == NULL || hid_dev->data == NULL) {
+        usb_kbd_t *kbd_dev = (usb_kbd_t *)fun->driver_data;
+        if (kbd_dev == NULL) {
                 usb_log_debug("default_connection_handler: "
                     "Missing parameter.\n");
@@ -260 +259 @@
                 return;
         }
-        
-        assert(hid_dev != NULL);
-        assert(hid_dev->data != NULL);
-        usb_kbd_t *kbd_dev = (usb_kbd_t *)hid_dev->data;
 
         if (method == IPC_M_CONNECT_TO_ME) {
@@ -663 +658 @@
  *     usb_hid_parse_report().
  */
-static void usb_kbd_process_data(usb_hid_dev_t *hid_dev,
+static void usb_kbd_process_data(usb_hid_dev_t *hid_dev, usb_kbd_t *kbd_dev,
                                  uint8_t *buffer, size_t actual_size)
 {
         assert(hid_dev->report != NULL);
         assert(hid_dev != NULL);
-        assert(hid_dev->data != NULL);
-        
-        usb_kbd_t *kbd_dev = (usb_kbd_t *)hid_dev->data;
+        assert(kbd_dev != NULL);
 
         usb_log_debug("Calling usb_hid_parse_report() with "
@@ -775 +768 @@
 /*----------------------------------------------------------------------------*/
 
-static int usb_kbd_create_function(usb_hid_dev_t *hid_dev)
+static int usb_kbd_create_function(usb_hid_dev_t *hid_dev, usb_kbd_t *kbd_dev)
 {
         assert(hid_dev != NULL);
         assert(hid_dev->usb_dev != NULL);
+        assert(kbd_dev != NULL);
         
         /* Create the function exposed under /dev/devices. */
@@ -793 +787 @@
          * to the DDF function.
          */
-        fun->ops = &hid_dev->ops;
-        fun->driver_data = hid_dev;   // TODO: maybe change to hid_dev->data
+        fun->ops = &kbd_dev->ops;
+        fun->driver_data = kbd_dev;
 
         int rc = ddf_fun_bind(fun);
@@ -841 +835 @@
  * @return Other value inherited from function usbhid_dev_init().
  */
-int usb_kbd_init(usb_hid_dev_t *hid_dev)
+int usb_kbd_init(usb_hid_dev_t *hid_dev, void **data)
 {
         usb_log_debug("Initializing HID/KBD structure...\n");
@@ -954 +948 @@
         
         // save the KBD device structure into the HID device structure
-        hid_dev->data = kbd_dev;
+        //hid_dev->data = kbd_dev;
+        *data = kbd_dev;
         
         // set handler for incoming calls
-        hid_dev->ops.default_handler = default_connection_handler;
+        kbd_dev->ops.default_handler = default_connection_handler;
         
         /*
@@ -982 +977 @@
         
         usb_log_debug("Creating KBD function...\n");
-        int rc = usb_kbd_create_function(hid_dev);
+        int rc = usb_kbd_create_function(hid_dev, kbd_dev);
         if (rc != EOK) {
                 usb_kbd_free(&kbd_dev);
@@ -993 +988 @@
 /*----------------------------------------------------------------------------*/
 
-bool usb_kbd_polling_callback(usb_hid_dev_t *hid_dev, uint8_t *buffer,
-     size_t buffer_size)
-{
-        if (hid_dev == NULL || buffer == NULL) {
+bool usb_kbd_polling_callback(usb_hid_dev_t *hid_dev, void *data, 
+     uint8_t *buffer, size_t buffer_size)
+{
+        if (hid_dev == NULL || buffer == NULL || data == NULL) {
                 // do not continue polling (???)
                 return false;
         }
         
+        usb_kbd_t *kbd_dev = (usb_kbd_t *)data;
+        assert(kbd_dev != NULL);
+        
         // TODO: add return value from this function
-        usb_kbd_process_data(hid_dev, buffer, buffer_size);
+        usb_kbd_process_data(hid_dev, kbd_dev, buffer, buffer_size);
         
         return true;
@@ -1065 +1063 @@
 /*----------------------------------------------------------------------------*/
 
-void usb_kbd_deinit(usb_hid_dev_t *hid_dev)
+void usb_kbd_deinit(usb_hid_dev_t *hid_dev, void *data)
 {
         if (hid_dev == NULL) {
@@ -1071 +1069 @@
         }
         
-        if (hid_dev->data != NULL) {
-                usb_kbd_t *kbd_dev = (usb_kbd_t *)hid_dev->data;
+        if (data != NULL) {
+                usb_kbd_t *kbd_dev = (usb_kbd_t *)data;
                 if (usb_kbd_is_initialized(kbd_dev)) {
                         usb_kbd_mark_unusable(kbd_dev);
                 } else {
                         usb_kbd_free(&kbd_dev);
-                        hid_dev->data = NULL;
                 }
         }