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: 779073 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3wtF354Fttz9s7M for ; Thu, 22 Jun 2017 05:22:25 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="SPk29lmN"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751133AbdFUTWW (ORCPT ); Wed, 21 Jun 2017 15:22:22 -0400 Received: from mail-lf0-f45.google.com ([209.85.215.45]:33447 "EHLO mail-lf0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752040AbdFUTWB (ORCPT ); Wed, 21 Jun 2017 15:22:01 -0400 Received: by mail-lf0-f45.google.com with SMTP id m77so102862750lfe.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=GlzaAobXWvbKcwLXWcX1Did4t857bAL7wwEZk8jICMRQYQAYW8DzFVFE+UpAXL1ArE a7OXQ45YWDiqPRgx84NArjqynzTyvRctVUCaf09M2+hx+XmJnnwGoJHl1tv8qrAgS/pU U1qdVAFeGVhSUGDeX3kRXqTKt7WDhn5xLKtgSNk4cQkNfBps55QvrTnBH7GjIaMMrU+N RRwY0hGCiPQTK8FDErMygX/NNctVDV797S4w+KXMkyV78am+c7zRNFYYmYw+za3Quphk AfjEkrekUlKTm/vZuOWz/MxvsidB1W3GAdEhRZL8ZzEMw6DnJLq+UiXbWVx6rkelhhTV J6UQ== X-Gm-Message-State: AKS2vOyTRR6FIb/XkzBo5CBabffdCY1VXCj5qUV+guUw3w5uDDZ4ubPQ +rsjKk5BJE4F4ZA3 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-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org 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)