From patchwork Wed Jun 21 19:21:28 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ulf Hansson X-Patchwork-Id: 9802761 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 6C98D601C0 for ; Wed, 21 Jun 2017 19:22:30 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5B520285E8 for ; Wed, 21 Jun 2017 19:22:30 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 599AF2866B; Wed, 21 Jun 2017 19:22:30 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 1329028653 for ; Wed, 21 Jun 2017 19:22:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751106AbdFUTWW (ORCPT ); Wed, 21 Jun 2017 15:22:22 -0400 Received: from mail-lf0-f42.google.com ([209.85.215.42]:34489 "EHLO mail-lf0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752007AbdFUTWB (ORCPT ); Wed, 21 Jun 2017 15:22:01 -0400 Received: by mail-lf0-f42.google.com with SMTP id l13so43023877lfl.1 for ; Wed, 21 Jun 2017 12:22:00 -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:in-reply-to:references; bh=gQ0z6uQZXEMVLPfg2GRn6Y71LvqWlZHeak/YrTUGnt4=; b=SPk29lmNM/olTnoR6riPwVRdcsRAEOODzwFzN7MaybsAnsZhxnNYpc41QGwDSXhk9u uGuIFltxl3D7vuzACyyGt8MkpvMV4+MQxJZ7OdZLMtfkyKx8P8VB8BdslWqqoorfhvIt xUfj2GF0Qjd9Y7xrZQlqPo/d1+QWALNgbyOr0= 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:in-reply-to :references; bh=gQ0z6uQZXEMVLPfg2GRn6Y71LvqWlZHeak/YrTUGnt4=; b=crhJaiKS5IZs+VvXUKygC0VkPpFqvHZx7CB7WFBo0ZdxAj/Rcbp0fj45mEFhqYHA75 9pobKld9G0+0ruXu/2fHpnO3heyTUSnfPZgDhecEKCuEvdtX4E4Q4Gf5E9sOXTD27pKa /JBV3vsg8Zq0AbsgdUOlm7SBW6vbpDsFO2iZHynW6qx0knJmwi7DhzgqFXj00bHWBNGF kqdSxViTttC7/ESDi4bVeg623CDbGx+s8lWSRfzQIgupBiWymOz/CtxzNp4/YEUKi1ef 7nyg670i772x5icTZTg6eO3IT3JEi174bhA9IlPn0y/y8gSKE1HQcOpNrukJQCLqFrYp /EZw== X-Gm-Message-State: AKS2vOzfqf4QiQJSBNrxVOoz5dKhC1ACi8SVDcHgLrEacw0bCLo0hhII Rhjf1iq5p1djP5r4 X-Received: by 10.46.69.212 with SMTP id s203mr11199810lja.67.1498072919669; Wed, 21 Jun 2017 12:21:59 -0700 (PDT) Received: from localhost.localdomain (h-158-174-22-67.NA.cust.bahnhof.se. [158.174.22.67]) by smtp.gmail.com with ESMTPSA id t24sm3920047ljd.1.2017.06.21.12.21.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 21 Jun 2017 12:21:58 -0700 (PDT) From: Ulf Hansson To: Wolfram Sang , "Rafael J . Wysocki" , Len Brown , linux-acpi@vger.kernel.org, linux-pm@vger.kernel.org Cc: Kevin Hilman , Jarkko Nikula , Andy Shevchenko , Mika Westerberg , Jisheng Zhang , John Stultz , Guodong Xu , Sumit Semwal , Haojian Zhuang , linux-arm-kernel@lists.infradead.org, linux-i2c@vger.kernel.org, Ulf Hansson Subject: [PATCH 9/9] i2c: designware: Deploy the runtime PM centric approach for system sleep Date: Wed, 21 Jun 2017 21:21:28 +0200 Message-Id: <1498072888-14782-10-git-send-email-ulf.hansson@linaro.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1498072888-14782-1-git-send-email-ulf.hansson@linaro.org> References: <1498072888-14782-1-git-send-email-ulf.hansson@linaro.org> Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Currently we runtime resume the device in the ->prepare() callback to make sure we manage system sleep properly. The earlier attempt to avoid that, as it's surely a waste in many situations, was based upon using the direct_complete path during system sleep. However, that failed because we could end up having the ->suspend() callback being invoked when the device was runtime suspended, causing calls for clock enable/disable and clock prepare/unprepare to become unbalanced. Deploying the runtime PM centric approach, via using the pm_runtime_force_suspend|resume() helpers as the system sleep callbacks, improves the behavior in the following regards: *) It avoids runtime resuming the device in the ->prepare() callback. **) The device remains runtime PM enabled during the device_suspend() phase, which thus makes it possible for users to send i2c transfers this entire phase. ***) It doesn't unnecessary runtime resume the device during system suspend, but allows the device to remain runtime suspended if that is possible. ****) It avoids to bring the device back to full power during system resume, unless really needed. Instead this task may get postponed to be managed by runtime PM. In case when the i2c device is attached to the ACPI PM domain, we need to inform ACPI to adapt its behavior for the runtime PM centric path during system sleep, which is done by calling acpi_dev_disable_direct_complete(). Worth to notice, comparing the earlier attempt of using the direct_complete() approach is that **) couldn't be supported and ****) relied on the device to be runtime suspended in the device_suspend() phase. Signed-off-by: Ulf Hansson --- drivers/i2c/busses/i2c-designware-platdrv.c | 25 ++++++++++--------------- 1 file changed, 10 insertions(+), 15 deletions(-) diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c index 2b7fa75..b6e05ba 100644 --- a/drivers/i2c/busses/i2c-designware-platdrv.c +++ b/drivers/i2c/busses/i2c-designware-platdrv.c @@ -331,6 +331,8 @@ static int dw_i2c_plat_probe(struct platform_device *pdev) if (r) goto exit_probe; + acpi_dev_disable_direct_complete(&pdev->dev); + return r; exit_probe: @@ -346,6 +348,7 @@ static int dw_i2c_plat_remove(struct platform_device *pdev) { struct dw_i2c_dev *dev = platform_get_drvdata(pdev); + acpi_dev_enable_direct_complete(&pdev->dev); pm_runtime_get_sync(&pdev->dev); i2c_del_adapter(&dev->adapter); @@ -372,18 +375,8 @@ static const struct of_device_id dw_i2c_of_match[] = { MODULE_DEVICE_TABLE(of, dw_i2c_of_match); #endif -#ifdef CONFIG_PM_SLEEP -static int dw_i2c_plat_prepare(struct device *dev) -{ - pm_runtime_resume(dev); - return 0; -} -#else -#define dw_i2c_plat_prepare NULL -#endif - #ifdef CONFIG_PM -static int dw_i2c_plat_suspend(struct device *dev) +static int dw_i2c_plat_runtime_suspend(struct device *dev) { struct platform_device *pdev = to_platform_device(dev); struct dw_i2c_dev *i_dev = platform_get_drvdata(pdev); @@ -394,7 +387,7 @@ static int dw_i2c_plat_suspend(struct device *dev) return 0; } -static int dw_i2c_plat_resume(struct device *dev) +static int dw_i2c_plat_runtime_resume(struct device *dev) { struct platform_device *pdev = to_platform_device(dev); struct dw_i2c_dev *i_dev = platform_get_drvdata(pdev); @@ -406,9 +399,11 @@ static int dw_i2c_plat_resume(struct device *dev) } static const struct dev_pm_ops dw_i2c_dev_pm_ops = { - .prepare = dw_i2c_plat_prepare, - SET_SYSTEM_SLEEP_PM_OPS(dw_i2c_plat_suspend, dw_i2c_plat_resume) - SET_RUNTIME_PM_OPS(dw_i2c_plat_suspend, dw_i2c_plat_resume, NULL) + SET_LATE_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, + pm_runtime_force_resume) + SET_RUNTIME_PM_OPS(dw_i2c_plat_runtime_suspend, + dw_i2c_plat_runtime_resume, + NULL) }; #define DW_I2C_DEV_PMOPS (&dw_i2c_dev_pm_ops)