USB: Drop unnecessary continue in a few drivers
authorJulia Lawall <julia@diku.dk>
Wed, 14 Nov 2007 08:15:16 +0000 (09:15 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 1 Feb 2008 22:34:51 +0000 (14:34 -0800)
Continue is not needed at the bottom of a loop.

The semantic patch implementing this change is as follows:

@@
@@

for (...;...;...) {
   ...
   if (...) {
     ...
-   continue;
   }
}

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/host/isp116x-hcd.c
drivers/usb/image/mdc800.c

index c27417f5b9d8635320a491bd5fbdad1c8f4b366c..0130fd8571e4018549db9a5003bd432b72645878 100644 (file)
@@ -918,7 +918,6 @@ static int isp116x_hub_status_data(struct usb_hcd *hcd, char *buf)
                              | RH_PS_OCIC | RH_PS_PRSC)) {
                        changed = 1;
                        buf[0] |= 1 << (i + 1);
-                       continue;
                }
        }
        spin_unlock_irqrestore(&isp116x->lock, flags);
index d1131a87a5b13ad72e40e4f573a6a7ace8979445..0fb114ca1eba9d598da09483f7565451b351822a 100644 (file)
@@ -478,8 +478,6 @@ static int mdc800_usb_probe (struct usb_interface *intf,
                                {
                                        irq_interval=intf_desc->endpoint [j].desc.bInterval;
                                }
-
-                               continue;
                        }
                }
                if (mdc800->endpoint[i] == -1)