Changes in uspace/lib/usbdev/src/devdrv.c [fb422312:70452dd4] in mainline

File:

• uspace/lib/usbdev/src/devdrv.c (modified) (9 diffs)

uspace/lib/usbdev/src/devdrv.c

@@ -41 +41 @@
 #include <assert.h>
 
-static int generic_device_add(ddf_dev_t *);
-static int generic_device_remove(ddf_dev_t *);
-static int generic_device_gone(ddf_dev_t *);
+static int generic_add_device(ddf_dev_t *);
 
 static driver_ops_t generic_driver_ops = {
-        .add_device = generic_device_add,
-        .dev_remove = generic_device_remove,
-        .dev_gone = generic_device_gone,
+        .add_device = generic_add_device
 };
 static driver_t generic_driver = {
@@ -127 +123 @@
         return EOK;
 }
-/*----------------------------------------------------------------------------*/
-/** Callback when a new device is supposed to be controlled by this driver.
- *
- * This callback is a wrapper for USB specific version of @c device_add.
+
+/** Callback when new device is supposed to be controlled by this driver.
+ *
+ * This callback is a wrapper for USB specific version of @c add_device.
  *
  * @param gen_dev Device structure as prepared by DDF.
  * @return Error code.
  */
-int generic_device_add(ddf_dev_t *gen_dev)
+int generic_add_device(ddf_dev_t *gen_dev)
 {
         assert(driver);
         assert(driver->ops);
-        assert(driver->ops->device_add);
+        assert(driver->ops->add_device);
 
         int rc;
@@ -151 +147 @@
                 return rc;
         }
-        gen_dev->driver_data = dev;
-
-        return driver->ops->device_add(dev);
-}
-/*----------------------------------------------------------------------------*/
-/** Callback when a device is supposed to be removed from the system.
- *
- * This callback is a wrapper for USB specific version of @c device_remove.
- *
- * @param gen_dev Device structure as prepared by DDF.
- * @return Error code.
- */
-int generic_device_remove(ddf_dev_t *gen_dev)
-{
-        assert(driver);
-        assert(driver->ops);
-        if (driver->ops->device_rem == NULL)
-                return ENOTSUP;
-        /* Just tell the driver to stop whatever it is doing, keep structures */
-        return driver->ops->device_rem(gen_dev->driver_data);
-}
-/*----------------------------------------------------------------------------*/
-/** Callback when a device was removed from the system.
- *
- * This callback is a wrapper for USB specific version of @c device_gone.
- *
- * @param gen_dev Device structure as prepared by DDF.
- * @return Error code.
- */
-int generic_device_gone(ddf_dev_t *gen_dev)
-{
-        assert(driver);
-        assert(driver->ops);
-        if (driver->ops->device_gone == NULL)
-                return ENOTSUP;
-        const int ret = driver->ops->device_gone(gen_dev->driver_data);
-        if (ret == EOK)
-                usb_device_destroy(gen_dev->driver_data);
-
-        return ret;
-}
-/*----------------------------------------------------------------------------*/
+
+        return driver->ops->add_device(dev);
+}
+
 /** Destroy existing pipes of a USB device.
  *
@@ -317 +275 @@
  * @return Error code.
  */
-int usb_device_create_pipes(const ddf_dev_t *dev, usb_device_connection_t *wire,
+int usb_device_create_pipes(ddf_dev_t *dev, usb_device_connection_t *wire,
     usb_endpoint_description_t **endpoints,
     uint8_t *config_descr, size_t config_descr_size,
@@ -391 +349 @@
         }
 
-        if (usb_hc_connection_close(&hc_conn) != EOK)
-                usb_log_warning("usb_device_create_pipes(): "
-                    "Failed to close connection.\n");
+        usb_hc_connection_close(&hc_conn);
 
         *pipes_ptr = pipes;
@@ -415 +371 @@
         }
 
-        if (usb_hc_connection_close(&hc_conn) != EOK)
-                usb_log_warning("usb_device_create_pipes(): "
-                    "Failed to close connection.\n");
+        usb_hc_connection_close(&hc_conn);
 
         /*
@@ -441 +395 @@
  * @param[in] pipes_count Number of endpoints.
  */
-int usb_device_destroy_pipes(const ddf_dev_t *dev,
+int usb_device_destroy_pipes(ddf_dev_t *dev,
     usb_endpoint_mapping_t *pipes, size_t pipes_count)
 {
@@ -472 +426 @@
         }
 
-        if (usb_hc_connection_close(&hc_conn) != EOK)
-                usb_log_warning("usb_device_destroy_pipes(): "
-                    "Failed to close connection.\n");
+        usb_hc_connection_close(&hc_conn);
 
         free(pipes);
@@ -593 +545 @@
         /* Ignore errors and hope for the best. */
         usb_device_destroy_pipes(dev->ddf_dev, dev->pipes, dev->pipes_count);
-        free(dev->descriptors.configuration);
+        if (dev->descriptors.configuration != NULL) {
+                free(dev->descriptors.configuration);
+        }
 
         if (dev->alternate_interfaces != NULL) {
-                free(dev->alternate_interfaces->alternatives);
-        }
-        free(dev->alternate_interfaces);
+                if (dev->alternate_interfaces->alternatives != NULL) {
+                        free(dev->alternate_interfaces->alternatives);
+                }
+                free(dev->alternate_interfaces);
+        }
 
         free(dev);