diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2018-12-03 11:28:47 +0200 |
|---|---|---|
| committer | Felipe Balbi <felipe.balbi@linux.intel.com> | 2018-12-07 07:30:18 +0200 |
| commit | 54d48183d21e03f780053d7129312049cb5dd591 (patch) | |
| tree | 13510ca56ea4d16d872d093da0c0ded41a144edf | |
| parent | 6abfa0f5bb7cce98c89e2c28fcea31c17200890e (diff) | |
| download | linux-54d48183d21e03f780053d7129312049cb5dd591.tar.gz linux-54d48183d21e03f780053d7129312049cb5dd591.tar.bz2 linux-54d48183d21e03f780053d7129312049cb5dd591.zip | |
usb: dwc3: trace: add missing break statement to make compiler happy
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 <felipe.balbi@linux.intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
| -rw-r--r-- | drivers/usb/dwc3/trace.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/usb/dwc3/trace.h b/drivers/usb/dwc3/trace.h index 50fb6f2d92dd..e97a00593dda 100644 --- a/drivers/usb/dwc3/trace.h +++ b/drivers/usb/dwc3/trace.h @@ -251,9 +251,11 @@ DECLARE_EVENT_CLASS(dwc3_log_trb, s = "2x "; break; case 3: + default: s = "3x "; break; } + break; default: s = ""; } s; }), |