On Mon, Dec 31, 2012 at 11:16:57AM +0530, chetan cr123 wrote: > On Sat, Dec 29, 2012 at 8:05 PM, Michal Nazarewicz <mina86@xxxxxxxxxx> wrote: > > On Sat, Dec 29 2012, chetan cr123 wrote: > >> usb_gadget_unregister_driver and usb_gadget_probe_driver functions are > >> exported (EXPORT_SYMBOL)from my gadget driver ((its in path > >> kernel/drivers/staging...)) > > > > Ah, missed this remark in my previous response. > > > > This is not correct. usb_gadget_unregister_driver and > > usb_gadget_probe_driverusb_gadget_probe_driver are exported by udc-core.c and UDC driver must > > instead use usb_add_gadget_udc() function to register as a UDC. > > > Ok.. i dont have udc-core.c file in my source code. > > I have these file related to udc : fsl_udc_core.c > fusb300_udc.c(since > we cant export 2 times > > usb_gadget_unregister_driver/usb_gadget_probe_driver) I have stopped > compiling fusb300_udc.c and musb_hdrc.c file which was exporting the > above register functions. > > Since i am exporting > usb_gadget_unregister_driver/usb_gadget_probe_driver in my driver. > > > is this a right way. ? > > i am using cypress mass storage driver (in path > /drivers/staging/westbridge/ben../gadget/cywbgadget.c ) That driver is very strange, and I don't even know if it works or not. You should contact the authors of it for any help you need with it, as no one else that I know of can even build that code, let alone run it. It's also marked to be removed from the kernel source tree in a release or so, so if you are using it, please work with the developers to get it cleaned up and moved to the proper location in the kernel source tree (out of drivers/staging/) so that the whole community can help out with it. thanks, greg k-h -- 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