From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2CA18C433C1 for ; Thu, 25 Mar 2021 08:00:56 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 14BAE619FB for ; Thu, 25 Mar 2021 08:00:55 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 14BAE619FB Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=samsung.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:References:In-Reply-To:MIME-Version:Date:Message-ID: From:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=DXP0g4v802blPDMefnefmhizVDbvFca/YN7KEm8pJLs=; b=geQtQvNSBM13Xvz13/J1f41RI gKS5SfVypTxvqPzEnLhKRf1wvbpBD2Rz1JtAFQ6O2cr9mBFahVaNHRH3oe45B/L5Oddu5POsF+crr th0HvYYShHRqyUMubTaLUqRH4tMqLPVlM/Qq6/E7K8DDFQj8bxM+T01umi8GWRES2CYrogJVGSebW cyySM+1Fj8Dd0z/w//yefSxNF51Q4QpIC3ryplQ0JSSY/o1ZQiqamXAdP6BCaYh/RAde4X9x7XOnX wk3Pv+ybwscwa784d7RdhVui+s0F/3aIpu262/nqvubaWJznCZxKZXa+/w1zzFbtCpqtqA8xrhwti Bup334CNw==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lPKs2-000r9Y-L9; Thu, 25 Mar 2021 07:57:35 +0000 Received: from mailout1.samsung.com ([203.254.224.24]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lPKoy-000q2i-El for linux-arm-kernel@lists.infradead.org; Thu, 25 Mar 2021 07:54:31 +0000 Received: from epcas1p3.samsung.com (unknown [182.195.41.47]) by mailout1.samsung.com (KnoxPortal) with ESMTP id 20210325075420epoutp01d18e81da047159ac5f7637efd8eb2f5f~vhqr69KRN1597715977epoutp01d for ; Thu, 25 Mar 2021 07:54:20 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout1.samsung.com 20210325075420epoutp01d18e81da047159ac5f7637efd8eb2f5f~vhqr69KRN1597715977epoutp01d DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1616658860; bh=EGyBSH8NZBTYbbk/1sMrOpaP/1IMDBXkBGpRs/amQ1Y=; h=Subject:To:Cc:From:Date:In-Reply-To:References:From; b=UalK5CGc6PQqIwhI57c6HMOz4wsPA3K/1y7JRdSvJTw53kfzEh1/FiXgBLsJxHxyV OMsWrm10mxf9Zm8ErJVkG6yVl9L4xikC7KwkvH+PDaESF9l0Na9/8h8bC5w4dWD9VF Ui+xYm+2gDr7sgwgC6PpqhUxPyJsJ3ICL/6fRTCE= Received: from epsnrtp2.localdomain (unknown [182.195.42.163]) by epcas1p2.samsung.com (KnoxPortal) with ESMTP id 20210325075419epcas1p27c0af01ce1945caa0447bfa300ae55a1~vhqrIiu1w1592515925epcas1p2M; Thu, 25 Mar 2021 07:54:19 +0000 (GMT) Received: from epsmges1p2.samsung.com (unknown [182.195.40.152]) by epsnrtp2.localdomain (Postfix) with ESMTP id 4F5cnM2Y6xz4x9QC; Thu, 25 Mar 2021 07:54:15 +0000 (GMT) Received: from epcas1p4.samsung.com ( [182.195.41.48]) by epsmges1p2.samsung.com (Symantec Messaging Gateway) with SMTP id 34.74.02277.7A14C506; Thu, 25 Mar 2021 16:54:15 +0900 (KST) Received: from epsmtrp1.samsung.com (unknown [182.195.40.13]) by epcas1p1.samsung.com (KnoxPortal) with ESMTPA id 20210325075414epcas1p13b6c9d842bf53d230c010ca1499d6fd5~vhqmwIO_O1796117961epcas1p1n; Thu, 25 Mar 2021 07:54:14 +0000 (GMT) Received: from epsmgms1p1new.samsung.com (unknown [182.195.42.41]) by epsmtrp1.samsung.com (KnoxPortal) with ESMTP id 20210325075414epsmtrp16860517764eed4e4eea85aa4afaf019d~vhqmvCBlV2520325203epsmtrp1D; Thu, 25 Mar 2021 07:54:14 +0000 (GMT) X-AuditID: b6c32a36-4d7ff700000108e5-73-605c41a73ba1 Received: from epsmtip2.samsung.com ( [182.195.34.31]) by epsmgms1p1new.samsung.com (Symantec Messaging Gateway) with SMTP id F5.10.33967.6A14C506; Thu, 25 Mar 2021 16:54:14 +0900 (KST) Received: from [10.113.221.102] (unknown [10.113.221.102]) by epsmtip2.samsung.com (KnoxPortal) with ESMTPA id 20210325075414epsmtip239f47d0ca454637badb10dda42ce692c~vhqmZ9BCn2664326643epsmtip2d; Thu, 25 Mar 2021 07:54:14 +0000 (GMT) Subject: Re: [PATCH V8 7/8] devfreq: mediatek: cci devfreq register opp notification for SVS support To: "Andrew-sh.Cheng" , MyungJoo Ham , Kyungmin Park , Rob Herring , Mark Rutland , Matthias Brugger , "Rafael J. Wysocki" , Viresh Kumar , Nishanth Menon , Stephen Boyd , Liam Girdwood , Mark Brown Cc: linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, srv_heupstream@mediatek.com From: Chanwoo Choi Organization: Samsung Electronics Message-ID: <6f1f7bf5-54b0-d3e2-e93c-fa4a837c9222@samsung.com> Date: Thu, 25 Mar 2021 17:11:14 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:59.0) Gecko/20100101 Thunderbird/59.0 MIME-Version: 1.0 In-Reply-To: <1616499241-4906-8-git-send-email-andrew-sh.cheng@mediatek.com> Content-Language: en-US X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrOJsWRmVeSWpSXmKPExsWy7bCmge5yx5gEg5WHBCy2r3/BajH14RM2 i/lHzrFanG16w27x7UoHk8Wmx9dYLS7vmsNmcbn5IqPF594jjBZLr19ksmhqMba43biCzeLN j7NMFmdOX2K1aN17hN3i37WNLBbT7wpZbPzq4SDksWbeGkaPnbPusntsWtXJ5nHn2h42j81L 6j1aTu5n8dhytZ3Fo2/LKkaP4ze2M3l83iQXwBWVbZORmpiSWqSQmpecn5KZl26r5B0c7xxv amZgqGtoaWGupJCXmJtqq+TiE6DrlpkD9JKSQlliTilQKCCxuFhJ386mKL+0JFUhI7+4xFYp tSAlp8CyQK84Mbe4NC9dLzk/18rQwMDIFKgwITuj/8srtoKDYhUnZjcxNjD+Fuxi5OSQEDCR 2DdlO1MXIxeHkMAORomX3ROgnE+MEn9Xn2GHcL4xSjTP3cIK07Jy8Xmoqr2MEp8mPWGGcN4z Stx63MgGUiUskCbx4SNIBxeHiMAtZol9B+aDVTELHGSUmH7pIhNIFZuAlsT+FzfAOvgFFCWu /njMCGLzCthJ/FvxDayGRUBV4mXfMmYQW1QgTOLkthaoGkGJkzOfsIDYnAI+Ev0/zoHZzALi EreezGeCsOUltr+dA7ZYQqCfU2Ltu+mMEE+4SDR/u8oOYQtLvDq+BcqWknjZ3wZlV0usPHmE DaK5g1Fiy/4L0BAwlti/dDLQBg6gDZoS63fpQ4QVJXb+nssIsZhP4t3XHlaQEgkBXomONiGI EmWJyw/uMkHYkhKL2zvZJjAqzULyziwkL8xC8sIshGULGFlWMYqlFhTnpqcWGxYYIUf4JkZw itcy28E46e0HvUOMTByMhxglOJiVRHhbwiMShHhTEiurUovy44tKc1KLDzGaAgN4IrOUaHI+ MMvklcQbmhoZGxtbmBiamRoaKonzJho8iBcSSE8sSc1OTS1ILYLpY+LglGpgst0b+5Fjnsbz rSx2nIbbdloLrz74TcDv/dEeo8Q9WxVabzWt53f0vrmtekHfY+29AvtL2Jrff+w3EDZ7mOh/ qrvZ6PaktV/8BWM/LTsqOy/zqI7tCbcbyluMWjd8SOKbprbqpO/5X1+rRXl97ljWtNZddI18 ueE+S9I6Zan4F2o1r+T3sK8UZv+1TTDl3QS23hsHZh+ztj58N+H/Os7l8tHzdj0IU/oyuSTj hsYxkQsHPsxrfJOhF3w3ZeZZl7VbFzFNviCyv9359qXj6w+WxMs6fjR+8ku8PetU5PsZhclc kUW9/0sOuhhu3axvNVvpeHhymx93t4s98824Uw+aWjY9yXp0nO9x/lRetVUBj5VYijMSDbWY i4oTAdXw3Lx6BAAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrIIsWRmVeSWpSXmKPExsWy7bCSvO4yx5gEg8U75C22r3/BajH14RM2 i/lHzrFanG16w27x7UoHk8Wmx9dYLS7vmsNmcbn5IqPF594jjBZLr19ksmhqMba43biCzeLN j7NMFmdOX2K1aN17hN3i37WNLBbT7wpZbPzq4SDksWbeGkaPnbPusntsWtXJ5nHn2h42j81L 6j1aTu5n8dhytZ3Fo2/LKkaP4ze2M3l83iQXwBXFZZOSmpNZllqkb5fAldH/5RVbwUGxihOz mxgbGH8LdjFyckgImEisXHyeqYuRi0NIYDejxOb288wQCUmJaRePAtkcQLawxOHDxRA1bxkl 3vU1soLUCAukSXz4uIUVJCEicI9Z4t2U38wgDrPAQUaJZ91/2CFaHjJKfFu6mQWkhU1AS2L/ ixtsIDa/gKLE1R+PGUFsXgE7iX8rvjGB2CwCqhIv+5aBnSEqECaxc8ljJogaQYmTM5+AzeEU 8JHo/3EOzGYWUJf4M+8SM4QtLnHryXwmCFteYvvbOcwTGIVnIWmfhaRlFpKWWUhaFjCyrGKU TC0ozk3PLTYsMMxLLdcrTswtLs1L10vOz93ECI51Lc0djNtXfdA7xMjEwXiIUYKDWUmEtyU8 IkGINyWxsiq1KD++qDQntfgQozQHi5I474Wuk/FCAumJJanZqakFqUUwWSYOTqkGpsKyjPri VgPBO++WBkat2WrzjdHkl2XaxDqvaSEyZ3RXlgVqlTx7tMljo3PMf90FYZPadk5siPmb6Te5 w0KEJWt66Pkoye0nU1tMO319vzBIHQ9z602s0hFNZo++b5PqJfPgr5/saeYDWnXPP1Ve51u4 MnNBgt/JxPmuM8xrPsStLT8duOPoq4wD8fYHLy+v3/L3m8/ElTO73yvaTrr7/9D8KdP2XfuT 4udqs55XJvWM/LQ7h2vDOm9ect3YpaRY8ztIb9eGhk3qXhOXMLUZ//dsFbT0NQhebfzw9umV bYUzrsyUVHq6tsDxrdj5wP5UVVHmeP5nN+uyn2qoyB4Nnnxllv75M68kAgXktGb3KrEUZyQa ajEXFScCANIkt+9kAwAA X-CMS-MailID: 20210325075414epcas1p13b6c9d842bf53d230c010ca1499d6fd5 X-Msg-Generator: CA X-Sendblock-Type: SVC_REQ_APPROVE CMS-TYPE: 101P DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20210323113413epcas1p1d3acc9ac2539da96b0757a0159bdcfc7 References: <1616499241-4906-1-git-send-email-andrew-sh.cheng@mediatek.com> <1616499241-4906-8-git-send-email-andrew-sh.cheng@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210325_075425_553848_BC40DC78 X-CRM114-Status: GOOD ( 35.88 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, I think that you can squash this patch to patch4. On 3/23/21 8:34 PM, Andrew-sh.Cheng wrote: > From: "Andrew-sh.Cheng" > > SVS will change the voltage of opp item. What it the full name of SVS? > CCI devfreq need to react to change frequency. > > Signed-off-by: Andrew-sh.Cheng > --- > drivers/devfreq/mt8183-cci-devfreq.c | 27 +++++++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > > diff --git a/drivers/devfreq/mt8183-cci-devfreq.c b/drivers/devfreq/mt8183-cci-devfreq.c > index 018543db7bae..6942a48f3f4f 100644 > --- a/drivers/devfreq/mt8183-cci-devfreq.c > +++ b/drivers/devfreq/mt8183-cci-devfreq.c > @@ -21,6 +21,7 @@ struct cci_devfreq { > struct clk *cci_clk; > int old_vproc; > unsigned long old_freq; > + struct notifier_block opp_nb; > }; > > static int mtk_cci_set_voltage(struct cci_devfreq *cci_df, int vproc) > @@ -89,6 +90,26 @@ static int mtk_cci_devfreq_target(struct device *dev, unsigned long *freq, > return 0; > } > > +static int ccidevfreq_opp_notifier(struct notifier_block *nb, I think that you better to change the function name as following: ccidevfreq_opp_notifier -> mtk_cci_devfreq_opp_notifier > + unsigned long event, void *data) > +{ > + struct dev_pm_opp *opp = data; > + struct cci_devfreq *cci_df = container_of(nb, struct cci_devfreq, > + opp_nb); > + unsigned long freq, volt; > + > + if (event == OPP_EVENT_ADJUST_VOLTAGE) { > + freq = dev_pm_opp_get_freq(opp); > + /* current opp item is changed */ > + if (freq == cci_df->old_freq) { > + volt = dev_pm_opp_get_voltage(opp); > + mtk_cci_set_voltage(cci_df, volt); > + } > + } > + > + return 0; > +} > + > static struct devfreq_dev_profile cci_devfreq_profile = { > .target = mtk_cci_devfreq_target, > }; > @@ -98,12 +119,15 @@ static int mtk_cci_devfreq_probe(struct platform_device *pdev) > struct device *cci_dev = &pdev->dev; > struct cci_devfreq *cci_df; > struct devfreq_passive_data *passive_data; > + struct notifier_block *opp_nb; > int ret; > > cci_df = devm_kzalloc(cci_dev, sizeof(*cci_df), GFP_KERNEL); > if (!cci_df) > return -ENOMEM; > > + opp_nb = &cci_df->opp_nb; Just move this code at the neighborhood of 'opp_nb->notifier_call' init code. > + > cci_df->cci_clk = devm_clk_get(cci_dev, "cci_clock"); > ret = PTR_ERR_OR_ZERO(cci_df->cci_clk); > if (ret) { > @@ -152,6 +176,9 @@ static int mtk_cci_devfreq_probe(struct platform_device *pdev) > goto err_opp; > } > > + opp_nb->notifier_call = ccidevfreq_opp_notifier; > + dev_pm_opp_register_notifier(cci_dev, opp_nb); Need to check whether return value is valid or not. > + > return 0; > > err_opp: > -- Best Regards, Chanwoo Choi Samsung Electronics _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel