USB: ehci-au1xxx: fix suspend callback
authorManuel Lauss <manuel.lauss@googlemail.com>
Thu, 5 May 2011 14:35:24 +0000 (16:35 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sat, 7 May 2011 01:16:39 +0000 (18:16 -0700)
Remove a stray 'return 0' at the top of the suspend callback,
and move au1xxx_stop_ehc() out of the ehci spinlock since it takes
some time to complete.

Signed-off-by: Manuel Lauss <manuel.lauss@googlemail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/host/ehci-au1xxx.c

index 40b002869ac2602415682c3b030474069b027ff5..42ae57409908750b329385e27ebb3ec36d5c9c24 100644 (file)
@@ -216,10 +216,7 @@ static int ehci_hcd_au1xxx_drv_suspend(struct device *dev)
        struct usb_hcd *hcd = dev_get_drvdata(dev);
        struct ehci_hcd *ehci = hcd_to_ehci(hcd);
        unsigned long flags;
-       int rc;
-
-       return 0;
-       rc = 0;
+       int rc = 0;
 
        if (time_before(jiffies, ehci->next_statechange))
                msleep(10);
@@ -234,13 +231,13 @@ static int ehci_hcd_au1xxx_drv_suspend(struct device *dev)
        (void)ehci_readl(ehci, &ehci->regs->intr_enable);
 
        clear_bit(HCD_FLAG_HW_ACCESSIBLE, &hcd->flags);
-
-       au1xxx_stop_ehc();
        spin_unlock_irqrestore(&ehci->lock, flags);
 
        // could save FLADJ in case of Vaux power loss
        // ... we'd only use it to handle clock skew
 
+       au1xxx_stop_ehc();
+
        return rc;
 }