Changeset c54b898 in mainline for uspace/drv

Timestamp:

2018-01-05T16:30:46Z (8 years ago)

Author:

Petr Manek <petr.manek@…>

Branches:

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

Children:

b60944b

Parents:

11d4c747

git-author:

Petr Manek <petr.manek@…> (2018-01-05 16:21:03)

git-committer:

Petr Manek <petr.manek@…> (2018-01-05 16:30:46)

Message:

usbdev: refactoring

The device_rem driver callback was renamed to device_remove. Also,
a new device_removed callback was introduced. This callback will fire
after all device pipes (endpoints, internally) are unregistered from
the HC device after a call to device_remove. It is semantically
a better opportunity for the USB device driver to free resources and
join all fibrils, since endpoint unregistration will force abort all
active transfers, waking up their issuers.

Location:

uspace/drv

Files:

• block/usbmast/main.c (modified) (1 diff)
• bus/usb/usbdiag/main.c (modified) (1 diff)
• bus/usb/usbflbk/main.c (modified) (1 diff)
• bus/usb/usbhub/main.c (modified) (1 diff)
• bus/usb/usbmid/main.c (modified) (1 diff)
• hid/usbhid/main.c (modified) (2 diffs)

uspace/drv/block/usbmast/main.c

@@ -390 +390 @@
 static const usb_driver_ops_t usbmast_driver_ops = {
         .device_add = usbmast_device_add,
-        .device_rem = usbmast_device_remove,
+        .device_remove = usbmast_device_remove,
         .device_gone = usbmast_device_gone,
 };

uspace/drv/bus/usb/usbdiag/main.c

@@ -186 +186 @@
 static const usb_driver_ops_t diag_driver_ops = {
         .device_add = device_add,
-        .device_rem = device_remove,
+        .device_remove = device_remove,
         .device_gone = device_gone,
         .function_online = function_online,

uspace/drv/bus/usb/usbflbk/main.c

@@ -68 +68 @@
 static const usb_driver_ops_t usbfallback_driver_ops = {
         .device_add = usbfallback_device_add,
-        .device_rem = usbfallback_device_gone,
+        .device_remove = usbfallback_device_gone,
         .device_gone = usbfallback_device_gone,
 };

uspace/drv/bus/usb/usbhub/main.c

@@ -48 +48 @@
 static const usb_driver_ops_t usb_hub_driver_ops = {
         .device_add = usb_hub_device_add,
-//      .device_rem = usb_hub_device_remove,
+//      .device_remove = usb_hub_device_remove,
         .device_gone = usb_hub_device_gone,
 };

uspace/drv/bus/usb/usbmid/main.c

@@ -162 +162 @@
 static const usb_driver_ops_t mid_driver_ops = {
         .device_add = usbmid_device_add,
-        .device_rem = usbmid_device_remove,
+        .device_remove = usbmid_device_remove,
         .device_gone = usbmid_device_gone,
         .function_online = usbmid_function_online,

uspace/drv/hid/usbhid/main.c

@@ -122 +122 @@
  * @return Error code.
  */
-static int usb_hid_device_rem(usb_device_t *dev)
+static int usb_hid_device_remove(usb_device_t *dev)
 {
         assert(dev);
@@ -164 +164 @@
 static const usb_driver_ops_t usb_hid_driver_ops = {
         .device_add = usb_hid_device_add,
-        .device_rem = usb_hid_device_rem,
+        .device_remove = usb_hid_device_remove,
         .device_gone = usb_hid_device_gone,
 };