Commit 01f58191 authored by Dave Jones's avatar Dave Jones

[CPUFREQ] Use the unified cpufreq debug infrastructure in the longhaul driver.

Modified existing dprintks, and removed the driver-only debug module
parameter.
Signed-off-by: default avatarDominik Brodowski <linux@brodo.de>
Signed-off-by: default avatarDave Jones <davej@redhat.com>
parent 2ac79696
...@@ -58,21 +58,9 @@ static int vrmrev; ...@@ -58,21 +58,9 @@ static int vrmrev;
/* Module parameters */ /* Module parameters */
static int dont_scale_voltage; static int dont_scale_voltage;
static int debug;
static void dprintk(const char *fmt, ...)
{
char s[256];
va_list args;
if (debug == 0) #define dprintk(msg...) cpufreq_debug_printk(CPUFREQ_DEBUG_DRIVER, "longhaul", msg)
return;
va_start(args, fmt);
vsprintf(s, fmt, args);
printk(s);
va_end(args);
}
#define __hlt() __asm__ __volatile__("hlt": : :"memory") #define __hlt() __asm__ __volatile__("hlt": : :"memory")
...@@ -192,7 +180,7 @@ static void longhaul_setstate(unsigned int clock_ratio_index) ...@@ -192,7 +180,7 @@ static void longhaul_setstate(unsigned int clock_ratio_index)
cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE); cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE);
dprintk (KERN_INFO PFX "Setting to FSB:%dMHz Mult:%d.%dx (%s)\n", dprintk ("Setting to FSB:%dMHz Mult:%d.%dx (%s)\n",
fsb, mult/10, mult%10, print_speed(speed/1000)); fsb, mult/10, mult%10, print_speed(speed/1000));
switch (longhaul_version) { switch (longhaul_version) {
...@@ -356,7 +344,7 @@ static int __init longhaul_get_ranges(void) ...@@ -356,7 +344,7 @@ static int __init longhaul_get_ranges(void)
} }
} }
dprintk (KERN_INFO PFX "MinMult:%d.%dx MaxMult:%d.%dx\n", dprintk ("MinMult:%d.%dx MaxMult:%d.%dx\n",
minmult/10, minmult%10, maxmult/10, maxmult%10); minmult/10, minmult%10, maxmult/10, maxmult%10);
if (fsb == -1) { if (fsb == -1) {
...@@ -366,9 +354,9 @@ static int __init longhaul_get_ranges(void) ...@@ -366,9 +354,9 @@ static int __init longhaul_get_ranges(void)
highest_speed = calc_speed(maxmult); highest_speed = calc_speed(maxmult);
lowest_speed = calc_speed(minmult); lowest_speed = calc_speed(minmult);
dprintk (KERN_INFO PFX "FSB:%dMHz ", fsb); dprintk ("FSB:%dMHz Lowest speed: %s Highest speed:%s\n", fsb,
dprintk ("Lowest speed:%s ", print_speed(lowest_speed/1000)); print_speed(lowest_speed/1000),
dprintk ("Highest speed:%s\n", print_speed(highest_speed/1000)); print_speed(highest_speed/1000));
if (lowest_speed == highest_speed) { if (lowest_speed == highest_speed) {
printk (KERN_INFO PFX "highestspeed == lowest, aborting.\n"); printk (KERN_INFO PFX "highestspeed == lowest, aborting.\n");
...@@ -434,11 +422,11 @@ static void __init longhaul_setup_voltagescaling(void) ...@@ -434,11 +422,11 @@ static void __init longhaul_setup_voltagescaling(void)
} }
if (vrmrev==0) { if (vrmrev==0) {
dprintk (KERN_INFO PFX "VRM 8.5 : "); dprintk ("VRM 8.5 \n");
memcpy (voltage_table, vrm85scales, sizeof(voltage_table)); memcpy (voltage_table, vrm85scales, sizeof(voltage_table));
numvscales = (voltage_table[maxvid]-voltage_table[minvid])/25; numvscales = (voltage_table[maxvid]-voltage_table[minvid])/25;
} else { } else {
dprintk (KERN_INFO PFX "Mobile VRM : "); dprintk ("Mobile VRM \n");
memcpy (voltage_table, mobilevrmscales, sizeof(voltage_table)); memcpy (voltage_table, mobilevrmscales, sizeof(voltage_table));
numvscales = (voltage_table[maxvid]-voltage_table[minvid])/5; numvscales = (voltage_table[maxvid]-voltage_table[minvid])/5;
} }
...@@ -658,9 +646,6 @@ static void __exit longhaul_exit(void) ...@@ -658,9 +646,6 @@ static void __exit longhaul_exit(void)
module_param (dont_scale_voltage, int, 0644); module_param (dont_scale_voltage, int, 0644);
MODULE_PARM_DESC(dont_scale_voltage, "Don't scale voltage of processor"); MODULE_PARM_DESC(dont_scale_voltage, "Don't scale voltage of processor");
module_param (debug, int, 0644);
MODULE_PARM_DESC(debug, "Dump debugging information.");
MODULE_AUTHOR ("Dave Jones <davej@codemonkey.org.uk>"); MODULE_AUTHOR ("Dave Jones <davej@codemonkey.org.uk>");
MODULE_DESCRIPTION ("Longhaul driver for VIA Cyrix processors."); MODULE_DESCRIPTION ("Longhaul driver for VIA Cyrix processors.");
MODULE_LICENSE ("GPL"); MODULE_LICENSE ("GPL");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment