kirkwood devicetree respin
Arnd Bergmann
arnd at arndb.de
Wed Mar 21 06:15:07 EST 2012
On Tuesday 20 March 2012, Jason Cooper wrote:
> diff --git a/drivers/usb/host/ehci-orion.c b/drivers/usb/host/ehci-orion.c
> index 6c6a5a3..0808417 100644
> --- a/drivers/usb/host/ehci-orion.c
> +++ b/drivers/usb/host/ehci-orion.c
> @@ -318,3 +318,5 @@ static struct platform_driver ehci_orion_driver = {
> .shutdown = usb_hcd_platform_shutdown,
> .driver.name = "orion-ehci",
> };
> +
> +module_platform_driver(ehci_orion_driver);
>
This actually looks wrong to me, because the ehci driver has an
interesting way of pulling in platform specific drivers.
I'm pretty sure that it will break modular builds, which can only
have one module_init function in them. It's not clear to me why this
patch actually fixes the problem either. Could it be that you have
multiple platform driver back-ends for ehci enabled at the same time?
That should not be possible in theory and at least give a warning, but
the ehci probing method is a bit fragile so I would not be too surprised.
Arnd
More information about the devicetree-discuss
mailing list