Hi, On 24 November 2016 at 17:30, Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> wrote: > Hello. > > On 11/24/2016 11:13 AM, Baolin Wang wrote: > >> Since these 'return' statements are not generally useful in void function, >> remove them. >> >> Signed-off-by: Baolin Wang <baolin.wang@xxxxxxxxxx> > > [...] >> >> diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c >> index 48a26d378..d6f59a3 100644 >> --- a/drivers/usb/host/xhci-mem.c >> +++ b/drivers/usb/host/xhci-mem.c >> @@ -1146,7 +1146,6 @@ int xhci_setup_addressable_virt_dev(struct xhci_hcd >> *xhci, struct usb_device *ud >> case USB_SPEED_WIRELESS: >> xhci_dbg(xhci, "FIXME xHCI doesn't support wireless >> speeds\n"); >> return -EINVAL; >> - break; > > > Here you're removing *break* instead but not mentioning this in the patch > description... OK. I will add description to explain it in new patch. Thanks. -- Baolin.wang Best Regards -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
