Changes in uspace/drv/bus/usb/usbhid/multimedia/multimedia.c [5203e256:5f6e25e] in mainline

File:

• uspace/drv/bus/usb/usbhid/multimedia/multimedia.c (modified) (5 diffs)

uspace/drv/bus/usb/usbhid/multimedia/multimedia.c

@@ -54 +54 @@
 
 // FIXME: remove this header
-#include <kernel/ipc/ipc_methods.h>
+#include <abi/ipc/methods.h>
 
 #define NAME "multimedia-keys"
@@ -167 +167 @@
 /*----------------------------------------------------------------------------*/
 
-static void usb_multimedia_free(usb_multimedia_t **multim_dev)
-{
-        if (multim_dev == NULL || *multim_dev == NULL) {
-                return;
-        }
-        
-        // hangup phone to the console
-        async_obsolete_hangup((*multim_dev)->console_phone);
-
-        free(*multim_dev);
-        *multim_dev = NULL;
-}
-
-/*----------------------------------------------------------------------------*/
-
 static int usb_multimedia_create_function(usb_hid_dev_t *hid_dev, 
     usb_multimedia_t *multim_dev)
 {
-        /* Create the function exposed under /dev/devices. */
+        /* Create the exposed function. */
         ddf_fun_t *fun = ddf_fun_create(hid_dev->usb_dev->ddf_dev, fun_exposed, 
             NAME);
@@ -205 +190 @@
         }
         
-        usb_log_debug("%s function created (jandle: %" PRIun ").\n",
+        usb_log_debug("%s function created (handle: %" PRIun ").\n",
             NAME, fun->handle);
         
-        rc = ddf_fun_add_to_class(fun, "keyboard");
+        rc = ddf_fun_add_to_category(fun, "keyboard");
         if (rc != EOK) {
                 usb_log_error(
-                    "Could not add DDF function to class 'keyboard': %s.\n",
+                    "Could not add DDF function to category 'keyboard': %s.\n",
                     str_error(rc));
                 // TODO: Can / should I destroy the DDF function?
@@ -247 +232 @@
         
         int rc = usb_multimedia_create_function(hid_dev, multim_dev);
-        if (rc != EOK) {
-                usb_multimedia_free(&multim_dev);
+        if (rc != EOK)
                 return rc;
-        }
         
         usb_log_debug(NAME " HID/multimedia structure initialized.\n");
@@ -267 +250 @@
         if (data != NULL) {
                 usb_multimedia_t *multim_dev = (usb_multimedia_t *)data;
-                usb_multimedia_free(&multim_dev);
+                // hangup phone to the console
+                async_obsolete_hangup(multim_dev->console_phone);
         }
 }