staging: usbip: add break to default case in switch statements
For consistency, a break statement is added to all default cases that
do not jump to a label.
Signed-off-by: matt mooney <mfm@muteddisk.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
diff --git a/drivers/staging/usbip/usbip_common.c b/drivers/staging/usbip/usbip_common.c
index cd66d56..a6ae67c 100644
--- a/drivers/staging/usbip/usbip_common.c
+++ b/drivers/staging/usbip/usbip_common.c
@@ -91,6 +91,7 @@
break;
default:
printk(KERN_DEBUG "ERR");
+ break;
}
printk(KERN_DEBUG "\n");
@@ -120,6 +121,7 @@
break;
default:
printk(KERN_DEBUG " SPD_ERROR");
+ break;
}
printk(KERN_DEBUG " tt %p, ttport %d", udev->tt, udev->ttport);
@@ -187,6 +189,7 @@
break;
default:
printk(KERN_DEBUG "------");
+ break;
}
}
@@ -244,6 +247,7 @@
break;
default:
printk(KERN_DEBUG "REQ(%02X) ", cmd->bRequest);
+ break;
}
printk(KERN_DEBUG " ");
@@ -342,7 +346,8 @@
break;
default:
/* NOT REACHED */
- usbip_udbg("UNKNOWN\n");
+ usbip_udbg("unknown command\n");
+ break;
}
}
EXPORT_SYMBOL_GPL(usbip_dump_header);
@@ -537,9 +542,10 @@
usbip_pack_ret_submit(pdu, urb, pack);
break;
default:
+ /* NOT REACHED */
err("unknown command");
- /* NOTREACHED */
/* BUG(); */
+ break;
}
}
EXPORT_SYMBOL_GPL(usbip_pack_pdu);
@@ -643,9 +649,10 @@
correct_endian_ret_unlink(&pdu->u.ret_unlink, send);
break;
default:
- /* NOTREACHED */
+ /* NOT REACHED */
err("unknown command in pdu header: %d", cmd);
/* BUG(); */
+ break;
}
}
EXPORT_SYMBOL_GPL(usbip_header_correct_endian);