From patchwork Thu Mar 30 03:26:37 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "\(Exiting\) Baolin Wang" X-Patchwork-Id: 96219 Delivered-To: patch@linaro.org Received: by 10.140.89.233 with SMTP id v96csp48819qgd; Wed, 29 Mar 2017 20:27:20 -0700 (PDT) X-Received: by 10.98.252.6 with SMTP id e6mr3838037pfh.267.1490844440868; Wed, 29 Mar 2017 20:27:20 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e4si781021plj.236.2017.03.29.20.27.20; Wed, 29 Mar 2017 20:27:20 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755453AbdC3D1T (ORCPT + 21 others); Wed, 29 Mar 2017 23:27:19 -0400 Received: from mail-pg0-f51.google.com ([74.125.83.51]:36846 "EHLO mail-pg0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754908AbdC3D1Q (ORCPT ); Wed, 29 Mar 2017 23:27:16 -0400 Received: by mail-pg0-f51.google.com with SMTP id g2so25433354pge.3 for ; Wed, 29 Mar 2017 20:27:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=G3t2QIfaKlRYtfCG/bdlZIBrVe1nLzptKC04308OY+w=; b=ER8loEz4CaIZZM8+aysVmuVvvmlU6KUc7St8wQh93eFzjiAM+AXOdWkJB870VubWe1 IXIOcjRw4sgC2JFNhzfw0V4RIyckkM+cs/PldXb9dnVeXyHA15NNs6XFdM/3t+btx0HT ULg/HrvXEiwrqwuqmKDNibwFFtOVroV/128rA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=G3t2QIfaKlRYtfCG/bdlZIBrVe1nLzptKC04308OY+w=; b=OY49hCfTuMToAaHCawIb6WpvDAB5iXPfPYbgDBGHQM4sukxFegaJZXfrPHSWg6TU3Q UrhBdOoseT27LsYtvbIyu9y0o0z8weDnYk3M3xEZscLWEgEm5ORiCUMov65imrL/wPWh uSVIV0OZ9pBp1DGf6V5ABIdTcESQs1KHEEfJeDp7Eqftp2FJbR3EhFt2ElF13ciJQcM/ 3nHvheOQ+ssTv0cq4zsC26HdaqMGKawXATQYUZwn9NWjNH5gFhYq4ObhnvPovkw9IPjJ 9NltL5pp8zTKMdezpopWZewJzZcjA9jjJn+YHFzd/qS/EhCc9KgqnyTBDRjiCUyzz9QL wjTw== X-Gm-Message-State: AFeK/H2aITp1LQ2Cu2F2LN3FQrQVFW/h7mfyCeXGn1KN235kZXjHS7old3qWFQf+fdacebPe X-Received: by 10.98.198.138 with SMTP id x10mr3809458pfk.85.1490844435415; Wed, 29 Mar 2017 20:27:15 -0700 (PDT) Received: from baolinwangubtpc.spreadtrum.com ([117.18.48.82]) by smtp.gmail.com with ESMTPSA id g75sm823669pfj.107.2017.03.29.20.27.13 (version=TLS1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 29 Mar 2017 20:27:14 -0700 (PDT) From: Baolin Wang To: mathias.nyman@intel.com Cc: gregkh@linuxfoundation.org, broonie@kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, baolin.wang@linaro.org Subject: [PATCH] usb: host: plat: Enable xHCI plat runtime PM Date: Thu, 30 Mar 2017 11:26:37 +0800 Message-Id: <50a6c7681d2731a72d7488d4c362833ceeac8c8b.1490841752.git.baolin.wang@linaro.org> X-Mailer: git-send-email 1.7.9.5 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Enable the xHCI plat runtime PM for parent device to suspend/resume xHCI. Also call pm_runtime_forbid() in probe() function to force users to explicitly enable runtime pm using power/control in sysfs, in case some parent devices didn't implement runtime PM callbacks. Signed-off-by: Baolin Wang --- drivers/usb/host/xhci-plat.c | 54 ++++++++++++++++++++++++++++++++++++------ 1 file changed, 47 insertions(+), 7 deletions(-) -- 1.7.9.5 diff --git a/drivers/usb/host/xhci-plat.c b/drivers/usb/host/xhci-plat.c index bd02a6c..2036c24 100644 --- a/drivers/usb/host/xhci-plat.c +++ b/drivers/usb/host/xhci-plat.c @@ -179,9 +179,15 @@ static int xhci_plat_probe(struct platform_device *pdev) return ret; } + pm_runtime_set_active(&pdev->dev); + pm_runtime_enable(&pdev->dev); + pm_runtime_get_noresume(&pdev->dev); + hcd = usb_create_hcd(driver, &pdev->dev, dev_name(&pdev->dev)); - if (!hcd) - return -ENOMEM; + if (!hcd) { + ret = -ENOMEM; + goto disable_runtime; + } res = platform_get_resource(pdev, IORESOURCE_MEM, 0); hcd->regs = devm_ioremap_resource(&pdev->dev, res); @@ -258,6 +264,14 @@ static int xhci_plat_probe(struct platform_device *pdev) if (ret) goto dealloc_usb2_hcd; + pm_runtime_put_noidle(&pdev->dev); + + /* + * Prevent runtime pm from being on as default, users should enable + * runtime pm using power/control in sysfs. + */ + pm_runtime_forbid(&pdev->dev); + return 0; @@ -277,6 +291,10 @@ static int xhci_plat_probe(struct platform_device *pdev) put_hcd: usb_put_hcd(hcd); +disable_runtime: + pm_runtime_put_noidle(&pdev->dev); + pm_runtime_disable(&pdev->dev); + return ret; } @@ -298,6 +316,9 @@ static int xhci_plat_remove(struct platform_device *dev) clk_disable_unprepare(clk); usb_put_hcd(hcd); + pm_runtime_set_suspended(&dev->dev); + pm_runtime_disable(&dev->dev); + return 0; } @@ -325,14 +346,33 @@ static int xhci_plat_resume(struct device *dev) return xhci_resume(xhci, 0); } +#endif /* CONFIG_PM_SLEEP */ + +#ifdef CONFIG_PM +static int xhci_plat_runtime_suspend(struct device *dev) +{ + struct usb_hcd *hcd = dev_get_drvdata(dev); + struct xhci_hcd *xhci = hcd_to_xhci(hcd); + + return xhci_suspend(xhci, device_may_wakeup(dev)); +} + +static int xhci_plat_runtime_resume(struct device *dev) +{ + struct usb_hcd *hcd = dev_get_drvdata(dev); + struct xhci_hcd *xhci = hcd_to_xhci(hcd); + + return xhci_resume(xhci, 0); +} +#endif /* CONFIG_PM */ static const struct dev_pm_ops xhci_plat_pm_ops = { SET_SYSTEM_SLEEP_PM_OPS(xhci_plat_suspend, xhci_plat_resume) + + SET_RUNTIME_PM_OPS(xhci_plat_runtime_suspend, + xhci_plat_runtime_resume, + NULL) }; -#define DEV_PM_OPS (&xhci_plat_pm_ops) -#else -#define DEV_PM_OPS NULL -#endif /* CONFIG_PM */ static const struct acpi_device_id usb_xhci_acpi_match[] = { /* XHCI-compliant USB Controller */ @@ -346,7 +386,7 @@ static int xhci_plat_resume(struct device *dev) .remove = xhci_plat_remove, .driver = { .name = "xhci-hcd", - .pm = DEV_PM_OPS, + .pm = &xhci_plat_pm_ops, .of_match_table = of_match_ptr(usb_xhci_of_match), .acpi_match_table = ACPI_PTR(usb_xhci_acpi_match), },