linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* CONFIG_CPU_FREQ_DEBUG
@ 2011-11-09 19:17 Paul Bolle
  2012-02-10 15:21 ` [PATCH] cpufreq: get rid of remaining references to CONFIG_CPU_FREQ_DEBUG Dominik Brodowski
  0 siblings, 1 reply; 2+ messages in thread
From: Paul Bolle @ 2011-11-09 19:17 UTC (permalink / raw)
  To: linux-arm-kernel

0) Commit 2d06d8c49afdcc9bb35a85039fa50f0fe35bd40e ("[CPUFREQ] use
dynamic debug instead of custom infrastructure") dropped config
CPU_FREQ_DEBUG. But currently there are still two references to
CONFIG_CPU_FREQ_DEBUG (both in arch/arm).

1) I've stared a bit at the (trivial) code still relying on
CONFIG_CPU_FREQ_DEBUG but wasn't sure how it could be rewritten. Perhaps
one of you does know how to do that.


Paul Bolle

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [PATCH] cpufreq: get rid of remaining references to CONFIG_CPU_FREQ_DEBUG
  2011-11-09 19:17 CONFIG_CPU_FREQ_DEBUG Paul Bolle
@ 2012-02-10 15:21 ` Dominik Brodowski
  0 siblings, 0 replies; 2+ messages in thread
From: Dominik Brodowski @ 2012-02-10 15:21 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Nov 09, 2011 at 08:17:05PM +0100, Paul Bolle wrote:
> 0) Commit 2d06d8c49afdcc9bb35a85039fa50f0fe35bd40e ("[CPUFREQ] use
> dynamic debug instead of custom infrastructure") dropped config
> CPU_FREQ_DEBUG. But currently there are still two references to
> CONFIG_CPU_FREQ_DEBUG (both in arch/arm).

Thanks for noting this. These two instances can and should be updated
accordingly.

> 1) I've stared a bit at the (trivial) code still relying on
> CONFIG_CPU_FREQ_DEBUG but wasn't sure how it could be rewritten. Perhaps
> one of you does know how to do that.

Well, as we now use dynamic debug, we can use the generic debug calls used
througout the kernel ( pr_debug(...) ), and do not need to wrap the debug
message in an #ifdef . That's the whole story, actually. Russell, do you
want to take this through the ARM tree?

Best,
	Dominik

---
cpufreq: get rid of remaining references to CONFIG_CPU_FREQ_DEBUG

As noted by Paul Bolle <pebolle@tiscali.nl>, there are still two references
to CONFIG_CPU_FREQ_DEBUG which can and should be removed.

Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>

diff --git a/arch/arm/configs/ezx_defconfig b/arch/arm/configs/ezx_defconfig
index d95763d..872d96e 100644
--- a/arch/arm/configs/ezx_defconfig
+++ b/arch/arm/configs/ezx_defconfig
@@ -28,7 +28,6 @@ CONFIG_ZBOOT_ROM_BSS=0x0
 CONFIG_CMDLINE="console=tty1 root=/dev/mmcblk0p2 rootfstype=ext2 rootdelay=3 ip=192.168.0.202:192.168.0.200:192.168.0.200:255.255.255.0 debug"
 CONFIG_KEXEC=y
 CONFIG_CPU_FREQ=y
-CONFIG_CPU_FREQ_DEBUG=y
 CONFIG_CPU_FREQ_GOV_POWERSAVE=m
 CONFIG_CPU_FREQ_GOV_USERSPACE=m
 CONFIG_CPU_FREQ_GOV_ONDEMAND=m
diff --git a/arch/arm/mach-tegra/cpu-tegra.c b/arch/arm/mach-tegra/cpu-tegra.c
index bb5ce39..55b827f 100644
--- a/arch/arm/mach-tegra/cpu-tegra.c
+++ b/arch/arm/mach-tegra/cpu-tegra.c
@@ -97,10 +97,8 @@ static int tegra_update_cpu_speed(unsigned long rate)
 	for_each_online_cpu(freqs.cpu)
 		cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE);
 
-#ifdef CONFIG_CPU_FREQ_DEBUG
-	printk(KERN_DEBUG "cpufreq-tegra: transition: %u --> %u\n",
+	pr_debug("cpufreq-tegra: transition: %u --> %u\n",
 	       freqs.old, freqs.new);
-#endif
 
 	ret = clk_set_rate(cpu_clk, freqs.new * 1000);
 	if (ret) {

^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-02-10 15:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-09 19:17 CONFIG_CPU_FREQ_DEBUG Paul Bolle
2012-02-10 15:21 ` [PATCH] cpufreq: get rid of remaining references to CONFIG_CPU_FREQ_DEBUG Dominik Brodowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).