]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
usb: dwc3: trace: print out event status too
authorFelipe Balbi <felipe.balbi@linux.intel.com>
Fri, 6 Apr 2018 08:03:19 +0000 (11:03 +0300)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Mon, 21 May 2018 07:00:56 +0000 (10:00 +0300)
This will make it easier to figure out the reason for the event. That
information really helps debugging certain problems.

Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/dwc3/debug.h

index bfb90c52d8fc9220f9acc6d520b9c6cd87ac52a4..0be6a554be5765107bdabf2fb5e3f1471e07bc72 100644 (file)
@@ -475,21 +475,33 @@ dwc3_ep_event_string(char *str, const struct dwc3_event_depevt *event,
        if (ret < 0)
                return "UNKNOWN";
 
+       status = event->status;
+
        switch (event->endpoint_event) {
        case DWC3_DEPEVT_XFERCOMPLETE:
-               strcat(str, "Transfer Complete");
+               len = strlen(str);
+               sprintf(str + len, "Transfer Complete (%c%c%c)",
+                               status & DEPEVT_STATUS_SHORT ? 'S' : 's',
+                               status & DEPEVT_STATUS_IOC ? 'I' : 'i',
+                               status & DEPEVT_STATUS_LST ? 'L' : 'l');
+
                len = strlen(str);
 
                if (epnum <= 1)
                        sprintf(str + len, " [%s]", dwc3_ep0_state_string(ep0state));
                break;
        case DWC3_DEPEVT_XFERINPROGRESS:
-               strcat(str, "Transfer In-Progress");
+               len = strlen(str);
+
+               sprintf(str + len, "Transfer In Progress (%c%c%c)",
+                               status & DEPEVT_STATUS_SHORT ? 'S' : 's',
+                               status & DEPEVT_STATUS_IOC ? 'I' : 'i',
+                               status & DEPEVT_STATUS_LST ? 'M' : 'm');
                break;
        case DWC3_DEPEVT_XFERNOTREADY:
                strcat(str, "Transfer Not Ready");
-               status = event->status & DEPEVT_STATUS_TRANSFER_ACTIVE;
-               strcat(str, status ? " (Active)" : " (Not Active)");
+               strcat(str, status & DEPEVT_STATUS_TRANSFER_ACTIVE ?
+                               " (Active)" : " (Not Active)");
 
                /* Control Endpoints */
                if (epnum <= 1) {