Changeset aab02fb in mainline for uspace/lib/usbvirt/ctrlpipe.c

Timestamp:

2010-10-26T22:21:45Z (13 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

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

Children:

efedee77

Parents:

fa2a361

Message:

Debugging support for libusbvirt

Added methods debug and lib_debug to device structure to simplify
debugging. Each method gets as a parameter debug level describing
verbosity of the message and also message tag that allows filtering
based on message type (e.g. messages related to transactions, to
device requests etc.).

File:

• uspace/lib/usbvirt/ctrlpipe.c (modified) (4 diffs)

uspace/lib/usbvirt/ctrlpipe.c

@@ -45 +45 @@
         ((value & GET_MIDBITS_MASK(size, shift)) >> shift)
 
-usb_address_t dev_new_address = -1;
+
+static const char *str_request_type(int type)
+{
+        switch (type) {
+                case REQUEST_TYPE_STANDARD:
+                        return "standard";
+                case REQUEST_TYPE_CLASS:
+                        return "class";
+                default:
+                        return "unknown";
+        }
+}
 
 /** Tell request type.
@@ -59 +70 @@
 int control_pipe(usbvirt_device_t *device, usbvirt_control_transfer_t *transfer)
 {
+        device->lib_debug(device, 1, USBVIRT_DEBUGTAG_CONTROL_PIPE_ZERO,
+            "op on control pipe zero (request_size=%u)", transfer->request_size);
+        
         if (transfer->request_size < sizeof(usb_device_request_setup_packet_t)) {
                 return ENOMEM;
@@ -69 +83 @@
         
         int rc = EOK;
+        
+        device->lib_debug(device, 2, USBVIRT_DEBUGTAG_CONTROL_PIPE_ZERO,
+            "request type: %s", str_request_type(type));
         
         switch (type) {
@@ -97 +114 @@
                 
                 device->new_address = -1;
+                
+                device->lib_debug(device, 2, USBVIRT_DEBUGTAG_CONTROL_PIPE_ZERO,
+                    "device address changed to %d (state %s)",
+                    device->address, str_device_state(device->state));
         }