From: matt mooney Date: Fri, 6 May 2011 10:47:54 +0000 (-0700) Subject: staging: usbip: add break to default case in switch statements X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=49aecefcdef5a26a7fb036d4c57573f0e0e2089b;p=mv-sheeva.git 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 Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/usbip/stub_rx.c b/drivers/staging/usbip/stub_rx.c index ac4dd4dae23..e11ac2a5c64 100644 --- a/drivers/staging/usbip/stub_rx.c +++ b/drivers/staging/usbip/stub_rx.c @@ -594,7 +594,7 @@ static void stub_rx_pdu(struct usbip_device *ud) /* NOTREACHED */ dev_err(dev, "unknown pdu\n"); usbip_event_add(ud, SDEV_EVENT_ERROR_TCP); - return; + break; } } diff --git a/drivers/staging/usbip/stub_tx.c b/drivers/staging/usbip/stub_tx.c index bbf207db598..76d82b203e1 100644 --- a/drivers/staging/usbip/stub_tx.c +++ b/drivers/staging/usbip/stub_tx.c @@ -91,6 +91,7 @@ void stub_complete(struct urb *urb) default: usbip_uinfo("urb completion with non-zero status %d\n", urb->status); + break; } /* link a urb to the queue of tx. */ diff --git a/drivers/staging/usbip/usbip_common.c b/drivers/staging/usbip/usbip_common.c index cd66d56b5ef..a6ae67c7457 100644 --- a/drivers/staging/usbip/usbip_common.c +++ b/drivers/staging/usbip/usbip_common.c @@ -91,6 +91,7 @@ static void usbip_dump_pipe(unsigned int p) break; default: printk(KERN_DEBUG "ERR"); + break; } printk(KERN_DEBUG "\n"); @@ -120,6 +121,7 @@ static void usbip_dump_usb_device(struct usb_device *udev) break; default: printk(KERN_DEBUG " SPD_ERROR"); + break; } printk(KERN_DEBUG " tt %p, ttport %d", udev->tt, udev->ttport); @@ -187,6 +189,7 @@ static void usbip_dump_request_type(__u8 rt) break; default: printk(KERN_DEBUG "------"); + break; } } @@ -244,6 +247,7 @@ static void usbip_dump_usb_ctrlrequest(struct usb_ctrlrequest *cmd) break; default: printk(KERN_DEBUG "REQ(%02X) ", cmd->bRequest); + break; } printk(KERN_DEBUG " "); @@ -342,7 +346,8 @@ void usbip_dump_header(struct usbip_header *pdu) break; default: /* NOT REACHED */ - usbip_udbg("UNKNOWN\n"); + usbip_udbg("unknown command\n"); + break; } } EXPORT_SYMBOL_GPL(usbip_dump_header); @@ -537,9 +542,10 @@ void usbip_pack_pdu(struct usbip_header *pdu, struct urb *urb, int cmd, 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 @@ void usbip_header_correct_endian(struct usbip_header *pdu, int send) 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); diff --git a/drivers/staging/usbip/vhci_hcd.c b/drivers/staging/usbip/vhci_hcd.c index d2566980a57..a94c2782b50 100644 --- a/drivers/staging/usbip/vhci_hcd.c +++ b/drivers/staging/usbip/vhci_hcd.c @@ -320,6 +320,7 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, usbip_dbg_vhci_rh(" ClearPortFeature: default %x\n", wValue); dum->port_status[rhport] &= ~(1 << wValue); + break; } break; case GetHubDescriptor: @@ -451,6 +452,7 @@ static int vhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, usbip_dbg_vhci_rh(" SetPortFeature: default %d\n", wValue); dum->port_status[rhport] |= (1 << wValue); + break; } break; diff --git a/drivers/staging/usbip/vhci_rx.c b/drivers/staging/usbip/vhci_rx.c index 61b9fc702b5..46f2ee5ec09 100644 --- a/drivers/staging/usbip/vhci_rx.c +++ b/drivers/staging/usbip/vhci_rx.c @@ -247,10 +247,11 @@ static void vhci_rx_pdu(struct usbip_device *ud) vhci_recv_ret_unlink(vdev, &pdu); break; default: - /* NOTREACHED */ + /* NOT REACHED */ usbip_uerr("unknown pdu %u\n", pdu.base.command); usbip_dump_header(&pdu); usbip_event_add(ud, VDEV_EVENT_ERROR_TCP); + break; } }