Changeset 65b458c4 in mainline for uspace/drv/usbhid/mouse/mousedev.c

Timestamp:

2011-05-13T12:44:37Z (13 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/mouse/mousedev.c (modified) (12 diffs)

uspace/drv/usbhid/mouse/mousedev.c

@@ -122 +122 @@
         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_mouse_t *mouse_dev = (usb_mouse_t *)fun->driver_data;
+        
+        if (mouse_dev == NULL) {
                 usb_log_debug("default_connection_handler: Missing "
                     "parameters.\n");
@@ -131 +131 @@
         }
         
-        assert(hid_dev != NULL);
-        assert(hid_dev->data != NULL);
-        usb_mouse_t *mouse_dev = (usb_mouse_t *)hid_dev->data;
-        
         int *phone = (str_cmp(fun->name, HID_MOUSE_FUN_NAME) == 0) 
                      ? &mouse_dev->mouse_phone : &mouse_dev->wheel_phone;
@@ -145 +141 @@
                             "phone to mouse already set.\n");
                         async_answer_0(icallid, ELIMIT);
-                        //async_answer_0(icallid, EOK);
                         return;
                 }
@@ -224 +219 @@
 /*----------------------------------------------------------------------------*/
 
-static bool usb_mouse_process_report(usb_hid_dev_t *hid_dev, uint8_t *buffer,
-    size_t buffer_size)
-{
-        usb_mouse_t *mouse_dev = (usb_mouse_t *)hid_dev->data;
+static bool usb_mouse_process_report(usb_hid_dev_t *hid_dev, 
+                                     usb_mouse_t *mouse_dev, uint8_t *buffer,
+                                     size_t buffer_size)
+{
+        assert(mouse_dev != NULL);
         
         usb_log_debug2("got buffer: %s.\n",
@@ -378 +374 @@
 /*----------------------------------------------------------------------------*/
 
-static int usb_mouse_create_function(usb_hid_dev_t *hid_dev)
-{
+static int usb_mouse_create_function(usb_hid_dev_t *hid_dev, usb_mouse_t *mouse)
+{
+        assert(hid_dev != NULL);
+        assert(mouse != NULL);
+        
         /* Create the function exposed under /dev/devices. */
         usb_log_debug("Creating DDF function %s...\n", HID_MOUSE_FUN_NAME);
@@ -389 +388 @@
         }
         
-        /*
-         * 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
+        fun->ops = &mouse->ops;
+        fun->driver_data = mouse;   // TODO: maybe change to hid_dev->data
 
         int rc = ddf_fun_bind(fun);
@@ -431 +426 @@
          * to the DDF function.
          */
-        fun->ops = &hid_dev->ops;
+        fun->ops = &mouse->ops;
         fun->driver_data = hid_dev;   // TODO: maybe change to hid_dev->data
 
@@ -458 +453 @@
 /*----------------------------------------------------------------------------*/
 
-int usb_mouse_init(usb_hid_dev_t *hid_dev)
+int usb_mouse_init(usb_hid_dev_t *hid_dev, void **data)
 {
         usb_log_debug("Initializing HID/Mouse structure...\n");
@@ -485 +480 @@
         
         // save the Mouse device structure into the HID device structure
-        hid_dev->data = mouse_dev;
+        *data = mouse_dev;
         
         // set handler for incoming calls
-        hid_dev->ops.default_handler = default_connection_handler;
+        // TODO: must be one for each subdriver!!
+        mouse_dev->ops.default_handler = default_connection_handler;
         
         // TODO: how to know if the device supports the request???
@@ -494 +490 @@
 //          hid_dev->usb_dev->interface_no, IDLE_RATE);
         
-        int rc = usb_mouse_create_function(hid_dev);
+        int rc = usb_mouse_create_function(hid_dev, mouse_dev);
         if (rc != EOK) {
                 usb_mouse_free(&mouse_dev);
@@ -505 +501 @@
 /*----------------------------------------------------------------------------*/
 
-bool usb_mouse_polling_callback(usb_hid_dev_t *hid_dev, uint8_t *buffer,
-     size_t buffer_size)
+bool usb_mouse_polling_callback(usb_hid_dev_t *hid_dev, void *data, 
+     uint8_t *buffer, size_t buffer_size)
 {
         usb_log_debug("usb_mouse_polling_callback()\n");
         usb_debug_str_buffer(buffer, buffer_size, 0);
         
-        if (hid_dev == NULL) {
+        if (hid_dev == NULL || data == NULL) {
                 usb_log_error("Missing argument to the mouse polling callback."
                     "\n");
@@ -517 +513 @@
         }
         
-        if (hid_dev->data == NULL) {
-                usb_log_error("Wrong argument to the mouse polling callback."
-                    "\n");
-                return false;
-        }
-        
-        return usb_mouse_process_report(hid_dev, buffer, buffer_size);
-}
-
-/*----------------------------------------------------------------------------*/
-
-void usb_mouse_deinit(usb_hid_dev_t *hid_dev)
-{
-        usb_mouse_free((usb_mouse_t **)&hid_dev->data);
+        usb_mouse_t *mouse_dev = (usb_mouse_t *)data;
+                
+        return usb_mouse_process_report(hid_dev, mouse_dev, buffer, 
+                                        buffer_size);
+}
+
+/*----------------------------------------------------------------------------*/
+
+void usb_mouse_deinit(usb_hid_dev_t *hid_dev, void *data)
+{
+        if (data != NULL) {
+                usb_mouse_free((usb_mouse_t **)&data);
+        }
 }