Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752888Ab2JEHmE (ORCPT ); Fri, 5 Oct 2012 03:42:04 -0400 Received: from eusmtp01.atmel.com ([212.144.249.243]:20530 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752730Ab2JEHmB (ORCPT ); Fri, 5 Oct 2012 03:42:01 -0400 Message-ID: <506E8F48.8040205@atmel.com> Date: Fri, 5 Oct 2012 09:42:00 +0200 From: Nicolas Ferre Organization: atmel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120912 Thunderbird/15.0.1 MIME-Version: 1.0 To: Florian Fainelli CC: , Kukjin Kim , "Greg Kroah-Hartman" , , , Felipe Balbi , "Lennert Buytenhek" , , "Ben Dooks" , , Subject: Re: [PATCH 14/24] USB: ohci: merge ohci_finish_controller_resume with ohci_resume References: <1349363872-27004-1-git-send-email-florian@openwrt.org> <1349363872-27004-15-git-send-email-florian@openwrt.org> In-Reply-To: <1349363872-27004-15-git-send-email-florian@openwrt.org> X-Enigmail-Version: 1.4.4 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.161.30.18] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 8940 Lines: 270 On 10/04/2012 05:17 PM, Florian Fainelli : > Merge ohci_finish_controller_resume with ohci_resume as suggested by Alan > Stern. Since ohci_finish_controller_resume no longer exists, update the > various OHCI drivers to call ohci_resume() instead. Some drivers used to set > themselves the bit HCD_FLAG_HW_ACCESSIBLE, which is now handled by > ohci_resume(). > > Signed-off-by: Florian Fainelli > --- > drivers/usb/host/ohci-at91.c | 2 +- Seems ok for AT91, so Acked-by: Nicolas Ferre Thanks Florian, bye, > drivers/usb/host/ohci-ep93xx.c | 2 +- > drivers/usb/host/ohci-exynos.c | 5 +---- > drivers/usb/host/ohci-hcd.c | 41 +++++++++++++++++++++++++++++++++++-- > drivers/usb/host/ohci-hub.c | 42 -------------------------------------- > drivers/usb/host/ohci-omap.c | 2 +- > drivers/usb/host/ohci-platform.c | 2 +- > drivers/usb/host/ohci-pxa27x.c | 2 +- > drivers/usb/host/ohci-s3c2410.c | 3 +-- > drivers/usb/host/ohci-spear.c | 2 +- > drivers/usb/host/ohci-tmio.c | 2 +- > 11 files changed, 48 insertions(+), 57 deletions(-) > > diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci-at91.c > index 0bf72f9..908d84a 100644 > --- a/drivers/usb/host/ohci-at91.c > +++ b/drivers/usb/host/ohci-at91.c > @@ -705,7 +705,7 @@ static int ohci_hcd_at91_drv_resume(struct platform_device *pdev) > if (!clocked) > at91_start_clock(); > > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > return 0; > } > #else > diff --git a/drivers/usb/host/ohci-ep93xx.c b/drivers/usb/host/ohci-ep93xx.c > index dbfbd1d..a982f04 100644 > --- a/drivers/usb/host/ohci-ep93xx.c > +++ b/drivers/usb/host/ohci-ep93xx.c > @@ -194,7 +194,7 @@ static int ohci_hcd_ep93xx_drv_resume(struct platform_device *pdev) > > ep93xx_start_hc(&pdev->dev); > > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > return 0; > } > #endif > diff --git a/drivers/usb/host/ohci-exynos.c b/drivers/usb/host/ohci-exynos.c > index fc3091b..53c5a989 100644 > --- a/drivers/usb/host/ohci-exynos.c > +++ b/drivers/usb/host/ohci-exynos.c > @@ -252,10 +252,7 @@ static int exynos_ohci_resume(struct device *dev) > if (pdata && pdata->phy_init) > pdata->phy_init(pdev, S5P_USB_PHY_HOST); > > - /* Mark hardware accessible again as we are out of D3 state by now */ > - set_bit(HCD_FLAG_HW_ACCESSIBLE, &hcd->flags); > - > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > > return 0; > } > diff --git a/drivers/usb/host/ohci-hcd.c b/drivers/usb/host/ohci-hcd.c > index 5d30992..568bdb3 100644 > --- a/drivers/usb/host/ohci-hcd.c > +++ b/drivers/usb/host/ohci-hcd.c > @@ -1003,13 +1003,50 @@ static int __maybe_unused ohci_suspend(struct usb_hcd *hcd, bool do_wakeup) > > static int __maybe_unused ohci_resume(struct usb_hcd *hcd, bool hibernated) > { > + struct ohci_hcd *ohci = hcd_to_ohci(hcd); > + int port; > + bool need_reinit = false; > + > set_bit(HCD_FLAG_HW_ACCESSIBLE, &hcd->flags); > > /* Make sure resume from hibernation re-enumerates everything */ > if (hibernated) > - ohci_usb_reset(hcd_to_ohci(hcd)); > + ohci_usb_reset(ohci); > + > + /* See if the controller is already running or has been reset */ > + ohci->hc_control = ohci_readl(ohci, &ohci->regs->control); > + if (ohci->hc_control & (OHCI_CTRL_IR | OHCI_SCHED_ENABLES)) { > + need_reinit = true; > + } else { > + switch (ohci->hc_control & OHCI_CTRL_HCFS) { > + case OHCI_USB_OPER: > + case OHCI_USB_RESET: > + need_reinit = true; > + } > + } > + > + /* If needed, reinitialize and suspend the root hub */ > + if (need_reinit) { > + spin_lock_irq(&ohci->lock); > + ohci_rh_resume(ohci); > + ohci_rh_suspend(ohci, 0); > + spin_unlock_irq(&ohci->lock); > + } > + > + /* Normally just turn on port power and enable interrupts */ > + else { > + ohci_dbg(ohci, "powerup ports\n"); > + for (port = 0; port < ohci->num_ports; port++) > + ohci_writel(ohci, RH_PS_PPS, > + &ohci->regs->roothub.portstatus[port]); > + > + ohci_writel(ohci, OHCI_INTR_MIE, &ohci->regs->intrenable); > + ohci_readl(ohci, &ohci->regs->intrenable); > + msleep(20); > + } > + > + usb_hcd_resume_root_hub(hcd); > > - ohci_finish_controller_resume(hcd); > return 0; > } > > diff --git a/drivers/usb/host/ohci-hub.c b/drivers/usb/host/ohci-hub.c > index 2f3619e..db09dae 100644 > --- a/drivers/usb/host/ohci-hub.c > +++ b/drivers/usb/host/ohci-hub.c > @@ -316,48 +316,6 @@ static int ohci_bus_resume (struct usb_hcd *hcd) > return rc; > } > > -/* Carry out the final steps of resuming the controller device */ > -static void __maybe_unused ohci_finish_controller_resume(struct usb_hcd *hcd) > -{ > - struct ohci_hcd *ohci = hcd_to_ohci(hcd); > - int port; > - bool need_reinit = false; > - > - /* See if the controller is already running or has been reset */ > - ohci->hc_control = ohci_readl(ohci, &ohci->regs->control); > - if (ohci->hc_control & (OHCI_CTRL_IR | OHCI_SCHED_ENABLES)) { > - need_reinit = true; > - } else { > - switch (ohci->hc_control & OHCI_CTRL_HCFS) { > - case OHCI_USB_OPER: > - case OHCI_USB_RESET: > - need_reinit = true; > - } > - } > - > - /* If needed, reinitialize and suspend the root hub */ > - if (need_reinit) { > - spin_lock_irq(&ohci->lock); > - ohci_rh_resume(ohci); > - ohci_rh_suspend(ohci, 0); > - spin_unlock_irq(&ohci->lock); > - } > - > - /* Normally just turn on port power and enable interrupts */ > - else { > - ohci_dbg(ohci, "powerup ports\n"); > - for (port = 0; port < ohci->num_ports; port++) > - ohci_writel(ohci, RH_PS_PPS, > - &ohci->regs->roothub.portstatus[port]); > - > - ohci_writel(ohci, OHCI_INTR_MIE, &ohci->regs->intrenable); > - ohci_readl(ohci, &ohci->regs->intrenable); > - msleep(20); > - } > - > - usb_hcd_resume_root_hub(hcd); > -} > - > /* Carry out polling-, autostop-, and autoresume-related state changes */ > static int ohci_root_hub_state_changes(struct ohci_hcd *ohci, int changed, > int any_connected, int rhsc_status) > diff --git a/drivers/usb/host/ohci-omap.c b/drivers/usb/host/ohci-omap.c > index f8b2d91..9fab4d8 100644 > --- a/drivers/usb/host/ohci-omap.c > +++ b/drivers/usb/host/ohci-omap.c > @@ -530,7 +530,7 @@ static int ohci_omap_resume(struct platform_device *dev) > ohci->next_statechange = jiffies; > > omap_ohci_clock_power(1); > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > return 0; > } > > diff --git a/drivers/usb/host/ohci-platform.c b/drivers/usb/host/ohci-platform.c > index 1caaf65..99d1755 100644 > --- a/drivers/usb/host/ohci-platform.c > +++ b/drivers/usb/host/ohci-platform.c > @@ -203,7 +203,7 @@ static int ohci_platform_resume(struct device *dev) > return err; > } > > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > return 0; > } > > diff --git a/drivers/usb/host/ohci-pxa27x.c b/drivers/usb/host/ohci-pxa27x.c > index 77f4402..8bffde3 100644 > --- a/drivers/usb/host/ohci-pxa27x.c > +++ b/drivers/usb/host/ohci-pxa27x.c > @@ -591,7 +591,7 @@ static int ohci_hcd_pxa27x_drv_resume(struct device *dev) > /* Select Power Management Mode */ > pxa27x_ohci_select_pmm(ohci, inf->port_mode); > > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > return 0; > } > > diff --git a/drivers/usb/host/ohci-s3c2410.c b/drivers/usb/host/ohci-s3c2410.c > index 664c869..8af53c6 100644 > --- a/drivers/usb/host/ohci-s3c2410.c > +++ b/drivers/usb/host/ohci-s3c2410.c > @@ -524,8 +524,7 @@ static int ohci_hcd_s3c2410_drv_resume(struct device *dev) > > s3c2410_start_hc(pdev, hcd); > > - set_bit(HCD_FLAG_HW_ACCESSIBLE, &hcd->flags); > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > > return 0; > } > diff --git a/drivers/usb/host/ohci-spear.c b/drivers/usb/host/ohci-spear.c > index fc7305e..d607be3 100644 > --- a/drivers/usb/host/ohci-spear.c > +++ b/drivers/usb/host/ohci-spear.c > @@ -231,7 +231,7 @@ static int spear_ohci_hcd_drv_resume(struct platform_device *dev) > ohci->next_statechange = jiffies; > > spear_start_ohci(ohci_p); > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > return 0; > } > #endif > diff --git a/drivers/usb/host/ohci-tmio.c b/drivers/usb/host/ohci-tmio.c > index 60c2b07..2c9ab8f 100644 > --- a/drivers/usb/host/ohci-tmio.c > +++ b/drivers/usb/host/ohci-tmio.c > @@ -352,7 +352,7 @@ static int ohci_hcd_tmio_drv_resume(struct platform_device *dev) > > spin_unlock_irqrestore(&tmio->lock, flags); > > - ohci_finish_controller_resume(hcd); > + ohci_resume(hcd, false); > > return 0; > } > -- Nicolas Ferre -- 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/