From patchwork Wed Oct 11 03:09:23 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chanwoo Choi X-Patchwork-Id: 9998623 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 779D260230 for ; Wed, 11 Oct 2017 03:11:05 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 70AAD28860 for ; Wed, 11 Oct 2017 03:11:05 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 656EF28863; Wed, 11 Oct 2017 03:11:05 +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=-4.2 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 002CF28860 for ; Wed, 11 Oct 2017 03:11:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:References: In-reply-to:Message-id:Date:Subject:To:From:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=TRUg6Plwpsftp6SVprJjE/0+PkGh16ew6tEb6olxOSs=; b=CJSyeb4m5ZdTLtn49IJGvb4J7o vXN0maj7D1/z/pQGTsx2syt5jmYBrGm3ebWW2upZ789FRoXRpckl1TfoDRJe5UfWSIeziB9w5HT1E 1m8EfXvuRBJr8Z6Pw8uMNuAK0xtHbfAWjGdjd9yJSTkCJ0CQoomIKluNzrvvc/LPQCRKqH7Xj2Qeo 3FuWPrIpFHlwnbZh9juwqy4kT+4SPH2eVQsiyIMbLh8SR8xj9Fk3o4YIKBAW5YAwvRnrz6viFhSJK uNibMZkyr50+uvsR7wrzszmvQUozHtaDb43KKcP1On+h9myS6420JmULw33tIIp1M/vheSkvU29Vy PDsXG02g==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1e27Pr-0003OT-Hr; Wed, 11 Oct 2017 03:10:39 +0000 Received: from mailout2.samsung.com ([203.254.224.25]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1e27PR-0001gY-C5 for linux-arm-kernel@lists.infradead.org; Wed, 11 Oct 2017 03:10:16 +0000 Received: from epcas1p1.samsung.com (unknown [182.195.41.45]) by mailout2.samsung.com (KnoxPortal) with ESMTP id 20171011030948epoutp02d79c363403ad040494ba56875638acf6~sZVRY6cfC1079910799epoutp02z; Wed, 11 Oct 2017 03:09:48 +0000 (GMT) Received: from epsmges1p5.samsung.com (unknown [182.195.40.67]) by epcas1p2.samsung.com (KnoxPortal) with ESMTP id 20171011030947epcas1p201eee214cb29e56e0c4c9b688dac3621~sZVREcCKA3212232122epcas1p2a; Wed, 11 Oct 2017 03:09:47 +0000 (GMT) Received: from epcas1p3.samsung.com ( [182.195.41.47]) by epsmges1p5.samsung.com (Symantec Messaging Gateway) with SMTP id C4.91.04255.B7B8DD95; Wed, 11 Oct 2017 12:09:47 +0900 (KST) Received: from epsmgms2p1new.samsung.com (unknown [182.195.42.142]) by epcas1p3.samsung.com (KnoxPortal) with ESMTP id 20171011030947epcas1p3fead28d00a52590b344b728dfca59138~sZVQgkaqD2921829218epcas1p3r; Wed, 11 Oct 2017 03:09:47 +0000 (GMT) X-AuditID: b6c32a39-a57ff7000000109f-89-59dd8b7bc651 Received: from epmmp1.local.host ( [203.254.227.16]) by epsmgms2p1new.samsung.com (Symantec Messaging Gateway) with SMTP id 3E.CD.06995.B7B8DD95; Wed, 11 Oct 2017 12:09:47 +0900 (KST) Received: from localhost.localdomain ([10.113.62.212]) by mmp1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id <0OXN00GRY24AU990@mmp1.samsung.com>; Wed, 11 Oct 2017 12:09:46 +0900 (KST) From: Chanwoo Choi To: myungjoo.ham@samsung.com, kyungmin.park@samsung.com, cw00.choi@samsung.com Subject: [PATCH v3 7/8] PM / devfreq: Define the constant governor name Date: Wed, 11 Oct 2017 12:09:23 +0900 Message-id: <1507691364-3899-8-git-send-email-cw00.choi@samsung.com> X-Mailer: git-send-email 1.8.0 In-reply-to: <1507691364-3899-1-git-send-email-cw00.choi@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrAKsWRmVeSWpSXmKPExsWy7bCmvm51991Ig459uhYTb1xhsbj+5Tmr xaT7E1gs+h+/ZrY4f34Du8XZpjfsFpseX2O1uLxrDpvF594jjBYzzu9jsrjduILN4vGKt+wO PB6L97xk8ti0qpPNY/OSeo++LasYPT5vkgtgjUq1yUhNTEktUkjNS85PycxLt1XyDo53jjc1 MzDUNbS0MFdSyEvMTbVVcvEJ0HXLzAE6T0mhLDGnFCgUkFhcrKRvZ1OUX1qSqpCRX1xiqxRt aGikZ2hgrmdkZKRnYhxrZWQKVJKQmnF792/2gjm6FVuXf2duYDyk1sXIwSEhYCLx43l1FyMX h5DADkaJ6zsXsEA43xklbq5pB3I4wYq2Hn/HCJHYzShx7PEidgjnB6NET8tcdpAqNgEtif0v brCB2CICARKda1cxgRQxC0xlktg2dSYjSEJYwF1i66U2NpDdLAKqEm9OZ4GEeQVcJOY2TmKF 2CYn8WHPI7CZnAKuEpNnnWKHiG9gk9h4NBrCdpH49+QIG4QtLPHq+BZ2iHekJS4dtQVZKyHQ ziixec49Fging1Hi/spGqAXGEqe6GplAbGYBPol3X3tYIZp5JTrahCBKPCS2rb4CNd9R4uuN e0wQD89glOg4cp1tAqPUAkaGVYxiqQXFuempxYYFpnrFibnFpXnpesn5uZsYwalHy3IH47Fz PocYBTgYlXh4d1y5EynEmlhWXJl7iFGCg1lJhNei8G6kEG9KYmVValF+fFFpTmrxIUZTYNBM ZJYSTc4HpsW8knhDE0sDEzMjYFqwNDRUEucVXX8tQkggPbEkNTs1tSC1CKaPiYNTqoGx6uGH h2H3TnWn/7GaqBxZv9Rl66ZnQennnpUWLF70W0Lm6dJj9wy+TYh14PK9M+cvq/Fl8Yy9XtfT /aTEOqtzFzrlqweFRT0VDLv5TS041fOpSLhRUYznR7fekI3Hu78lBX8QLiiZd/7D+Z6oEvan ldsvTll+Xb4/6uihrdtOdecuyX+/+a6mEktxRqKhFnNRcSIA2jWIEVMDAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrCLMWRmVeSWpSXmKPExsVy+t9jAd3q7ruRBqvv8lpMvHGFxeL6l+es FpPuT2Cx6H/8mtni/PkN7BZnm96wW2x6fI3V4vKuOWwWn3uPMFrMOL+PyeJ24wo2i8cr3rI7 8Hgs3vOSyWPTqk42j81L6j36tqxi9Pi8SS6ANYrLJiU1J7MstUjfLoEr4/bu3+wFc3Qrti7/ ztzAeEiti5GTQ0LARGLr8XeMXYxcHEICOxkl5nzezgLh/GCU2PJ4OhNIFZuAlsT+FzfYQGwR AT+J7fPXg3UwC0xlkvjaeAKsSFjAXWLrpTagIg4OFgFViTens0DCvAIuEnMbJ7FCbJOT+LDn ETuIzSngKjF51ikwWwio5sfql4wTGHkWMDKsYpRMLSjOTc8tNiowzEst1ytOzC0uzUvXS87P 3cQIDLtth7X6djDeXxJ/iFGAg1GJh1fg+p1IIdbEsuLK3EOMEhzMSiK8FoV3I4V4UxIrq1KL 8uOLSnNSiw8xSnOwKInz3s47FikkkJ5YkpqdmlqQWgSTZeLglGpgtHFRO/9R6JR3A9c2V6cd HLvvHtM8eOWTjmCx4269JfvWV8aw8O68s8o2sSzQotDkHuvtBwpPAy9nW877EmKypO6lxD6H Ty179aUL5906F7taPubzpfV6y6o+NKjLdfvnrDMN2nNLk2Hr4qR50woknIymbtE6qdfYzDtl aaPWrrWbKo3Eyw0tlFiKMxINtZiLihMByTNjpjcCAAA= X-CMS-MailID: 20171011030947epcas1p3fead28d00a52590b344b728dfca59138 X-Msg-Generator: CA X-Sender-IP: 182.195.42.142 X-Local-Sender: =?UTF-8?B?7LWc7LCs7JqwG1RpemVuIFBsYXRmb3JtIExhYihTL1fshLw=?= =?UTF-8?B?7YSwKRvsgrzshLHsoITsnpAbU2VuaW9yIEVuZ2luZWVy?= X-Global-Sender: =?UTF-8?B?Q2hhbndvbyBDaG9pG1RpemVuIFBsYXRmb3JtIExhYi4bU2Ft?= =?UTF-8?B?c3VuZyBFbGVjdHJvbmljcxtTZW5pb3IgRW5naW5lZXI=?= X-Sender-Code: =?UTF-8?B?QzEwG1RFTEUbQzEwVjgxMTE=?= CMS-TYPE: 101P DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20171011030947epcas1p3fead28d00a52590b344b728dfca59138 X-RootMTR: 20171011030947epcas1p3fead28d00a52590b344b728dfca59138 References: <1507691364-3899-1-git-send-email-cw00.choi@samsung.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20171010_201013_779804_57438824 X-CRM114-Status: GOOD ( 14.76 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-samsung-soc@vger.kernel.org, linux-pm@vger.kernel.org, rafael.j.wysocki@intel.com, linux-kernel@vger.kernel.org, Krzysztof Kozlowski , inki.dae@samsung.com, Kukjin Kim , chanwoo@kernel.org, linux-arm-kernel@lists.infradead.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Prior to that, the devfreq device uses the governor name when adding the itself. In order to prevent the mistake used the wrong governor name, this patch defines the governor name as a constant and then uses them instead of using the string directly. Signed-off-by: Chanwoo Choi Cc: Kukjin Kim Cc: Krzysztof Kozlowski Cc: linux-samsung-soc@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org --- drivers/devfreq/exynos-bus.c | 5 +++-- drivers/devfreq/governor_passive.c | 2 +- drivers/devfreq/governor_performance.c | 2 +- drivers/devfreq/governor_powersave.c | 2 +- drivers/devfreq/governor_simpleondemand.c | 2 +- drivers/devfreq/governor_userspace.c | 2 +- drivers/devfreq/rk3399_dmc.c | 2 +- include/linux/devfreq.h | 7 +++++++ 8 files changed, 16 insertions(+), 8 deletions(-) diff --git a/drivers/devfreq/exynos-bus.c b/drivers/devfreq/exynos-bus.c index 49f68929e024..c25658b26598 100644 --- a/drivers/devfreq/exynos-bus.c +++ b/drivers/devfreq/exynos-bus.c @@ -436,7 +436,8 @@ static int exynos_bus_probe(struct platform_device *pdev) ondemand_data->downdifferential = 5; /* Add devfreq device to monitor and handle the exynos bus */ - bus->devfreq = devm_devfreq_add_device(dev, profile, "simple_ondemand", + bus->devfreq = devm_devfreq_add_device(dev, profile, + DEVFREQ_GOV_SIMPLE_ONDEMAND, ondemand_data); if (IS_ERR(bus->devfreq)) { dev_err(dev, "failed to add devfreq device\n"); @@ -488,7 +489,7 @@ static int exynos_bus_probe(struct platform_device *pdev) passive_data->parent = parent_devfreq; /* Add devfreq device for exynos bus with passive governor */ - bus->devfreq = devm_devfreq_add_device(dev, profile, "passive", + bus->devfreq = devm_devfreq_add_device(dev, profile, DEVFREQ_GOV_PASSIVE, passive_data); if (IS_ERR(bus->devfreq)) { dev_err(dev, diff --git a/drivers/devfreq/governor_passive.c b/drivers/devfreq/governor_passive.c index 673ad8cc9a1d..3bc29acbd54e 100644 --- a/drivers/devfreq/governor_passive.c +++ b/drivers/devfreq/governor_passive.c @@ -183,7 +183,7 @@ static int devfreq_passive_event_handler(struct devfreq *devfreq, } static struct devfreq_governor devfreq_passive = { - .name = "passive", + .name = DEVFREQ_GOV_PASSIVE, .immutable = 1, .get_target_freq = devfreq_passive_get_target_freq, .event_handler = devfreq_passive_event_handler, diff --git a/drivers/devfreq/governor_performance.c b/drivers/devfreq/governor_performance.c index c72f942f30a8..4d23ecfbd948 100644 --- a/drivers/devfreq/governor_performance.c +++ b/drivers/devfreq/governor_performance.c @@ -42,7 +42,7 @@ static int devfreq_performance_handler(struct devfreq *devfreq, } static struct devfreq_governor devfreq_performance = { - .name = "performance", + .name = DEVFREQ_GOV_PERFORMANCE, .get_target_freq = devfreq_performance_func, .event_handler = devfreq_performance_handler, }; diff --git a/drivers/devfreq/governor_powersave.c b/drivers/devfreq/governor_powersave.c index 0c6bed567e6d..0c42f23249ef 100644 --- a/drivers/devfreq/governor_powersave.c +++ b/drivers/devfreq/governor_powersave.c @@ -39,7 +39,7 @@ static int devfreq_powersave_handler(struct devfreq *devfreq, } static struct devfreq_governor devfreq_powersave = { - .name = "powersave", + .name = DEVFREQ_GOV_POWERSAVE, .get_target_freq = devfreq_powersave_func, .event_handler = devfreq_powersave_handler, }; diff --git a/drivers/devfreq/governor_simpleondemand.c b/drivers/devfreq/governor_simpleondemand.c index ae72ba5e78df..28e0f2de7100 100644 --- a/drivers/devfreq/governor_simpleondemand.c +++ b/drivers/devfreq/governor_simpleondemand.c @@ -125,7 +125,7 @@ static int devfreq_simple_ondemand_handler(struct devfreq *devfreq, } static struct devfreq_governor devfreq_simple_ondemand = { - .name = "simple_ondemand", + .name = DEVFREQ_GOV_SIMPLE_ONDEMAND, .get_target_freq = devfreq_simple_ondemand_func, .event_handler = devfreq_simple_ondemand_handler, }; diff --git a/drivers/devfreq/governor_userspace.c b/drivers/devfreq/governor_userspace.c index 77028c27593c..080607c3f34d 100644 --- a/drivers/devfreq/governor_userspace.c +++ b/drivers/devfreq/governor_userspace.c @@ -87,7 +87,7 @@ static ssize_t show_freq(struct device *dev, struct device_attribute *attr, NULL, }; static const struct attribute_group dev_attr_group = { - .name = "userspace", + .name = DEVFREQ_GOV_USERSPACE, .attrs = dev_entries, }; diff --git a/drivers/devfreq/rk3399_dmc.c b/drivers/devfreq/rk3399_dmc.c index 1b89ebbad02c..5dfbfa3cc878 100644 --- a/drivers/devfreq/rk3399_dmc.c +++ b/drivers/devfreq/rk3399_dmc.c @@ -431,7 +431,7 @@ static int rk3399_dmcfreq_probe(struct platform_device *pdev) data->devfreq = devm_devfreq_add_device(dev, &rk3399_devfreq_dmc_profile, - "simple_ondemand", + DEVFREQ_GOV_SIMPLE_ONDEMAND, &data->ondemand_data); if (IS_ERR(data->devfreq)) return PTR_ERR(data->devfreq); diff --git a/include/linux/devfreq.h b/include/linux/devfreq.h index 597294e0cc40..a3ad2074bebe 100644 --- a/include/linux/devfreq.h +++ b/include/linux/devfreq.h @@ -19,6 +19,13 @@ #define DEVFREQ_NAME_LEN 16 +/* DEVFREQ governor name */ +#define DEVFREQ_GOV_SIMPLE_ONDEMAND "simple_ondemand" +#define DEVFREQ_GOV_PERFORMANCE "performance" +#define DEVFREQ_GOV_POWERSAVE "powersave" +#define DEVFREQ_GOV_USERSPACE "userspace" +#define DEVFREQ_GOV_PASSIVE "passive" + /* DEVFREQ notifier interface */ #define DEVFREQ_TRANSITION_NOTIFIER (0)