wireguard-linux/drivers/usb/dwc3/trace.h, branch jd/unified-crypt-queue

usb: dwc3: trace: print enqueue/dequeue pointers too

2020-03-15T09:06:39Z

By printing enqueue/dequeue pointers, we can make sure that our TRB handling is correct. We've had a recent situation where we were not always dequeueing all TRBs in an SG list and this helped figure out what the problem was. Signed-off-by: Felipe Balbi

usb: common: Separated decoding functions from dwc3 driver.

2019-08-29T07:52:27Z

Patch moves some decoding functions from driver/usb/dwc3/debug.h driver to driver/usb/common/debug.c file. These moved functions include: dwc3_decode_get_status dwc3_decode_set_clear_feature dwc3_decode_set_address dwc3_decode_get_set_descriptor dwc3_decode_get_configuration dwc3_decode_set_configuration dwc3_decode_get_intf dwc3_decode_set_intf dwc3_decode_synch_frame dwc3_decode_set_sel dwc3_decode_set_isoch_delay dwc3_decode_ctrl These functions are used also in inroduced cdns3 driver. All functions prefixes were changed from dwc3 to usb. Also, function's parameters has been extended according to the name of fields in standard SETUP packet. Additionally, patch adds usb_decode_ctrl function to include/linux/usb/ch9.h file. Signed-off-by: Pawel Laszczak Signed-off-by: Felipe Balbi

usb: dwc3: trace: pass trace buffer size to decoding functions

2019-02-04T13:43:38Z

Instead of assuming that our buffer is big enough, let's pass the buffer size around so printing functions can make sure they won't overflow the buffer. Signed-off-by: Felipe Balbi

usb: dwc3: gadget: remove DWC3_EP_END_TRANSFER_PENDING

2019-02-04T06:46:22Z

Now that we modified the code to fix a race condition, it's clear that DWC3_EP_END_TRANSFER_PENDING is unnecessary, considering that DWC3_EP_TRANSFER_STARTED will remain set until End Transfer completes. Tested-by: Thinh Nguyen Signed-off-by: Felipe Balbi

usb: dwc3: trace: add missing break statement to make compiler happy

2018-12-07T05:30:18Z

The missed break statement in the outer switch makes the code fall through always and thus always same value will be printed. Besides that, compiler warns about missed fall through marker: drivers/usb/dwc3/./trace.h: In function ‘trace_raw_output_dwc3_log_trb’: drivers/usb/dwc3/./trace.h:246:4: warning: this statement may fall through [-Wimplicit-fallthrough=] switch (pcm) { ^~~~~~ Add the missing break statement to work correctly without compilation warnings. Fixes: fa8d965d736b ("usb: dwc3: trace: pretty print high-bandwidth transfers too") Cc: Felipe Balbi Signed-off-by: Andy Shevchenko Signed-off-by: Felipe Balbi

usb: dwc3: trace: log ep commands in hex

2018-11-26T07:14:03Z

They are much more useful in hexadecimal than in decimal. Moreover, generic commands are already logged in hex. Signed-off-by: Felipe Balbi

usb: dwc3: gadget: remove duplicated missed isoc handling

2018-05-21T07:00:53Z

Now, this part of the code is duplicated and brings no extra value to the driver. Let's remove it. Signed-off-by: Felipe Balbi

usb: dwc3: gadget: remove DWC3_EP_BUSY flag

2018-05-21T07:00:52Z

It has no use anymore. Signed-off-by: Felipe Balbi

usb: dwc3: gadget: remove allocated/queued request tracking

2018-05-21T07:00:50Z

That has never proven useful in any way. Just remove it. Signed-off-by: Felipe Balbi

usb: dwc3: Make sparse a bit happier

2017-12-11T10:36:49Z

Fixes the following warnings: drivers/usb/dwc3/io.h:43:31: warning: incorrect type in argument 1 (different address spaces) drivers/usb/dwc3/io.h:43:31: expected void *base drivers/usb/dwc3/io.h:43:31: got void [noderef] * drivers/usb/dwc3/io.h:62:32: warning: incorrect type in argument 1 (different address spaces) drivers/usb/dwc3/io.h:62:32: expected void *base drivers/usb/dwc3/io.h:62:32: got void [noderef] * This fixes the noisiest ones as they get emitted multiple times. A few warnings remain, for which the proper fix is less clear. No behaviour change is expected. Signed-off-by: Vincent Pelletier Signed-off-by: Felipe Balbi