[{"id":1766239,"web_url":"http://patchwork.ozlabs.org/comment/1766239/","msgid":"<57153767-6dee-a5b5-548c-f7b0dc104b46@ti.com>","list_archive_url":null,"date":"2017-09-11T11:29:10","subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","submitter":{"id":3234,"url":"http://patchwork.ozlabs.org/api/people/3234/","name":"Sekhar Nori","email":"nsekhar@ti.com"},"content":"On Friday 08 September 2017 11:24 PM, Franklin S Cooper Jr wrote:\n> 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain\n> like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime\n\nunlike ?\n\n> is required to insure the power domain used by the specific I2C instance is\n> properly turned on along with its functional clock.\n> \n> Signed-off-by: Franklin S Cooper Jr \n> ---\n> Version 3 changes:\n> Remove several statements that set clk to NULL\n> Fix error path\n> \n> drivers/i2c/busses/i2c-davinci.c | 64 +++++++++++++++++++++++++++++++++-------\n> 1 file changed, 53 insertions(+), 11 deletions(-)\n\n> @@ -802,12 +821,22 @@ static int davinci_i2c_probe(struct platform_device *pdev)\n> \tdev->clk = devm_clk_get(&pdev->dev, NULL);\n> \tif (IS_ERR(dev->clk))\n> \t\treturn PTR_ERR(dev->clk);\n> -\tclk_prepare_enable(dev->clk);\n> \n> \tmem = platform_get_resource(pdev, IORESOURCE_MEM, 0);\n> \tdev->base = devm_ioremap_resource(&pdev->dev, mem);\n> \tif (IS_ERR(dev->base)) {\n> -\t\tr = PTR_ERR(dev->base);\n> +\t\treturn PTR_ERR(dev->base);\n> +\t}\n> +\n> +\tpm_runtime_set_autosuspend_delay(dev->dev,\n> +\t\t\t\t\t DAVINCI_I2C_PM_TIMEOUT);\n> +\tpm_runtime_use_autosuspend(dev->dev);\n> +\n> +\tpm_runtime_enable(dev->dev);\n> +\n> +\tr = pm_runtime_get_sync(dev->dev);\n> +\tif (r < 0) {\n> +\t\tdev_err(dev->dev, \"failed to runtime_get device: %d\\n\", r);\n> \t\tgoto err_unuse_clocks;\n\nYou end up doing a pm_runtime_put_sync() on failure here, instead of\npm_runtime_put_noidle() like rest of the patch.\n\nMay be handle this failure here instead of relying on the goto path.\n\n> \t}\n> \n> @@ -849,27 +878,40 @@ static int davinci_i2c_probe(struct platform_device *pdev)\n> \tif (r)\n> \t\tgoto err_unuse_clocks;\n> \n> +\tpm_runtime_mark_last_busy(dev->dev);\n> +\tpm_runtime_put_autosuspend(dev->dev);\n> +\n> \treturn 0;\n> \n> err_unuse_clocks:\n> -\tclk_disable_unprepare(dev->clk);\n> -\tdev->clk = NULL;\n> +\tpm_runtime_dont_use_autosuspend(dev->dev);\n> +\tpm_runtime_put_sync(dev->dev);\n> +\tpm_runtime_disable(dev->dev);\n> +\n> \treturn r;\n> }\n\nThanks,\nSekhar","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"STzZz2Mr\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ti.com header.i=@ti.com header.b=\"eS836+iu\"; \n\tdkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xrQh25SN1z9sNV\n\tfor ;\n\tMon, 11 Sep 2017 21:29:54 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drMuV-0001eo-8E; Mon, 11 Sep 2017 11:29:51 +0000","from lelnx194.ext.ti.com ([198.47.27.80])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drMuQ-0001bw-Pa for linux-arm-kernel@lists.infradead.org;\n\tMon, 11 Sep 2017 11:29:48 +0000","from dflxv15.itg.ti.com ([128.247.5.124])\n\tby lelnx194.ext.ti.com (8.15.1/8.15.1) with ESMTP id v8BBTExK017552; \n\tMon, 11 Sep 2017 06:29:14 -0500","from DFLE105.ent.ti.com (dfle105.ent.ti.com [10.64.6.26])\n\tby dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8BBTEFc004393;\n\tMon, 11 Sep 2017 06:29:14 -0500","from DFLE115.ent.ti.com (10.64.6.36) by DFLE105.ent.ti.com\n\t(10.64.6.26) with Microsoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.845.34;\n\tMon, 11 Sep 2017 06:29:14 -0500","from dflp33.itg.ti.com (10.64.6.16) by DFLE115.ent.ti.com\n\t(10.64.6.36) with Microsoft SMTP Server (version=TLS1_0,\n\tcipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.845.34 via Frontend\n\tTransport; Mon, 11 Sep 2017 06:29:14 -0500","from [172.24.190.171] (ileax41-snat.itg.ti.com [10.172.224.153])\n\tby dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8BBTBVf010450;\n\tMon, 11 Sep 2017 06:29:11 -0500"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Cc:Content-ID:\n\tContent-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc\n\t:Resent-Message-ID:List-Owner;\n\tbh=kiRTcfngG4bv8IW5tEOy709ywBr+ywgUrzBf2iPa9+k=;\n\tb=STzZz2MrNvXVpgsNnaV+tp9T4U\n\tBh59+8SoY12VErVkwyRyUwYX3nApCwEstUv5D7IIcXRPuIr84eMaLcmVDYloFNgwfc7Y+VIicLCPD\n\ttsM6A5Ad2ipe5PkYt3DQeOerUNMYhcq+wINIP4DKZvbIMvW50adPj8FcDqw7F54Y+vPffxUT/nrX9\n\tx9feCGFAmqY8OBOZIDw7yeY46J6ZtSytfwfdOAdsqgKddslTX9CJpWIfGpbgcem8afgGTc7XjzPja\n\teJ9Jz8NRMjaOYVY5zVmF+2PlLor53QHuIOo6ZRv0UXG+k3dQzSWTo4HsmkJmY3iXi2HVzp6wE4yQ7\n\tunn6NYqg==;","v=1; a=rsa-sha256; c=relaxed/simple; d=ti.com;\n\ts=ti-com-17Q1; t=1505129354;\n\tbh=4ppH4/R7zSBgKJljFL5NZaUMXuTTJtGxx6FM+vOAJpE=;\n\th=Subject:To:References:From:Date:In-Reply-To;\n\tb=eS836+iuBN3bx5N9h0vppEXIvTN+lcvl+PUn/AAy7QoNpNn/GiJA78h9NcXs/8tdA\n\taOmAnYu6yCqPLAjtNWXiqeGiGohu5vPGbX08ZztvHQpxFrzTZS+MfjN0It9FrNAdwi\n\tNpFiUXVClgzjh0+2SVLLy3MTh9lVNL0IPUG2Vuwk="],"Subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","To":"Franklin S Cooper Jr , ,\n\t, ,\n\t, ,\n\t, ,\n\t, ","References":"<20170908175448.3367-1-fcooper@ti.com>\n\t<20170908175448.3367-2-fcooper@ti.com>","From":"Sekhar Nori ","Message-ID":"<57153767-6dee-a5b5-548c-f7b0dc104b46@ti.com>","Date":"Mon, 11 Sep 2017 16:59:10 +0530","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170908175448.3367-2-fcooper@ti.com>","Content-Language":"en-US","X-EXCLAIMER-MD-CONFIG":"e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170911_042946_981853_0207DC50 ","X-CRM114-Status":"GOOD ( 12.70 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [198.47.27.80 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1766497,"web_url":"http://patchwork.ozlabs.org/comment/1766497/","msgid":"","list_archive_url":null,"date":"2017-09-11T20:07:15","subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","submitter":{"id":64326,"url":"http://patchwork.ozlabs.org/api/people/64326/","name":"Franklin S Cooper Jr","email":"fcooper@ti.com"},"content":"On 09/11/2017 06:29 AM, Sekhar Nori wrote:\n> On Friday 08 September 2017 11:24 PM, Franklin S Cooper Jr wrote:\n>> 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain\n>> like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime\n> \n> unlike ?\n> \n>> is required to insure the power domain used by the specific I2C instance is\n>> properly turned on along with its functional clock.\n>>\n>> Signed-off-by: Franklin S Cooper Jr \n>> ---\n>> Version 3 changes:\n>> Remove several statements that set clk to NULL\n>> Fix error path\n>>\n>> drivers/i2c/busses/i2c-davinci.c | 64 +++++++++++++++++++++++++++++++++-------\n>> 1 file changed, 53 insertions(+), 11 deletions(-)\n> \n>> @@ -802,12 +821,22 @@ static int davinci_i2c_probe(struct platform_device *pdev)\n>> \tdev->clk = devm_clk_get(&pdev->dev, NULL);\n>> \tif (IS_ERR(dev->clk))\n>> \t\treturn PTR_ERR(dev->clk);\n>> -\tclk_prepare_enable(dev->clk);\n>> \n>> \tmem = platform_get_resource(pdev, IORESOURCE_MEM, 0);\n>> \tdev->base = devm_ioremap_resource(&pdev->dev, mem);\n>> \tif (IS_ERR(dev->base)) {\n>> -\t\tr = PTR_ERR(dev->base);\n>> +\t\treturn PTR_ERR(dev->base);\n>> +\t}\n>> +\n>> +\tpm_runtime_set_autosuspend_delay(dev->dev,\n>> +\t\t\t\t\t DAVINCI_I2C_PM_TIMEOUT);\n>> +\tpm_runtime_use_autosuspend(dev->dev);\n>> +\n>> +\tpm_runtime_enable(dev->dev);\n>> +\n>> +\tr = pm_runtime_get_sync(dev->dev);\n>> +\tif (r < 0) {\n>> +\t\tdev_err(dev->dev, \"failed to runtime_get device: %d\\n\", r);\n>> \t\tgoto err_unuse_clocks;\n> \n> You end up doing a pm_runtime_put_sync() on failure here, instead of\n> pm_runtime_put_noidle() like rest of the patch.\n> \n> May be handle this failure here instead of relying on the goto path.\n\nOk\n> \n>> \t}\n>> \n>> @@ -849,27 +878,40 @@ static int davinci_i2c_probe(struct platform_device *pdev)\n>> \tif (r)\n>> \t\tgoto err_unuse_clocks;\n>> \n>> +\tpm_runtime_mark_last_busy(dev->dev);\n>> +\tpm_runtime_put_autosuspend(dev->dev);\n>> +\n>> \treturn 0;\n>> \n>> err_unuse_clocks:\n>> -\tclk_disable_unprepare(dev->clk);\n>> -\tdev->clk = NULL;\n>> +\tpm_runtime_dont_use_autosuspend(dev->dev);\n>> +\tpm_runtime_put_sync(dev->dev);\n>> +\tpm_runtime_disable(dev->dev);\n>> +\n>> \treturn r;\n>> }\n> \n> Thanks,\n> Sekhar\n>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"b+fNr5s2\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ti.com header.i=@ti.com header.b=\"uGIZ0sTP\"; \n\tdkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xrf9l2yh7z9sBZ\n\tfor ;\n\tTue, 12 Sep 2017 06:07:55 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drUzn-0000Jb-VG; Mon, 11 Sep 2017 20:07:51 +0000","from lelnx193.ext.ti.com ([198.47.27.77])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drUzj-0000HZ-OL for linux-arm-kernel@lists.infradead.org;\n\tMon, 11 Sep 2017 20:07:49 +0000","from dlelxv90.itg.ti.com ([172.17.2.17])\n\tby lelnx193.ext.ti.com (8.15.1/8.15.1) with ESMTP id v8BK7FGj010969; \n\tMon, 11 Sep 2017 15:07:15 -0500","from DLEE70.ent.ti.com (dlee70.ent.ti.com [157.170.170.113])\n\tby dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8BK7FeI030318; \n\tMon, 11 Sep 2017 15:07:15 -0500","from [128.247.59.135] (128.247.59.135) by DLEE70.ent.ti.com\n\t(157.170.170.113) with Microsoft SMTP Server id 14.3.294.0;\n\tMon, 11 Sep 2017 15:07:15 -0500"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Cc:Content-ID:\n\tContent-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc\n\t:Resent-Message-ID:List-Owner;\n\tbh=lnIGeUzZ8RHhJUs/gi9p6Gy05R/HIDS27Dy7BA3IFw0=;\n\tb=b+fNr5s2R4fCxLBBLUPQf5dlc6\n\tZssm3m2au1OEtkhs1O0H0b6TBTw3nC+8y3I6ietmZHecLqQsqGOnJohFB76wSbs3aSC6AKAn4WtZE\n\tqWbaUpn5ygYWjZMz71Yzp51pedn5Ihy2uymmi9k1gMtRgyT5Tfnepe0o0ApPeoE8hz1g/uZO+zNwu\n\t/5Oa3x1AcR6RFVi0tpImK6ByudrCn0KGitqCLl7GrpxvoeWUUwWu9gbx+SnbooEkGgQl5DJiGJ2rU\n\tmsblBx6UVAwuZEyGxhcsY5RUAkAlN0EFHcNmKC4hxUYKPFv+6khw+z2ZqE+AkRFYuQlyvQx02o/aX\n\twvpMvWmg==;","v=1; a=rsa-sha256; c=relaxed/simple; d=ti.com;\n\ts=ti-com-17Q1; t=1505160435;\n\tbh=ZKXfmVZ90wY0PUWhN+jjUV/kdKXiCRrfunr1xXx7U8E=;\n\th=Subject:To:References:From:Date:In-Reply-To;\n\tb=uGIZ0sTPMZjZpLp+U00CurTtp4DUdTFnc0K9jMrZnj0JR03dZ5O5uy/RckgjqR+pK\n\t7zJnrIrVFlEzLC37W1XAQBuX0m4bY/BB0t8uS5B6tMtJxFS8WRwoS0OQOKVBHO3yht\n\tGXWoiQpbF1wsqsdbyHMrVXBE3b6xO87C3cuPq1g8="],"Subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","To":"Sekhar Nori , , , \n\t, ,\n\t, ,\n\t, ,\n\t","References":"<20170908175448.3367-1-fcooper@ti.com>\n\t<20170908175448.3367-2-fcooper@ti.com>\n\t<57153767-6dee-a5b5-548c-f7b0dc104b46@ti.com>","From":"Franklin S Cooper Jr ","Message-ID":"","Date":"Mon, 11 Sep 2017 15:07:15 -0500","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<57153767-6dee-a5b5-548c-f7b0dc104b46@ti.com>","Content-Language":"en-US","X-Originating-IP":"[128.247.59.135]","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170911_130747_945296_83D57369 ","X-CRM114-Status":"GOOD ( 12.58 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [198.47.27.77 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1766501,"web_url":"http://patchwork.ozlabs.org/comment/1766501/","msgid":"<94b7a185-4cde-03e6-cc17-f64d5d28c9f2@ti.com>","list_archive_url":null,"date":"2017-09-11T20:11:42","subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","submitter":{"id":25084,"url":"http://patchwork.ozlabs.org/api/people/25084/","name":"Grygorii Strashko","email":"grygorii.strashko@ti.com"},"content":"On 09/11/2017 03:07 PM, Franklin S Cooper Jr wrote:\n> \n> \n> On 09/11/2017 06:29 AM, Sekhar Nori wrote:\n>> On Friday 08 September 2017 11:24 PM, Franklin S Cooper Jr wrote:\n>>> 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain\n>>> like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime\n>>\n>> unlike ?\n>>\n>>> is required to insure the power domain used by the specific I2C instance is\n>>> properly turned on along with its functional clock.\n>>>\n>>> Signed-off-by: Franklin S Cooper Jr \n>>> ---\n>>> Version 3 changes:\n>>> Remove several statements that set clk to NULL\n>>> Fix error path\n>>>\n>>> drivers/i2c/busses/i2c-davinci.c | 64 +++++++++++++++++++++++++++++++++-------\n>>> 1 file changed, 53 insertions(+), 11 deletions(-)\n>>\n>>> @@ -802,12 +821,22 @@ static int davinci_i2c_probe(struct platform_device *pdev)\n>>> \tdev->clk = devm_clk_get(&pdev->dev, NULL);\n>>> \tif (IS_ERR(dev->clk))\n>>> \t\treturn PTR_ERR(dev->clk);\n>>> -\tclk_prepare_enable(dev->clk);\n>>> \n>>> \tmem = platform_get_resource(pdev, IORESOURCE_MEM, 0);\n>>> \tdev->base = devm_ioremap_resource(&pdev->dev, mem);\n>>> \tif (IS_ERR(dev->base)) {\n>>> -\t\tr = PTR_ERR(dev->base);\n>>> +\t\treturn PTR_ERR(dev->base);\n>>> +\t}\n>>> +\n>>> +\tpm_runtime_set_autosuspend_delay(dev->dev,\n>>> +\t\t\t\t\t DAVINCI_I2C_PM_TIMEOUT);\n>>> +\tpm_runtime_use_autosuspend(dev->dev);\n>>> +\n>>> +\tpm_runtime_enable(dev->dev);\n>>> +\n>>> +\tr = pm_runtime_get_sync(dev->dev);\n>>> +\tif (r < 0) {\n>>> +\t\tdev_err(dev->dev, \"failed to runtime_get device: %d\\n\", r);\n>>> \t\tgoto err_unuse_clocks;\n>>\n>> You end up doing a pm_runtime_put_sync() on failure here, instead of\n>> pm_runtime_put_noidle() like rest of the patch.\n>>\n>> May be handle this failure here instead of relying on the goto path.\n> \n> Ok\n\nI think, it's not necessary - pm_runtime_put_sync() can be used in this case\n (and used the same way in many other drivers).\nIn case, of failure in this place - pm_runtime_put_sync() will just decrement usage counter.\n\n\n>>\n>>> \t}\n>>> \n>>> @@ -849,27 +878,40 @@ static int davinci_i2c_probe(struct platform_device *pdev)\n>>> \tif (r)\n>>> \t\tgoto err_unuse_clocks;\n>>> \n>>> +\tpm_runtime_mark_last_busy(dev->dev);\n>>> +\tpm_runtime_put_autosuspend(dev->dev);\n>>> +\n>>> \treturn 0;\n>>> \n>>> err_unuse_clocks:\n>>> -\tclk_disable_unprepare(dev->clk);\n>>> -\tdev->clk = NULL;\n>>> +\tpm_runtime_dont_use_autosuspend(dev->dev);\n>>> +\tpm_runtime_put_sync(dev->dev);\n>>> +\tpm_runtime_disable(dev->dev);\n>>> +\n>>> \treturn r;\n>>> }\n>>\n>> Thanks,\n>> Sekhar\n>>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"UnSLv5hL\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ti.com header.i=@ti.com header.b=\"J8CMEPUD\"; \n\tdkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xrfHq28dcz9s8J\n\tfor ;\n\tTue, 12 Sep 2017 06:13:11 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drV4r-0002t9-1D; Mon, 11 Sep 2017 20:13:05 +0000","from fllnx209.ext.ti.com ([198.47.19.16])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drV4m-0002az-DC for linux-arm-kernel@lists.infradead.org;\n\tMon, 11 Sep 2017 20:13:04 +0000","from dflxv15.itg.ti.com ([128.247.5.124])\n\tby fllnx209.ext.ti.com (8.15.1/8.15.1) with ESMTP id v8BKBllI014465; \n\tMon, 11 Sep 2017 15:11:47 -0500","from DLEE70.ent.ti.com (dlee70.ent.ti.com [157.170.170.113])\n\tby dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8BKBgOe024355;\n\tMon, 11 Sep 2017 15:11:42 -0500","from [128.247.59.147] (128.247.59.147) by DLEE70.ent.ti.com\n\t(157.170.170.113) with Microsoft SMTP Server id 14.3.294.0;\n\tMon, 11 Sep 2017 15:11:42 -0500"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Cc:Content-ID:\n\tContent-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc\n\t:Resent-Message-ID:List-Owner;\n\tbh=G9bc0DylbfMBRnlBZr8A/uuYcon+SLypOKzUeFkUClI=;\n\tb=UnSLv5hLVL6cisdVF6j4fQZEW6\n\tWvQghyFYZ9uNypYlX4L6Yo1rdgiqyPa6yvP+LAi/kA6Sb/qGLlxNN3fisNTwlYRNRZgA6PjIwojBx\n\tcNAPG3ZmfCVCztTT7pIqV0EZA63iw6jZOtbD3sRP6t9SU2O8aBv23Abt2OAcLjpwzpZhqcKmBnzdY\n\tG7NCKGkbJuK4pIZt5srdziRAN9ve5zekIO8XFkOtkSMR+MvxZg4pYFHXaIyOvLPGbZboRTsElfYLu\n\t+sVXV0xnDwwHe3NCEbOGAxkVss6uPYKOp+NwNMSbg6TEZRMuIo7avgdFWEc3hmxy6SFGqa5nkHJNZ\n\tHGjDHxRQ==;","v=1; a=rsa-sha256; c=relaxed/simple; d=ti.com;\n\ts=ti-com-17Q1; t=1505160707;\n\tbh=PjmBID5mWy8b/C5Xa9VeeFRsDNkmtE/7XtbwQPlOE3U=;\n\th=Subject:To:References:From:Date:In-Reply-To;\n\tb=J8CMEPUDplzLK7IHR7oYaBErzVcFOSsDgfQg/64CRMEHmcbBaArAtutzvog0YADmc\n\tYhEN5BKeCqCTpKIzMNqUwmSVs+3TW2wo7Fbbow3gdFdyy+JnyZx+U/T27Bpz9A2/Tf\n\tFF1SMJnzmjslR2v/Fbnfnx0Sdlf9qF76/EUdYRIo="],"Subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","To":"Franklin S Cooper Jr , Sekhar Nori ,\n\t, , ,\n\t, ,\n\t, ,\n\t","References":"<20170908175448.3367-1-fcooper@ti.com>\n\t<20170908175448.3367-2-fcooper@ti.com>\n\t<57153767-6dee-a5b5-548c-f7b0dc104b46@ti.com>\n\t","From":"Grygorii Strashko ","Message-ID":"<94b7a185-4cde-03e6-cc17-f64d5d28c9f2@ti.com>","Date":"Mon, 11 Sep 2017 15:11:42 -0500","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","X-Originating-IP":"[128.247.59.147]","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170911_131300_646372_7796BBC7 ","X-CRM114-Status":"GOOD ( 12.11 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [198.47.19.16 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1766752,"web_url":"http://patchwork.ozlabs.org/comment/1766752/","msgid":"<5d8778d8-281c-8926-fec3-f9acf82e342c@ti.com>","list_archive_url":null,"date":"2017-09-12T08:25:52","subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","submitter":{"id":3234,"url":"http://patchwork.ozlabs.org/api/people/3234/","name":"Sekhar Nori","email":"nsekhar@ti.com"},"content":"On Tuesday 12 September 2017 01:41 AM, Grygorii Strashko wrote:\n> \n> \n> On 09/11/2017 03:07 PM, Franklin S Cooper Jr wrote:\n>>\n>>\n>> On 09/11/2017 06:29 AM, Sekhar Nori wrote:\n>>> On Friday 08 September 2017 11:24 PM, Franklin S Cooper Jr wrote:\n>>>> 66AK2G has I2C instances that are not apart of the ALWAYS_ON power domain\n>>>> like other Keystone 2 SoCs and OMAPL138. Therefore, pm_runtime\n>>>\n>>> unlike ?\n>>>\n>>>> is required to insure the power domain used by the specific I2C instance is\n>>>> properly turned on along with its functional clock.\n>>>>\n>>>> Signed-off-by: Franklin S Cooper Jr \n>>>> ---\n>>>> Version 3 changes:\n>>>> Remove several statements that set clk to NULL\n>>>> Fix error path\n>>>>\n>>>> drivers/i2c/busses/i2c-davinci.c | 64 +++++++++++++++++++++++++++++++++-------\n>>>> 1 file changed, 53 insertions(+), 11 deletions(-)\n>>>\n>>>> @@ -802,12 +821,22 @@ static int davinci_i2c_probe(struct platform_device *pdev)\n>>>> \tdev->clk = devm_clk_get(&pdev->dev, NULL);\n>>>> \tif (IS_ERR(dev->clk))\n>>>> \t\treturn PTR_ERR(dev->clk);\n>>>> -\tclk_prepare_enable(dev->clk);\n>>>> \n>>>> \tmem = platform_get_resource(pdev, IORESOURCE_MEM, 0);\n>>>> \tdev->base = devm_ioremap_resource(&pdev->dev, mem);\n>>>> \tif (IS_ERR(dev->base)) {\n>>>> -\t\tr = PTR_ERR(dev->base);\n>>>> +\t\treturn PTR_ERR(dev->base);\n>>>> +\t}\n>>>> +\n>>>> +\tpm_runtime_set_autosuspend_delay(dev->dev,\n>>>> +\t\t\t\t\t DAVINCI_I2C_PM_TIMEOUT);\n>>>> +\tpm_runtime_use_autosuspend(dev->dev);\n>>>> +\n>>>> +\tpm_runtime_enable(dev->dev);\n>>>> +\n>>>> +\tr = pm_runtime_get_sync(dev->dev);\n>>>> +\tif (r < 0) {\n>>>> +\t\tdev_err(dev->dev, \"failed to runtime_get device: %d\\n\", r);\n>>>> \t\tgoto err_unuse_clocks;\n>>>\n>>> You end up doing a pm_runtime_put_sync() on failure here, instead of\n>>> pm_runtime_put_noidle() like rest of the patch.\n>>>\n>>> May be handle this failure here instead of relying on the goto path.\n>>\n>> Ok\n> \n> I think, it's not necessary - pm_runtime_put_sync() can be used in this case\n> (and used the same way in many other drivers).\n> In case, of failure in this place - pm_runtime_put_sync() will just decrement usage counter.\n\nCan you please explain why this is the case? At least on DA850, I see\nthe runtime_idle callback invoked from rpm_idle() if\npm_runtime_put_sync() is used in the failure path.\n\nThanks\nSekhar","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"WJp0LNDP\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ti.com header.i=@ti.com header.b=\"DuPRMgqA\"; \n\tdkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xryZF1BNJz9s7g\n\tfor ;\n\tTue, 12 Sep 2017 18:26:42 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drgWk-0001lV-Ge; Tue, 12 Sep 2017 08:26:38 +0000","from lelnx194.ext.ti.com ([198.47.27.80])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1drgWg-0001gS-4i for linux-arm-kernel@lists.infradead.org;\n\tTue, 12 Sep 2017 08:26:36 +0000","from dflxv15.itg.ti.com ([128.247.5.124])\n\tby lelnx194.ext.ti.com (8.15.1/8.15.1) with ESMTP id v8C8PvV0026737; \n\tTue, 12 Sep 2017 03:25:57 -0500","from DFLE104.ent.ti.com (dfle104.ent.ti.com [10.64.6.25])\n\tby dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8C8Pvwj020963;\n\tTue, 12 Sep 2017 03:25:57 -0500","from DFLE100.ent.ti.com (10.64.6.21) by DFLE104.ent.ti.com\n\t(10.64.6.25) with Microsoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.845.34;\n\tTue, 12 Sep 2017 03:25:56 -0500","from dflp33.itg.ti.com (10.64.6.16) by DFLE100.ent.ti.com\n\t(10.64.6.21) with Microsoft SMTP Server (version=TLS1_0,\n\tcipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.845.34 via Frontend\n\tTransport; Tue, 12 Sep 2017 03:25:56 -0500","from [172.24.190.171] (ileax41-snat.itg.ti.com [10.172.224.153])\n\tby dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8C8PrFd031610;\n\tTue, 12 Sep 2017 03:25:54 -0500"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Cc:Content-ID:\n\tContent-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc\n\t:Resent-Message-ID:List-Owner;\n\tbh=4jH9ttdlKeS/GTZ1ChTiwtJCoU5ufsgS3p3f3AkEBT8=;\n\tb=WJp0LNDPXjzbI6XTYkMNM+hgbe\n\tpRF8BieRzVZGsSwPK4FewFHChew34RAJm27QUPrYu2HmjB4UP4HXOCNbI+EjTgaNLDPr6CNOcAjy+\n\th/k6pjHiqOPMQ+96XY8vwkvbzE+SvUOkpJZF0UPPWeC2kkF58hu4cCGDKOkWU7DbRmtDdEawXD3fV\n\tYhYRyADE66jChVgvDG0eQE71LNCanVNJZXPm0pPcAEhrbpE+M3P7rfb8mst3PaFql8++2DOxTce2Z\n\tFfGeN8NXlm/X8kOl7e7okBxrEa5EtXWlzlCz/gvsHDufzZA8wi2VwhQdUdGMaw2MiM0EICoW4lMV4\n\teQ+CDO2A==;","v=1; a=rsa-sha256; c=relaxed/simple; d=ti.com;\n\ts=ti-com-17Q1; t=1505204757;\n\tbh=DAfWoveJrxhgfnqNfVloGqFd9QtyfmKXgnOCTiXV5jg=;\n\th=Subject:To:References:From:Date:In-Reply-To;\n\tb=DuPRMgqAGlrz1NS2iqLV5IeWItEZIO9urQaaFBZups3wU85zpBca53kD93emEAr/i\n\tow6AtF9WkpBMMLlkuE/lm0CeRO2GCRLFzwlDhlzl6tmTXojFqgYNTnYTrJAeTewYKi\n\tUNh0kWK3AFG7Pg8r9+Cdh/HfLzJFTd/o4revZS34="],"Subject":"Re: [PATCH v3 1/2] i2c: davinci: Add PM Runtime Support","To":"Grygorii Strashko , Franklin S Cooper Jr\n\t, ,\n\t, ,\n\t, ,\n\t, ,\n\t","References":"<20170908175448.3367-1-fcooper@ti.com>\n\t<20170908175448.3367-2-fcooper@ti.com>\n\t<57153767-6dee-a5b5-548c-f7b0dc104b46@ti.com>\n\t\n\t<94b7a185-4cde-03e6-cc17-f64d5d28c9f2@ti.com>","From":"Sekhar Nori ","Message-ID":"<5d8778d8-281c-8926-fec3-f9acf82e342c@ti.com>","Date":"Tue, 12 Sep 2017 13:55:52 +0530","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<94b7a185-4cde-03e6-cc17-f64d5d28c9f2@ti.com>","Content-Language":"en-US","X-EXCLAIMER-MD-CONFIG":"e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170912_012634_342941_AFB4D1BB ","X-CRM114-Status":"GOOD ( 12.81 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [198.47.27.80 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]