public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] EHCI USB: Don't use generic shutdown function
@ 2009-09-16  0:10 Olof Johansson
  2009-09-16  6:39 ` Felipe Balbi
  2009-09-22  1:14 ` [APPLIED] [PATCH] EHCI USB: Don't " Tony Lindgren
  0 siblings, 2 replies; 3+ messages in thread
From: Olof Johansson @ 2009-09-16  0:10 UTC (permalink / raw)
  To: linux-omap; +Cc: Felipe Balbi, Tony Lindgren

The OMAP EHCI glue code has a layer of driver state struct between
the platform_device and usb_hcd. So it can't use the generic
usb_hcd_platform_shutdown.

This fixes a panic at reboot time.


Signed-off-by: Olof Johansson <olof@lixom.net>

diff --git a/drivers/usb/host/ehci-omap.c b/drivers/usb/host/ehci-omap.c
index 8f8f022..bca3f0c 100644
--- a/drivers/usb/host/ehci-omap.c
+++ b/drivers/usb/host/ehci-omap.c
@@ -701,10 +701,19 @@ static int ehci_hcd_omap_remove(struct platform_device *pdev)
 	return 0;
 }
 
+static void ehci_hcd_omap_shutdown(struct platform_device *pdev)
+{
+	struct ehci_hcd_omap *omap = platform_get_drvdata(pdev);
+	struct usb_hcd *hcd = ehci_to_hcd(omap->ehci);
+
+	if (hcd->driver->shutdown)
+		hcd->driver->shutdown(hcd);
+}
+
 static struct platform_driver ehci_hcd_omap_driver = {
 	.probe			= ehci_hcd_omap_probe,
 	.remove			= ehci_hcd_omap_remove,
-	.shutdown		= usb_hcd_platform_shutdown,
+	.shutdown		= ehci_hcd_omap_shutdown,
 	/*.suspend		= ehci_hcd_omap_suspend, */
 	/*.resume		= ehci_hcd_omap_resume, */
 	.driver = {

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] EHCI USB: Don't use generic shutdown function
  2009-09-16  0:10 [PATCH] EHCI USB: Don't use generic shutdown function Olof Johansson
@ 2009-09-16  6:39 ` Felipe Balbi
  2009-09-22  1:14 ` [APPLIED] [PATCH] EHCI USB: Don&#39;t " Tony Lindgren
  1 sibling, 0 replies; 3+ messages in thread
From: Felipe Balbi @ 2009-09-16  6:39 UTC (permalink / raw)
  To: ext Olof Johansson
  Cc: linux-omap@vger.kernel.org, Felipe Balbi, Tony Lindgren

On Wed, Sep 16, 2009 at 02:10:55AM +0200, ext Olof Johansson wrote:
> The OMAP EHCI glue code has a layer of driver state struct between
> the platform_device and usb_hcd. So it can't use the generic
> usb_hcd_platform_shutdown.
> 
> This fixes a panic at reboot time.
> 
> 
> Signed-off-by: Olof Johansson <olof@lixom.net>

Acked-by: Felipe Balbi <felipe.balbi@nokia.com>

-- 
balbi

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [APPLIED] [PATCH] EHCI USB: Don&#39;t use generic shutdown function
  2009-09-16  0:10 [PATCH] EHCI USB: Don't use generic shutdown function Olof Johansson
  2009-09-16  6:39 ` Felipe Balbi
@ 2009-09-22  1:14 ` Tony Lindgren
  1 sibling, 0 replies; 3+ messages in thread
From: Tony Lindgren @ 2009-09-22  1:14 UTC (permalink / raw)
  To: linux-omap

This patch has been applied to the linux-omap
by youw fwiendly patch wobot.

Branch in linux-omap: ehci

Initial commit ID (Likely to change): 53e4096a17681ef7cd1ad0d97a55c9df51898594

PatchWorks
http://patchwork.kernel.org/patch/47783/

Git (Likely to change, and takes a while to get mirrored)
http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=53e4096a17681ef7cd1ad0d97a55c9df51898594



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-09-22  1:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-09-16  0:10 [PATCH] EHCI USB: Don't use generic shutdown function Olof Johansson
2009-09-16  6:39 ` Felipe Balbi
2009-09-22  1:14 ` [APPLIED] [PATCH] EHCI USB: Don&#39;t " Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox