Hello Thanks for reviewing. I will fix it and resend it on the next version of the patchset to avoid spamming the ML I guess it is ok to add your Reviewed-by Thanks! On Fri, Nov 28, 2014 at 3:46 PM, Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> wrote: > Hello. > > On 11/28/2014 04:50 PM, Ricardo Ribalda Delgado wrote: > >> restart_dma is not used before it is declaration. Therefore we can > > > s/it is/its/. > >> remove this definition. > > > You're removing the declaration, not definition. > >> Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@xxxxxxxxx> >> --- >> drivers/usb/gadget/udc/net2280.c | 2 -- >> 1 file changed, 2 deletions(-) > > >> diff --git a/drivers/usb/gadget/udc/net2280.c >> b/drivers/usb/gadget/udc/net2280.c >> index 0340462..e8c36db 100644 >> --- a/drivers/usb/gadget/udc/net2280.c >> +++ b/drivers/usb/gadget/udc/net2280.c >> @@ -1043,8 +1043,6 @@ dma_done(struct net2280_ep *ep, struct >> net2280_request *req, u32 dmacount, >> done(ep, req, status); >> } >> >> -static void restart_dma(struct net2280_ep *ep); >> - >> static void scan_dma_completions(struct net2280_ep *ep) >> { >> /* only look at descriptors that were "naturally" retired, > > > WBR, Sergei > -- Ricardo Ribalda -- 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