Hi,
On Thu, Nov 08, 2012 at 11:11:37AM -0500, Alan Stern wrote:
> On Thu, 8 Nov 2012, Felipe Balbi wrote:
>
> > > Mark the *_overrides structures as __initdata.
> >
> > I just have one question here: What about all the ehci-*.c with their
> > probe outside of .init.text ?
> >
> > $ git grep -e __devinit drivers/usb/host/ehci*.c
> > drivers/usb/host/ehci-atmel.c:static int __devinit ehci_atmel_drv_probe(struct platform_device *pdev)
> > drivers/usb/host/ehci-grlib.c:static int __devinit ehci_hcd_grlib_probe(struct platform_device *op)
> > drivers/usb/host/ehci-orion.c:static int __devinit ehci_orion_drv_probe(struct platform_device *pdev)
> > drivers/usb/host/ehci-platform.c:static int __devinit ehci_platform_probe(struct platform_device *dev)
> > drivers/usb/host/ehci-ppc-of.c:static int __devinit
> > drivers/usb/host/ehci-ppc-of.c:static int __devinit ehci_hcd_ppc_of_probe(struct platform_device *op)
> > drivers/usb/host/ehci-ps3.c:static int __devinit ps3_ehci_probe(struct ps3_system_bus_device *dev)
> > drivers/usb/host/ehci-s5p.c:static int __devinit s5p_ehci_probe(struct platform_device *pdev)
> > drivers/usb/host/ehci-spear.c:static struct of_device_id spear_ehci_id_table[] __devinitdata = {
> > drivers/usb/host/ehci-tegra.c:static struct of_device_id tegra_ehci_of_match[] __devinitdata = {
> > drivers/usb/host/ehci-w90x900.c:static int __devinit usb_w90x900_probe(const struct hc_driver *driver,
> > drivers/usb/host/ehci-w90x900.c:static int __devinit ehci_w90x900_probe(struct platform_device *pdev)
> > drivers/usb/host/ehci-xilinx-of.c:static int __devinit ehci_hcd_xilinx_of_probe(struct platform_device *op)
>
> These drivers, when they are converted, will need a new ehci_*_init()
> routine (separate from their probe routine) that calls
> ehci_driver_init(). The new routine will be marked with __init and
> will have a module_init() line.
>
> This is the normal sort of thing that has to be done when converting a
> driver to a standalone module. If you have a particular example you
fair enough, I'm happy with that :-)
> can test, I'll write a patch to do the conversion.
unfortunately not, OMAP isn't part of that list :-(
--
balbi
Attachment:
signature.asc
Description: Digital signature
