From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934050AbcDFKGQ (ORCPT ); Wed, 6 Apr 2016 06:06:16 -0400 Received: from mailout1.samsung.com ([203.254.224.24]:51884 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754822AbcDFKGO (ORCPT ); Wed, 6 Apr 2016 06:06:14 -0400 X-AuditID: cbfee68d-f79e86d0000012da-e1-5704df92e216 Date: Wed, 06 Apr 2016 10:06:10 +0000 (GMT) From: MyungJoo Ham Subject: Re: [PATCH v7 05/20] PM / devfreq: Add new passive governor To: =?utf-8?Q?=EC=B5=9C=EC=B0=AC=EC=9A=B0?= , =?utf-8?Q?=EB=B0=95=EA=B2=BD=EB=AF=BC?= , =?utf-8?Q?=ED=81=AC=EC=89=AC=EC=8B=9C=ED=86=A0=ED=94=84?= , "kgene@kernel.org" Cc: "rjw@rjwysocki.net" , "robh+dt@kernel.org" , "pawel.moll@arm.com" , "mark.rutland@arm.com" , "ijc+devicetree@hellion.org.uk" , "galak@codeaurora.org" , "linux@arm.linux.org.uk" , "linux.amoon@gmail.com" , "m.reichl@fivetechno.de" , "tjakobi@math.uni-bielefeld.de" , =?utf-8?Q?=EB=8C=80=EC=9D=B8=EA=B8=B0?= , "linux-kernel@vger.kernel.org" , "linux-pm@vger.kernel.org" , "linux-samsung-soc@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" Reply-to: myungjoo.ham@samsung.com MIME-version: 1.0 X-MTR: 20160406100556711@myungjoo.ham Msgkey: 20160406100556711@myungjoo.ham X-EPLocale: ko_KR.utf-8 X-Priority: 3 X-EPWebmail-Msg-Type: personal X-EPWebmail-Reply-Demand: 0 X-EPApproval-Locale: X-EPHeader: ML X-MLAttribute: X-RootMTR: 20160406100556711@myungjoo.ham X-ParentMTR: X-ArchiveUser: X-CPGSPASS: N X-ConfirmMail: N,general Content-type: text/plain; charset=utf-8 MIME-version: 1.0 Message-id: <1156624937.169371459937164322.JavaMail.weblogic@ep2mlwas02a> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrAIsWRmVeSWpSXmKPExsWyRsSkUHfSfZZwg20XWC0u75rD5sDo8XmT XABjFJdNSmpOZllqkb5dAlfGrJtNjAVPxCp29Uxma2CcItbFyMkhJKAusWjJSTYQW0LAROJx 13pWCFtM4sK99UBxLqCapYwSGxcfZoUpmnhxMjtE8xxGia3TNUFsFgEVifebdwPFOTjYBPQk Zn5OBjGFBVwklr61BqkQEZjAJLH4XwHISGaBhewST35cYoYYoySxZt8rFhCbV0BQ4uTMJywg vRICqhKLl2qAmLwCahK7V5RDHCAucWHuJXYIm1diRvtTFghbTmLa1zXMELa0xPlZGxhhPln8 /TFUnF/i2O0dTBC2gMTUMweharQkNj28AGXzSaxZ+JYFpn7XqeXMMLvub5kL1SshsbXlCThA mAUUJaZ0PwR7nFlAU2L9Ln1Uj4DYnhJbP/WxgnwuITCVQ2LXtLdsExiVZiGpm4Vk1CyEUchK FjCyrGIUTS1ILihOSi8y1CtOzC0uzUvXS87P3cQITAin/z3r3cF4+4D1IUYBDkYlHl4BK5Zw IdbEsuLK3EOMpsAomsgsJZqcD0w7eSXxhsZmRhamJqbGRuaWZkrivIpSP4OFBNITS1KzU1ML Uovii0pzUosPMTJxcEo1MMoLX3pibNHvIHTEc0tq8b10XZmNsZNfHNqq4bKXaUf6tchZ60yy DvJs8ee8c275+k3LmH/9SrR5/n/asl0hf/2KZuQpfTLZyPRt0RIjNq9Sn8/FPFmH9pZH/70X kCyrtkXY/FDIgjWhb5I0euI7mFq/+asfEfb5tHeh7U7XIPWgWo+Mr+t2qCmxFGckGmoxFxUn AgCJkHbEAwMAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrAKsWRmVeSWpSXmKPExsVy+t/tft1J91nCDf7XWlzeNYfNgdHj8ya5 AMaoNJuM1MSU1CKF1Lzk/JTMvHRbJe/geOd4UzMDQ11DSwtzJYW8xNxUWyUXnwBdt8wcoJlK CmWJOaVAoYDE4mIlfTubovzSklSFjPziElulaENzIz0jAz1TIz1D41grQwMDI1OgmoS0jFk3 mxgLnohV7OqZzNbAOEWsi5GTQ0hAXWLRkpNsILaEgInExIuT2SFsMYkL99azQdTMYZTYOl0T xGYRUJF4v3k3UA0HB5uAnsTMz8kgprCAi8TSt9YgFSICE5gkFv8r6GLk4mAWWMgu8eTHJWaI MUoSa/a9YgGxeQUEJU7OfMIC0ishoCqxeKkGiMkroCaxe0U5xAHiEhfmXoI6hldiRvtTFghb TmLa1zXMELa0xPlZGxhhDl78/TFUnF/i2O0dTBC2gMTUMweharQkNj28AGXzSaxZ+JYFpn7X qeXMMLvub5kL1SshsbXlCSuIzSygKDGl+yHY48wCmhLrd+mjegTE9pTY+qmPdQKj7CwkqVlI umchdCMrWcDIsopRNLUguaA4Kb3CWK84Mbe4NC9dLzk/dxMjOPU8W7yD8f9560OMAhyMSjy8 AlYs4UKsiWXFlbmHGCU4mJVEeGXvAYV4UxIrq1KL8uOLSnNSiw8xmgIjbCKzlGhyPjAt5pXE Gxobm5iZmFqaWBiYmiuJ8wb8XRcmJJCeWJKanZpakFoE08fEwSnVwBgm6Ot4UTNg/XuzrdKf F15wTl1xR1e+8sxGQZO3r82WmEjJPTzcNzv218WrJ0PPqyVWLDY/3uz1/+Oyhxffv+dqm3w3 vnGq7okP675oGpUcqexZfH5jyhuFitt+ARJ7bKqVXcSXPjl9a+q0hjvGz9/tWvfpmHPK6j9+ mSVOma/2TNC7d0mA9WKmEktxRqKhFnNRcSIAQM60ElMDAAA= DLP-Filter: Pass X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u36A6QGD001570 > > This patch adds the new passive governor for DEVFREQ framework. The following > governors are already present and used for DVFS (Dynamic Voltage and Frequency > Scaling) drivers. The following governors are independently used for one device [] > Suggested-by: Myungjoo Ham > Signed-off-by: Chanwoo Choi > [tjakobi: Reported RCU locking issue and cw00.choi fix it.] > Reported-by: Tobias Jakobi Signed-off-by: MyungJoo Ham Please just edit the comment below and push to your git repo so that I can simply pull your repo a day or two later. > --- > drivers/devfreq/Kconfig | 8 ++ > drivers/devfreq/Makefile | 1 + > drivers/devfreq/governor_passive.c | 206 +++++++++++++++++++++++++++++++++++++ > include/linux/devfreq.h | 33 ++++++ > 4 files changed, 248 insertions(+) > create mode 100644 drivers/devfreq/governor_passive.c > > diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig > index cedda8f1e5eb..d260cd0219f6 100644 [] > diff --git a/drivers/devfreq/Makefile b/drivers/devfreq/Makefile > index 8af8aaf922a8..2633087d5c63 100644 [] > diff --git a/drivers/devfreq/governor_passive.c b/drivers/devfreq/governor_passive.c > + /* > + * If the parent and passive devfreq device uses the OPP table, > + * get the next frequency by using the OPP table. Doesn't need comments from here > + * - parent devfreq device uses the governors except for passive. > + * - passive devfreq device uses the passive governor. > + */ > + > + /* To here. > + * Each devfreq has the OPP table. After deciding the new frequency > + * from the governor of parent devfreq device, the passive governor > + * need to get the index of new frequency on OPP table of parent > + * device. And then the index is used for getting the suitable > + * new frequency for passive devfreq device. > + */ [] > diff --git a/include/linux/devfreq.h b/include/linux/devfreq.h > index 98c699304e12..2de4e2eea180 100644 []