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: 9802765 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 82B2C6089D 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 717A428653 for ; Wed, 21 Jun 2017 19:22:30 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6F550285E8; 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=-7.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, RCVD_IN_DNSWL_HI autolearn=unavailable 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 CA37D286BC for ; Wed, 21 Jun 2017 19:22:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751689AbdFUTWI (ORCPT ); Wed, 21 Jun 2017 15:22:08 -0400 Received: from mail-lf0-f43.google.com ([209.85.215.43]:33447 "EHLO mail-lf0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752042AbdFUTWB (ORCPT ); Wed, 21 Jun 2017 15:22:01 -0400 Received: by mail-lf0-f43.google.com with SMTP id m77so102862748lfe.0 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=E1sCUYe8tOpZ/QDKWJ8zSP8k9j5hzsccRf1IyVfqRMhDhaYwJpJJG3pg5lcEuEA3ZK fD/dzIkdpKVeTKFJxbmGymj+etxI+eZ8VF82C5S8zfG8xMW7uocl5hWvzauOZD95C+Xp w78JNJK80fkgFJ8XdJoLM43Dr1FFtUBKVeSDfRaVJLsLSv51GYdngvfSgYLkIEVg4tnY H76RzWOuGnAPPyPW9G9vcoyv5jygQeNPavqeSPpYqX4xB/NaBaz2qymZ6OIYGAjieHCN NdlHLJOf3bQfJy9vlJA2PHESe2SvKESj+4yaGdJlLpzXbXaKgcts0PQ8YY942zOvo4PK Rizw== X-Gm-Message-State: AKS2vOwYYlzPvcHnia5oXt0i+pqq3cqoYyc2tEhbFVZwo05yD7lshhJ3 Z7jTfXaRjPYH37cx 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-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@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)