Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1228239 > unrolled thread

[PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for OF platform driver

Started byLuis de Bethencourt <luis@debethencourt.com>
First post2015-09-18 21:20 +0200
Last post2015-09-19 07:00 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for OF  platform driver Luis de Bethencourt <luis@debethencourt.com> - 2015-09-18 21:20 +0200
    Re: [PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for  OF platform driver Alan Stern <stern@rowland.harvard.edu> - 2015-09-18 21:40 +0200
    Re: [PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for OF  platform driver Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2015-09-19 07:00 +0200

#1228239 — [PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for OF platform driver

FromLuis de Bethencourt <luis@debethencourt.com>
Date2015-09-18 21:20 +0200
Subject[PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for OF platform driver
Message-ID<qa9mq-3gO-9@gated-at.bofh.it>
This platform driver has a OF device ID table but the OF module
alias information is not created so module autoloading won't work.

Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
---
 drivers/usb/host/ehci-spear.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/usb/host/ehci-spear.c b/drivers/usb/host/ehci-spear.c
index 34e1474..3c4e525 100644
--- a/drivers/usb/host/ehci-spear.c
+++ b/drivers/usb/host/ehci-spear.c
@@ -149,6 +149,7 @@ static const struct of_device_id spear_ehci_id_table[] = {
 	{ .compatible = "st,spear600-ehci", },
 	{ },
 };
+MODULE_DEVICE_TABLE(of, spear_ehci_id_table);
 
 static struct platform_driver spear_ehci_hcd_driver = {
 	.probe		= spear_ehci_hcd_drv_probe,
-- 
2.4.6

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1228245 — Re: [PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for OF platform driver

FromAlan Stern <stern@rowland.harvard.edu>
Date2015-09-18 21:40 +0200
SubjectRe: [PATCH v2 1/4] usb: host: ehci-spear: Fix module autoload for OF platform driver
Message-ID<qa9FL-3Dm-13@gated-at.bofh.it>
In reply to#1228239
On Fri, 18 Sep 2015, Luis de Bethencourt wrote:

> This platform driver has a OF device ID table but the OF module
> alias information is not created so module autoloading won't work.
> 
> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> ---
>  drivers/usb/host/ehci-spear.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/usb/host/ehci-spear.c b/drivers/usb/host/ehci-spear.c
> index 34e1474..3c4e525 100644
> --- a/drivers/usb/host/ehci-spear.c
> +++ b/drivers/usb/host/ehci-spear.c
> @@ -149,6 +149,7 @@ static const struct of_device_id spear_ehci_id_table[] = {
>  	{ .compatible = "st,spear600-ehci", },
>  	{ },
>  };
> +MODULE_DEVICE_TABLE(of, spear_ehci_id_table);
>  
>  static struct platform_driver spear_ehci_hcd_driver = {
>  	.probe		= spear_ehci_hcd_drv_probe,
> 

Acked-by: Alan Stern <stern@rowland.harvard.edu>

Also for V2 of patches 3/4 and 4/4.

Alan Stern


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1228390

FromGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Date2015-09-19 07:00 +0200
Message-ID<qaipH-7VZ-3@gated-at.bofh.it>
In reply to#1228239
On Fri, Sep 18, 2015 at 09:14:52PM +0200, Luis de Bethencourt wrote:
> This platform driver has a OF device ID table but the OF module
> alias information is not created so module autoloading won't work.
> 
> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> ---
>  drivers/usb/host/ehci-spear.c | 1 +
>  1 file changed, 1 insertion(+)

Ugh, what a mess, please resend the whole series when you fix up a patch
in it as this is hard to figure out what to do.

And can you "thread" the patch series properly?  That makes it much
easier for me to apply them as I can group them correctly.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web