On 11/11/2016 2:53 AM, Felipe Balbi wrote: > > Hi, > > John Youn <johnyoun@xxxxxxxxxxxx> writes: >> From: Vahram Aharonyan <vahrama@xxxxxxxxxxxx> >> >> Add dwc2_gadget_get_desc_params() function to define descriptor entry >> parameters based on the endpoint type. >> >> Add dwc2_gadget_config_nonisoc_xfer_ddma() function, which programs DDMA >> chain entries with corresponding values based on the received DMA buffer >> and transfer length in case of non-isochronous endpoint. Isochronous >> endpoints' DMA descriptors will be filled separately. >> >> Add dwc2_gadget_get_xfersize_ddma() function to iterate over DDMA chain >> and get info on bytes remaining in descriptor status field after >> transfer complete. >> >> Signed-off-by: Vahram Aharonyan <vahrama@xxxxxxxxxxxx> >> Signed-off-by: John Youn <johnyoun@xxxxxxxxxxxx> > > This patch adds build warnings: > > CC [M] drivers/usb/dwc2/gadget.o > drivers/usb/dwc2/gadget.c:1825:21: warning: ‘dwc2_gadget_get_xfersize_ddma’ defined but not used [-Wunused-function] > static unsigned int dwc2_gadget_get_xfersize_ddma(struct dwc2_hsotg_ep *hs_ep) > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > drivers/usb/dwc2/gadget.c:682:13: warning: ‘dwc2_gadget_config_nonisoc_xfer_ddma’ defined but not used [-Wunused-function] > static void dwc2_gadget_config_nonisoc_xfer_ddma(struct dwc2_hsotg_ep *hs_ep, > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > LD [M] drivers/usb/dwc2/dwc2.o > > I have applied up to the patch right before this one. Please fix the > warning and rebase on testing/next > Ok. I'll resend next week. Regards, John -- 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