> -----Original Message----- > From: Felipe Balbi [mailto:[email protected]] > Sent: Saturday, March 14, 2015 6:11 PM > To: John Youn > Cc: Kaukab, Yousaf; [email protected]; [email protected]; Herrero, > Gregory; [email protected]; [email protected]; > [email protected] > Subject: Re: [PATCH 01/19] usb: dwc2: move debugfs code to a separate file > > On Mon, Mar 09, 2015 at 06:42:36PM -0700, John Youn wrote: > > On 3/9/2015 8:04 AM, Mian Yousaf Kaukab wrote: > > > + > > > +int dwc2_debugfs_init(struct dwc2_hsotg *hsotg) { > > > + int ret; > > > + > > > + hsotg->debug_root = debugfs_create_dir(dev_name(hsotg- > >dev), NULL); > > > + if (!hsotg->debug_root) { > > > + ret = -ENOMEM; > > > + goto err0; > > > + } > > > + > > > + /* Add gadget debugfs nodes */ > > > + s3c_hsotg_create_debug(hsotg); > > > +err0: > > > + return ret; > > > +} > > > > Need export for this function when dwc2-platform is configured as a module. > > the file is still part of the same binary, right ? EXPORT_SYMBOL*() are only > needed when functions are exposed to other modules. Usually, > EXPORT_SYMBOL*() in a driver (not in the framework) is an indication that > something's wrong ;-)
EXPORT_SYMBOL* are needed because the driver is built as three different modules dwc2.ko dwc2_platform.ko and dwc2_pci.ko. As dwc2_pci is now exporting platform device as well, we can get rid of all EXPORT_SYMBOLS* by having only two modules dwc2.ko and dwc2_pci.ko. That is by including platform.o in dwc2.ko I will include the missing EXPORT_SYMBOLS* in this patch. I have another patch read to merge dwc2.ko and dwc2_platform.ko and remove all EXPORT_SYMBOLS*. If John agrees, then I can include this in the next revision of this patch-set. > > -- > balbi BR, Yousaf -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
