From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chanwoo Choi Subject: Re: [PATCH] cpufreq: Remove CONFIG_CPU_FREQ_STAT_DETAILS config option Date: Thu, 05 Jan 2017 19:12:29 +0900 Message-ID: <586E1C0D.20102@samsung.com> References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="Boundary_(ID_nvHJ5WsuTttLH/XkBof27w)" Return-path: In-reply-to: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Viresh Kumar , Rafael Wysocki , Kukjin Kim , Krzysztof Kozlowski , Javier Martinez Canillas , Jason Cooper , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Simon Horman , Magnus Damm Cc: linux-mips@linux-mips.org, linaro-kernel@lists.linaro.org, Vincent Guittot , linux-pm@vger.kernel.org, linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-samsung-soc@vger.kernel.org, arnd.bergmann@linaro.org, linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org List-Id: linux-pm@vger.kernel.org This is a multi-part message in MIME format. --Boundary_(ID_nvHJ5WsuTttLH/XkBof27w) Content-type: text/plain; charset=UTF-8 Content-transfer-encoding: 8BIT Hi Viresh, On 2017년 01월 05일 17:27, Viresh Kumar wrote: > This doesn't have any benefit apart from saving a small amount of memory > when it is disabled. The ifdef hackery in the code makes it dirty > unnecessarily. > > Clean it up by removing the Kconfig option completely. Few defconfigs > are also updated and CONFIG_CPU_FREQ_STAT_DETAILS is replaced with > CONFIG_CPU_FREQ_STAT now in them, as users wanted stats to be enabled. > > Signed-off-by: Viresh Kumar > --- > arch/arm/configs/exynos_defconfig | 2 +- > arch/arm/configs/multi_v5_defconfig | 2 +- > arch/arm/configs/multi_v7_defconfig | 2 +- > arch/arm/configs/mvebu_v5_defconfig | 2 +- > arch/arm/configs/pxa_defconfig | 2 +- > arch/arm/configs/shmobile_defconfig | 2 +- > arch/mips/configs/lemote2f_defconfig | 1 - > arch/powerpc/configs/ppc6xx_defconfig | 1 - > arch/sh/configs/sh7785lcr_32bit_defconfig | 2 +- > drivers/cpufreq/Kconfig | 8 -------- > drivers/cpufreq/cpufreq_stats.c | 14 -------------- > 11 files changed, 7 insertions(+), 31 deletions(-) I agree. Looks good to me. Reviewed-by: Chanwoo Choi [snip] -- Best Regards, Chanwoo Choi Samsung Electronics --Boundary_(ID_nvHJ5WsuTttLH/XkBof27w) Content-type: text/x-vcard; CHARSET=EUC-KR; name=cw00_choi.vcf Content-transfer-encoding: base64 Content-disposition: attachment; filename=cw00_choi.vcf bnVsbA0K --Boundary_(ID_nvHJ5WsuTttLH/XkBof27w) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --Boundary_(ID_nvHJ5WsuTttLH/XkBof27w)--